refactor: cleanups and formatting

Author: okaziya

backend/jest.config.js

@@ -5,12 +5,7 @@
 
 /** @type {import('jest').Config} */
 const config = {
-  // The test environment that will be used for testing
-  // testEnvironment: "jest-environment-node",
   testEnvironment: 'node',
-
-  // A map from regular expressions to paths to transformers
-  // transform: undefined,
   transform: {
     '^.+\\.jsx?$': 'babel-jest',
   },

backend/src/__test__/app.test.js

@@ -173,13 +173,13 @@ describe('App API Endpoints', () => {
     })
 
     it('should keep list completed status as false when not all items are completed', async () => {
-      const itemOneRes = await createItem( {
+      const itemOneRes = await createItem({
         itemTitle: 'Incomplete Item',
         completed: false,
       })
       expect(itemOneRes.status).toBe(200)
 
-      const itemTwoRes = await createItem( {
+      const itemTwoRes = await createItem({
         itemTitle: 'Completed Item',
         completed: true,
       })
@@ -210,22 +210,18 @@ describe('App API Endpoints', () => {
     })
 
     it('should update list completed status to true if all remaining items are completed after deletion', async () => {
+      const testTodoList = { title: 'The POST list!', completed: true }
 
-       const testTodoList = { title: 'The POST list!', completed: true }
-
-
-         const res = await request(app).post('/api/todo-list').send(testTodoList)
-         expect(res.status).toBe(200)
-
-        const testListId = res.body.id
+      const res = await request(app).post('/api/todo-list').send(testTodoList)
+      expect(res.status).toBe(200)
 
+      const testListId = res.body.id
 
       const itemRes = await request(app)
         .post(`/api/todo-list/${testListId}/item`)
         .send({ itemTitle: 'Completed Item' })
       expect(itemRes.status).toBe(200)
 
-
       const deleteRes = await request(app).delete(
         `/api/todo-list/${testListId}/item/${itemRes.body.id}`
       )

backend/src/app.js

@@ -6,7 +6,8 @@ import {
   createTodoItem,
   fetchTodoItemById,
   updateTodoItem,
-  deleteTodoItem, createTodoList
+  deleteTodoItem,
+  createTodoList,
 } from './todo-service.js'
 
 const app = express()

backend/src/todo-service.js

@@ -77,22 +77,18 @@ const handleListCompletionStatus = async (todoListRepository, listId, itemComple
   if (itemCompleted) {
     await markListAsCompletedIfAllItemsCompleted(todoListRepository, listId)
   } else {
-    console.log('NOW MARKING THE LIST AS NOT COMPLETED!')
     await todoListRepository.update({ id: listId }, { completed: false })
   }
 }
 
-
 export const updateTodoItem = async (listId, itemId, updateData) => {
-  console.log('*** updateTodoItem', listId, itemId, updateData)
   const AppDataSource = await getAppDataSource()
   const todoRepository = AppDataSource.getRepository(TodoItem)
   const todoListRepository = AppDataSource.getRepository(TodoList)
 
   const todoItem = await fetchAndValidateTodoItem(todoRepository, listId, itemId)
 
   const updatedTodoItem = await todoRepository.save(todoRepository.merge(todoItem, updateData))
-  console.log('updatedTodoItem in memory and in database:', updatedTodoItem)
 
   await handleListCompletionStatus(todoListRepository, listId, updateData.completed)
 
@@ -108,18 +104,7 @@ export const deleteTodoItem = async (listId, itemId) => {
 
   await todoRepository.delete(itemId)
 
-  console.log(">BEFORE",  await todoListRepository.findOne({
-    where: { id: listId },
-    relations: ['items'],
-  })
-  )
   await markListAsCompletedIfAllItemsCompleted(todoListRepository, listId)
 
-  console.log(">AFTER",  await todoListRepository.findOne({
-      where: { id: listId },
-      relations: ['items'],
-    })
-  )
-
   return todoItem
 }