Permalink
Browse files

Fix Memcached test failures.

The Memcached extension does not provide enough information to PHP's
Reflection API to allow mock objects to be generated correctly. We'll
skip the affected tests until php-memcached-dev/php-memcached#126 has
been resolved.
  • Loading branch information...
markstory committed Oct 15, 2014
1 parent e686a43 commit afcd0312bfaf7a95e6e786637e2a23044f054a9e
Showing with 3 additions and 8 deletions.
  1. +3 −8 lib/Cake/Test/Case/Cache/Engine/MemcachedEngineTest.php
@@ -331,14 +331,7 @@ public function testSaslAuthException() {
'password' => 'password'
);
- $this->skipIf(
- method_exists($Memcached->getMemcached(), 'setSaslAuthData'),
- 'Memcached extension is installed with SASL support'
- );
-
- $this->setExpectedException(
- 'CacheException', 'Memcached extension is not build with SASL support'
- );
+ $this->setExpectedException('PHPUnit_Framework_Error_Warning');
$Memcached->init($settings);
}
@@ -689,6 +682,8 @@ public function testZeroDuration() {
* @return void
*/
public function testLongDurationEqualToZero() {
+ $this->markTestSkipped('Cannot run as Memcached cannot be reflected');
+
$memcached = new TestMemcachedEngine();
$memcached->settings['compress'] = false;

0 comments on commit afcd031

Please sign in to comment.