Skip to content

Commit 208c4f7

Browse files
committed
fixes to Edit functionality
1 parent 23fffc0 commit 208c4f7

File tree

4 files changed

+4
-4
lines changed

4 files changed

+4
-4
lines changed

Lesson2-Exercise2-Delete/solution/src/tracker.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ def edit_expense(self, expense_id, new_amount=None, new_category=None):
2929
expense = self.get_expense(expense_id)
3030
if expense:
3131
if new_amount is not None:
32-
expense["amount"] = amount
32+
expense["amount"] = new_amount
3333
if new_category is not None:
3434
expense["category"] = new_category
3535

Lesson2-Exercise2-Delete/solution/tests/test_tracker.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ def test_edit_expense(tracker):
5555
edited_expense = tracker.get_expense(expense_id)
5656
# Assert
5757
assert tracker.total == 12.50
58-
assert edit_expense["category"] == "dining"
58+
assert edited_expense["category"] == "dining"
5959

6060
def test_delete_expense(tracker):
6161
# Arrange: Add an expense to delete

Lesson2-Exercise2-Delete/starter/src/tracker.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ def edit_expense(self, expense_id, new_amount=None, new_category=None):
2929
expense = self.get_expense(expense_id)
3030
if expense:
3131
if new_amount is not None:
32-
expense["amount"] = amount
32+
expense["amount"] = new_amount
3333
if new_category is not None:
3434
expense["category"] = new_category
3535

Lesson2-Exercise2-Delete/starter/tests/test_tracker.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ def test_edit_expense(tracker):
4747
edited_expense = tracker.get_expense(expense_id)
4848
# Assert
4949
assert tracker.total == 12.50
50-
assert edit_expense["category"] == "dining"
50+
assert edited_expense["category"] == "dining"
5151

5252
def test_list_expenses_by_category_returns_only_requested_category(tracker):
5353
# Arrange

0 commit comments

Comments
 (0)