profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/indraneelr/events. GitMemory does not store any data, but only uses NGINX to cache data for a period of time. The idea behind GitMemory is simply to give users a better reading experience.

cbgaindia/story-generator 17

Budget Visualization Tool to explore and analyse major fiscal indicators across various states in India

ICT4H/atomfeed 15

An ATOM implementation in Java to broadcast and consume events.

CoffeeBeansLabs/SimDB 2

A vector similarity database

ICT4H/simplefeed 2

A simple Atom-based protocol for broadcasting a feed of events.

ICT4H/atomfeed-standalone 1

A server that creates and publishes events using the ATOM protocol.

ICT4H/Cordova-SQLitePlugin 1

Cordova-SQLitePlugin used by dcs-mobile

ICT4H/dcs-mobile 1

DCS mobile app

ICT4H/dcs-web 1

Forked from datawinners

ICT4H/dcs-api 0

(Can be deleted since code merged to dcs-web)Python app to expose api's to be used by dcs-app

Pull request review commentopenmrs/openmrs-distro-referenceapplication

RATEST-143: Fix record merge issue test

 public void setUp() throws Exception {         patient1 = new TestPatient();     } -    @Ignore+    @Test+    @Category(BuildTests.class)     public void recordMergeIssueTest() throws Exception {-        homePage.goToRegisterPatientApp();+        homePage.goToRegisterPatientApp().waitForPage();

After the test invoking many pages during the test setUp(), the patient registration page takes a bit longer to be loaded by the classLoader. Adding waitforPage() resolves the delay Otherwise the actions that follow are not effective and the patient data is not saved in db instance

kdaud

comment created time in 5 hours

Pull request review commentopenmrs/openmrs-distro-referenceapplication

RATEST-143: Fix record merge issue test

 public void tearDown() throws Exception {         p.uuid = patient.uuid;         deletePatient(p);         waitForPatientDeletion(patient.uuid);-        headerPage.logOut();

After deleting the test patient data from the db instance there is no need to invoke headerPage.logOut() within tearDown() function

kdaud

comment created time in 5 hours

Pull request review commentopenmrs/openmrs-distro-referenceapplication

RATEST-74: Failing Test: ignored AddFormTests Follow up

     private static final By MANAGE_MODULES = By.cssSelector("#legacyui-manageModules a");     private final static By MANAGE_VISIT_TYPES = By.cssSelector("#legacyui-manageVisitTypes a");     private static final By MANAGE_PROVIDERS = By.cssSelector("#content a[href='/openmrs/admin/provider/index.htm']");-    private static final By MANAGE_PERSONS = By.cssSelector("#content a[href=\"/openmrs/admin/person/index.htm\"]");+    private static final By MANAGE_PERSONS = By.cssSelector("#content a[href='/openmrs/admin/person/index.htm']");+    private static final By MANAGE_HTMLFORMS_PAGE = By.cssSelector("#content a[href*='/module/htmlformentry/htmlForms.list']");

Resolved thanks

sherrif10

comment created time in 7 hours

PR opened openmrs/openmrs-distro-referenceapplication

RATEST-143: Fix record merge issue test

Ticket ID Description -> Fix record merge issue test

+10 -5

0 comment

1 changed file

pr created time in 7 hours

Pull request review commentopenmrs/openmrs-distro-referenceapplication

RATEST-74: Failing Test: ignored AddFormTests Follow up

 public void addFormTest() throws Exception {         manageForm.formIdFromUrl();         manageForm.save();         headerPage.clickOnHomeIcon();-        homePage.goToActiveVisitPatient();

This is not needed with in this code, it will only break the automation perhaps it also doesnt add up within the workflow

sherrif10

comment created time in 8 hours

Pull request review commentopenmrs/openmrs-distro-referenceapplication

fixed RATEST-149:Failing Test: Check Modules Test

     /**      * Check the list of modules to be sure they are all started.      */-    @Ignore+    @Test     @Category(org.openmrs.reference.groups.BuildTests.class)

@category(buildTest.class)

I think @sherrif10 meant @Category(BuildTest.class)

insookwa

comment created time in 8 hours

Pull request review commentopenmrs/openmrs-module-reportingrest

Fix swagger error by changing webservices dependency version to imple…

 public void testEvaluatingSerializedCohortDefintionWithParams() throws Exception 	private void assertCohortMembers(Object evaluated, String json, String[] expectedUuids) throws Exception { 		assertEquals(expectedUuids.length, ((List) path(evaluated, "members")).size()); 		for (String expected : expectedUuids) {

Does this mean the resource was changed from patient to person? Could you point me to where that was done?

logeshvar

comment created time in 8 hours

Pull request review commentopenmrs/openmrs-distro-referenceapplication

fixed RATEST-149:Failing Test: Check Modules Test

     /**      * Check the list of modules to be sure they are all started.      */-    @Ignore+    @Test     @Category(org.openmrs.reference.groups.BuildTests.class)

You can make it as @Category(buildTest.class) not org.openmrs.reference.groups.buildTests.class simply because the full package is being defined in pom class

insookwa

comment created time in 8 hours

Pull request review commentopenmrs/openmrs-distro-referenceapplication

fixed RATEST-149:Failing Test: Check Modules Test

     /**      * Check the list of modules to be sure they are all started.      */-    @Ignore+    @Test

the @category is just below the @Test

insookwa

comment created time in 8 hours

pull request commentopenmrs/openmrs-module-webservices.rest

RESTWS-823: Add REST resource for Alert Management

@dkayiwa Can you see why the snyk job has failed? I can't access it

BartlomiejRasztabiga

comment created time in 17 hours

Pull request review commentopenmrs/openmrs-module-reportingrest

Fix swagger error by changing webservices dependency version to imple…

 public void testEvaluatingSerializedCohortDefintionWithParams() throws Exception 	private void assertCohortMembers(Object evaluated, String json, String[] expectedUuids) throws Exception { 		assertEquals(expectedUuids.length, ((List) path(evaluated, "members")).size()); 		for (String expected : expectedUuids) {

Yes, due to change in dependency version, there is some change in the internals that cause this.. so had to change that. :)

logeshvar

comment created time in a day

CommitCommentEvent

pull request commentopenmrs/openmrs-distro-referenceapplication

RATEST-93:Failing Test: Delete Form Test

Please fix https://ci.openmrs.org/browse/REFAPP-UI-257

sherrif10

comment created time in a day

push eventopenmrs/openmrs-distro-referenceapplication

k-joseph

commit sha 4478682c4b004a08c1d5da4ea91d4e6f8a003cc4

Revert "RATEST-93:Failing Test: Delete Form Test (#423)" (#460) This reverts commit 896ae4eb57f953ade5e512c7d909fb41d1a3eebb.

view details

push time in a day

push eventopenmrs/openmrs-distro-referenceapplication

MAGEMBE SHARIF

commit sha 896ae4eb57f953ade5e512c7d909fb41d1a3eebb

RATEST-93:Failing Test: Delete Form Test (#423) * RATEST-93:Failing Test: Delete Form Test * rebuilding the index

view details

push time in a day

Pull request review commentopenmrs/openmrs-distro-referenceapplication

RATEST-74: Failing Test: ignored AddFormTests Follow up

 public void addFormTest() throws Exception {         manageForm.formIdFromUrl();         manageForm.save();         headerPage.clickOnHomeIcon();-        homePage.goToActiveVisitPatient();

once again, clarify why this was removed

sherrif10

comment created time in a day

Pull request review commentopenmrs/openmrs-distro-referenceapplication

RATEST-74: Failing Test: ignored AddFormTests Follow up

 public void deletePath() {         DELETE_FORM_PATH = By.xpath(DELETE_FORM + id + REST_PATH + id + REST_PATH_2);         clickOn(DELETE_FORM_PATH);     }--    //    Eye Form+//  Eye Form

this looks to be miss formatted

sherrif10

comment created time in a day

Pull request review commentopenmrs/openmrs-distro-referenceapplication

RATEST-74: Failing Test: ignored AddFormTests Follow up

     private static final By MANAGE_MODULES = By.cssSelector("#legacyui-manageModules a");     private final static By MANAGE_VISIT_TYPES = By.cssSelector("#legacyui-manageVisitTypes a");     private static final By MANAGE_PROVIDERS = By.cssSelector("#content a[href='/openmrs/admin/provider/index.htm']");-    private static final By MANAGE_PERSONS = By.cssSelector("#content a[href=\"/openmrs/admin/person/index.htm\"]");+    private static final By MANAGE_PERSONS = By.cssSelector("#content a[href='/openmrs/admin/person/index.htm']");+    private static final By MANAGE_HTMLFORMS_PAGE = By.cssSelector("#content a[href*='/module/htmlformentry/htmlForms.list']");

any reason why you need the asteric after href? you can also get rid of #content

sherrif10

comment created time in a day

pull request commentopenmrs/openmrs-distro-referenceapplication

RATEST-143: Fix record merge issue test

@kdaud please fix https://ci.openmrs.org/browse/REFAPP-UI-255

kdaud

comment created time in a day

push eventopenmrs/openmrs-distro-referenceapplication

k-joseph

commit sha 87c703a8bcfad0053293798fee6af0922321013c

Revert "RATEST-143: Fix record merge issue test (#448)" (#459) This reverts commit 1c6c4295a070119abba7df8b6a08cf47b34808e2.

view details

push time in a day

PR closed openmrs/openmrs-distro-referenceapplication

RATEST-91: Fix book request appointment test

Ticket ID Description -> Fix book request appointment test

+6 -5

0 comment

2 changed files

kdaud

pr closed time in a day

pull request commentopenmrs/openmrs-module-webservices.rest

RESTWS-821: Add REST resource with links to administration pages of installed modules

Why not add the resource to the omod-2.0 submodule? We no longer support 1.8

I moved it to omod-2.0 but the PersonResource test is still failing. https://github.com/openmrs/openmrs-module-webservices.rest/pull/484/commits/76a01380b3c7101d85a040f627cd7cec3cb188ba

BartlomiejRasztabiga

comment created time in a day

push eventopenmrs/openmrs-distro-referenceapplication

Kakumirizi Daud

commit sha 1c6c4295a070119abba7df8b6a08cf47b34808e2

RATEST-143: Fix record merge issue test (#448) * RATEST-143: Fix record merge issue test * RATEST-143: Refactoring code base for quality assurance

view details

push time in a day