Remove placeholder tests
authorChad Horohoe <chadh@wikimedia.org>
Fri, 4 Oct 2013 19:33:58 +0000 (12:33 -0700)
committerChad Horohoe <chadh@wikimedia.org>
Fri, 4 Oct 2013 19:35:51 +0000 (12:35 -0700)
They give a false sense of test coverage and nobody's shown any
interest in implementing them.

Change-Id: Id15ddb555e059bb6a76c69ca57d09c91e96e91ce

tests/phpunit/includes/api/ApiEditPageTest.php
tests/phpunit/includes/api/ApiWatchTest.php

index e680af6..b0efd19 100644 (file)
@@ -193,10 +193,6 @@ class ApiEditPageTest extends ApiTestCase {
                $this->assertEquals( $expected, $text );
        }
 
-       function testEditSection() {
-               $this->markTestIncomplete( "not yet implemented" );
-       }
-
        /**
         * Test action=edit&section=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++;
index 78bb151..8253989 100644 (file)
@@ -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' );
-       }
 }