Novactive / NovaCollection

tests/Tests/FirstCollectionTest.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
 {
     public function testFirstReturnsFirstItemInCollection()
     {
-        $coll = Factory::create($this->fixtures['names']);
+        $coll = Factory::create($this->fixtures[ 'names' ]);
         $this->assertEquals('Chelsea', $coll->first());
     }
 }

tests/Tests/SetCollectionTest.php

Spacing +3 added lines, -3 removed lines

@@ -19,14 +19,14 @@
 {
     public function testSetWillSetValueOnCollectionInPlace()
     {
-        $exp  = $this->fixtures['assoc'];
+        $exp  = $this->fixtures[ 'assoc' ];
         $coll = Factory::create($exp);
         $this->assertEquals($exp, $coll->toArray());
         $this->assertSame($coll, $coll->set('1st', 'worst'), 'Collection::set() should return the collection itself.');
-        $exp['1st'] = 'worst';
+        $exp[ '1st' ] = 'worst';
         $this->assertEquals($exp, $coll->toArray());
         $coll->set('first', 'worst');
-        $exp['first'] = 'worst';
+        $exp[ 'first' ] = 'worst';
         $this->assertEquals($exp, $coll->toArray());
     }
 }

tests/Tests/AddCollectionTest.php

Spacing +2 added lines, -2 removed lines

