ambika manually merges PR 1044 Luis unit test inventory controller ge…

…t all inv in project

requirements/inventoryController/getAllInvInProject.md

@@ -0,0 +1,17 @@
+Check mark: ✅
+Cross Mark: ❌
+
+# Post Badge
+
+> ## Positive case
+
+1. ❌ Receives a POST request in the **/api/userProfile** route
+2. ❌ Returns 200 if successfully fetch inventory data
+
+   > ## Negative case
+
+3. ❌ Returns error 404 if the API does not exist
+4. ❌ Returns error code 403 if the user is not authorized to view the inventory data
+5. ❌ Returns error code 404 if an error occurs when populating or saving.
+
+> ## Edge case

requirements/inventoryController/getAllInvInProjectWBS.md

@@ -0,0 +1,17 @@
+Check mark: ✅
+Cross Mark: ❌
+
+# Post Badge
+
+> ## Positive case
+
+1. ❌ Receives a POST request in the **/api/userProfile** route
+2. ✅ Returns 200 if successfully found data
+
+> ## Negative case
+
+1. ❌ Returns error 404 if the API does not exist
+2. ✅ Returns 403 if user is not authorized to view inventory data
+3. ✅ Returns 404 if an error occurs while fetching data
+
+> ## Edge case

requirements/inventoryController/postInvInProjectWBS.md

@@ -0,0 +1,19 @@
+Check mark: ✅
+Cross Mark: ❌
+
+# Post Badge
+
+> ## Positive case
+
+1. ❌ Receives a POST request in the **/api/userProfile** route
+2. ✅ Returns status code 201, if the inventory was successfully created and saved
+3. ✅ Returns status code 201, if the inventory item was succesfully updated and saved.
+
+> ## Negative case
+
+1. ❌ Returns error 404 if the API does not exist
+2. ✅ Returns error 403 if the user is not authorized to view data
+3. ✅ Returns error 500 if an error occurs when saving
+4. ✅ Returns error 400 if a valid project was found but quantity and type id were missing
+
+> ## Edge case

src/controllers/inventoryController.js

@@ -13,7 +13,7 @@ const inventoryController = function (Item, ItemType) {
     // use req.params.projectId and wbsId
     // Run a mongo query on the Item model to find all items with both the project and wbs
     // sort the mongo query so that the Wasted false items are listed first
-    return Item.find({
+    await Item.find({
       project: mongoose.Types.ObjectId(req.params.projectId),
       wbs:
         req.params.wbsId && req.params.wbsId !== 'Unassigned'
@@ -283,9 +283,9 @@ const inventoryController = function (Item, ItemType) {
     }
 
     // update the original item by decreasing by the quantity and adding a note
-    if (req.body.quantity && req.params.invId && projectExists && wbsExists) {
+    if (req.body.quantity && req.param.invId && projectExists && wbsExists) {
       return Item.findByIdAndUpdate(
-        req.params.invId,
+        req.param.invId,
         {
           $decr: { quantity: req.body.quantity },
           $push: {
@@ -797,4 +797,4 @@ const inventoryController = function (Item, ItemType) {
   };
 };
 
-module.exports = inventoryController;
+module.exports = inventoryController;