From 4e63cbf52be79db519b596a1592f4a67f71fcd16 Mon Sep 17 00:00:00 2001 From: Aaron Date: Thu, 23 Aug 2012 14:13:20 -0700 Subject: [PATCH] Split up doOperations() test to be more robust when tests fail. Change-Id: If7c0f1476e694feeb791754608ea964899be0c38 --- .../includes/filerepo/FileBackendTest.php | 46 ++++++++++--------- 1 file changed, 25 insertions(+), 21 deletions(-) diff --git a/tests/phpunit/includes/filerepo/FileBackendTest.php b/tests/phpunit/includes/filerepo/FileBackendTest.php index 6fb7acef75..c22a86760c 100644 --- a/tests/phpunit/includes/filerepo/FileBackendTest.php +++ b/tests/phpunit/includes/filerepo/FileBackendTest.php @@ -1147,26 +1147,6 @@ class FileBackendTest extends MediaWikiTestCase { $this->tearDownFiles(); $this->doTestDoOperations(); $this->tearDownFiles(); - - $this->backend = $this->singleBackend; - $this->tearDownFiles(); - $this->doTestDoOperations2(); - $this->tearDownFiles(); - - $this->backend = $this->multiBackend; - $this->tearDownFiles(); - $this->doTestDoOperations2(); - $this->tearDownFiles(); - - $this->backend = $this->singleBackend; - $this->tearDownFiles(); - $this->doTestDoOperationsFailing(); - $this->tearDownFiles(); - - $this->backend = $this->multiBackend; - $this->tearDownFiles(); - $this->doTestDoOperationsFailing(); - $this->tearDownFiles(); } private function doTestDoOperations() { @@ -1242,8 +1222,20 @@ class FileBackendTest extends MediaWikiTestCase { "Correct file SHA-1 of $fileC" ); } + public function testDoOperationsPipeline() { + $this->backend = $this->singleBackend; + $this->tearDownFiles(); + $this->doTestDoOperationsPipeline(); + $this->tearDownFiles(); + + $this->backend = $this->multiBackend; + $this->tearDownFiles(); + $this->doTestDoOperationsPipeline(); + $this->tearDownFiles(); + } + // concurrency orientated - private function doTestDoOperations2() { + private function doTestDoOperationsPipeline() { $base = $this->baseStorePath(); $fileAContents = '3tqtmoeatmn4wg4qe-mg3qt3 tq'; @@ -1329,6 +1321,18 @@ class FileBackendTest extends MediaWikiTestCase { "Correct file SHA-1 of $fileC" ); } + public function testDoOperationsFailing() { + $this->backend = $this->singleBackend; + $this->tearDownFiles(); + $this->doTestDoOperationsFailing(); + $this->tearDownFiles(); + + $this->backend = $this->multiBackend; + $this->tearDownFiles(); + $this->doTestDoOperationsFailing(); + $this->tearDownFiles(); + } + private function doTestDoOperationsFailing() { $base = $this->baseStorePath(); -- 2.20.1