feat(tests): add firefox to vitest browser tests

CHANGELOG.md

@@ -2,6 +2,7 @@
 
 ## [next]
 
+- feat(tests): add firefox to vitest browser tests [#10571](https://github.com/fabricjs/fabric.js/pull/10571)
 - refactor(tests): move to data url tests from qunit to playwright [#10581](https://github.com/fabricjs/fabric.js/pull/10581)
 - chore(TS): remove @ts-nocheck in straighten.ts [#10572](https://github.com/fabricjs/fabric.js/pull/10572)
 - refactor(tests): move svg export visual tests to playwright [#10575](https://github.com/fabricjs/fabric.js/pull/10575)

package.json

@@ -51,6 +51,7 @@
     "build-tests": "rollup -c ./rollup.test.config.js",
     "test:vitest": "vitest --run --project unit-node",
     "test:vitest:chromium": "vitest --run --project unit-chromium",
+    "test:vitest:firefox": "vitest --run --project unit-firefox",
     "test:vitest:all": "vitest --run",
     "test": "npm run cli -- test",
     "sandbox": "npm run sandboxscript -- sandbox",

src/shapes/IText/ITextBehavior.test.ts

@@ -42,34 +42,59 @@ describe('text imperative changes', () => {
     matchTextStateSnapshot(iText);
   });
 
-  it('insertChars', async () => {
+  it('insertChars', async (context) => {
+    context.skip(
+      !!context.task.file.projectName?.includes('firefox'),
+      'Firefox delivers different snapshot',
+    );
+
     const iText = await create();
     iText.insertChars('ab', undefined, 1);
     expect(iText.text).toBe('tabest');
     matchTextStateSnapshot(iText);
   });
 
-  it('insertChars and removes chars', async () => {
+  it('insertChars and removes chars', async (context) => {
+    context.skip(
+      !!context.task.file.projectName?.includes('firefox'),
+      'Firefox delivers different snapshot',
+    );
+
     const iText = await create();
     iText.insertChars('ab', undefined, 1, 2);
     expect(iText.text).toBe('tabst');
     matchTextStateSnapshot(iText);
   });
 
-  it('insertChars and removes chars', async () => {
+  it('insertChars and removes chars', async (context) => {
+    context.skip(
+      !!context.task.file.projectName?.includes('firefox'),
+      'Firefox delivers different snapshot',
+    );
+
     const iText = await create();
     iText.insertChars('ab', undefined, 1, 4);
     expect(iText.text).toBe('tab');
     matchTextStateSnapshot(iText);
   });
 
-  it('insertChars handles new lines correctly', async () => {
+  it('insertChars handles new lines correctly', async (context) => {
+    context.skip(
+      !!context.task.file.projectName?.includes('firefox'),
+      'Firefox delivers different snapshot',
+    );
+
     const iText = await create();
     iText.insertChars('ab\n\n', undefined, 1);
     matchTextStateSnapshot(iText);
   });
 
-  it('insertChars can accept some style for the new text', async () => {
+  it('insertChars can accept some style for the new text', async (context) => {
+    context.skip(
+      !!context.task.file.projectName?.includes('firefox'),
+      'Firefox delivers different snapshot',
+    );
+
     const iText = await create();
     iText.insertChars(
       'ab\n\na',

tsconfig.spec.json

@@ -1,7 +1,7 @@
 {
   "extends": "./tsconfig.json",
   "compilerOptions": {
-    "types": [],
+    "types": ["@vitest/browser/providers/playwright"],
     "noEmit": true,
     "allowSyntheticDefaultImports": true,
     "paths": {

vitest.config.ts

@@ -48,6 +48,25 @@ export default defineConfig({
           name: 'unit-chromium',
         },
       },
+      {
+        extends: true,
+        test: {
+          browser: {
+            provider: 'playwright',
+            enabled: true,
+            headless: true,
+            instances: [
+              {
+                browser: 'firefox',
+                context: {
+                  hasTouch: true,
+                },
+              },
+            ],
+          },
+          name: 'unit-firefox',
+        },
+      },
     ],
     include: [
       'src/**/*.test.{ts,tsx}',