Document why tests are split by responsibility

sahilds1 · sahilds1 · commit 8198574dc63e · 2026-03-26T15:24:12.000-04:00
diff --git a/server/api/services/test_embedding_services.py b/server/api/services/test_embedding_services.py
@@ -10,6 +10,11 @@
     log_usage,
 )
 
+# Each function is tested one responsibility at a time. One test for the whole
+# function collapses all responsibilities into a single assertion block — when
+# it fails you know something is broken but not which responsibility. You have
+# to debug to find out.
+
 # ---------------------------------------------------------------------------
 # build_query tests
 #

Original file line number	Diff line number	Diff line change
`@@ -10,6 +10,11 @@`
`10`	`10`	`log_usage,`
`11`	`11`	`)`
`12`	`12`
	`13`	`+# Each function is tested one responsibility at a time. One test for the whole`
	`14`	`+# function collapses all responsibilities into a single assertion block — when`
	`15`	`+# it fails you know something is broken but not which responsibility. You have`
	`16`	`+# to debug to find out.`
	`17`	`+`
`13`	`18`	`# ---------------------------------------------------------------------------`
`14`	`19`	`# build_query tests`
`15`	`20`	`#`