Skip to content

Re-enable integration tests - #762

Draft
martin-helmich wants to merge 11 commits into
masterfrom
task/re-enable-integration-tests
Draft

Re-enable integration tests#762
martin-helmich wants to merge 11 commits into
masterfrom
task/re-enable-integration-tests

Conversation

@martin-helmich

Copy link
Copy Markdown
Member

No description provided.

@martin-helmich

Copy link
Copy Markdown
Member Author

@mfal As announced, this is something that I had difficulties with. In the current setup (Jest + ts-jest), oclif seems to have issues importing the concrete command files (which just hangs indefinitely, with the tests eventually timing out). I also tried other setups (like using Babel1), but ran into other problems with those (for example, not processing the TSX files or difficulties with ESM imports).

Any thoughts?

Footnotes

  1. https://jestjs.io/docs/getting-started#via-babel

@mfal mfal self-assigned this Sep 9, 2024
@gandie
gandie force-pushed the task/re-enable-integration-tests branch from 8f08d0b to 2d31255 Compare July 27, 2026 10:27
@gandie
gandie marked this pull request as draft July 27, 2026 10:27
@gandie

gandie commented Jul 27, 2026

Copy link
Copy Markdown
Member

@martin-helmich brought this one back to life ( to make it a draft then 😆 ).
Would love to see such tests running again, as they would greatly increase my confidence in changes. First step was to rebase on top of current master and creating a plan how integration tests might look like in the future ( see plan document i've added ).

tl;dr : Use mockoon ( or something else ) plus already prepared environment variable switch to alter mittwald API base URL. This would allow real HTTP requests to run during tests, no mocking for that needed. By altering and specializing the mockoon schema we could provide HTTP responses matching our test cases ( if an example response does not fit already ).

@gandie
gandie force-pushed the task/re-enable-integration-tests branch from 2d31255 to 8e0fe12 Compare July 28, 2026 07:36
@gandie

gandie commented Jul 28, 2026

Copy link
Copy Markdown
Member

Getting somewhere:

[run-all] execution complete: 185 commands
[run-all] statistics: successful=100, failed=85, total=185

Failed commands mostly come from missing arguments, the command discovery misses some.
Yet, at least two errors from current error log caught my eye already, will analyze them when fog has lifted.

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.

3 participants