Modify

Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#14495 closed defect (fixed)

StyleCacheTest failing

Reported by: bastiK Owned by: michael2402
Priority: normal Milestone: 17.03
Component: Unit tests Version:
Keywords: test Cc:

Description

StyleCacheTest.testStyleCacheInternPool test is failing in Jenkins.

There is a problem with this test that it uses the current global MapCSS style, which is modified by other tests. So it is kind of random what this method tests. Whatever style it uses at the moment, is causing a test error.

Attachments (0)

Change History (10)

comment:1 by stoecker, 7 years ago

Means JOSMFixture.init() must be fixed?

comment:2 by Don-vip, 7 years ago

Cc: michael2402 added
Component: Core mappaintUnit tests

comment:3 by Don-vip, 7 years ago

Cc: michael2402 removed
Owner: changed from team to michael2402

3 other tests must be fixed:

org.openstreetmap.josm.gui.dialogs.InspectPrimitiveDialogTest.testBuildMapPaintText
org.openstreetmap.josm.gui.mappaint.MapCSSRendererTest.testRender[area-text]
org.openstreetmap.josm.data.validation.tests.MultipolygonTestTest.testTicket10469

comment:4 by michael2402, 7 years ago

In 11776/josm:

Reduce startup time of StyleCacheTest, use test rules, do not leak memory. See #14495

comment:5 by michael2402, 7 years ago

In 11777/josm:

See #14495 - move style source reset to JOSM test rules.

comment:6 by bastiK, 7 years ago

In 11797/josm:

see #14495 - fix immutable class non-constant field (by removing it)

comment:7 by bastiK, 7 years ago

In 11805/josm:

see #14495 - add debugging output

comment:8 by bastiK, 7 years ago

Resolution: fixed
Status: newclosed

In 11818/josm:

fixed #14495 - stale ElemStyles.defaultLinesIdx

comment:9 by bastiK, 7 years ago

I will get rid of all the public fields in org.openstreetmap.josm.gui.mappaint.styleelement.*, this is really not such a brilliant idea for immutable classes.

comment:10 by michael2402, 7 years ago

Agreed, I have not found a plugin that uses these fields.

Modify Ticket

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