fix: address code review - Alpine.js $set, SQLAlchemy bool filter style

Agent-Logs-Url: https://github.com/christianlouis/dmarq/sessions/f141cd3b-8d0a-4c7e-b479-ee973c209c11

Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com>

backend/app/templates/mail_sources.html

@@ -454,7 +454,7 @@ function mailSourcesApp() {
         },
 
         async testSource(id) {
-            this.$set(this.testing, id, true);
+            this.testing[id] = true;
             this.feedback = { message: '', type: '' };
             try {
                 const resp = await fetch(`/api/v1/mail-sources/${id}/test`, { method: 'POST' });
@@ -474,7 +474,7 @@ function mailSourcesApp() {
             } catch (e) {
                 this.feedback = { message: `Test error: ${e.message}`, type: 'error' };
             } finally {
-                this.$set(this.testing, id, false);
+                this.testing[id] = false;
             }
         },
 

backend/app/tests/test_mail_sources.py

@@ -147,9 +147,7 @@ class TestMailSourcesAPI:
         db_session.add(MailSource(name="Disabled", method="IMAP", enabled=False))
         db_session.commit()
 
-        enabled = (
-            db_session.query(MailSource).filter(MailSource.enabled == True).all()  # noqa: E712
-        )
+        enabled = db_session.query(MailSource).filter(MailSource.enabled).all()
         assert len(enabled) == 2
         names = {s.name for s in enabled}
         assert "Enabled A" in names