Remove notes about being destructive. We work properly on cloned tables now

diff --git a/tests/phpunit/TODO b/tests/phpunit/TODO
index 76f38c1..b2fa7fb 100644 (file)
--- a/tests/phpunit/TODO
+++ b/tests/phpunit/TODO
@@ -1,9 +1,4 @@
 == Things To Do ==
-
-* DEFAULT TESTS NEED TO MADE NON-DESTRUCTIVE. Any destructive tests which alter the contents of the live wiki need to
-be protected with an explicit confirmation so people exploring their system don't accidentally destroy their main page
-or register user accounts with default passwords.
-
 * Most of the tests are named poorly; naming should describe a use case in story-like language, not simply identify the
 unit under test. An example would be the difference between testCalculate and testAddingIntegersTogetherWorks.
 * Many of the tests make multiple assertions, and are thus not unitary tests. By using data-providers and more use-case

diff --git a/tests/phpunit/includes/ArticleTablesTest.php b/tests/phpunit/includes/ArticleTablesTest.php
index 362cddf..01776c9 100644 (file)
--- a/tests/phpunit/includes/ArticleTablesTest.php
+++ b/tests/phpunit/includes/ArticleTablesTest.php
@@ -2,7 +2,6 @@
 
 /**
  * @group Database
- * @group Destructive
  */
 class ArticleTablesTest extends MediaWikiLangTestCase {
 

diff --git a/tests/phpunit/includes/api/ApiBlockTest.php b/tests/phpunit/includes/api/ApiBlockTest.php
index 325a30f..227555e 100644 (file)
--- a/tests/phpunit/includes/api/ApiBlockTest.php
+++ b/tests/phpunit/includes/api/ApiBlockTest.php
@@ -2,7 +2,6 @@
 
 /**
  * @group Database
- * @group Destructive
  */
 class ApiBlockTest extends ApiTestCase {
 

diff --git a/tests/phpunit/includes/api/ApiTest.php b/tests/phpunit/includes/api/ApiTest.php
index e2215ac..a587e6b 100644 (file)
--- a/tests/phpunit/includes/api/ApiTest.php
+++ b/tests/phpunit/includes/api/ApiTest.php
@@ -2,7 +2,6 @@
 
 /**
  * @group Database
- * @group Destructive
  */
 class ApiTest extends ApiTestCase {
 

diff --git a/tests/phpunit/includes/api/ApiUploadTest.php b/tests/phpunit/includes/api/ApiUploadTest.php
index 8ce6258..e2a3150 100644 (file)
--- a/tests/phpunit/includes/api/ApiUploadTest.php
+++ b/tests/phpunit/includes/api/ApiUploadTest.php
@@ -2,7 +2,6 @@
 
 /**
  * @group Database
- * @group Destructive
  */
 
 /**
@@ -20,7 +19,6 @@ require_once( 'ApiTestCaseUpload.php' );
 
 /**
  * @group Database
- * @group Destructive
  *
  * This is pretty sucky... needs to be prettified.
  */

diff --git a/tests/phpunit/includes/api/ApiWatchTest.php b/tests/phpunit/includes/api/ApiWatchTest.php
index 7fca538..3c7ff30 100644 (file)
--- a/tests/phpunit/includes/api/ApiWatchTest.php
+++ b/tests/phpunit/includes/api/ApiWatchTest.php
@@ -2,7 +2,6 @@
 
 /**
  * @group Database
- * @group Destructive
  * @todo This test suite is severly broken and need a full review 
  */
 class ApiWatchTest extends ApiTestCase {