diff --git a/.github/workflows/linters.yml b/.github/workflows/linters.yml
index 6701673cc7f..37d8363beaa 100644
--- a/.github/workflows/linters.yml
+++ b/.github/workflows/linters.yml
@@ -43,3 +43,6 @@ jobs:
 
       - name: Run Semgrep rules
         run: semgrep ci --config ./frappe-semgrep-rules/rules --config r/python.lang.correctness
+
+      - name: Semgrep for Test Correctness
+        run: semgrep ci --include=**/test_*.py --config ./semgrep/test-correctness.yml
diff --git a/semgrep/test-correctness.yml b/semgrep/test-correctness.yml
new file mode 100644
index 00000000000..34eb82fa1d6
--- /dev/null
+++ b/semgrep/test-correctness.yml
@@ -0,0 +1,18 @@
+rules:
+- id: Dont-commit
+  pattern: frappe.db.commit()
+  message: Commiting inside test breaks idempotency.
+  languages: [python]
+  severity: ERROR
+- id: Implicit-commit
+  pattern: frappe.db.truncate()
+  message: DB truncation does implict commit which breaks test idempotency.
+  languages: [python]
+  severity: ERROR
+- id: Dont-override-teardown
+  pattern: |
+    def tearDown(...):
+      ...
+  message: ERPNextTestSuite forces rollback on each tearDown, which ensures idempotency. Don't override tearDown.
+  languages: [python]
+  severity: ERROR