Skip to content

Commit 6c2cb6a

Browse files
committed
Fix assertions on Sets
1 parent 86636bf commit 6c2cb6a

File tree

4 files changed

+20
-20
lines changed

4 files changed

+20
-20
lines changed

tests/GroupsTest.php

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ public function testGraphEmpty()
1414
$this->assertEquals(array(), $alg->getGroups());
1515
$this->assertEquals(0, $alg->getNumberOfGroups());
1616

17-
$this->assertEquals(array(), $alg->getVerticesGroup(123));
17+
$this->assertTrue($alg->getVerticesGroup(123)->isEmpty());
1818

1919
$this->assertFalse($alg->isBipartit());
2020
}
@@ -32,8 +32,8 @@ public function testGraphPairIsBipartit()
3232
$this->assertEquals(array(1, 2), $alg->getGroups());
3333
$this->assertEquals(2, $alg->getNumberOfGroups());
3434

35-
$this->assertEquals(array(), $alg->getVerticesGroup(123));
36-
$this->assertEquals(array(1 => $v1), $alg->getVerticesGroup(1));
35+
$this->assertTrue($alg->getVerticesGroup(123)->isEmpty());
36+
$this->assertEquals(array(1 => $v1), $alg->getVerticesGroup(1)->getMap());
3737

3838
$this->assertTrue($alg->isBipartit());
3939
}
@@ -54,8 +54,8 @@ public function testGraphTriangleCycleIsNotBipartit()
5454
$this->assertEquals(array(1, 2), $alg->getGroups());
5555
$this->assertEquals(2, $alg->getNumberOfGroups());
5656

57-
$this->assertEquals(array(), $alg->getVerticesGroup(123));
58-
$this->assertEquals(array(1 => $v1, 3 => $v3), $alg->getVerticesGroup(1));
57+
$this->assertTrue($alg->getVerticesGroup(123)->isEmpty());
58+
$this->assertEquals(array(1 => $v1, 3 => $v3), $alg->getVerticesGroup(1)->getMap());
5959

6060
$this->assertFalse($alg->isBipartit());
6161
}

tests/ParallelTest.php

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -27,8 +27,8 @@ public function testDirectedCycleIsNotConsideredParallel()
2727
$alg = new AlgorithmParallel($graph);
2828

2929
$this->assertFalse($alg->hasEdgeParallel());
30-
$this->assertEquals(array(), $alg->getEdgesParallelEdge($e1));
31-
$this->assertEquals(array(), $alg->getEdgesParallelEdge($e2));
30+
$this->assertEquals(array(), $alg->getEdgesParallelEdge($e1)->getVector());
31+
$this->assertEquals(array(), $alg->getEdgesParallelEdge($e2)->getVector());
3232
}
3333

3434
public function testDirectedParallelEdge()
@@ -44,8 +44,8 @@ public function testDirectedParallelEdge()
4444
$alg = new AlgorithmParallel($graph);
4545

4646
$this->assertTrue($alg->hasEdgeParallel());
47-
$this->assertEquals(array($e2), $alg->getEdgesParallelEdge($e1));
48-
$this->assertEquals(array($e1), $alg->getEdgesParallelEdge($e2));
47+
$this->assertEquals(array($e2), $alg->getEdgesParallelEdge($e1)->getVector());
48+
$this->assertEquals(array($e1), $alg->getEdgesParallelEdge($e2)->getVector());
4949
}
5050

5151
public function testMixedParallelEdge()
@@ -61,8 +61,8 @@ public function testMixedParallelEdge()
6161
$alg = new AlgorithmParallel($graph);
6262

6363
$this->assertTrue($alg->hasEdgeParallel());
64-
$this->assertEquals(array($e2), $alg->getEdgesParallelEdge($e1));
65-
$this->assertEquals(array($e1), $alg->getEdgesParallelEdge($e2));
64+
$this->assertEquals(array($e2), $alg->getEdgesParallelEdge($e1)->getVector());
65+
$this->assertEquals(array($e1), $alg->getEdgesParallelEdge($e2)->getVector());
6666
}
6767

6868
public function testMixedParallelEdgesMultiple()
@@ -86,12 +86,12 @@ public function testMixedParallelEdgesMultiple()
8686
$alg = new AlgorithmParallel($graph);
8787

8888
$this->assertTrue($alg->hasEdgeParallel());
89-
$this->assertEquals(array($e2, $e3, $e4), $alg->getEdgesParallelEdge($e1));
90-
$this->assertEquals(array($e1, $e3, $e4), $alg->getEdgesParallelEdge($e2));
91-
$this->assertEquals(array($e1, $e2, $e4, $e5, $e6), $alg->getEdgesParallelEdge($e3));
92-
$this->assertEquals(array($e1, $e2, $e3, $e5, $e6), $alg->getEdgesParallelEdge($e4));
93-
$this->assertEquals(array($e3, $e4, $e6), $alg->getEdgesParallelEdge($e5));
94-
$this->assertEquals(array($e3, $e4, $e5), $alg->getEdgesParallelEdge($e6));
89+
$this->assertEquals(array($e2, $e3, $e4), $alg->getEdgesParallelEdge($e1)->getVector());
90+
$this->assertEquals(array($e1, $e3, $e4), $alg->getEdgesParallelEdge($e2)->getVector());
91+
$this->assertEquals(array($e1, $e2, $e4, $e5, $e6), $alg->getEdgesParallelEdge($e3)->getVector());
92+
$this->assertEquals(array($e1, $e2, $e3, $e5, $e6), $alg->getEdgesParallelEdge($e4)->getVector());
93+
$this->assertEquals(array($e3, $e4, $e6), $alg->getEdgesParallelEdge($e5)->getVector());
94+
$this->assertEquals(array($e3, $e4, $e5), $alg->getEdgesParallelEdge($e6)->getVector());
9595
}
9696

9797
}

tests/Property/PropertyGraphTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ class PropertyGraphTest extends TestCase
77
{
88
public function testEmptyIsEdgeless()
99
{
10-
$graph = new Graph($graph);
10+
$graph = new Graph();
1111

1212
$alg = new GraphProperty($graph);
1313

tests/Property/WalkPropertyTest.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,7 @@ public function testCircuit()
9494
// 1 -> 2 -> 2 -> 1
9595
$walk = Walk::factoryFromEdges(array($e1, $e3, $e2), $v1);
9696

97-
$this->assertEquals(array(1, 2, 2, 1), $walk->getVerticesSequenceId());
97+
$this->assertEquals(array(1, 2, 2, 1), $walk->getVertices()->getIds());
9898

9999
$alg = new WalkProperty($walk);
100100

@@ -116,7 +116,7 @@ public function testNonCircuit()
116116
// 1 -> 2 -> 2 -> 2 -> 1
117117
$walk = Walk::factoryFromEdges(array($e1, $e3, $e3, $e2), $v1);
118118

119-
$this->assertEquals(array(1, 2, 2, 2, 1), $walk->getVerticesSequenceId());
119+
$this->assertEquals(array(1, 2, 2, 2, 1), $walk->getVertices()->getIds());
120120

121121
$alg = new WalkProperty($walk);
122122

0 commit comments

Comments
 (0)