From 9f670961b4d69d1010b8caebe0924c0a39f37b4f Mon Sep 17 00:00:00 2001
From: Kevin Bost <kitokeboo@gmail.com>
Date: Sat, 30 May 2026 13:58:52 -0700
Subject: [PATCH 1/2] Use MSTest v4 throw assertions

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
---
 SimplyBudgetSharedTests/Data/ExpenseCategoryItemTests.cs | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/SimplyBudgetSharedTests/Data/ExpenseCategoryItemTests.cs b/SimplyBudgetSharedTests/Data/ExpenseCategoryItemTests.cs
index 1c86507..e2bad0c 100644
--- a/SimplyBudgetSharedTests/Data/ExpenseCategoryItemTests.cs
+++ b/SimplyBudgetSharedTests/Data/ExpenseCategoryItemTests.cs
@@ -66,7 +66,7 @@ public void SetIgnoreBudget_WithNull_Throws()
     {
         var item = new ExpenseCategoryItem();
 
-        var ex = Assert.ThrowsException<ArgumentNullException>(() => item.IgnoreBudget = null);
+        var ex = Assert.ThrowsExactly<ArgumentNullException>(() => item.IgnoreBudget = null);
         Assert.AreEqual("value", ex.ParamName);
     }
 
@@ -75,7 +75,7 @@ public void SetIgnoreBudget_True_Throws()
     {
         var item = new ExpenseCategoryItem();
 
-        var ex = Assert.ThrowsException<ArgumentNullException>(() => item.IgnoreBudget = null);
+        var ex = Assert.ThrowsExactly<ArgumentNullException>(() => item.IgnoreBudget = null);
         Assert.AreEqual("value", ex.ParamName);
     }
 

From 72616f4ed266bb7f46f84b59700b252f38aa6f7c Mon Sep 17 00:00:00 2001
From: Kevin Bost <kitokeboo@gmail.com>
Date: Sat, 30 May 2026 14:05:15 -0700
Subject: [PATCH 2/2] Fix progress indicator accessibility labels

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
---
 SimplyBudgetWeb.Web/src/components/AddTransactionDialog.tsx | 2 +-
 SimplyBudgetWeb.Web/src/pages/Accounts.tsx                  | 2 +-
 SimplyBudgetWeb.Web/src/pages/Budget.tsx                    | 2 +-
 SimplyBudgetWeb.Web/src/pages/History.tsx                   | 2 +-
 SimplyBudgetWeb.Web/src/pages/Import.tsx                    | 4 ++--
 SimplyBudgetWeb.Web/src/pages/Settings.tsx                  | 2 +-
 6 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/SimplyBudgetWeb.Web/src/components/AddTransactionDialog.tsx b/SimplyBudgetWeb.Web/src/components/AddTransactionDialog.tsx
