Fix scroller plugin with tests

Author: yajra

src/Html/Options/Plugins/Scroller.php

@@ -12,29 +12,33 @@
 trait Scroller
 {
     /**
-     * Set scroller option value.
+     * Set scroller boundaryScale option value.
      *
-     * @param  bool|array  $value
+     * @param  float  $value
      * @return $this
-     * @see https://datatables.net/reference/option/scroller
+     * @see https://datatables.net/reference/option/scroller.boundaryScale
      */
-    public function scroller(bool|array $value = true): static
+    public function scrollerBoundaryScale(float $value = 0.5): static
     {
-        $this->attributes['scroller'] = $value;
-
-        return $this;
+        return $this->scroller(['boundaryScale' => $value]);
     }
 
     /**
-     * Set scroller boundaryScale option value.
+     * Set scroller option value.
      *
-     * @param  float  $value
+     * @param  bool|array  $value
      * @return $this
-     * @see https://datatables.net/reference/option/scroller.boundaryScale
+     * @see https://datatables.net/reference/option/scroller
      */
-    public function scrollerBoundaryScale(float $value = 0.5): static
+    public function scroller(bool|array $value = true): static
     {
-        return $this->scroller(['boundaryScale' => $value]);
+        if (is_array($value)) {
+            $this->attributes['scroller'] = array_merge((array) $this->attributes['scroller'], $value);
+        } else {
+            $this->attributes['scroller'] = $value;
+        }
+
+        return $this;
     }
 
     /**
@@ -84,4 +88,17 @@ public function scrollerServerWait(int $value = 200): static
     {
         return $this->scroller(['serverWait' => $value]);
     }
+
+    /**
+     * @param  string|null  $key
+     * @return mixed
+     */
+    public function getScroller(string $key = null): mixed
+    {
+        if (is_null($key)) {
+            return $this->attributes['scroller'] ?? true;
+        }
+
+        return $this->attributes['scroller'][$key] ?? false;
+    }
 }

tests/BuilderOptionsPluginsTest.php

@@ -231,8 +231,28 @@ public function it_has_row_reorder_plugin()
         $this->assertEquals('td:first-child', $builder->getRowReorder('selector'));
         $this->assertEquals(true, $builder->getRowReorder('snapX'));
         $this->assertEquals(true, $builder->getRowReorder('update'));
+    }
 
-
+    /** @test */
+    public function it_has_scroller_plugin()
+    {
+        $builder = $this->getHtmlBuilder();
+        $builder->scroller();
+
+        $this->assertTrue($builder->getAttribute('scroller'));
+        $this->assertTrue($builder->getScroller());
+
+        $builder->scrollerBoundaryScale()
+                ->scrollerDisplayBuffer()
+                ->scrollerLoadingIndicator()
+                ->scrollerRowHeight()
+                ->scrollerServerWait();
+
+        $this->assertEquals(0.5, $builder->getScroller('boundaryScale'));
+        $this->assertEquals(9, $builder->getScroller('displayBuffer'));
+        $this->assertEquals(true, $builder->getScroller('loadingIndicator'));
+        $this->assertEquals('auto', $builder->getScroller('rowHeight'));
+        $this->assertEquals(200, $builder->getScroller('serverWait'));
     }