Modify

Opened 5 years ago

Closed 5 years ago

#18576 closed defect (fixed)

PluginHandlerTest#testBuildListOfPluginsToLoad is broken

Reported by: simon04 Owned by: Don-vip
Priority: normal Milestone: 20.01
Component: Unit tests Version:
Keywords: Cc:

Description

Before r15717, the test set the following system property causing the error "JOSM could not find information about the following plugins"

josm.plugins=org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@2f1cfecf,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@b2414e30,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@67f89e51,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@a8a2fdbf,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@67f8a766,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@67f8b2e7,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@67f8cc36,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@67f8d415,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@67f8d4cf,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@c395b966,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@cf27964d,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@894f8e72,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@3f2a3ab4,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@a811abdd,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@3c7eeac0,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@85e5d4bf,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@777589ef,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@57c3a841,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@5f34d421,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@ff9053ef,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@10e4286d,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@7df7a262,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@c5ae19ff,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@13afa32c,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@c1d27406,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@5b05539a,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@950a8a6f,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@98339bfe,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@55b3341,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@67a8e8c9,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@2ebb3b42,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@9ca54e81,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@5beca769,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@53cb26dd,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@968c7407,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@b42fdee2,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@d6f15503,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@382f9bc5,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@179b66b8,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@8a4e4470,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@5428d528,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@6d3331,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@5ff695f,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@e3233dd,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@61067956,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@2423fb6a,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@447b2626,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@b5d64e9e,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@64de8479,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@92ac640,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@7dc0b6c,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@18404457,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@c6756b75,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@fef61b85,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@6af757b3,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@9fda5cb3,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@c9b6e3eb,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@d2badfe6,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@11db38ca,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@362ac678,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@6e617b0a,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@f7b27d7f,org.openstreetmap.josm.plugins.PluginHandler$DeprecatedPlugin@a6abf147,[irsrectify, surveyor2, gpsbabelgui, Intersect_way, ContourOverlappingMerge, LaneConnector, Remove.redundant.points]

This is due calling toString on DeprecatedPlugin which is not implemented.

When setting the following system property (after r15717 and r15723), the test fails:

josm.plugins=mappaint,unglueplugin,lang-de,lang-en_GB,lang-fr,lang-it,lang-pl,lang-ro,lang-ru,ewmsplugin,ywms,tways-0.2,geotagged,landsat,namefinder,waypoints,slippy_map_chooser,tcx-support,usertools,AgPifoJ,utilsplugin,ghost,validator,multipoly,multipoly-convert,remotecontrol,imagery,slippymap,wmsplugin,ParallelWay,dumbutils,ImproveWayAccuracy,Curves,epsg31287,licensechange,restart,wayselector,openstreetbugs,nearclick,notes,mirrored_download,ImageryCache,commons-imaging,missingRoads,trafficFlowDirection,kendzi3d-jogl,josm-geojson,proj4j,OpenStreetView,imageryadjust,walkingpapers,czechaddress,kendzi3d_Improved_by_Andrei,videomapping,public_transport_layer,lakewalker,download_along,plastic_laf,osmarender,geojson,gpxfilter,tag2link,rapid,irsrectify,surveyor2,gpsbabelgui,Intersect_way,ContourOverlappingMerge,LaneConnector,Remove.redundant.points

Attachments (0)

Change History (2)

comment:1 by Don-vip, 5 years ago

Owner: changed from team to Don-vip
Status: newassigned

comment:2 by Don-vip, 5 years ago

Resolution: fixed
Status: assignedclosed

In 15737/josm:

fix #18576 - fix PluginHandlerTest#testBuildListOfPluginsToLoad unit test

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Don-vip.
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.