From d42a417d6ee11051f7c9ef8269947e77813cab42 Mon Sep 17 00:00:00 2001 From: Chad Horohoe Date: Fri, 4 Oct 2013 12:33:58 -0700 Subject: [PATCH] Remove placeholder tests They give a false sense of test coverage and nobody's shown any interest in implementing them. Change-Id: Id15ddb555e059bb6a76c69ca57d09c91e96e91ce --- .../phpunit/includes/api/ApiEditPageTest.php | 8 -------- tests/phpunit/includes/api/ApiWatchTest.php | 19 ------------------- 2 files changed, 27 deletions(-) diff --git a/tests/phpunit/includes/api/ApiEditPageTest.php b/tests/phpunit/includes/api/ApiEditPageTest.php index e680af628d..b0efd19dc7 100644 --- a/tests/phpunit/includes/api/ApiEditPageTest.php +++ b/tests/phpunit/includes/api/ApiEditPageTest.php @@ -193,10 +193,6 @@ class ApiEditPageTest extends ApiTestCase { $this->assertEquals( $expected, $text ); } - function testEditSection() { - $this->markTestIncomplete( "not yet implemented" ); - } - /** * Test action=edit§ion=new * Run it twice so we test adding a new section on a @@ -236,10 +232,6 @@ class ApiEditPageTest extends ApiTestCase { $this->assertEquals( $text, "== header ==\n\ntest\n\n== header ==\n\ntest" ); } - function testUndo() { - $this->markTestIncomplete( "not yet implemented" ); - } - function testEditConflict() { static $count = 0; $count++; diff --git a/tests/phpunit/includes/api/ApiWatchTest.php b/tests/phpunit/includes/api/ApiWatchTest.php index 78bb1515f8..825398958e 100644 --- a/tests/phpunit/includes/api/ApiWatchTest.php +++ b/tests/phpunit/includes/api/ApiWatchTest.php @@ -145,23 +145,4 @@ class ApiWatchTest extends ApiTestCase { } } } - - /** - */ - function testWatchDelete() { - $tokens = $this->getTokens(); - - $data = $this->doApiRequest( array( - 'action' => 'delete', - 'token' => $tokens['deletetoken'], - 'title' => 'Help:UTPage' ) ); - $this->assertArrayHasKey( 'delete', $data[0] ); - $this->assertArrayHasKey( 'title', $data[0]['delete'] ); - - $this->doApiRequest( array( - 'action' => 'query', - 'list' => 'watchlist' ) ); - - $this->markTestIncomplete( 'This test needs to verify the deleted article was added to the users watchlist' ); - } } -- 2.20.1