Skip to content

Integration test for start command#4

Merged
carole-lavillonniere merged 1 commit intomainfrom
integ-tests
Jan 30, 2026
Merged

Integration test for start command#4
carole-lavillonniere merged 1 commit intomainfrom
integ-tests

Conversation

@carole-lavillonniere
Copy link
Collaborator

@carole-lavillonniere carole-lavillonniere commented Jan 29, 2026

Adds integration tests for the lstk start command, written in go.

Considered bats to write the integration tests suite, but:

  • bats is not type-safe
  • bats it more complicated to write components with, to avoid code redundancy
  • bats is harder to debug
  • bats would bring more complexity by adding a 2nd language in the project

@carole-lavillonniere carole-lavillonniere marked this pull request as ready for review January 30, 2026 09:14
Copy link
Member

@silv-io silv-io left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Glad we're not using bats here :P

@carole-lavillonniere carole-lavillonniere merged commit 3df7e75 into main Jan 30, 2026
@carole-lavillonniere carole-lavillonniere deleted the integ-tests branch January 30, 2026 10:24
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.

2 participants