Debug tests

jderusse · jderusse · commit ecc7ca5e796b · 2017-09-03T14:38:46.000+02:00
diff --git a/.travis.yml b/.travis.yml
@@ -115,12 +115,9 @@ before_install:
       fi
 
       # Matrix lines for intermediate PHP versions are skipped for pull requests
-      if [[ ! $deps && ! $PHP = ${MIN_PHP%.*} && ! $PHP = hhvm* && $TRAVIS_PULL_REQUEST != false ]]; then
-          deps=skip
-          skip=1
-      else
-          COMPONENTS=$(find src/Symfony -mindepth 3 -type f -name phpunit.xml.dist -printf '%h\n')
-      fi
+      # COMPONENTS=$(printf 'src/Symfony/Bridge/Doctrine\nsrc/Symfony/Bundle/FrameworkBundle\nsrc/Symfony/Component/Cache\nsrc/Symfony/Component/HttpFoundation\nsrc/Symfony/Component/Lock\n')
+      # COMPONENTS=$(find src/Symfony -mindepth 3 -type f -name phpunit.xml.dist -printf '%h\n'|grep Lock)
+      COMPONENTS=$(find src/Symfony -mindepth 3 -type f -name phpunit.xml.dist -printf '%h\n')
 
     - |
       # Install sigchild-enabled PHP to test the Process component on the lowest PHP matrix line
diff --git a/src/Symfony/Component/Cache/Tests/Adapter/AbstractRedisAdapterTest.php b/src/Symfony/Component/Cache/Tests/Adapter/AbstractRedisAdapterTest.php
@@ -41,7 +41,6 @@ public static function setupBeforeClass()
 
     public static function tearDownAfterClass()
     {
-        self::$redis->flushDB();
         self::$redis = null;
     }
 }
diff --git a/src/Symfony/Component/Cache/Tests/Adapter/PredisClusterAdapterTest.php b/src/Symfony/Component/Cache/Tests/Adapter/PredisClusterAdapterTest.php
@@ -21,7 +21,6 @@ public static function setupBeforeClass()
 
     public static function tearDownAfterClass()
     {
-        self::$redis->getConnection()->getConnectionByKey('foo')->executeCommand(self::$redis->createCommand('FLUSHDB'));
         self::$redis = null;
     }
 }
diff --git a/src/Symfony/Component/Cache/Tests/Simple/AbstractRedisCacheTest.php b/src/Symfony/Component/Cache/Tests/Simple/AbstractRedisCacheTest.php
@@ -41,7 +41,6 @@ public static function setupBeforeClass()
 
     public static function tearDownAfterClass()
     {
-        self::$redis->flushDB();
         self::$redis = null;
     }
 }
diff --git a/src/Symfony/Component/Lock/Tests/Store/AbstractStoreTest.php b/src/Symfony/Component/Lock/Tests/Store/AbstractStoreTest.php
@@ -39,6 +39,9 @@ public function testSave()
         $this->assertFalse($store->exists($key));
     }
 
+    /**
+     * @dataProvider lotOfData
+     */
     public function testSaveWithDifferentResources()
     {
         $store = $this->getStore();
@@ -63,6 +66,9 @@ public function testSaveWithDifferentResources()
         $this->assertFalse($store->exists($key2));
     }
 
+    /**
+     * @dataProvider lotOfData
+     */
     public function testSaveWithDifferentKeysOnSameResources()
     {
         $store = $this->getStore();
@@ -112,4 +118,9 @@ public function testSaveTwice()
 
         $store->delete($key);
     }
+
+    public function lotOfData()
+    {
+        return array_fill(0, 1000, []);
+    }
 }
diff --git a/src/Symfony/Component/Lock/Tests/Store/MemcachedStoreTest.php b/src/Symfony/Component/Lock/Tests/Store/MemcachedStoreTest.php
@@ -11,6 +11,8 @@
 
 namespace Symfony\Component\Lock\Tests\Store;
 
+use Symfony\Component\Lock\Exception\LockConflictedException;
+use Symfony\Component\Lock\Key;
 use Symfony\Component\Lock\Store\MemcachedStore;
 
 /**

Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,6 @@ public static function setupBeforeClass()`
`41`	`41`
`42`	`42`	`public static function tearDownAfterClass()`
`43`	`43`	`{`
`44`		`- self::$redis->flushDB();`
`45`	`44`	`self::$redis = null;`
`46`	`45`	`}`
`47`	`46`	`}`
Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,6 @@ public static function setupBeforeClass()`
`21`	`21`
`22`	`22`	`public static function tearDownAfterClass()`
`23`	`23`	`{`
`24`		`- self::$redis->getConnection()->getConnectionByKey('foo')->executeCommand(self::$redis->createCommand('FLUSHDB'));`
`25`	`24`	`self::$redis = null;`
`26`	`25`	`}`
`27`	`26`	`}`
Original file line number	Diff line number	Diff line change
`@@ -39,6 +39,9 @@ public function testSave()`
`39`	`39`	`$this->assertFalse($store->exists($key));`
`40`	`40`	`}`
`41`	`41`
	`42`	`+ /**`
	`43`	`+ * @dataProvider lotOfData`
	`44`	`+ */`
`42`	`45`	`public function testSaveWithDifferentResources()`
`43`	`46`	`{`
`44`	`47`	`$store = $this->getStore();`
`@@ -63,6 +66,9 @@ public function testSaveWithDifferentResources()`
`63`	`66`	`$this->assertFalse($store->exists($key2));`
`64`	`67`	`}`
`65`	`68`
	`69`	`+ /**`
	`70`	`+ * @dataProvider lotOfData`
	`71`	`+ */`
`66`	`72`	`public function testSaveWithDifferentKeysOnSameResources()`
`67`	`73`	`{`
`68`	`74`	`$store = $this->getStore();`
`@@ -112,4 +118,9 @@ public function testSaveTwice()`
`112`	`118`
`113`	`119`	`$store->delete($key);`
`114`	`120`	`}`
	`121`	`+`
	`122`	`+ public function lotOfData()`
	`123`	`+ {`
	`124`	`+ return array_fill(0, 1000, []);`
	`125`	`+ }`
`115`	`126`	`}`