openmrs-core tests currently failing

Development
1

Hi,

there are 2 tests currently failing on master see https://travis-ci.org/github/openmrs/openmrs-core/jobs/716072053#L3816

I am not sure why yet. I am reverting changes I made today. So far that has not fixed it, which puzzles me. If anyone has time and a hunch please let me know. I am trying to fix this but am unsure when that will be. I am very sorry. Thanks for your patience in advance!

1 Like
2

I think I reverted all of today’s changes, the build was green yesterday. I don’t know what is causing this. Any help is very appreciated!

1 Like
3

@teleivo it fails locally too.

Screenshot from 2020-08-08 10-07-34
Screenshot from 2020-08-08 10-07-341645Ă—203 55.3 KB

4

not unexpected, would be even weirder if it did not.

If you wonder why I commit changes that most likely lead to this. Well the automated PRs, ran the tests and they were passing. I did not change this test or the implementation it is testing. I also did not change the dataset this test is relying on.

I also think I reverted all of my today’s changes so we should be back at a working state. Unfortunately, we are not.

I might be overlooking something super simple that is why I need others to help me figure this out as you have fresh eyes. Thanks a lot :heart:

5

the implementation hasnt changed in a long time see https://github.com/openmrs/openmrs-core/commits/master/api/src/main/java/org/openmrs/notification/db/hibernate/HibernateAlertDAO.java

6

Change the expiration date in the file “org/openmrs/api/db/hibernate/include/HibernateAlertDAOTestDataSet.xml” for alert_id=“2”, for example to 2021. :wink:

2 Likes
7

whaaooo , great @sacull

1 Like
8

related PR : https://github.com/openmrs/openmrs-core/pull/3473

9

Remember that, because this post will come back in a year. 2021 was an example, 2100 or similar would be better.

1 Like
10

thanks @sacull , 2100 is too far, 2025 will be better .

11

Any year that does not fail the test will be good. :nerd_face:

1 Like
12

thank you very much @sacull

1 Like
13

thank you so much! what a classic :joy: I guess I was the lucky one who got bitten by it. Well, next time I might remember when I go down the list of possibilities. Again thanknyou so much for looking into this :heart:

After the quick fix we should if possible create this data entry dynamically. So that we adjust the date and make future proof test.

2 Likes
14

Do we need to create a Ticket for this

@teleivo ?

15

https://issues.openmrs.org/browse/TRUNK-5914

16

we just merged the suggested fix :slight_smile: I reopened the ticket that added the test with a suggestion on how to improve it

17

so please don’t create a new ticket

18

builds are passing again! thanks a lot to @sacull @achilep !!

3 Likes
19

That was a nice catch @sacull :smile:

2 Likes