Modify

Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#16590 closed defect (fixed)

Troubleshoot DownloadWmsAlongTrackActionTest.testTMSLayer failure

Reported by: Don-vip Owned by: ris
Priority: normal Milestone: 18.08
Component: Unit tests Version:
Keywords: Cc: wiktorn

Description

Follow-up of #16010.

Test DownloadWmsAlongTrackActionTest.testTMSLayer fails on Jenkins since r14052:

org.awaitility.core.ConditionTimeoutException: Condition with org.openstreetmap.josm.gui.layer.gpx.DownloadWmsAlongTrackActionTest was not fulfilled within 10000 milliseconds.
	at org.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:145)
	at org.awaitility.core.CallableCondition.await(CallableCondition.java:79)
	at org.awaitility.core.CallableCondition.await(CallableCondition.java:27)
	at org.awaitility.core.ConditionFactory.until(ConditionFactory.java:902)
	at org.awaitility.core.ConditionFactory.until(ConditionFactory.java:860)
	at org.openstreetmap.josm.gui.layer.gpx.DownloadWmsAlongTrackActionTest.testTMSLayer(DownloadWmsAlongTrackActionTest.java:62)
	at org.openstreetmap.josm.testutils.JOSMTestRules$TimeoutThread.run(JOSMTestRules.java:689)

Until this is fixed, the test needs to be disabled to silence Jenkins notifications.

Change History (15)

comment:1 by Don-vip, 7 years ago

In 14116/josm:

see #16590 - ignore failing unit test

comment:2 by ris, 7 years ago

I've narrowed this down - this fails when the test is run preceded by both org.openstreetmap.josm.actions.AddImageryLayerActionTest and org.openstreetmap.josm.actions.ExitActionTest.

Last edited 7 years ago by ris (previous) (diff)

comment:3 by Don-vip, 7 years ago

Oh, great! :)

comment:4 by ris, 7 years ago

Ah, duh, MainApplication.shutdown() also shuts down JCSCacheManager. Will mock that out.

comment:6 by Don-vip, 7 years ago

Resolution: fixed
Status: assignedclosed

In 14126/josm:

fix #16590 - fix unit tests (patch by ris)

comment:7 by Don-vip, 7 years ago

Thanks! :)

comment:8 by ris, 7 years ago

Oh that's interesting - ExitActionTest itself failed now.

comment:9 by ris, 7 years ago

It would help if I'd included Main.java in the patch...

comment:10 by Don-vip, 7 years ago

In 14131/josm:

see #16590 - more fixes (patch by ris)
see #15229 - javadoc fixes

comment:11 by Don-vip, 7 years ago

In 14132/josm:

see #16590 - remove unused import

comment:12 by Don-vip, 7 years ago

In 14133/josm:

see #16590 - remove unused import

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain ris.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.