Add a new group destructive.

Spacing.

diff --git a/maintenance/tests/phpunit/includes/SampleTest.php b/maintenance/tests/phpunit/includes/SampleTest.php
index a9930eb..117ac1c 100644 (file)
--- a/maintenance/tests/phpunit/includes/SampleTest.php
+++ b/maintenance/tests/phpunit/includes/SampleTest.php
@@ -88,8 +88,8 @@ class TestSample extends PHPUnit_Framework_TestCase {
         * See http://www.phpunit.de/manual/3.4/en/appendixes.annotations.html#appendixes.annotations.expectedException
         */
        function testTitleObjectFromObject() {
-               $title = Title::newFromText(new Title("test"));
+               $title = Title::newFromText( new Title( "test" ) );
                $this->assertEquals( "Test", $title->isLocal() );
        }
-  }
+}
 

diff --git a/maintenance/tests/phpunit/includes/api/ApiSetup.php b/maintenance/tests/phpunit/includes/api/ApiSetup.php
index 7ffa807..45790d4 100644 (file)
--- a/maintenance/tests/phpunit/includes/api/ApiSetup.php
+++ b/maintenance/tests/phpunit/includes/api/ApiSetup.php
@@ -51,9 +51,9 @@ class UserWrapper {
 
        public function __construct( $userName, $password, $group = '' ) {
                $this->userName = $userName;
-           $this->password = $password;
+               $this->password = $password;
 
-           $this->user = User::newFromName( $this->userName );
+               $this->user = User::newFromName( $this->userName );
                if ( !$this->user->getID() ) {
                        $this->user = User::createNew( $this->userName, array(
                                "email" => "test@example.com",

diff --git a/maintenance/tests/phpunit/includes/api/ApiTest.php b/maintenance/tests/phpunit/includes/api/ApiTest.php
index 1f87602..5e83253 100644 (file)
--- a/maintenance/tests/phpunit/includes/api/ApiTest.php
+++ b/maintenance/tests/phpunit/includes/api/ApiTest.php
@@ -17,6 +17,9 @@ class MockApi extends ApiBase {
        }
 }
 
+/**
+ * @group destructive
+ */
 class ApiTest extends ApiTestSetup {
 
        function testRequireOnlyOneParameterDefault() {
@@ -28,8 +31,8 @@ class ApiTest extends ApiTestSetup {
        }
 
        /**
-     * @expectedException UsageException
-     */
+        * @expectedException UsageException
+        */
        function testRequireOnlyOneParameterZero() {
                $mock = new MockApi();
 
@@ -39,8 +42,8 @@ class ApiTest extends ApiTestSetup {
        }
 
        /**
-     * @expectedException UsageException
-     */
+        * @expectedException UsageException
+        */
        function testRequireOnlyOneParameterTrue() {
                $mock = new MockApi();
 

diff --git a/maintenance/tests/phpunit/includes/api/ApiWatchTest.php b/maintenance/tests/phpunit/includes/api/ApiWatchTest.php
index bf7e709..50bade6 100644 (file)
--- a/maintenance/tests/phpunit/includes/api/ApiWatchTest.php
+++ b/maintenance/tests/phpunit/includes/api/ApiWatchTest.php
@@ -2,6 +2,9 @@
 
 require_once dirname( __FILE__ ) . '/ApiSetup.php';
 
+/**
+ * @group destructive
+ */
 class ApiWatchTest extends ApiTestSetup {
 
        function setUp() {

diff --git a/maintenance/tests/phpunit/suites/ExtensionsTestSuite.php b/maintenance/tests/phpunit/suites/ExtensionsTestSuite.php
index bf1e42b..4bf3542 100644 (file)
--- a/maintenance/tests/phpunit/suites/ExtensionsTestSuite.php
+++ b/maintenance/tests/phpunit/suites/ExtensionsTestSuite.php
@@ -4,7 +4,7 @@
  * See http://www.mediawiki.org/wiki/Manual:Hooks/UnitTestsList for details of how to register your tests.
  */
 
- class ExtensionsTestSuite extends PHPUnit_Framework_TestSuite {
+class ExtensionsTestSuite extends PHPUnit_Framework_TestSuite {
        public function __construct() {
                parent::__construct();
                $files = array();