Merge lp:~yrke/tapaal/fix-1887512-SharedPlaceWithToken into lp:tapaal

Proposed by Kenneth Yrke Jørgensen
Status: Merged
Approved by: Kenneth Yrke Jørgensen
Approved revision: 1069
Merged at revision: 1068
Proposed branch: lp:~yrke/tapaal/fix-1887512-SharedPlaceWithToken
Merge into: lp:tapaal
Diff against target: 59 lines (+25/-2)
3 files modified
src/dk/aau/cs/io/TapnXmlLoader.java (+3/-2)
src/pipe/gui/GuiFrameController.java (+1/-0)
tests/dk/aau/cs/io/TapnXmlLoaderTest.kt (+21/-0)
To merge this branch: bzr merge lp:~yrke/tapaal/fix-1887512-SharedPlaceWithToken
Reviewer Review Type Date Requested Status
Peter Haahr Taankvist (community) Approve
Review via email: mp+387357@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Peter Haahr Taankvist (ptaank) wrote :

I cannot reproduce the error anymore.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/dk/aau/cs/io/TapnXmlLoader.java'
2--- src/dk/aau/cs/io/TapnXmlLoader.java 2020-06-21 13:33:43 +0000
3+++ src/dk/aau/cs/io/TapnXmlLoader.java 2020-07-14 11:27:23 +0000
4@@ -162,8 +162,9 @@
5 }
6
7 SharedPlace place = new SharedPlace(name, invariant);
8- place.addTokens(numberOfTokens);
9- place.setCurrentMarking(marking);
10+ place.setCurrentMarking(marking);
11+ place.addTokens(numberOfTokens);
12+
13
14 return place;
15 }
16
17=== modified file 'src/pipe/gui/GuiFrameController.java'
18--- src/pipe/gui/GuiFrameController.java 2020-06-22 08:20:47 +0000
19+++ src/pipe/gui/GuiFrameController.java 2020-07-14 11:27:23 +0000
20@@ -304,6 +304,7 @@
21 e.getMessage(),
22 "Error loading file",
23 JOptionPane.ERROR_MESSAGE);
24+ e.printStackTrace();
25 return;
26 }finally {
27 CreateGui.getAppGui().setCursor(Cursor.getPredefinedCursor(Cursor.DEFAULT_CURSOR));
28
29=== modified file 'tests/dk/aau/cs/io/TapnXmlLoaderTest.kt'
30--- tests/dk/aau/cs/io/TapnXmlLoaderTest.kt 2020-06-21 13:30:40 +0000
31+++ tests/dk/aau/cs/io/TapnXmlLoaderTest.kt 2020-07-14 11:27:23 +0000
32@@ -110,6 +110,27 @@
33 }
34
35 @Test
36+ fun `SharedPlace has a token bug#1887512`() {
37+ val net = """
38+ <?xml version="1.0" encoding="UTF-8" standalone="no"?>
39+ <pnml xmlns="http://www.informatik.hu-berlin.de/top/pnml/ptNetb">
40+ <shared-place initialMarking="1" invariant="&lt; inf" name="P0"/>
41+ <net active="true" id="IntroExample" type="P/T net">
42+ <place displayName="true" id="P0" initialMarking="1" invariant="&lt; inf" name="P0" nameOffsetX="0" nameOffsetY="0" positionX="105" positionY="30"/>
43+ </net>
44+ </pnml>
45+ """.trimIndent().asInpurtStream()
46+
47+ val tapnXmlLoader = TapnXmlLoader()
48+
49+ val model = tapnXmlLoader.load(net)
50+ val place = model.templates().first().guiModel().getPlaceByName("P0") as TimedPlaceComponent
51+
52+ Assertions.assertEquals(1, place.numberOfTokens)
53+ Assertions.assertEquals(BigDecimal.ZERO, place.underlyingPlace().tokens().first().age())
54+ }
55+
56+ @Test
57 fun `Place has 5 token`() {
58 val net = xmlNet("""
59 <place displayName="true" id="Start" initialMarking="5" invariant="&lt; inf" name="Start" nameOffsetX="-5.0" nameOffsetY="35.0" positionX="135.0" positionY="30.0"/>

Subscribers

People subscribed via source and target branches