delete and save button + report update

Author: magdalena-grubesa

my_studysmart.xlsx

@@ -43,6 +43,7 @@
                     dbc.Row([
                         dbc.Col(dbc.Button("💾 Save", id="save-button", color="secondary", size="sm")),
                         dbc.Col(dbc.Button("📂 Load", id="load-button", color="secondary", size="sm")),
+                        dbc.Col(html.Div(id="save-status"), width="auto"), 
                     ], className="mb-3"),
                     html.Div(id="exam-table")
                 ], width=6)
@@ -329,12 +330,17 @@ def add_exam(n_clicks, name, exam_date, hours, topics):
     })
 
     rows = []
-    for e in exams:
+    for i, e in enumerate(exams):
         rows.append(html.Tr([
             html.Td(e["name"]),
             html.Td(e["date"]),
             html.Td(f"{e['hours']}hrs"),
-            html.Td(", ".join(e["topics"]) if e["topics"] else "—")
+            html.Td(", ".join(e["topics"]) if e["topics"] else "—"),
+            html.Td(dbc.Button("🗑️",
+            id={"type": "delete-exam", "index": i},
+            color="danger",
+            size="sm"
+            ))
         ]))
 
     table = dbc.Table([
@@ -347,21 +353,56 @@ def add_exam(n_clicks, name, exam_date, hours, topics):
 
     return table
 
+@app.callback(
+    Output("exam-table", "children", allow_duplicate=True),
+    Input({"type": "delete-exam", "index": ALL}, "n_clicks"),
+    prevent_initial_call=True
+)
+def delete_exam(n_clicks_list):
+    if not any(n_clicks_list):
+        return dash.no_update
+
+    triggered_index = ctx.triggered_id["index"]
+    exams.pop(triggered_index)
+
+    rows = []
+    for i, e in enumerate(exams):
+        rows.append(html.Tr([
+            html.Td(e["name"]),
+            html.Td(e["date"]),
+            html.Td(f"{e['hours']}hrs"),
+            html.Td(", ".join(e["topics"]) if e["topics"] else "—"),
+            html.Td(dbc.Button("🗑️",
+                id={"type": "delete-exam", "index": i},
+                color="danger",
+                size="sm"
+            ))
+        ]))
+
+    table = dbc.Table([
+        html.Thead(html.Tr([
+            html.Th("Exam"), html.Th("Date"),
+            html.Th("Hours"), html.Th("Topics"), html.Th("")
+        ])),
+        html.Tbody(rows)
+    ], bordered=True, hover=True, striped=True, size="sm")
+
+    return table if exams else html.P("No exams added.")
 
 # Callback — save exams
 @app.callback(
     Output("save-button", "children"),
+    Output("save-status", "children"), 
     Input("save-button", "n_clicks"),
     prevent_initial_call=True
 )
 def save_exams(n_clicks):
     if not exams:
-        return "💾 Save"
+        return "💾 Save", ""
     df = pd.DataFrame(exams)
     df["topics"] = df["topics"].apply(lambda t: ", ".join(t) if isinstance(t, list) else "")
     df.to_excel("my_studysmart.xlsx", index=False)
-    return "✅ Saved!"
-
+    return "💾 Save", "✅ Saved!"
 
 # Callback — load exams
 @app.callback(
@@ -389,18 +430,23 @@ def load_exams(n_clicks):
             })
 
         rows = []
-        for e in exams:
+        for i, e in enumerate(exams):
             rows.append(html.Tr([
                 html.Td(e["name"]),
                 html.Td(e["date"]),
                 html.Td(f"{e['hours']}hrs"),
-                html.Td(", ".join(e["topics"]) if e["topics"] else "—")
-            ]))
+                html.Td(", ".join(e["topics"]) if e["topics"] else "—"),
+                html.Td(dbc.Button("🗑️",
+                id={"type": "delete-exam", "index": i},
+                color="danger",
+                size="sm"
+                ))
+             ])) 
 
         table = dbc.Table([
             html.Thead(html.Tr([
                 html.Th("Exam"), html.Th("Date"),
-                html.Th("Hours"), html.Th("Topics")
+                html.Th("Hours"), html.Th("Topics"), html.Th("")
             ])),
             html.Tbody(rows)
         ], bordered=True, hover=True, striped=True, size="sm")
@@ -629,5 +675,3 @@ def generate_schedule(n_clicks, spaced_repetition, default_hours, start_date):
 if __name__ == "__main__":
     app.run(debug=True)
 
-
-    

src/study_smart/app.py

@@ -9,6 +9,7 @@
 - build_commitment_list: commitment display
 - build_tips: study tip generation
 - add_exam callback: validation and global state
+- delete_exam callback: removal of exams
 - add_commitment callback: accumulation and validation
 - delete_commitment callback: removal and recompute
 - save_exams / load_exams: Excel persistence
@@ -29,6 +30,7 @@
     build_tips,
     build_commitment_list,
     add_exam,
+    delete_exam,
     add_commitment,
     delete_commitment,
     save_exams,
@@ -323,6 +325,19 @@ def test_add_exam_multiple_exams_all_appear_in_table():
     assert isinstance(result, dbc.Table)
     assert len(app.exams) == 2
 
+# Test for delete_exam callback
+
+def test_delete_exam_removes_correct_item():
+    """Test that deleting index 0 removes the first exam."""
+    app.exams.append({"name": "Stats", "date": "2026-06-01", "hours": 10, "topics": []})
+    app.exams.append({"name": "Math", "date": "2026-06-05", "hours": 8, "topics": []})
+
+    with patch("study_smart.app.ctx") as mock_ctx:
+        mock_ctx.triggered_id = {"index": 0}
+        delete_exam([1, None])
+
+    assert len(app.exams) == 1
+    assert app.exams[0]["name"] == "Math"
 
 
 # Tests for add_commitment callback
@@ -451,16 +466,18 @@ def test_build_weekly_view_no_rev_label_when_not_spaced():
 
 def test_save_exams_empty_returns_save_label():
     """Test that saving with no exams returns the original '💾 Save' label."""
-    result = save_exams(1)
-    assert result == "💾 Save"
+    button, status = save_exams(1)
+    assert button == "💾 Save"
+    assert status == ""
 
 
 def test_save_exams_with_exams_returns_saved_label():
     """Test that saving exams returns '✅ Saved!' label."""
     app.exams.append({"name": "Stats", "date": "2026-06-01", "hours": 10, "topics": ["Ch1"]})
     with patch.object(pd.DataFrame, "to_excel"):
-        result = save_exams(1)
-    assert result == "✅ Saved!"
+        button, status = save_exams(1)
+    assert button == "💾 Save"
+    assert status == "✅ Saved!"
 
 
 def test_save_exams_serialises_topics_as_string():