Skip to content

Commit

Permalink
Use always true assert instead
Browse files Browse the repository at this point in the history
  • Loading branch information
murrant committed Apr 26, 2023
1 parent 9be14af commit e7ca6de
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions tests/OSModulesTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -92,8 +92,6 @@ public function testDataIsValid($os, $variant, $modules)
*/
public function testOS($os, $variant, $modules)
{
$this->expectNotToPerformAssertions(); // successful tests will make no assertions due to checkTestData()

// Lock testing time
$this->travelTo(new \DateTime('2022-01-01 00:00:00'));
$this->requireSnmpsim(); // require snmpsim for tests
Expand Down Expand Up @@ -131,6 +129,8 @@ public function testOS($os, $variant, $modules)
$this->checkTestData($expected, $actual, 'Polled', $os, $module, $filename, $helper, $phpunit_debug);
}

$this->assertTrue(true, "Tested $os successfully"); // avoid no asserts error

DeviceCache::flush(); // clear cached devices
$this->travelBack();
}
Expand Down

0 comments on commit e7ca6de

Please sign in to comment.