@@ -19,7 +19,7 @@ 
 {
     public function testAddAppendsItemToCollectionWithNextNumericIndex()
     {
-        $array = Factory::create($this->fixtures['array']);
+        $array = Factory::create($this->fixtures[ 'array' ]);
         $this->assertEquals(
             [
                 0 => 'first',
@@ -50,7 +50,7 @@ 
             $array->toArray()
         );
 
-        $assoc = Factory::create($this->fixtures['assoc']);
+        $assoc = Factory::create($this->fixtures[ 'assoc' ]);
         $this->assertEquals(
             [
                 '1st' => 'first',

tests/Tests/LastCollectionTest.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
 {
     public function testLastReturnsLastItemInCollection()
     {
-        $coll = Factory::create($this->fixtures['names']);
+        $coll = Factory::create($this->fixtures[ 'names' ]);
         $this->assertEquals('Nakia', $coll->last());
     }
 }

tests/Tests/IteratorCollectionTest.php

Spacing +6 added lines, -6 removed lines

@@ -19,13 +19,13 @@ 
 {
     public function testCurrentReturnsCurrentValue()
     {
-        $coll = Factory::create($this->fixtures['names']);
+        $coll = Factory::create($this->fixtures[ 'names' ]);
         $this->assertEquals('Chelsea', $coll->current());
     }
 
     public function testNextMovesCollectionInternalPointer()
     {
-        $coll = Factory::create($this->fixtures['names']);
+        $coll = Factory::create($this->fixtures[ 'names' ]);
         $this->assertEquals(
             'Chelsea',
             $coll->current(),
@@ -70,7 +70,7 @@ 
 
     public function testKeyReturnsCurrentKeyInCollection()
     {
-        $coll = Factory::create($this->fixtures['names']);
+        $coll = Factory::create($this->fixtures[ 'names' ]);
         $this->assertSame(0, $coll->key(), 'Initial call to key() should return first item in collection.');
         $this->assertSame(0, $coll->key(), 'Subsequent calls to key() should continue to return the same value.');
         $coll->next();
@@ -109,7 +109,7 @@ 
 
     public function testValidReturnsFalseWhenCollectionHasBeenIteratedBeyondItsLastItem()
     {
-        $coll = Factory::create($this->fixtures['names']);
+        $coll = Factory::create($this->fixtures[ 'names' ]);
         $this->assertTrue($coll->valid(), 'Initial call to valid() should always return true.');
         $coll->next();
         $this->assertTrue($coll->valid(), 'Subsequent calls to valid() should continue to return true.');
@@ -132,7 +132,7 @@ 
 
     public function testRewindWillReturnInternalPointerToItsInitialPosition()
     {
-        $coll = Factory::create($this->fixtures['names']);
+        $coll = Factory::create($this->fixtures[ 'names' ]);
         $this->assertEquals('Chelsea', $coll->first());
         $this->assertTrue($coll->valid(), 'Initial call to valid() should always return true.');
         $coll->next();
@@ -172,7 +172,7 @@ 
 
     public function testInstantiationShouldRewindArrayArgsInternalPointer()
     {
-        $arr = $this->fixtures['names'];
+        $arr = $this->fixtures[ 'names' ];
         $this->assertEquals('Chelsea', current($arr));
         next($arr);
         $this->assertEquals('Adella', current($arr));

tests/Tests/RemoveCollectionTest.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
 {
     public function testRemoveRemovesItemByKey()
     {
-        $exp  = $this->fixtures['assoc'];
+        $exp  = $this->fixtures[ 'assoc' ];
         $coll = Factory::create($exp);
         $this->assertTrue($coll->containsKey('2nd'));
         $removed = $coll->remove('2nd');

tests/Tests/TransformCollectionTest.php

Spacing +2 added lines, -2 removed lines

@@ -20,9 +20,9 @@
 {
     public function testTransformTransformsCollectionInPlace()
     {
-        $coll        = Factory::create($this->fixtures['names']);
+        $coll        = Factory::create($this->fixtures[ 'names' ]);
         $transformed = $coll->transform(
-            function ($val, $key) {
+            function($val, $key) {
                 return $val.$key;
             }
         );

tests/Tests/PruneCollectionTest.php

Spacing +3 added lines, -3 removed lines

@@ -19,12 +19,12 @@ 
 {
     public function testPruneFiltersCollectionInPlace()
     {
-        $coll = Factory::create($this->fixtures['names']);
+        $coll = Factory::create($this->fixtures[ 'names' ]);
         $this->assertCount(10, $coll);
         $this->assertSame(
             $coll,
             $coll->prune(
-                function ($value, $key) {
+                function($value, $key) {
                     return $key % 2 == 0;
                 }
             )
@@ -33,7 +33,7 @@ 
         $this->assertSame(
             $coll,
             $coll->prune(
-                function ($value, $key) {
+                function($value, $key) {
                     return strlen($value) >= 6;
                 }
             )

tests/Tests/ArrayAccessCollectionTest.php

Spacing +20 added lines, -20 removed lines

@@ -19,26 +19,26 @@ 
 {
     public function testArrayAccessUnsetRemovesItemByKeyAndReturnsNull()
     {
-        $exp  = $this->fixtures['assoc'];
+        $exp  = $this->fixtures[ 'assoc' ];
         $coll = Factory::create($exp);
         $this->assertTrue($coll->containsKey('2nd'));
         $removed = $coll->offsetUnset('2nd');
         $this->assertFalse($coll->containsKey('2nd'));
         $this->assertNull($removed, 'The ArrayAccess interface expects offsetUnset to have no return value.');
         $this->assertTrue($coll->containsKey('3rd'));
-        unset($coll['3rd']);
+        unset($coll[ '3rd' ]);
         $this->assertFalse($coll->containsKey('3rd'));
     }
 
     public function testArrayAccessOffsetExistsAllowsIssetToWorkWithSquareBrackets()
     {
         // associative
-        $exp  = $this->fixtures['assoc'];
+        $exp  = $this->fixtures[ 'assoc' ];
         $coll = Factory::create($exp);
         $this->assertTrue($coll->containsKey('2nd'));
         $this->assertTrue($coll->offsetExists('2nd'), 'Collection::offsetExists should return true if index exists.');
         $this->assertTrue(
-            isset($coll['2nd']),
+            isset($coll[ '2nd' ]),
             'Collection::offsetExists should allow for the use of isset() on a collection using square brackets.'
         );
         $this->assertFalse($coll->containsKey('4th'));
@@ -47,13 +47,13 @@ 
             'Collection::offsetExists should return false if index does not exist.'
         );
         $this->assertFalse(
-            isset($coll['4th']),
+            isset($coll[ '4th' ]),
             'Collection::offsetExists should allow for the use of isset() on a '.
             'collection using square brackets for an index that does not exist.'
         );
 
         // numeric
-        $exp  = $this->fixtures['array'];
+        $exp  = $this->fixtures[ 'array' ];
         $coll = Factory::create($exp);
         $this->assertTrue($coll->containsKey(1));
         $this->assertTrue(
@@ -61,7 +61,7 @@ 
             'Collection::offsetExists should return true if numeric offset exists.'
         );
         $this->assertTrue(
-            isset($coll[1]),
+            isset($coll[ 1 ]),
             'Collection::offsetExists should allow for the use of isset() on a collection using '.
             'square brackets and numeric offset.'
         );
@@ -71,7 +71,7 @@ 
             'Collection::offsetExists should return false if numeric offset does not exist.'
         );
         $this->assertFalse(
-            isset($coll[5]),
+            isset($coll[ 5 ]),
             'Collection::offsetExists should allow for the use of isset() on a collection using square brackets '.
             'for a numeric offset that does not exist.'
         );
@@ -80,10 +80,10 @@ 
     public function testArrayAccessOffsetSetAllowsUseOfSquareBracketsForSetting()
     {
         // associative
-        $exp  = $this->fixtures['assoc'];
+        $exp  = $this->fixtures[ 'assoc' ];
         $coll = Factory::create($exp);
         $this->assertFalse($coll->containsKey('foo'));
-        $coll['foo'] = 'bar';
+        $coll[ 'foo' ] = 'bar';
         $this->assertTrue($coll->containsKey('foo'));
         $this->assertEquals('bar', $coll->get('foo'));
 
@@ -93,10 +93,10 @@ 
         $this->assertEquals('far', $coll->get('boo'));
 
         // numeric
-        $exp  = $this->fixtures['array'];
+        $exp  = $this->fixtures[ 'array' ];
         $coll = Factory::create($exp);
         $this->assertFalse($coll->containsKey(5));
-        $coll[5] = 'bar';
+        $coll[ 5 ] = 'bar';
         $this->assertTrue($coll->containsKey(5));
         $this->assertEquals('bar', $coll->get(5));
 
@@ -109,29 +109,29 @@ 
     public function testArrayAccessOffsetSetAllowsUseOfSquareBracketsForSettingWithoutIndex()
     {
         // associative
-        $assoc = $this->fixtures['assoc'];
+        $assoc = $this->fixtures[ 'assoc' ];
         $aColl = Factory::create($assoc);
         $this->assertFalse($aColl->containsKey(0));
-        $aColl[] = 'test';
+        $aColl[ ] = 'test';
         $this->assertTrue($aColl->containsKey(0));
 
         // numeric
-        $arr     = $this->fixtures['array'];
+        $arr     = $this->fixtures[ 'array' ];
         $arrColl = Factory::create($arr);
         $this->assertTrue($arrColl->containsKey(0));
         $this->assertTrue($arrColl->containsKey(1));
         $this->assertTrue($arrColl->containsKey(2));
         $this->assertFalse($arrColl->containsKey(3));
-        $arrColl[] = 'test';
+        $arrColl[ ] = 'test';
         $this->assertTrue($arrColl->containsKey(3));
     }
 
     public function testArrayAccessOffsetGetAllowsUseOfSquareBracketsForGetting()
     {
-        $exp  = $this->fixtures['assoc'];
+        $exp  = $this->fixtures[ 'assoc' ];
         $coll = Factory::create($exp);
         $this->assertEquals('first', $coll->offsetGet('1st'));
-        $this->assertEquals('first', $coll['1st']);
+        $this->assertEquals('first', $coll[ '1st' ]);
     }
 
     /**
@@ -140,8 +140,8 @@ 
      */
     public function testArrayAccessOffsetGetThrowsExceptionIfIndexDoesNotExist()
     {
-        $exp  = $this->fixtures['assoc'];
+        $exp  = $this->fixtures[ 'assoc' ];
         $coll = Factory::create($exp);
-        $foo  = $coll['foo'];
+        $foo  = $coll[ 'foo' ];
     }
 }