test

Author: vivekbopaliya

client/modules/IDE/components/SketchList.unit.test.jsx

@@ -81,16 +81,18 @@ describe('<Sketchlist />', () => {
 
   it('renders component correctly', () => {
     const { container } = subject();
-    
-    expect(container.querySelector('.sketches-table-container')).toBeInTheDocument();
+
+    expect(
+      container.querySelector('.sketches-table-container')
+    ).toBeInTheDocument();
     expect(container.querySelector('.sketches-table')).toBeInTheDocument();
     expect(container.querySelector('thead')).toBeInTheDocument();
     expect(container.querySelector('tbody')).toBeInTheDocument();
-    
+
     // expect(screen.getByText(/Sketch/i)).toBeInTheDocument();
     // expect(screen.getByText(/Date created/i)).toBeInTheDocument();
     // expect(screen.getByText(/Last updated/i)).toBeInTheDocument();
-    
+
     const sketchRows = container.querySelectorAll('tbody tr');
     expect(sketchRows.length).toBeGreaterThan(0);
   });

server/controllers/collection.controller/listCollections.js

@@ -30,7 +30,7 @@ export default async function listCollections(req, res) {
       return sendFailure({ code: 404, message: 'User not found' });
     }
 
-    let collections = await Collection.find({ owner: ownerId }).populate([
+    const collections = await Collection.find({ owner: ownerId }).populate([
       { path: 'owner', select: ['id', 'username'] },
       {
         path: 'items.project',
@@ -49,8 +49,8 @@ export default async function listCollections(req, res) {
     }
 
     const publicCollections = collections.map((collection) => {
-      let items = collection.items;
-      items = items.filter(
+      const { items: originalItems } = collection;
+      const items = originalItems.filter(
         (item) => item.project && item.project.visibility === 'Public'
       );
       return {
@@ -60,11 +60,11 @@ export default async function listCollections(req, res) {
       };
     });
 
-    sendSuccess(publicCollections);
+    return sendSuccess(publicCollections);
   } catch (error) {
-    sendFailure({
+    return sendFailure({
       code: error.code || 500,
       message: error.message || 'Something went wrong'
     });
   }
-}
+}