Skip to content

Commit

Permalink
Fix PHP CS Fixer errors
Browse files Browse the repository at this point in the history
  • Loading branch information
colinmollenhour committed Oct 26, 2023
1 parent 4572156 commit c985c8d
Show file tree
Hide file tree
Showing 3 changed files with 344 additions and 345 deletions.
3 changes: 1 addition & 2 deletions Client.php
Original file line number Diff line number Diff line change
Expand Up @@ -1391,7 +1391,6 @@ public function __call($name, $args)
$response = $typeMap[$response];
break;

// Handle scripting errors
case 'eval':
case 'evalsha':
case 'script':
Expand Down Expand Up @@ -1420,7 +1419,7 @@ public function __call($name, $args)
if (is_bool($response) && $response === true) {
$this->redis->clearLastError();
}
// no break
// no break
default:
$error = $this->redis->getLastError();
$this->redis->clearLastError();
Expand Down
28 changes: 14 additions & 14 deletions tests/CredisStandaloneTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -15,20 +15,20 @@ public function testPersistentConnectionsOnStandAloneTcpConnection()
$this->assertEquals('value', $this->credis->get('key'));
}

public function testPersistentvsNonPersistent()
{
$this->assertTrue(true);
}
public function testPersistentvsNonPersistent()
{
$this->assertTrue(true);
}

public function testStandAloneArgumentsExtra()
{
$this->assertTrue($this->credis->hMSet('hash', array('field1' => 'value1', 'field2' => 'value2'), 'field3', 'value3'));
$this->assertEquals(array('field1' => 'value1', 'field2' => 'value2', 'field3' =>'value3'), $this->credis->hMGet('hash', array('field1','field2','field3')));
}
public function testStandAloneArgumentsExtra()
{
$this->assertTrue($this->credis->hMSet('hash', array('field1' => 'value1', 'field2' => 'value2'), 'field3', 'value3'));
$this->assertEquals(array('field1' => 'value1', 'field2' => 'value2', 'field3' =>'value3'), $this->credis->hMGet('hash', array('field1','field2','field3')));
}

public function testStandAloneMultiPipelineThrowsException()
{
$this->setExpectedExceptionShim('CredisException', 'A pipeline is already in use and only one pipeline is supported.');
$this->credis->pipeline()->pipeline();
}
public function testStandAloneMultiPipelineThrowsException()
{
$this->setExpectedExceptionShim('CredisException', 'A pipeline is already in use and only one pipeline is supported.');
$this->credis->pipeline()->pipeline();
}
}

0 comments on commit c985c8d

Please sign in to comment.