Task #10845 (closed)
Opened 11 years ago
Closed 9 years ago
RFE: Update TestNG to 6.8.1
| Reported by: | bpindelski | Owned by: | sbesson |
|---|---|---|---|
| Priority: | minor | Milestone: | 5.x |
| Component: | Deployment | Version: | 5.0.0-beta1 |
| Keywords: | n.a. | Cc: | jamoore, mtbcarroll, sbesson |
| Resources: | n.a. | Referenced By: | n.a. |
| References: | n.a. | Remaining Time: | n.a. |
| Sprint: | n.a. |
Description
Currently TestNG 6.8 depends on guice.jar (as a bug). guice.jar has been added to the repository to allow for tests to run.
A long-term solution would be to update TestNG to 6.8.1 (fixes the guice bug), remove guice.jar and add JCommander jar as a dependency. This has to be done for Bio-Formats first (verify that tests pass). Then update ome-dsl/ivy.xml.
See https://groups.google.com/forum/#!topic/testng-dev/EsBF4wxTxs0
Change History (8)
comment:1 Changed 11 years ago by bpindelski
- Milestone changed from 5.0.0-beta1 to 5.0.0-beta2
- Priority changed from major to minor
- Version set to 5.0.0-beta1
comment:2 Changed 10 years ago by jburel
- Milestone changed from 5.0.0-beta2 to 5.0.0
comment:3 Changed 9 years ago by jburel
comment:4 Changed 9 years ago by jburel
- Cc sbesson added
cc'ing Sebastien.
An update on OMERO side will now be possible after decoupling
comment:5 Changed 9 years ago by sbesson
- Owner set to sbesson
Bumping TestNG and removing guice results in an error when running the dsl unit tests - see https://travis-ci.org/sbesson/openmicroscopy/jobs/78280074. Is this still valid?
comment:6 Changed 9 years ago by mtbcarroll
It seems likely we'll want to bump it up to some later version at some point, but I'm not aware of any urgency.
comment:7 Changed 9 years ago by jamoore
Even the latest version (http://jcenter.bintray.com/org/testng/testng/6.9.5/%testng-6.9.5.pom) has a dependency on guice. I don't mind bumping testng, but I think the rest of this ticket no longer holds.
comment:8 Changed 9 years ago by sbesson
- Resolution set to invalid
- Status changed from new to closed
Closing this ticket. We can bump TestNG independently when required.
Moving to 5, unless there is an urgent need to do it now.