Skip to content

test: give smoke agent a distinct name - #616

Merged
sasan1200 merged 1 commit into
mainfrom
feat/fixture-connection
Oct 22, 2026
Merged

sasan1200 merged 1 commit into
mainfrom
feat/fixture-connection

Conversation

@sasan1200

Copy link
Copy Markdown
Member

Summary

  • Give the smoke agent a dedicated executable name and update its process matcher.
  • Report network fixture readiness after establishing the connection.
  • Add a bounded regression that builds the smoke agent and checks a real non-loopback connection before starting the smoke suite.

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 🔒 Security enhancement
  • 📝 Documentation update
  • 🎨 Refactoring / Code style cleanups

Verification Checklist

  • go test ./... passes cleanly
  • python3 plugin/hooks/test_secret_guard.py and other python hook tests pass
  • bash packaging/swift_macos.sh test --package-path menubar passes
  • ./packaging/test/e2e_smoke.sh passes with Go 1.26.10
  • No secrets, tokens, or private keys are exposed or committed

Additional validation: the network regression failed before the fixture rename and passed afterward. Shell syntax and diff checks pass.

@sasan1200
sasan1200 merged commit e297ebc into main Oct 22, 2026
10 checks passed
@sasan1200
sasan1200 deleted the feat/fixture-connection branch October 22, 2026 04:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant