Test coverage

mitykas · Apr 22, 2022 · 0c88cff · 0c88cff
1 parent 4c7d931
commit 0c88cff
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 6 deletions.
diff --git a/packages/app/app/containers/PlayQueueContainer/PlayQueueContainer.test.tsx b/packages/app/app/containers/PlayQueueContainer/PlayQueueContainer.test.tsx
@@ -15,7 +15,7 @@ describe('Play Queue container', () => {
     store.clear();
   });
 
-  it('should display with track in queue', async () => {
+  it('should display tracks in the queue', async () => {
     const { component } = mountComponent();
     expect(component.asFragment()).toMatchSnapshot();
   });
@@ -46,7 +46,7 @@ describe('Play Queue container', () => {
     expect(component.asFragment()).toMatchSnapshot();
   });
 
-  it('should copy original track url to clipboard', async () => {
+  it('should copy the original track url to clipboard', async () => {
     const { component } = mountComponent();
     // eslint-disable-next-line @typescript-eslint/no-var-requires
     const clipboard = require('electron').clipboard;
@@ -57,7 +57,7 @@ describe('Play Queue container', () => {
     expect(clipboard.writeText).toHaveBeenCalledWith('https://test-track-original-url');
   });
 
-  it('should not display copy original track url button if the url is not included in the current stream', async () => {
+  it('should not display the copy original track url button if the url is not included in the current stream', async () => {
     const { component } = mountComponent();
     const track = component.getByTestId('queue-popup-uuid2');
     await waitFor(() => fireEvent.contextMenu(track));
@@ -123,6 +123,16 @@ describe('Play Queue container', () => {
     ]);
   });
 
+  it('should clear the queue', async () => {
+    const { component, store } = mountComponent();
+
+    await waitFor(() => component.getByTestId('queue-menu-more-container').click());
+    await waitFor(() => component.getByText(/Clear queue/i).click());
+
+    const state = store.getState();
+    expect(state.queue.queueItems).toEqual([]);
+  });
+
   const mountComponent = (initialStore?: AnyProps) => {
     const initialState = initialStore || buildStoreState()
       .withTracksInPlayQueue()

diff --git a/packages/ui/stories/components/searchbox.stories.tsx b/packages/ui/stories/components/searchbox.stories.tsx
@@ -20,7 +20,7 @@ storiesOf('Components/Search box', module)
       <SearchBox 
         {...commonProps} 
         selectedSearchProvider={selectedSearchProvider}
-        onSearchProviderSelect={(_, data) => onSearchProviderSelect(searchProviders.find((provider) => provider.value === data.value))}
+        onSearchProviderSelect={(data) => onSearchProviderSelect(searchProviders.find((provider) => provider.value === data.value))}
       />
     </div>;
   })
@@ -29,7 +29,7 @@ storiesOf('Components/Search box', module)
     return <div className='bg'><SearchBox 
       {...commonProps} 
       selectedSearchProvider={selectedSearchProvider}
-      onSearchProviderSelect={(_, data) => onSearchProviderSelect(searchProviders.find((provider) => provider.value === data.value))}
+      onSearchProviderSelect={(data) => onSearchProviderSelect(searchProviders.find((provider) => provider.value === data.value))}
       loading
     />
     </div>;
@@ -39,7 +39,7 @@ storiesOf('Components/Search box', module)
     return <div className='bg'><SearchBox 
       {...commonProps} 
       selectedSearchProvider={selectedSearchProvider}
-      onSearchProviderSelect={(_, data) => onSearchProviderSelect(searchProviders.find((provider) => provider.value === data.value))}
+      onSearchProviderSelect={(data) => onSearchProviderSelect(searchProviders.find((provider) => provider.value === data.value))}
       disabled
     />
     </div>;