diff options
| author | Joas Schilling <coding@schilljs.com> | 2018-01-25 11:23:12 +0100 |
|---|---|---|
| committer | Joas Schilling <coding@schilljs.com> | 2018-01-25 11:33:25 +0100 |
| commit | bf2be08c9f3b6957a3327929a169eb6204e84cd0 (patch) | |
| tree | 4738ade539309fea85c5db8e796783c2cdbc0438 /tests/lib/Hooks | |
| parent | 24af5c20bd65f8e657bcc495984234088c22e824 (diff) | |
| download | nextcloud-server-bf2be08c9f3b6957a3327929a169eb6204e84cd0.tar.gz nextcloud-server-bf2be08c9f3b6957a3327929a169eb6204e84cd0.zip | |
Fix risky tests without assertions
Signed-off-by: Joas Schilling <coding@schilljs.com>
Diffstat (limited to 'tests/lib/Hooks')
| -rw-r--r-- | tests/lib/Hooks/BasicEmitterTest.php | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/tests/lib/Hooks/BasicEmitterTest.php b/tests/lib/Hooks/BasicEmitterTest.php index 2e6bd8100e3..7b71bb8259e 100644 --- a/tests/lib/Hooks/BasicEmitterTest.php +++ b/tests/lib/Hooks/BasicEmitterTest.php @@ -157,7 +157,7 @@ class BasicEmitterTest extends \Test\TestCase { $this->emitter->removeListener('Test', 'test', $listener); $this->emitter->emitEvent('Test', 'test'); - $this->assertTrue(true); + $this->addToAssertionCount(1); } public function testRemoveWildcardListener() { @@ -172,7 +172,7 @@ class BasicEmitterTest extends \Test\TestCase { $this->emitter->removeListener('Test', 'test'); $this->emitter->emitEvent('Test', 'test'); - $this->assertTrue(true); + $this->addToAssertionCount(1); } public function testRemoveWildcardMethod() { @@ -185,7 +185,7 @@ class BasicEmitterTest extends \Test\TestCase { $this->emitter->emitEvent('Test', 'test'); $this->emitter->emitEvent('Test', 'foo'); - $this->assertTrue(true); + $this->addToAssertionCount(1); } public function testRemoveWildcardScope() { @@ -198,7 +198,7 @@ class BasicEmitterTest extends \Test\TestCase { $this->emitter->emitEvent('Test', 'test'); $this->emitter->emitEvent('Bar', 'test'); - $this->assertTrue(true); + $this->addToAssertionCount(1); } public function testRemoveWildcardScopeAndMethod() { @@ -213,7 +213,7 @@ class BasicEmitterTest extends \Test\TestCase { $this->emitter->emitEvent('Test', 'foo'); $this->emitter->emitEvent('Bar', 'foo'); - $this->assertTrue(true); + $this->addToAssertionCount(1); } /** @@ -231,7 +231,7 @@ class BasicEmitterTest extends \Test\TestCase { $this->emitter->removeListener('Test', 'test', $listener1); $this->emitter->emitEvent('Test', 'test'); - $this->assertTrue(true); + $this->addToAssertionCount(1); } /** @@ -246,7 +246,7 @@ class BasicEmitterTest extends \Test\TestCase { $this->emitter->removeListener('Test', 'foo', $listener); $this->emitter->emitEvent('Test', 'test'); - $this->assertTrue(true); + $this->addToAssertionCount(1); } /** @@ -261,7 +261,7 @@ class BasicEmitterTest extends \Test\TestCase { $this->emitter->removeListener('Bar', 'test', $listener); $this->emitter->emitEvent('Test', 'test'); - $this->assertTrue(true); + $this->addToAssertionCount(1); } /** @@ -275,6 +275,6 @@ class BasicEmitterTest extends \Test\TestCase { $this->emitter->removeListener('Bar', 'test', $listener); $this->emitter->emitEvent('Test', 'test'); - $this->assertTrue(true); + $this->addToAssertionCount(1); } } |