Skip to content

Commit 91c703b

Browse files
Copilotintel352
andauthored
fix: clarify TestPlugin_StepSchemaCoverage doc comment to match what it actually tests
The comment said "every step type advertised in StepTypes()" but the test reads from plugin.json's stepTypes array, not the Go StepTypes() method. Agent-Logs-Url: https://github.com/GoCodeAlone/workflow-plugin-data-engineering/sessions/7e0eb14e-5aa9-4dc7-a3b0-a6ae661d980c Co-authored-by: intel352 <77607+intel352@users.noreply.github.com>
1 parent c25e5a2 commit 91c703b

1 file changed

Lines changed: 4 additions & 3 deletions

File tree

internal/plugin_test.go

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -594,9 +594,10 @@ func repoRoot(t *testing.T) string {
594594
return filepath.Dir(filepath.Dir(file))
595595
}
596596

597-
// TestPlugin_StepSchemaCoverage verifies that every step type advertised in
598-
// StepTypes() has a corresponding entry in the stepSchemas field of plugin.json.
599-
// This ensures strict contract descriptors are present for all steps.
597+
// TestPlugin_StepSchemaCoverage verifies that every step type listed in the
598+
// stepTypes array of plugin.json has a corresponding entry in the stepSchemas
599+
// array of the same file, and that no orphan stepSchemas entries exist.
600+
// This ensures strict contract descriptors are present for all advertised steps.
600601
func TestPlugin_StepSchemaCoverage(t *testing.T) {
601602
root := repoRoot(t)
602603
data, err := os.ReadFile(filepath.Join(root, "plugin.json"))

0 commit comments

Comments
 (0)