Lint and test fixing for react-components

Signed-off-by: Aaron Chong <aaronchongth@gmail.com>

packages/dashboard/package.json

@@ -65,7 +65,7 @@
     "debug": "^4.3.4",
     "eventemitter3": "^5.0.1",
     "keycloak-js": "^23.0.1",
-    "node-vibrant": "3.2.1-alpha.1",
+    "node-vibrant": "3.1.6",
     "react": "^18.2.0",
     "react-components": "workspace:*",
     "react-customizable-progressbar": "^1.2.0",

packages/dashboard/src/components/trajectories-overlay.tsx

@@ -1,12 +1,3 @@
-import { Trajectory } from 'react-components';
+import { TrajectoryMarkerProps } from 'react-components';
 
-export interface TrajectoryData {
-  trajectory: Trajectory;
-  color: string;
-  conflict?: boolean;
-  loopAnimation?: boolean;
-  /**
-   * default: 1
-   */
-  animationScale?: number;
-}
+export type TrajectoryData = TrajectoryMarkerProps;

packages/react-components/lib/form-inputs.spec.tsx

@@ -7,8 +7,10 @@ describe('PositiveIntField', () => {
   it('does not allow negative numbers to be typed', async () => {
     const root = render(<PositiveIntField id="test" label="test" />);
     const inputEl = root.getByLabelText('test') as HTMLInputElement;
-    userEvent.clear(inputEl);
-    userEvent.type(inputEl, '-1');
+    const user = userEvent.setup();
+    user.clear(inputEl);
+    expect(inputEl.value).toBe('');
+    await user.type(inputEl, '-1');
     // '-' char should get "eaten", so should end up with "1".
     expect(inputEl.value).toBe('1');
   });

packages/react-components/lib/index.ts

@@ -22,7 +22,6 @@ export * from './map';
 export * from './navigation-bar';
 export * from './place';
 export * from './robots';
-export * from './simple-filter';
 export * from './simple-info';
 export * from './spotlight-accordion';
 export * from './stack-navigator';

packages/react-components/lib/lifts/lift-request-dialog.spec.tsx

@@ -10,7 +10,7 @@ describe('Lift request form', () => {
     const mockOnClose = jasmine.createSpy();
     const lift = makeLift();
     const liftState = makeLiftState();
-    render(
+    const root = render(
       <LiftRequestDialog
         currentLevel={liftState.current_floor}
         availableLevels={lift.levels}
@@ -20,8 +20,11 @@ describe('Lift request form', () => {
         onClose={mockOnClose}
       />,
     );
-    userEvent.type(screen.getByPlaceholderText('Pick a Destination'), '{selectall}{backspace}');
+    const inputEl = root.getByLabelText('Pick a Destination') as HTMLInputElement;
+    const user = userEvent.setup();
+    user.clear(inputEl);
+    expect(inputEl.value).toBe('');
     fireEvent.click(screen.getByText('Request'));
-    expect(screen.getByText('Destination cannot be empty')).toBeTruthy();
+    expect(root.getByText('Destination cannot be empty')).toBeTruthy();
   });
 });

packages/react-components/lib/map/index.tsx

@@ -2,3 +2,4 @@ export * from './lift-marker';
 export * from './utils';
 export * from './three-fiber';
 export * from './trajectory';
+export * from './trajectory-marker';