Modify

Opened 7 weeks ago

Closed 7 days ago

#19087 closed defect (fixed)

Jenkins JOSM-Plugins fails on plugin wikipedia target test-init

Reported by: GerdP Owned by: team
Priority: normal Milestone:
Component: Unit tests Version:
Keywords: Cc: simon04, floscher

Description

Seems the wikipedia plugin causes a problem:
https://josm.openstreetmap.de/jenkins/job/JOSM-Plugins/jdk=JDK8/1839/console

ant clean dist takes very long (Total time: 3 minutes 50 seconds) but works on my machine.

Attachments (0)

Change History (12)

comment:1 Changed 3 weeks ago by simon04

Is this now being discussed in ticket:18053#comment:26 ff.?

comment:2 Changed 3 weeks ago by GerdP

Yes, but it should be discussed here. Sorry.

comment:3 Changed 3 weeks ago by GerdP

Cc: simon04 added

Any idea what happens with the wikipedia plugin in the test phase?
See end of https://josm.openstreetmap.de/jenkins/job/JOSM-Plugins/jdk=JDK8/1855/console
I see the same on my machine. Is it even intended that the unit tests are executed in batch?

comment:4 Changed 2 weeks ago by GerdP

Shouldn't we fix the problems in the plugins before we continue changing JOSM core? We have lots of problems with the JOSM-Integration job as well and I wonder if anybody cares?
I tried to understand what's wrong but failed so far.

comment:5 Changed 2 weeks ago by simon04

Cc: floscher added
Summary: Jenkins: Projekt JOSM-Plugins failsJenkins JOSM-Plugins fails on plugin wikipedia target test-init

comment:7 Changed 2 weeks ago by GerdP

jdk8 seems to be fine :)
jdk11 still fails, but for the MicrosoftStreetside plugin:
https://josm.openstreetmap.de/jenkins/job/JOSM-Plugins/jdk=JDK11/1858/consoleText

comment:8 Changed 2 weeks ago by simon04

MicrosoftStreetside: I expected that, see ticket:19208#comment:18

comment:9 Changed 12 days ago by GerdP

wikipedia plugin fails again, the file ivy_settings.xml is missing. I assume it should have the same content as
josm\core\ivysettings.xml ?
Note the extra underscore in the name used in josm\plugins\build-common.xml
Is that intended? Should every plugin use its own file?

comment:10 in reply to:  9 Changed 12 days ago by simon04

Sorry, I forgot to git add
Done via https://gitlab.com/JOSM/plugin/wikipedia/-/commit/46a3d84e2d5a2c42ff829980ea650c6578b2e632

Is that intended? Should every plugin use its own file?

It is how it currently works. Not my preferred solution…

comment:11 Changed 7 days ago by GerdP

Component: CoreUnit tests

comment:12 Changed 7 days ago by GerdP

Resolution: fixed
Status: newclosed

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.