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 basic e2e tests for user scripts #136

Closed
PEZ opened this issue Jan 5, 2023 · 0 comments · Fixed by #137
Closed

Add basic e2e tests for user scripts #136

PEZ opened this issue Jan 5, 2023 · 0 comments · Fixed by #137

Comments

@PEZ
Copy link
Collaborator

PEZ commented Jan 5, 2023

Like we have for workspace scripts. I noticed while adding JS scripts support that it is entirely possible to make things work for workspace scripts, and fail for user scripts. 😄

PEZ added a commit that referenced this issue Jan 5, 2023
* Add smoke test for user-activate
* Add JS example script to default user scripts content

Addressing #136

TODO:

* [ ] Add more smoke tests
* [ ] Handle new my-lib location somehow
@PEZ PEZ mentioned this issue Jan 5, 2023
3 tasks
@PEZ PEZ closed this as completed in #137 Jan 6, 2023
PEZ added a commit that referenced this issue Jan 6, 2023
* Add configurable user scripts location via env var

* Add smoke test for user-activate
* Add JS example script to default user scripts content

Fixes #136

TODO:

* [ ] Add more smoke tests
* [ ] Handle new my-lib location somehow

* Make user JS scripts reload when run

* Create user example script exports work

* Add tests for running user scripts

* Consolidate test files some

* Add test for reloading js scripts

* Add check for minimum passed assertions

Fixes #138

* Only create getting started user content if activate is missing

Fixes #139

* Add test run launch config

* Open extension host in workspace 1 folder by default
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 a pull request may close this issue.

1 participant