Fix testifylint violations in lock_schema_test.go

[Copilot]

The lint-go workflow was failing on 10 testifylint violations in pkg/workflow/lock_schema_test.go.

Changes

• Error assertions: Changed assert.Error/assert.NoError → require.Error/require.NoError (9 instances)
  • testifylint enforces require.* for error assertions to fail fast on critical conditions
• Empty checks: Changed assert.Equal(t, "", metadata.StopTime) → assert.Empty(t, metadata.StopTime) (1 instance)
  • testifylint enforces semantic clarity for empty value assertions

// Before
assert.NoError(t, err, "Should parse legacy lock file")
assert.Equal(t, "", metadata.StopTime, "Stop time should be empty")

// After
require.NoError(t, err, "Should parse legacy lock file")
assert.Empty(t, metadata.StopTime, "Stop time should be empty")

All changes are linter compliance fixes with no functional impact.

Original prompt

Fix the failing GitHub Actions workflow lint-go
Analyze the workflow logs, identify the root cause of the failure, and implement a fix.
Job ID: 63926118092
Job URL: https://github.com/github/gh-aw/actions/runs/22116568757/job/63926118092

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[Copilot]

Pull request overview

This PR addresses lint-go failures by updating pkg/workflow/lock_schema_test.go to comply with testifylint’s preferred assertion patterns, without changing test behavior.

Changes:

• Switched error assertions from assert.Error/assert.NoError to require.Error/require.NoError where fail-fast behavior is expected.
• Replaced an empty-string equality check with assert.Empty for semantic clarity.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.