Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add integration tests #30

Merged
merged 11 commits into from
Oct 23, 2023
Merged

Add integration tests #30

merged 11 commits into from
Oct 23, 2023

Conversation

deribaucourt
Copy link
Member

No description provided.

@deribaucourt deribaucourt force-pushed the add-client-tests branch 2 times, most recently from 93bfbce to a166042 Compare October 20, 2023 15:17
Copy link
Member

@idillon-sfl idillon-sfl left a comment

Choose a reason for hiding this comment

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

looks cool!

src/integration-tests/README.md Outdated Show resolved Hide resolved
client/tsconfig.json Outdated Show resolved Hide resolved
client/package.json Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
Integration tests will allow running the extension in a live vscode
window. We'll have to use mocha instead of jest for these
because jest doesn't support programmatically being run in the
current process (always forks).
Because integration tests will need the server and the client at the
same time, their sources will be placed in the common `src` directory.

Setup the typescript configuration to build the integration tests.
This test runs a VSCode instance and checks that one bitbake setting is
properly read.
Add configurations to debug the integration tests.
This separates the Jest test script from the npm run test script.
When running in some environments, the test could be run before VSCode
had finished opening the workspace (happens asynchronously).
The GitHub action runner has no screen and VSCode wants one. Run it
in a virtual X server.
This makes it clear that the `src` folder does not provide any
functionality to the extension itself, but is only used for testing.
@deribaucourt
Copy link
Member Author

Thanks for your suggestions!

@deribaucourt deribaucourt merged commit 5202c0e into staging Oct 23, 2023
1 check passed
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