mozzy
(Moses Mutesasira)
June 18, 2020, 8:46pm
43
Thanks @ibacher for looking into this ,
@achilep most times a BeanCreationException
in Tests is caused by missing dependence on the class path at test run time and a carefull follow up of the error gives a crue which dependence is likely to be missing.
1 Like
achilep
(Pahonsi Bebeto Achile)
June 18, 2020, 10:27pm
44
thanks, @ibacher @mozzy i use this 2:
<dependency>
<groupId>jakarta.xml.bind</groupId>
<artifactId>jakarta.xml.bind-api</artifactId>
<version>3.0.0-RC3</version>
</dependency>
<dependency>
<groupId>com.sun.xml.bind</groupId>
<artifactId>jaxb-impl</artifactId>
<version>3.0.0-M4</version>
</dependency>
and is working without a problem , please is there a more precise way to test this 2 change.
1 Like
mozzy
(Moses Mutesasira)
June 19, 2020, 5:59am
45
<dependency>
<groupId>com.sun.xml.bind</groupId>
<artifactId>jaxb-impl</artifactId>
<version>3.0.0-M4</version>
</dependency>
does it still fail without this dependence ??
1 Like
mozzy
(Moses Mutesasira)
June 19, 2020, 6:10am
46
@achilep can you first resolve these merge conflicts ??
Its because your other PRs on the same pom.xml file were merged. So first pull all those merged commits into this PR.
1 Like
achilep
(Pahonsi Bebeto Achile)
June 19, 2020, 7:09am
47
its no longer failing. i’m just asking if there is a more precise way to test it, apart from the standalone reference application that i use for testing.
thanks @mozzy
mozzy
(Moses Mutesasira)
June 19, 2020, 7:18am
48
i was asking that doest it fail when you remove that other dependence ??
What do you mean by precise ?? you mean simpler ??
There other ways you can run the reff app ie
running a docker container
sdk
enterpise installtion
but it seems to me the standalone is the lightest and easiest way you can test your changes in the core
2 Likes
dkayiwa
(Daniel Kayiwa)
June 19, 2020, 8:16am
50
Lighter or easier than simply running mvn jetty:run
or mvn tomcat7:run
from the webapp
folder?
2 Likes
achilep
(Pahonsi Bebeto Achile)
June 19, 2020, 8:21am
51
thank you @mozzy , i have resolved the merge conflict?
1 Like
mozzy
(Moses Mutesasira)
June 19, 2020, 8:26am
53
@dkayiwa true ,
Thats seems to be the simplest ,
but a standalone seems to be the most lightest ,when it comes to processing
achilep
(Pahonsi Bebeto Achile)
June 19, 2020, 4:06pm
54
please can you have links which explains how contributes on the documentation.
thanks
achilep
(Pahonsi Bebeto Achile)
June 20, 2020, 9:43pm
55
hello @mozzy @ibacher @sharif please I need help to fix the problem on this PR:
openmrs:master
← achilep:TRUNK-5764
opened 09:19PM - 05 Jun 20 UTC
Indeed I want to update ca.uhn.hapi to version 2.3. but this version no longer supports MessageTypeRouter. and suddenly I try to replace it by ApplicationRouter
without succeeding.
achilep
(Pahonsi Bebeto Achile)
June 20, 2020, 10:34pm
56
hello @ibacher @teleivo
thanks for the resources, its a lot of help, but i’m still stuck at the conversion of BaseContextSensitiveTest and these subclasses. here is an example of what I tried to do but it doesn’t work:
BaseContextSensitiveTest in junit5 .BaseContextSensitiveTest - Pastebin.com
AllergenTest in junit5 : AllergenTest - Pastebin.com
BaseOpenmrsObjectTest in junit 5 : BaseOpenmrsObjectTest - Pastebin.com
I don’t know what I failed to do correctly. thank you
teleivo
(Ivo Ulrich)
June 23, 2020, 6:37am
57
@achilep I commented on github Its enough if you post here or on github I am watching both
2 Likes
achilep
(Pahonsi Bebeto Achile)
June 23, 2020, 7:04am
58
Thanks @teleivo I will make good use of it.
achilep
(Pahonsi Bebeto Achile)
June 23, 2020, 1:27pm
59
hello @ibacher @mozzy @dkayiwa @teleivo
please can you help me evaluate this ticket, i changed my approach for migrate from junit 4 to junit 5
thanks
TRUNK-5811 Step to Update Junit from 4.13 to Latest version 5.6.2
Type: Bug
Priority: TBD
achilep
(Pahonsi Bebeto Achile)
June 24, 2020, 5:56am
61
as we agree to support junit 4 until version 2.5 or 2.5 we can leave the seventh sub-task
@ibacher @dkayiwa @mozzy @burke
sharif
(Sharif Magembe)
June 24, 2020, 7:28am
62
We can also make them ready for work unless otherwise
1 Like