Skip to content

chore: migrate from karma-chai to direct chai imports (chai v6)#33

Merged
nikku merged 0 commit intorenovate/major-testfrom
copilot/sub-pr-26
Dec 22, 2025
Merged

chore: migrate from karma-chai to direct chai imports (chai v6)#33
nikku merged 0 commit intorenovate/major-testfrom
copilot/sub-pr-26

Conversation

Copy link

Copilot AI commented Dec 22, 2025

Updates test dependencies to major versions (chai ^4→^6, mocha ^10→^11) and migrates away from karma-chai plugin which is no longer needed with chai v6.

Changes

  • Removed karma-chai dependency - chai v6 drops support for the karma adapter
  • Created test/globals.js - exports chai's expect as global for backward compatibility
  • Updated karma.conf.js - removed 'chai' framework, added webpack rule to mark globals.js with sideEffects: true
  • Updated test/testBundle.js - imports globals to initialize test environment

Test files remain unchanged - expect continues to work as a global variable.

// test/globals.js
import { expect } from 'chai';
global.expect = expect;

Mirrors the migration pattern from bpmn-io/bpmn-js-token-simulation#176.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@bpmn-io-tasks bpmn-io-tasks bot added the in progress Currently worked on label Dec 22, 2025
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copy link
Author

Copilot AI commented Dec 22, 2025

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • googlechromelabs.github.io
    • Triggering command: /usr/local/bin/node node install.mjs (dns block)
  • https://storage.googleapis.com/chrome-for-testing-public/131.0.6778.264/linux64/chrome-headless-shell-linux64.zip
    • Triggering command: /usr/local/bin/node node install.mjs (http block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI changed the title [WIP] Update test dependencies to major versions chore: migrate from karma-chai to direct chai imports (chai v6) Dec 22, 2025
Copilot AI requested a review from nikku December 22, 2025 22:40
@nikku nikku force-pushed the copilot/sub-pr-26 branch from 67002b5 to 7742096 Compare December 22, 2025 22:42
@nikku nikku merged commit 7742096 into renovate/major-test Dec 22, 2025
1 check passed
@nikku nikku force-pushed the renovate/major-test branch from c8326eb to 7742096 Compare December 22, 2025 22:43
@nikku nikku deleted the copilot/sub-pr-26 branch December 22, 2025 22:43
@bpmn-io-tasks bpmn-io-tasks bot removed the in progress Currently worked on label Dec 22, 2025
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