Do not pass auto grid props to grid

Author: Artur-

packages/ts/react-grid/src/autogrid.tsx

@@ -82,14 +82,16 @@ const createColumns = (model: ModelConstructor<any, any>) => {
 };
 
 export function AutoGrid<TItem>(props: AutoGridProps<TItem>) {
+  const { service, model, ...gridProps } = props;
+
   const ref = useRef(null);
 
   useEffect(() => {
     const grid = ref.current as any as GridElement<TItem>;
-    grid.dataProvider = createDataProvider(grid, props.service);
+    grid.dataProvider = createDataProvider(grid, service);
   }, []);
 
-  const children = createColumns(props.model);
+  const children = createColumns(model);
 
-  return <Grid {...props} ref={ref} children={children}></Grid>;
+  return <Grid {...gridProps} ref={ref} children={children}></Grid>;
 }

packages/ts/react-grid/test/autogrid.spec.tsx

@@ -32,15 +32,9 @@ async function sleep(ms: number) {
   );
 }
 describe('@hilla/react-grid', () => {
-  //   type UseAutoGridSpy = sinon.SinonSpy<Parameters<typeof _useAutoGrid>, ReturnType<typeof _useAutoGrid>>;
-  //   const useAutoGrid = sinon.spy(_useAutoGrid) as typeof _useAutoGrid;
-
-  beforeEach(() => {
-    // (useAutoGrid as UseAutoGridSpy).resetHistory();
-  });
+  beforeEach(() => {});
 
   function TestAutoGrid() {
-    // const autoGrid = useAutoGrid(fakeEndpoint, PersonModel);
     return <AutoGrid service={fakeService} model={PersonModel}></AutoGrid>;
   }
   describe('useAutoGrid', () => {
@@ -69,5 +63,11 @@ describe('@hilla/react-grid', () => {
       expect(getBodyCellContent(grid, 1, 0).innerText).to.equal('Jane');
       expect(getBodyCellContent(grid, 1, 1).innerText).to.equal('Love');
     });
+    it('does not pass its own parameters to the underlying grid', async () => {
+      const result = render(<TestAutoGrid />);
+      const grid: GridElement = result.container.querySelector('vaadin-grid')!;
+      expect(grid.getAttribute("model")).to.be.null;
+      expect(grid.getAttribute("service")).to.be.null;
+    });
   });
 });