Merge pull request #178 from Arquisoft/develop

Develop - MainProductsTesting +80%
Arquisoft · May 2, 2022 · b710f39 · b710f39
2 parents 2d39821 + f72c654
commit b710f39
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/webapp/src/components/products/MainProducts.tsx b/webapp/src/components/products/MainProducts.tsx
@@ -219,7 +219,7 @@ function MainProducts(props: MainProductsProps): JSX.Element {
             label="color-filter"
           >
             <MenuItem>
-              <ListItem button key="all" onClick={() => { filterColor("All") }}>
+              <ListItem button data-testid="all-color" key="all" onClick={() => { filterColor("All") }}>
                 <ListItemIcon>
                   <ListItemText primary="All" />
                 </ListItemIcon>

diff --git a/webapp/src/components/products/tests/MainProducts.test.tsx b/webapp/src/components/products/tests/MainProducts.test.tsx
@@ -344,7 +344,9 @@ test("When listing products, use filter by color black works", async () => {
 
   //we cannot expect to see the content changed to red cars as we're mocking the API call..
   expect(getByColorBlack).toHaveBeenCalledWith("&color[eq]=black");
-
+  let allOpt = screen.getByTestId("all-color");
+  fireEvent.click(allOpt);
+  expect(getByColorBlack).toHaveBeenCalledWith("");
 });
 
 test("When listing products, use filter by brand works as expected", async () => {
@@ -383,6 +385,10 @@ test("When listing products, use filter by brand works as expected", async () =>
   //we cannot expect to see the content changed to Toyota cars as we're mocking the API call..
   //thus, we can only test the parameters passed to the API
   expect(getByBrand).toHaveBeenCalledWith("&brand[eq]=Toyota");
+
+  let allOpt = screen.getByTestId("All");
+  fireEvent.click(allOpt);
+  expect(getByBrand).toHaveBeenCalledWith("");
 });        
 
 test("When listing products, use filter by rating works as expected", async () => {