Skip to content

Enables testing and bundling#41

Merged
ufukty merged 25 commits into
mainfrom
feature/testing
Apr 22, 2026
Merged

Enables testing and bundling#41
ufukty merged 25 commits into
mainfrom
feature/testing

Conversation

@ufukty
Copy link
Copy Markdown
Owner

@ufukty ufukty commented Apr 19, 2026

@ufukty ufukty added the feature label Apr 19, 2026
@ufukty ufukty changed the title Enables testing Enables testing and bundling Apr 22, 2026
Copy link
Copy Markdown
Owner Author

@ufukty ufukty left a comment

Choose a reason for hiding this comment

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

todo:

  • maybe keep using mocha for integration testing along with the vitest used for unit tests
  • sort .vscodeignore

@ufukty
Copy link
Copy Markdown
Owner Author

ufukty commented Apr 22, 2026

Integration test seems to fail at CI. Options are tricking the tester (with xvfb-run?), skipping integration tests in CI or removing integration tests completely in favor of only using unit tests.

Run npm run test-integration

> dim@3.2.0 test-integration
> npx tsc -p . && npx eslint src && node esbuild.mjs && vscode-test

- Resolving version...
✔ Validated version: 1.117.0
- Found at https://update.code.visualstudio.com/1.117.0/linux-x64/stable?released=true
✔ Found at https://update.code.visualstudio.com/1.117.0/linux-x64/stable?released=true
- Downloading (203.55 MB)
✔ Downloaded VS Code into /home/runner/work/dim/dim/.vscode-test/vscode-linux-x64-1.117.0
[2445:0422/105136.491777:ERROR:ui/ozone/platform/x11/ozone_platform_x11.cc:249] Missing X server or $DISPLAY
[2445:0422/105136.491827:ERROR:ui/aura/env.cc:257] The platform failed to initialize.  Exiting.
Exit code:   SIGSEGV
Error: Process completed with exit code 1.

Unit tests do look like would be enough for the significant portion of breaking changes. Integration tests can be added later.

@ufukty
Copy link
Copy Markdown
Owner Author

ufukty commented Apr 22, 2026

CI seems using an outdated Node version, doesn't meet the requirements of some Vitest dep.

@ufukty ufukty merged commit 4eee3e7 into main Apr 22, 2026
2 checks passed
@ufukty ufukty deleted the feature/testing branch April 22, 2026 15:36
This was referenced Apr 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Testing Bundle

1 participant