mark the check up front

Emyrk · Emyrk · commit 771be40b6dc3 · 2025-04-07T13:08:46.000-05:00
diff --git a/provider/formtype_test.go b/provider/formtype_test.go
@@ -241,12 +241,12 @@ func TestValidateFormType(t *testing.T) {
 			t.Run(c.name, func(t *testing.T) {
 				t.Parallel()
 
+				formTypesChecked[c.config.String()] = struct{}{}
 				formTypeTest(t, c)
 				if _, ok := formTypesChecked[c.config.String()]; ok {
 					t.Log("Duplicated form type check, delete this extra test case")
 					t.Fatalf("form type %q already checked", c.config.String())
 				}
-				formTypesChecked[c.config.String()] = struct{}{}
 			})
 		}
 	})

Original file line number	Diff line number	Diff line change
`@@ -241,12 +241,12 @@ func TestValidateFormType(t *testing.T) {`
`241`	`241`	`t.Run(c.name, func(t *testing.T) {`
`242`	`242`	`t.Parallel()`
`243`	`243`
	`244`	`+ formTypesChecked[c.config.String()] = struct{}{}`
`244`	`245`	`formTypeTest(t, c)`
`245`	`246`	`if _, ok := formTypesChecked[c.config.String()]; ok {`
`246`	`247`	`t.Log("Duplicated form type check, delete this extra test case")`
`247`	`248`	`t.Fatalf("form type %q already checked", c.config.String())`
`248`	`249`	`}`
`249`		`- formTypesChecked[c.config.String()] = struct{}{}`
`250`	`250`	`})`
`251`	`251`	`}`
`252`	`252`	`})`