From 2843781db468678221804057f00bf399f2ef8fdd Mon Sep 17 00:00:00 2001 From: Joseph Doherty Date: Fri, 19 Jun 2026 01:57:02 -0400 Subject: [PATCH] =?UTF-8?q?docs(#70):=20correct=20SqliteInsertSql=20commen?= =?UTF-8?q?t=20=E2=80=94=20test=20drives=20EF=20SQL=20generator,=20not=20t?= =?UTF-8?q?he=20constant?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../Migrations/20260615171137_ResyncLdapGroupMappingSeed.cs | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/ZB.MOM.WW.ScadaBridge.ConfigurationDatabase/Migrations/20260615171137_ResyncLdapGroupMappingSeed.cs b/src/ZB.MOM.WW.ScadaBridge.ConfigurationDatabase/Migrations/20260615171137_ResyncLdapGroupMappingSeed.cs index 2eab897c..f236ae99 100644 --- a/src/ZB.MOM.WW.ScadaBridge.ConfigurationDatabase/Migrations/20260615171137_ResyncLdapGroupMappingSeed.cs +++ b/src/ZB.MOM.WW.ScadaBridge.ConfigurationDatabase/Migrations/20260615171137_ResyncLdapGroupMappingSeed.cs @@ -12,8 +12,9 @@ namespace ZB.MOM.WW.ScadaBridge.ConfigurationDatabase.Migrations /// INSERT OR IGNORE skips the row when the Id primary key already /// exists, so replaying this migration against a DB that already carries Id=5 /// (e.g. one seeded by a newer HasData baseline) is a no-op rather than a PK - /// violation. Exposed as a constant so the idempotency test can exercise the - /// exact SQL the migration emits on the SQLite branch. + /// violation. Kept as a named constant for readability and so the emitted + /// SQLite SQL is documented in one place; the idempotency test drives the + /// real migration through EF's SQL generator rather than reading this directly. /// internal const string SqliteInsertSql = "INSERT OR IGNORE INTO \"LdapGroupMappings\" (\"Id\", \"LdapGroupName\", \"Role\") " +