Skip to content

Add integration tests for saild, and thus also sail :)#43

Open
novedevo wants to merge 13 commits into
maintenancefrom
integration-tests
Open

Add integration tests for saild, and thus also sail :)#43
novedevo wants to merge 13 commits into
maintenancefrom
integration-tests

Conversation

@novedevo

@novedevo novedevo commented Oct 19, 2025

Copy link
Copy Markdown
Member

this is a solid integration testing strategy IMO, as long as we believe saild should be a flagship implementation / example, i.e. it supports most valid usecases, or enough to reasonably cover sail

todo:

  • more test cases
  • testcase (re)generator - supply the client data and the generator records saild's responses to make a new testcase
  • ignore text, only check response codes in integration tests

@novedevo
novedevo changed the base branch from maintenance to main October 19, 2025 01:10
@novedevo
novedevo marked this pull request as ready for review October 19, 2025 01:11
@novedevo
novedevo changed the base branch from main to maintenance October 19, 2025 01:26
@novedevo
novedevo marked this pull request as draft October 19, 2025 01:34
@novedevo
novedevo marked this pull request as ready for review October 30, 2025 22:34
@novedevo
novedevo force-pushed the integration-tests branch 2 times, most recently from e66062a to a29fa20 Compare November 1, 2025 19:43
@novedevo
novedevo requested a review from gennyble November 1, 2025 19:50
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