Skip to content

Commit 6abb9f3

Browse files
committed
Added test for scheduler queueing
1 parent b81f9ec commit 6abb9f3

File tree

2 files changed

+19
-2
lines changed

2 files changed

+19
-2
lines changed

pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44

55
<groupId>com.marginallyclever</groupId>
66
<artifactId>nodegraphcore</artifactId>
7-
<version>1.6.0</version>
7+
<version>1.6.1</version>
88

99
<name>NodeGraphCore</name>
1010
<description>Flow based programming in Java.</description>

src/test/java/com/marginallyclever/nodegraphcore/TestGraph.java

Lines changed: 18 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,12 +78,27 @@ private void buildAddTwoConstants() {
7878
public void testAddTwoConstants() {
7979
buildAddTwoConstants();
8080
ThreadPoolScheduler scheduler = new ThreadPoolScheduler();
81+
scheduler.setQueueDownstreamNodes(true);
8182
scheduler.submit(graph.getNodes().get(0));
8283
scheduler.submit(graph.getNodes().get(1));
8384
scheduler.run();
8485
assertEquals( 3.0, graph.getNodes().get(2).getPort(2).getValue() );
8586
}
8687

88+
/**
89+
* confirm adding two constants together via {@link Connection}s works as expected
90+
*/
91+
@Test
92+
public void testAddTwoConstantsFailsWithoutScheduler() {
93+
buildAddTwoConstants();
94+
ThreadPoolScheduler scheduler = new ThreadPoolScheduler();
95+
scheduler.setQueueDownstreamNodes(false);
96+
scheduler.submit(graph.getNodes().get(0));
97+
scheduler.submit(graph.getNodes().get(1));
98+
scheduler.run();
99+
assertNotEquals( 3.0, graph.getNodes().get(2).getPort(2).getValue() );
100+
}
101+
87102
/**
88103
* confirm adding two constants together via {@link Connection}s works as expected.
89104
* confirm {@link PrintToStdOut} works as expected.
@@ -95,6 +110,7 @@ public void testAddTwoConstantsAndReport() {
95110
Node report = graph.add(new PrintToStdOut());
96111
graph.add(new Connection(graph.getNodes().get(2),2,report,0));
97112

113+
scheduler.setQueueDownstreamNodes(true);
98114
scheduler.submit(graph.getNodes().get(0));
99115
scheduler.submit(graph.getNodes().get(1));
100116
scheduler.run();
@@ -275,8 +291,9 @@ public void testAddTwoGraphsTogether() {
275291
}
276292
assertEquals(4,count);
277293

294+
scheduler.setQueueDownstreamNodes(true);
278295
scheduler.run();
279-
296+
// check the values of the Adds
280297
assertEquals(9.0,m.getPort(2).getValue());
281298
}
282299
}

0 commit comments

Comments
 (0)