index 384f27e..ef34e70 100644
--- a/SimplyBudgetWeb.Web/src/components/AddTransactionDialog.tsx
+++ b/SimplyBudgetWeb.Web/src/components/AddTransactionDialog.tsx
@@ -207,7 +207,7 @@ export default function AddTransactionDialog({ open, onClose, categories, onSucc
       <DialogActions>
         <Button onClick={handleClose}>Cancel</Button>
         <Button variant="contained" onClick={handleSubmit} disabled={submitting}>
-          {submitting ? <CircularProgress size={20} /> : 'Save'}
+          {submitting ? <CircularProgress size={20} aria-label="Saving transaction" /> : 'Save'}
         </Button>
       </DialogActions>
     </Dialog>
diff --git a/SimplyBudgetWeb.Web/src/pages/Accounts.tsx b/SimplyBudgetWeb.Web/src/pages/Accounts.tsx
index efbc3a4..d2c4b02 100644
--- a/SimplyBudgetWeb.Web/src/pages/Accounts.tsx
+++ b/SimplyBudgetWeb.Web/src/pages/Accounts.tsx
@@ -75,7 +75,7 @@ export default function Accounts() {
       </Box>
 
       {loading ? (
-        <Box sx={{ display: 'flex', justifyContent: 'center', p: 4 }}><CircularProgress /></Box>
+        <Box sx={{ display: 'flex', justifyContent: 'center', p: 4 }}><CircularProgress aria-label="Loading accounts" /></Box>
       ) : (
         <List>
           {accounts.map(account => (
diff --git a/SimplyBudgetWeb.Web/src/pages/Budget.tsx b/SimplyBudgetWeb.Web/src/pages/Budget.tsx
index dcd4954..67c812a 100644
--- a/SimplyBudgetWeb.Web/src/pages/Budget.tsx
+++ b/SimplyBudgetWeb.Web/src/pages/Budget.tsx
@@ -76,7 +76,7 @@ export default function Budget() {
 
       {loading ? (
         <Box sx={{ display: 'flex', justifyContent: 'center', p: 4 }}>
-          <CircularProgress />
+          <CircularProgress aria-label="Loading budget" />
         </Box>
       ) : (
         <Box>
diff --git a/SimplyBudgetWeb.Web/src/pages/History.tsx b/SimplyBudgetWeb.Web/src/pages/History.tsx
index b59d1f2..4d30c60 100644
--- a/SimplyBudgetWeb.Web/src/pages/History.tsx
+++ b/SimplyBudgetWeb.Web/src/pages/History.tsx
@@ -103,7 +103,7 @@ export default function History() {
       </Paper>
 
       {loading ? (
-        <Box sx={{ display: 'flex', justifyContent: 'center', p: 4 }}><CircularProgress /></Box>
+        <Box sx={{ display: 'flex', justifyContent: 'center', p: 4 }}><CircularProgress aria-label="Loading history" /></Box>
       ) : (
         <List>
           {items.length === 0 && (
diff --git a/SimplyBudgetWeb.Web/src/pages/Import.tsx b/SimplyBudgetWeb.Web/src/pages/Import.tsx
index 54e8e5d..72149d0 100644
--- a/SimplyBudgetWeb.Web/src/pages/Import.tsx
+++ b/SimplyBudgetWeb.Web/src/pages/Import.tsx
@@ -74,7 +74,7 @@ export default function Import() {
           sx={{ mb: 2 }}
         />
         <Button variant="contained" onClick={handleParse} disabled={loading || !csvText.trim()}>
-          {loading ? <CircularProgress size={20} /> : 'Parse'}
+          {loading ? <CircularProgress size={20} aria-label="Parsing import data" /> : 'Parse'}
         </Button>
       </Paper>
 
@@ -133,7 +133,7 @@ export default function Import() {
             </Table>
           </Paper>
           <Button variant="contained" onClick={handleImport} disabled={submitting}>
-            {submitting ? <CircularProgress size={20} /> : 'Save Import'}
+            {submitting ? <CircularProgress size={20} aria-label="Saving import" /> : 'Save Import'}
           </Button>
         </>
       )}
diff --git a/SimplyBudgetWeb.Web/src/pages/Settings.tsx b/SimplyBudgetWeb.Web/src/pages/Settings.tsx
index fd9586c..3629b13 100644
--- a/SimplyBudgetWeb.Web/src/pages/Settings.tsx
+++ b/SimplyBudgetWeb.Web/src/pages/Settings.tsx
@@ -140,7 +140,7 @@ export default function Settings() {
       </Box>
 
       {loading ? (
-        <Box sx={{ display: 'flex', justifyContent: 'center', p: 4 }}><CircularProgress /></Box>
+        <Box sx={{ display: 'flex', justifyContent: 'center', p: 4 }}><CircularProgress aria-label="Loading import rules" /></Box>
       ) : (
         <List>
           {rules.length === 0 && <ListItem><ListItemText primary="No rules defined." /></ListItem>}