jenkins-bot has submitted this change and it was merged.

Change subject: Remove clutter
......................................................................


Remove clutter

Change-Id: Ie3983d57ed5eec78ddfe63d1a30b9b362c0395fb
---
M includes/dic/DependencyInjector.php
M includes/dic/SharedDependencyContainer.php
M includes/dic/SimpleDependencyBuilder.php
M includes/serializer/Deserializers/SemanticDataDeserializer.php
M includes/serializer/SerializerFactory.php
M includes/serializer/Serializers/QueryResultSerializer.php
M includes/serializer/Serializers/SemanticDataSerializer.php
M tests/phpunit/includes/context/BaseContextTest.php
M tests/phpunit/includes/context/EmptyContextTest.php
M tests/phpunit/includes/dic/DependencyInjectorTest.php
M tests/phpunit/includes/dic/SharedDependencyContainerTest.php
M tests/phpunit/includes/dic/SimpleDependencyBuilderTest.php
M tests/phpunit/includes/jobs/UpdateDispatcherJobTest.php
M tests/phpunit/includes/jobs/UpdateJobTest.php
M tests/phpunit/includes/serializer/QueryResultSerializerTest.php
M tests/phpunit/includes/serializer/SemanticDataDeserializerTest.php
M tests/phpunit/includes/serializer/SemanticDataSerializationRoundtripTest.php
M tests/phpunit/includes/serializer/SemanticDataSerializerTest.php
M tests/phpunit/includes/serializer/SerializerFactoryTest.php
19 files changed, 95 insertions(+), 245 deletions(-)

Approvals:
  Mwjames: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/includes/dic/DependencyInjector.php 
