Merge pull request #432 from DavidGoodwin/fix-redistest-driver-expectation

tedivm · web-flow · commit 2e1954b104cf · 2025-02-07T18:15:19.000-06:00
RedisTest throws \RedisException, so expect that, not Warning
diff --git a/tests/Stash/Test/Driver/RedisTest.php b/tests/Stash/Test/Driver/RedisTest.php
@@ -67,7 +67,7 @@ protected function getInvalidOptions()
 
     public function testBadDisconnect()
     {
-        $this->expectException('Warning');
+        $this->expectException("RedisException");
         if (defined('HHVM_VERSION')) {
             $this->markTestSkipped('This test can not run on HHVM as HHVM throws a different set of errors.');
         }

Original file line number	Diff line number	Diff line change
`@@ -67,7 +67,7 @@ protected function getInvalidOptions()`
`67`	`67`
`68`	`68`	`public function testBadDisconnect()`
`69`	`69`	`{`
`70`		`- $this->expectException('Warning');`
	`70`	`+ $this->expectException("RedisException");`
`71`	`71`	`if (defined('HHVM_VERSION')) {`
`72`	`72`	`$this->markTestSkipped('This test can not run on HHVM as HHVM throws a different set of errors.');`
`73`	`73`	`}`