Fix rowGroup plugin with tests

Author: yajra

src/Html/Options/Plugins/RowGroup.php

@@ -12,29 +12,33 @@
 trait RowGroup
 {
     /**
-     * Set rowGroup option value.
+     * Set rowGroup className option value.
      *
-     * @param  bool|array  $value
+     * @param  string  $value
      * @return $this
-     * @see https://datatables.net/reference/option/rowGroup
+     * @see https://datatables.net/reference/option/rowGroup.className
      */
-    public function rowGroup(bool|array $value = true): static
+    public function rowGroupUpdate(string $value = 'group'): static
     {
-        $this->attributes['rowGroup'] = $value;
-
-        return $this;
+        return $this->rowGroup(['className' => $value]);
     }
 
     /**
-     * Set rowGroup className option value.
+     * Set rowGroup option value.
      *
-     * @param  string  $value
+     * @param  bool|array  $value
      * @return $this
-     * @see https://datatables.net/reference/option/rowGroup.className
+     * @see https://datatables.net/reference/option/rowGroup
      */
-    public function rowGroupUpdate(string $value = 'group'): static
+    public function rowGroup(bool|array $value = true): static
     {
-        return $this->rowGroup(['className' => $value]);
+        if (is_array($value)) {
+            $this->attributes['rowGroup'] = array_merge((array) $this->attributes['rowGroup'], $value);
+        } else {
+            $this->attributes['rowGroup'] = $value;
+        }
+
+        return $this;
     }
 
     /**
@@ -120,4 +124,17 @@ public function rowGroupStartRender(string $value = null): static
     {
         return $this->rowGroup(['startRender' => $value]);
     }
+
+    /**
+     * @param  string|null  $key
+     * @return mixed
+     */
+    public function getRowGroup(string $key = null): mixed
+    {
+        if (is_null($key)) {
+            return $this->attributes['rowGroup'] ?? true;
+        }
+
+        return $this->attributes['rowGroup'][$key] ?? false;
+    }
 }

tests/BuilderOptionsPluginsTest.php

@@ -180,4 +180,32 @@ public function it_has_responsive_plugin()
         $this->assertEquals('type', $builder->getResponsive('details')['type']);
         $this->assertEquals('orthogonal', $builder->getResponsive('orthogonal'));
     }
+
+    /** @test */
+    public function it_has_row_group_plugin()
+    {
+        $builder = $this->getHtmlBuilder();
+        $builder->rowGroup();
+
+        $this->assertTrue($builder->getAttribute('rowGroup'));
+        $this->assertTrue($builder->getRowGroup());
+
+        $builder->rowGroupDataSrc([1])
+                ->rowGroupEmptyDataGroup()
+                ->rowGroupEnable()
+                ->rowGroupEndClassName()
+                ->rowGroupEndRender('fn')
+                ->rowGroupStartClassName()
+                ->rowGroupStartRender();
+
+        $this->assertEquals([1], $builder->getRowGroup('dataSrc'));
+        $this->assertEquals('No Group', $builder->getRowGroup('emptyDataGroup'));
+        $this->assertEquals(true, $builder->getRowGroup('enable'));
+        $this->assertEquals('group-end', $builder->getRowGroup('endClassName'));
+        $this->assertEquals('fn', $builder->getRowGroup('endRender'));
+        $this->assertEquals('group-start', $builder->getRowGroup('startClassName'));
+        $this->assertEquals(null, $builder->getRowGroup('startRender'));
+    }
+
+
 }