b/includes/dic/DependencyInjector.php
index a1479b5..9b112d6 100644
--- a/includes/dic/DependencyInjector.php
+++ b/includes/dic/DependencyInjector.php
@@ -3,17 +3,6 @@
 namespace SMW;
 
 /**
- * Abstract class that implements the DependencyRequestor
- *
- * @file
- *
- * @license GNU GPL v2+
- * @since   1.9
- *
- * @author mwjames
- */
-
-/**
  * Abstract class that implements the DependencyRequestor to enable convenience
  * access to an injected DependencyBuilder
  *
@@ -29,9 +18,12 @@
  *  $fooClass->getDependencyBuilder()->newObject( 'Bar' );
  * @endcode
  *
- * @since  1.9
- *
  * @ingroup DependencyRequestor
+ *
+ * @licence GNU GPL v2+
+ * @since 1.9
+ *
+ * @author mwjames
  */
 abstract class DependencyInjector implements DependencyRequestor {
 
diff --git a/includes/dic/SharedDependencyContainer.php 
b/includes/dic/SharedDependencyContainer.php
index 0a6821b..507b094 100644
--- a/includes/dic/SharedDependencyContainer.php
+++ b/includes/dic/SharedDependencyContainer.php
@@ -3,21 +3,15 @@
 namespace SMW;
 
 /**
- * Extends the BaseDependencyContainer
- *
- * @file
- *
- * @license GNU GPL v2+
- * @since   1.9
- *
- * @author mwjames
- */
-
-/**
  * Extends the BaseDependencyContainer to provide general purpose dependency
  * object definitions
  *
  * @ingroup DependencyContainer
+ *
+ * @licence GNU GPL v2+
+ * @since 1.9
+ *
+ * @author mwjames
  */
 class SharedDependencyContainer extends BaseDependencyContainer {
 
diff --git a/includes/dic/SimpleDependencyBuilder.php 
b/includes/dic/SimpleDependencyBuilder.php
index 0a85c3a..6c695bc 100644
--- a/includes/dic/SimpleDependencyBuilder.php
+++ b/includes/dic/SimpleDependencyBuilder.php
@@ -8,22 +8,16 @@
 /**
  * Implements a basic DependencyBuilder
  *
- * @file
- *
- * @license GNU GPL v2+
- * @since   1.9
- *
- * @author mwjames
- */
-
-/**
- * Implements a basic DependencyBuilder
- *
  * @par Example:
  * For a more exhaustive description and examples on how to "work with
  * a DependencyBuilder/Container", see /dic/README.md
  *
  * @ingroup DependencyBuilder
+ *
+ * @licence GNU GPL v2+
+ * @since 1.9
+ *
+ * @author mwjames
  */
 class SimpleDependencyBuilder implements DependencyBuilder {
 
diff --git a/includes/serializer/Deserializers/SemanticDataDeserializer.php 
b/includes/serializer/Deserializers/SemanticDataDeserializer.php
index 0cec0f4..4e8db4f 100644
--- a/includes/serializer/Deserializers/SemanticDataDeserializer.php
+++ b/includes/serializer/Deserializers/SemanticDataDeserializer.php
@@ -17,18 +17,12 @@
 /**
  * SemanticData deserializer
  *
- * @file
+ * @ingroup Deserializers
  *
- * @license GNU GPL v2+
- * @since   1.9
+ * @licence GNU GPL v2+
+ * @since 1.9
  *
  * @author mwjames
- */
-
-/**
- * SemanticData deserializer
- *
- * @ingroup SMW
  */
 class SemanticDataDeserializer implements Deserializer {
 
diff --git a/includes/serializer/SerializerFactory.php 
b/includes/serializer/SerializerFactory.php
index d7f32d9..a53a491 100644
--- a/includes/serializer/SerializerFactory.php
+++ b/includes/serializer/SerializerFactory.php
@@ -12,21 +12,17 @@
 use OutOfBoundsException;
 
 /**
- * Serializer handler for a serializable object
+ * Factory class for a serializable object
  *
- * @file
- *
- * @license GNU GPL v2+
- * @since   1.9
- *
- * @author mwjames
- */
-
-/**
  * A factory class that assigns registered serializers to an object and
  * identifies an unserializer based on the invoked array.
  *
- * @ingroup SMW
+ * @ingroup Serializers
+ *
+ * @licence GNU GPL v2+
+ * @since 1.9
+ *
+ * @author mwjames
  */
 class SerializerFactory {
 
diff --git a/includes/serializer/Serializers/QueryResultSerializer.php 
b/includes/serializer/Serializers/QueryResultSerializer.php
index 4304931..709af7c 100644
--- a/includes/serializer/Serializers/QueryResultSerializer.php
+++ b/includes/serializer/Serializers/QueryResultSerializer.php
@@ -18,9 +18,11 @@
  * This class is distinct from SMWSerializer and the SMWExpData object
  * it takes, in that here semantic context is lost.
  *
+ * @ingroup Serializers
+ *
+ * @licence GNU GPL v2+
  * @since 1.7
  *
- * @licence GNU GPL v2 or later
  * @author Jeroen De Dauw < [email protected] >
  */
 class QueryResultSerializer implements Serializer {
diff --git a/includes/serializer/Serializers/SemanticDataSerializer.php 
b/includes/serializer/Serializers/SemanticDataSerializer.php
index 23bb21a..5d6733b 100644
--- a/includes/serializer/Serializers/SemanticDataSerializer.php
+++ b/includes/serializer/Serializers/SemanticDataSerializer.php
@@ -8,9 +8,13 @@
 use OutOfBoundsException;
 
 /**
- * @since 1.9
+ * SemanticData serializer
+ *
+ * @ingroup Serializers
  *
  * @licence GNU GPL v2+
+ * @since 1.9
+ *
  * @author mwjames
  */
 class SemanticDataSerializer implements Serializer {
diff --git a/tests/phpunit/includes/context/BaseContextTest.php 
b/tests/phpunit/includes/context/BaseContextTest.php
index 9f1e19e..16b9923 100644
--- a/tests/phpunit/includes/context/BaseContextTest.php
+++ b/tests/phpunit/includes/context/BaseContextTest.php
@@ -7,19 +7,19 @@
 /**
  * @covers \SMW\BaseContext
  *
- * @licence GNU GPL v2+
- * @since 1.9
+ * @ingroup Test
  *
  * @group SMW
  * @group SMWExtension
+ *
+ * @licence GNU GPL v2+
+ * @since 1.9
  *
  * @author mwjames
  */
 class BaseContextTest extends SemanticMediaWikiTestCase {
 
        /**
-        * Returns the name of the class to be tested
-        *
         * @return string|false
         */
        public function getClass() {
@@ -27,9 +27,9 @@
        }
 
        /**
-        * Helper method that returns a BaseContext object
-        *
         * @since 1.9
+        *
+        * @return BaseContext
         */
        private function newInstance( $builder = null ) {
                return new BaseContext( $builder );
diff --git a/tests/phpunit/includes/context/EmptyContextTest.php 
b/tests/phpunit/includes/context/EmptyContextTest.php
index deccc98..e57a2b5 100644
--- a/tests/phpunit/includes/context/EmptyContextTest.php
+++ b/tests/phpunit/includes/context/EmptyContextTest.php
@@ -7,19 +7,19 @@
 /**
  * @covers \SMW\EmptyContext
  *
- * @licence GNU GPL v2+
- * @since 1.9
+ * @ingroup Test
  *
  * @group SMW
  * @group SMWExtension
+ *
+ * @licence GNU GPL v2+
+ * @since 1.9
  *
  * @author mwjames
  */
 class EmptyContextTest extends SemanticMediaWikiTestCase {
 
        /**
-        * Returns the name of the class to be tested
-        *
         * @return string|false
         */
        public function getClass() {
@@ -27,9 +27,9 @@
        }
 
        /**
-        * Helper method that returns a EmptyContext object
-        *
         * @since 1.9
+        *
+        * @return EmptyContext
         */
        private function newInstance() {
                return new EmptyContext();
diff --git a/tests/phpunit/includes/dic/DependencyInjectorTest.php 
b/tests/phpunit/includes/dic/DependencyInjectorTest.php
index fa275de..2e91401 100644
--- a/tests/phpunit/includes/dic/DependencyInjectorTest.php
+++ b/tests/phpunit/includes/dic/DependencyInjectorTest.php
@@ -5,29 +5,21 @@
 use SMW\DependencyInjector;
 
 /**
- * Tests for the DependencyInjector
- *
- * @file
- *
- * @license GNU GPL v2+
- * @since   1.9
- *
- * @author mwjames
- */
-
-/**
  * @covers \SMW\DependencyInjector
  *
  * @ingroup Test
  *
  * @group SMW
  * @group SMWExtension
+ *
+ * @licence GNU GPL v2+
+ * @since 1.9
+ *
+ * @author mwjames
  */
 class DependencyInjectorTest extends SemanticMediaWikiTestCase {
 
        /**
-        * Returns the name of the class to be tested
-        *
         * @return string|false
         */
        public function getClass() {
@@ -35,8 +27,6 @@
        }
 
        /**
-        * Helper method that returns a DependencyBuilder object
-        *
         * @since 1.9
         *
         * @return DependencyBuilder
@@ -52,35 +42,27 @@
        }
 
        /**
-        * Helper method that returns a DependencyInjector object
-        *
         * @since 1.9
-        *
-        * @param $data
         *
         * @return DependencyInjector
         */
-       private function getInstance() {
+       private function newInstance() {
                return $this->getMockForAbstractClass( $this->getClass() );
        }
 
        /**
-        * @test DependencyInjector::__construct
-        *
         * @since 1.9
         */
        public function testConstructor() {
-               $this->assertInstanceOf( $this->getClass(), 
$this->getInstance() );
+               $this->assertInstanceOf( $this->getClass(), 
$this->newInstance() );
        }
 
        /**
-        * @test DependencyInjector::newObject
-        *
         * @since 1.9
         */
        public function testSetGet() {
 
-               $instance = $this->getInstance();
+               $instance = $this->newInstance();
                $builder  = $this->newMockDependencyBuilder();
 
                $instance->setDependencyBuilder( $builder );
diff --git a/tests/phpunit/includes/dic/SharedDependencyContainerTest.php 
b/tests/phpunit/includes/dic/SharedDependencyContainerTest.php
index 6870854..401c040 100644
--- a/tests/phpunit/includes/dic/SharedDependencyContainerTest.php
+++ b/tests/phpunit/includes/dic/SharedDependencyContainerTest.php
@@ -13,12 +13,14 @@
  * @covers \SMW\SimpleDependencyBuilder
  * @covers \SMW\BaseDependencyContainer
  *
- * @licence GNU GPL v2+
- * @since 1.9
+ * @ingroup Test
  *
  * @group SMW
  * @group SMWExtension
  *
+ * @licence GNU GPL v2+
+ * @since 1.9
+ *
  * @author mwjames
  */
 class SharedDependencyContainerTest extends SemanticMediaWikiTestCase {
diff --git a/tests/phpunit/includes/dic/SimpleDependencyBuilderTest.php 
b/tests/phpunit/includes/dic/SimpleDependencyBuilderTest.php
index bc8e006..ce8aeae 100644
--- a/tests/phpunit/includes/dic/SimpleDependencyBuilderTest.php
+++ b/tests/phpunit/includes/dic/SimpleDependencyBuilderTest.php
@@ -10,17 +10,6 @@
 use Title;
 
 /**
- * Tests for the SimpleDependencyBuilder
- *
- * @file
- *
- * @license GNU GPL v2+
- * @since   1.9
- *
- * @author mwjames
- */
-
-/**
  * @covers \SMW\SimpleDependencyBuilder
  * @covers \SMW\BaseDependencyContainer
  * @covers \SMW\DependencyInjector
@@ -29,12 +18,15 @@
  *
  * @group SMW
  * @group SMWExtension
+ *
+ * @licence GNU GPL v2+
+ * @since 1.9
+ *
+ * @author mwjames
  */
 class SimpleDependencyBuilderTest extends SemanticMediaWikiTestCase {
 
        /**
-        * Returns the name of the class to be tested
-        *
         * @return string|false
         */
        public function getClass() {
@@ -42,8 +34,6 @@
        }
 
        /**
-        * Helper method that returns a scope definition
-        *
         * @since 1.9
         *
         * @param $data
@@ -54,11 +44,7 @@
        }
 
        /**
-        * Helper method that returns a DependencyContainer object
-        *
         * @since 1.9
-        *
-        * @param $data
         *
         * @return DependencyContainer
         */
@@ -88,11 +74,7 @@
        }
 
        /**
-        * Helper method that returns a SimpleDependencyBuilder object
-        *
         * @since 1.9
-        *
-        * @param $data
         *
         * @return SimpleDependencyBuilder
         */
@@ -101,8 +83,6 @@
        }
 
        /**
-        * @test SimpleDependencyBuilder::__construct
-        *
         * @since 1.9
         */
        public function testConstructor() {
@@ -110,9 +90,6 @@
        }
 
        /**
-        * @test SimpleDependencyBuilder::registerContainer
-        * @test SimpleDependencyBuilder::newObject
-        *
         * @since 1.9
         */
        public function testRegisterContainer() {
@@ -146,9 +123,6 @@
        }
 
        /**
-        * @test SimpleDependencyBuilder::registerContainer
-        * @test SimpleDependencyBuilder::newObject
-        *
         * Register another container containing the same identifier but
         * with a different definition
         *
@@ -177,10 +151,6 @@
        }
 
        /**
-        * @test SimpleDependencyBuilder::getContainer
-        * @test SimpleDependencyBuilder::registerObject
-        * @test SimpleDependencyBuilder::newObject
-        *
         * @since 1.9
         */
        public function testRegisterObject() {
@@ -198,9 +168,6 @@
        }
 
        /**
-        * @test SimpleDependencyBuilder::registerObject
-        * @test SimpleDependencyBuilder::newObject
-        *
         * @since 1.9
         */
        public function testRegisterObjectUsingMagicMethodEagerLoading() {
@@ -222,9 +189,6 @@
        }
 
        /**
-        * @test SimpleDependencyBuilder::registerObject
-        * @test SimpleDependencyBuilder::newObject
-        *
         * @since 1.9
         */
        public function testRegisterObjectUsingMagicMethodLazyLoading() {
@@ -263,9 +227,6 @@
        }
 
        /**
-        * @test SimpleDependencyBuilder::registerObject
-        * @test SimpleDependencyBuilder::newObject
-        *
         * @since 1.9
         */
        public function testRegisterObjectUsingMagicMethodViaBuilder() {
@@ -304,10 +265,6 @@
        }
 
        /**
-        * @test SimpleDependencyBuilder::getContainer
-        * @test SimpleDependencyBuilder::addArgument
-        * @test SimpleDependencyBuilder::getArgument
-        *
         * @since 1.9
         */
        public function testAddGetArguments() {
@@ -330,10 +287,6 @@
        }
 
        /**
-        * @test SimpleDependencyBuilder::getContainer
-        * @test SimpleDependencyBuilder::addArgument
-        * @test SimpleDependencyBuilder::getArgument
-        *
         * @since 1.9
         */
        public function testAddGetArgumentsOnMockObject() {
@@ -367,7 +320,6 @@
        }
 
        /**
-        * @test SimpleDependencyBuilder::newObject
         * @dataProvider autoArgumentsDataProvider
         *
         * @since 1.9
@@ -395,13 +347,9 @@
        }
 
        /**
-        * @test SimpleDependencyBuilder::newObject
         * @dataProvider scopeDataProvider
         *
         * @since 1.9
-        *
-        * @param $setup
-        * @param $expected
         */
        public function testCompareScope( $setup, $expected ) {
 
@@ -442,13 +390,7 @@
        }
 
        /**
-        * @test SimpleDependencyBuilder::newObject
-        * @test SimpleDependencyBuilder::setScope
-        *
         * @since 1.9
-        *
-        * @param $setup
-        * @param $expected
         */
        public function testResetScopeLazyLoading() {
 
@@ -495,13 +437,7 @@
        }
 
        /**
-        * @test SimpleDependencyBuilder::newObject
-        * @test SimpleDependencyBuilder::setScope
-        *
         * @since 1.9
-        *
-        * @param $setup
-        * @param $expected
         */
        public function testSetScope() {
 
@@ -537,8 +473,6 @@
        }
 
        /**
-        * @test SimpleDependencyBuilder::newObject
-        *
         * @since 1.9
         */
        public function testSetCall() {
@@ -559,8 +493,6 @@
        }
 
        /**
-        * @test SimpleDependencyBuilder::newObject
-        *
         * @since 1.9
         */
        public function testSetCallMultipleArguments() {
@@ -591,13 +523,9 @@
        }
 
        /**
-        * @test SimpleDependencyBuilder::newObject
         * @dataProvider scopeDataProvider
         *
         * @since 1.9
-        *
-        * @param $setup
-        * @param $expected
         */
        public function testSetCallMagicWordScope( $setup, $expected ) {
 
@@ -629,8 +557,6 @@
        }
 
        /**
-        * @test SimpleDependencyBuilder::registerObject
-        *
         * @since 1.9
         */
        public function testRegisterObjectAndRemove() {
@@ -659,13 +585,9 @@
        }
 
        /**
-        * @test SimpleDependencyBuilder::newObject
         * @dataProvider dependencyObjectDataProvider
         *
         * @since 1.9
-        *
-        * @param $setup
-        * @param $expected
         */
        public function testDeferredLoading( $setup, $expected ) {
 
@@ -683,8 +605,6 @@
        }
 
        /**
-        * @test SimpleDependencyBuilder::newObject
-        *
         * A object definition depends on previous definitions in order to 
build a
         * requested object instance
         *
@@ -731,8 +651,6 @@
 
 
        /**
-        * @test SimpleDependencyBuilder::getArgument
-        *
         * @since 1.9
         */
        public function testGetArgumentOutOfBoundsException() {
@@ -743,8 +661,6 @@
        }
 
        /**
-        * @test SimpleDependencyBuilder::hasArgument
-        *
         * @since 1.9
         */
        public function testHasArgumentInvalidArgument() {
@@ -755,8 +671,6 @@
        }
 
        /**
-        * @test SimpleDependencyBuilder::addArgument
-        *
         * @since 1.9
         */
        public function testAddArgumentInvalidArgument() {
@@ -767,8 +681,6 @@
        }
 
        /**
-        * @test SimpleDependencyBuilder::newObject
-        *
         * @since 1.9
         */
        public function testNewObjectInvalidArgument() {
@@ -779,8 +691,6 @@
        }
 
        /**
-        * @test SimpleDependencyBuilder::newObject
-        *
         * @since 1.9
         */
        public function testNewObjectArgumentsInvalidArgument() {
@@ -791,8 +701,6 @@
        }
 
        /**
-        * @test SimpleDependencyBuilder::newObject
-        *
         * @since 1.9
         */
        public function testNewObjectUnknownObject() {
@@ -803,8 +711,6 @@
        }
 
        /**
-        * @test SimpleDependencyBuilder::newObject
-        *
         * @since 1.9
         */
        public function testNewObjectDeferredLoadingUnknownObject() {
@@ -818,8 +724,6 @@
        }
 
        /**
-        * @test SimpleDependencyBuilder::newObject
-        *
         * @since 1.9
         */
        public function 
testSingletonCircularReferenceDetectionOutOfBoundsException() {
@@ -836,8 +740,6 @@
        }
 
        /**
-        * @test SimpleDependencyBuilder::newObject
-        *
         * @since 1.9
         */
        public function 
testPrototypeCircularReferenceDetectionOutOfBoundsException() {
@@ -854,8 +756,6 @@
        }
 
        /**
-        * @test SimpleDependencyBuilder::newObject
-        *
         * @since 1.9
         */
        public function 
testDeferredPrototypeCircularReferenceDetectionOutOfBoundsException() {
diff --git a/tests/phpunit/includes/jobs/UpdateDispatcherJobTest.php 
b/tests/phpunit/includes/jobs/UpdateDispatcherJobTest.php
index c0a5020..0a3f889 100644
--- a/tests/phpunit/includes/jobs/UpdateDispatcherJobTest.php
+++ b/tests/phpunit/includes/jobs/UpdateDispatcherJobTest.php
@@ -11,12 +11,14 @@
 /**
  * @covers \SMW\UpdateDispatcherJob
  *
- * @licence GNU GPL v2+
- * @since 1.9
+ * @ingroup Test
  *
  * @group SMW
  * @group SMWExtension
  *
+ * @licence GNU GPL v2+
+ * @since 1.9
+ *
  * @author mwjames
  */
 class UpdateDispatcherJobTest extends SemanticMediaWikiTestCase {
diff --git a/tests/phpunit/includes/jobs/UpdateJobTest.php 
b/tests/phpunit/includes/jobs/UpdateJobTest.php
index 6ac9fe5..6122542 100644
--- a/tests/phpunit/includes/jobs/UpdateJobTest.php
+++ b/tests/phpunit/includes/jobs/UpdateJobTest.php
@@ -11,12 +11,14 @@
  * @covers \SMW\UpdateJob
  * @covers \SMW\JobBase
  *
- * @licence GNU GPL v2+
- * @since 1.9
+ * @ingroup Test
  *
  * @group SMW
  * @group SMWExtension
  *
+ * @licence GNU GPL v2+
+ * @since 1.9
+ *
  * @author mwjames
  */
 class UpdateJobTest extends ParserTestCase {
diff --git a/tests/phpunit/includes/serializer/QueryResultSerializerTest.php 
b/tests/phpunit/includes/serializer/QueryResultSerializerTest.php
index bbe2f69..81ec3f0 100644
--- a/tests/phpunit/includes/serializer/QueryResultSerializerTest.php
+++ b/tests/phpunit/includes/serializer/QueryResultSerializerTest.php
@@ -10,10 +10,14 @@
 /**
  * @covers \SMW\Serializers\QueryResultSerializer
  *
+ * @ingroup Test
+ *
  * @group SMW
  * @group SMWExtension
  *
- * @license GNU GPL v2+
+ * @licence GNU GPL v2+
+ * @since 1.9
+ *
  * @author mwjames
  */
 class QueryResultSerializerTest extends SemanticMediaWikiTestCase {
diff --git a/tests/phpunit/includes/serializer/SemanticDataDeserializerTest.php 
b/tests/phpunit/includes/serializer/SemanticDataDeserializerTest.php
index d79b39c..ba303a2 100644
--- a/tests/phpunit/includes/serializer/SemanticDataDeserializerTest.php
+++ b/tests/phpunit/includes/serializer/SemanticDataDeserializerTest.php
@@ -5,21 +5,17 @@
 use SMW\Deserializers\SemanticDataDeserializer;
 
 /**
- * @file
- *
- * @license GNU GPL v2+
- * @since   1.9
- *
- * @author mwjames
- */
-
-/**
  * @covers \SMW\Deserializers\SemanticDataDeserializer
  *
  * @ingroup Test
  *
  * @group SMW
  * @group SMWExtension
+ *
+ * @licence GNU GPL v2+
+ * @since 1.9
+ *
+ * @author mwjames
  */
 class SemanticDataDeserializerTest extends SemanticMediaWikiTestCase {
 
diff --git 
a/tests/phpunit/includes/serializer/SemanticDataSerializationRoundtripTest.php 
b/tests/phpunit/includes/serializer/SemanticDataSerializationRoundtripTest.php
index 5b4e08b..3826744 100644
--- 
a/tests/phpunit/includes/serializer/SemanticDataSerializationRoundtripTest.php
+++ 
b/tests/phpunit/includes/serializer/SemanticDataSerializationRoundtripTest.php
@@ -11,15 +11,6 @@
 use SMW\Subobject;
 
 /**
- * @file
- *
- * @license GNU GPL v2+
- * @since   1.9
- *
- * @author mwjames
- */
-
-/**
  * @covers \SMW\Deserializers\SemanticDataDeserializer
  * @covers \SMW\Serializers\SemanticDataSerializer
  *
@@ -27,6 +18,11 @@
  *
  * @group SMW
  * @group SMWExtension
+ *
+ * @licence GNU GPL v2+
+ * @since 1.9
+ *
+ * @author mwjames
  */
 class SemanticDataSerializationRoundtripTest extends SemanticMediaWikiTestCase 
{
 
diff --git a/tests/phpunit/includes/serializer/SemanticDataSerializerTest.php 
b/tests/phpunit/includes/serializer/SemanticDataSerializerTest.php
index e54c796..27da891 100644
--- a/tests/phpunit/includes/serializer/SemanticDataSerializerTest.php
+++ b/tests/phpunit/includes/serializer/SemanticDataSerializerTest.php
@@ -10,21 +10,17 @@
 use SMW\Subobject;
 
 /**
- * @file
- *
- * @license GNU GPL v2+
- * @since   1.9
- *
- * @author mwjames
- */
-
-/**
  * @covers \SMW\Serializers\SemanticDataSerializer
  *
  * @ingroup Test
  *
  * @group SMW
  * @group SMWExtension
+ *
+ * @licence GNU GPL v2+
+ * @since 1.9
+ *
+ * @author mwjames
  */
 class SemanticDataSerializerTest extends SemanticMediaWikiTestCase {
 
diff --git a/tests/phpunit/includes/serializer/SerializerFactoryTest.php 
b/tests/phpunit/includes/serializer/SerializerFactoryTest.php
index dee6026..77938e6 100644
--- a/tests/phpunit/includes/serializer/SerializerFactoryTest.php
+++ b/tests/phpunit/includes/serializer/SerializerFactoryTest.php
@@ -7,23 +7,17 @@
 use SMW\DIWikiPage;
 
 /**
- * Tests for the SerializerFactory class
- *
- * @file
- *
- * @license GNU GPL v2+
- * @since   1.9
- *
- * @author mwjames
- */
-
-/**
  * @covers \SMW\SerializerFactory
  *
  * @ingroup Test
  *
  * @group SMW
  * @group SMWExtension
+ *
+ * @licence GNU GPL v2+
+ * @since 1.9
+ *
+ * @author mwjames
  */
 class SerializerFactoryTest extends SemanticMediaWikiTestCase {
 

-- 
To view, visit https://gerrit.wikimedia.org/r/90737
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie3983d57ed5eec78ddfe63d1a30b9b362c0395fb
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/SemanticMediaWiki
Gerrit-Branch: master
Gerrit-Owner: Mwjames <[email protected]>
Gerrit-Reviewer: Mwjames <[email protected]>
Gerrit-Reviewer: jenkins-bot

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to