Skip to content

Fix unit tests#358

Open
roguisharcanetrickster wants to merge 2 commits into
masterfrom
fix_unit_tests
Open

Fix unit tests#358
roguisharcanetrickster wants to merge 2 commits into
masterfrom
fix_unit_tests

Conversation

@roguisharcanetrickster

@roguisharcanetrickster roguisharcanetrickster commented Jun 8, 2026

Copy link
Copy Markdown
Contributor
  1. add unit tests to git actions
  2. update the unit tests
  3. add badge to README

While reviewing keep this in mind. Most of this is generated: I took the best look at it I could and found no glaring issues

Adds sinon, which has 9 million weekly downloads
https://www.npmjs.com/package/sinon

@roguisharcanetrickster roguisharcanetrickster added the minor Tag Pull Requests to trigger a minor version update label Jun 8, 2026
@roguisharcanetrickster roguisharcanetrickster self-assigned this Jun 8, 2026
@github-actions

github-actions Bot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Please add one release label:
major (for breaking changes), minor (for new features), patch (for bug fixes) or skip-release (to skip the auto release process).

Comment thread test/rootPages/Designer/ui_work_datacollection.test.js Fixed
roguisharcanetrickster and others added 2 commits June 16, 2026 13:30
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>

@benthongtiang benthongtiang left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@johnny-hausman johnny-hausman left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

OK!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

minor Tag Pull Requests to trigger a minor version update

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants