Skip to content

Commit daed7e1

Browse files
authored
Merge pull request #21 from kraftjs/section-4.1.3-patch
Add assertion to check inventory and update addItemToCart function to pass
2 parents d7273c6 + 11f74f8 commit daed7e1

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

chapter4/1_setting_up_a_test_environment/1_exposing_modules/3_unit_tests/cartController.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
const { removeFromInventory } = require("./inventoryController");
1+
const { inventory, removeFromInventory } = require("./inventoryController");
22
const logger = require("./logger");
33

44
const carts = new Map();
@@ -8,6 +8,7 @@ const addItemToCart = (username, item) => {
88
const newItems = (carts.get(username) || []).concat(item);
99

1010
if (!compliesToItemLimit(newItems)) {
11+
inventory.set(item, inventory.get(item) + 1);
1112
const limitError = new Error(
1213
"You can't have more than three units of an item in your cart"
1314
);

chapter4/1_setting_up_a_test_environment/1_exposing_modules/3_unit_tests/cartController.test.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,8 @@ describe("addItemToCart", () => {
4747
}
4848

4949
expect(carts.get("test_user")).toEqual(initialCartContent);
50-
expect.assertions(2);
50+
expect(inventory.get('cheesecake')).toEqual(1);
51+
expect.assertions(3);
5152
});
5253

5354
test("logging added items", () => {

0 commit comments

Comments
 (0)