refactor: consolidate assertions and add rounding boundary tests

- Consolidate repetitive undefined checks using Object.values().every()
- Add explicit rounding boundary tests (.49 and .50) for formatCurrency
- All 212 tests passing with improved test maintainability

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

Author: Copilot

src/constants/appConstants.test.ts

@@ -170,21 +170,13 @@ describe("Application Constants", () => {
     it("returns undefined values when options is null", () => {
       const result = mapOptionsToConstants(null as unknown as Record<string, CIADetails>, "description");
 
-      expect(result.NONE).toBeUndefined();
-      expect(result.LOW).toBeUndefined();
-      expect(result.MODERATE).toBeUndefined();
-      expect(result.HIGH).toBeUndefined();
-      expect(result.VERY_HIGH).toBeUndefined();
+      expect(Object.values(result).every(v => v === undefined)).toBe(true);
     });
 
     it("returns undefined values when options is undefined", () => {
       const result = mapOptionsToConstants(undefined as unknown as Record<string, CIADetails>, "description");
 
-      expect(result.NONE).toBeUndefined();
-      expect(result.LOW).toBeUndefined();
-      expect(result.MODERATE).toBeUndefined();
-      expect(result.HIGH).toBeUndefined();
-      expect(result.VERY_HIGH).toBeUndefined();
+      expect(Object.values(result).every(v => v === undefined)).toBe(true);
     });
 
     it("maps description field correctly", () => {

src/constants/costConstants.test.ts

@@ -310,6 +310,18 @@ describe("costConstants", () => {
       expect(formatted).toContain("1,235");
     });
 
+    it("rounds down correctly at .49 boundary", () => {
+      const formatted = formatCurrency(1234.49);
+      expect(formatted).toContain("$");
+      expect(formatted).toContain("1,234");
+    });
+
+    it("rounds up correctly at .50 boundary", () => {
+      const formatted = formatCurrency(1234.50);
+      expect(formatted).toContain("$");
+      expect(formatted).toContain("1,235");
+    });
+
     it("uses default USD when no currency specified", () => {
       const formatted = formatCurrency(500);
       expect(formatted).toContain("$");