BUG Implement peer review feedback,

silverstripe · Feb 26, 2019 · 6ff319a · 6ff319a
1 parent 9a59f2f
commit 6ff319a
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 10 deletions.
diff --git a/tests/php/ORM/DataObjectTest/MockDynamicAssignmentDBField.php b/tests/php/ORM/DataObjectTest/MockDynamicAssignmentDBField.php
@@ -2,6 +2,7 @@
 
 namespace SilverStripe\ORM\Tests\DataObjectTest;
 
+use SilverStripe\Dev\TestOnly;
 use SilverStripe\ORM\FieldType\DBBoolean;
 use SilverStripe\ORM\FieldType\DBField;
 
@@ -13,16 +14,16 @@
  * If the field is set to false, it will try to do a plain assignment. This is so you can save the initial value no
  * matter what. If the field is set to true, it will try to do a dynamic assignment.
  */
-class MockDynamicAssignmentDBField extends DBBoolean
+class MockDynamicAssignmentDBField extends DBBoolean implements TestOnly
 {
 
     private $scalarOnly;
     private $dynamicAssignment;
 
     /**
-     * @param $name
-     * @param $scalarOnly Whether our fake field should be scalar only
-     * @param $dynamicAssigment Wheter our fake field will try to do a dynamic assignement
+     * @param string $name
+     * @param boolean $scalarOnly Whether our fake field should be scalar only.
+     * @param boolean $dynamicAssignment Whether our fake field will try to do a dynamic assignment.
      */
     public function __construct($name = '', $scalarOnly = false, $dynamicAssignment = false)
     {
@@ -32,15 +33,15 @@ public function __construct($name = '', $scalarOnly = false, $dynamicAssignment
     }
 
     /**
-     * If the field value and dynamicAssignment are true, we'll try to do a dynamic assignement
+     * If the field value and $dynamicAssignment are true, we'll try to do a dynamic assignment.
      * @param $value
-     * @return array|int|mixed
+     * @return array|int
      */
     public function prepValueForDB($value)
     {
         if ($value) {
             return $this->dynamicAssignment
-                ? ['GREATEST(?, ?)' => [0, 1]]
+                ? ['ABS(?)' => [1]]
                 : 1;
         }
 

diff --git a/tests/php/ORM/DataObjectTest/MockDynamicAssignmentDataObject.php b/tests/php/ORM/DataObjectTest/MockDynamicAssignmentDataObject.php
@@ -30,11 +30,11 @@ class MockDynamicAssignmentDataObject extends DataObject implements TestOnly
     ];
 
     private static $many_many = [
-        "MockManyMany" => self::class,
+        'MockManyMany' => self::class,
     ];
 
     private static $belongs_many_many = [
-        "MockBelongsManyMany" => self::class,
+        'MockBelongsManyMany' => self::class,
     ];
 
     private static $many_many_extraFields = [

diff --git a/tests/php/ORM/ManyManyListTest.php b/tests/php/ORM/ManyManyListTest.php
@@ -10,6 +10,7 @@
 use SilverStripe\ORM\Tests\DataObjectTest\Team;
 use SilverStripe\ORM\Tests\ManyManyListTest\ExtraFieldsObject;
 use SilverStripe\ORM\Tests\ManyManyListTest\Product;
+use InvalidArgumentException;
 
 class ManyManyListTest extends SapphireTest
 {
@@ -402,7 +403,7 @@ public function testWriteManipulationWithNonScalarValuesAllowed()
 
     public function testWriteManipulationWithNonScalarValuesDisallowed()
     {
-        $this->expectException(\InvalidArgumentException::class);
+        $this->expectException(InvalidArgumentException::class);
 
         $left = DataObjectTest\MockDynamicAssignmentDataObject::create();
         $left->write();