chore(shared): Migrate tests to vitest#7014

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.

Sign up for GitHub

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

Merged
nikosdouvlis merged 3 commits into from
Oct 17, 2025
Merged

chore(shared): Migrate tests to vitest #7014

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
acad25b
chore(shared): Migrate tests to vitest
nikosdouvlis Oct 16, 2025
e70ddad
fix cicd
nikosdouvlis Oct 16, 2025
5fa2f97
Create breezy-flowers-float.md
nikosdouvlis Oct 16, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions .changeset/breezy-flowers-float.md
View file
Open in desktop
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
---
---
9 changes: 0 additions & 9 deletions packages/shared/customJSDOMEnvironment.ts
View file
Open in desktop

This file was deleted.

28 changes: 0 additions & 28 deletions packages/shared/jest.config.js
View file
Open in desktop

This file was deleted.

45 changes: 0 additions & 45 deletions packages/shared/jest.setup.ts
View file
Open in desktop

This file was deleted.

7 changes: 3 additions & 4 deletions packages/shared/package.json
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -148,10 +148,9 @@
"lint:attw": "attw --pack . --profile node16",
"lint:publint": "publint",
"publish:local": "pnpm yalc push --replace --sig",
"test": "jest && vitest",
"test:cache:clear": "jest --clearCache --useStderr",
"test:ci": "jest --maxWorkers=70%",
"test:coverage": "jest --collectCoverage && open coverage/lcov-report/index.html"
"test": "vitest",
"test:ci": "vitest --maxWorkers=70%",
"test:coverage": "vitest --collectCoverage && open coverage/lcov-report/index.html"
},
"dependencies": {
"@clerk/types": "workspace:^",
Expand Down
2 changes: 2 additions & 0 deletions ..._tests__/apiUrlFromPublishableKey.test.ts → ..._tests__/apiUrlFromPublishableKey.spec.ts
View file
Open in desktop
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
import { describe, expect, test } from 'vitest';

import { apiUrlFromPublishableKey } from '../apiUrlFromPublishableKey';

describe('apiUrlFromPublishableKey', () => {
Expand Down
45 changes: 35 additions & 10 deletions ...ages/shared/src/__tests__/browser.test.ts → ...ages/shared/src/__tests__/browser.spec.ts
View file
Open in desktop
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,17 @@
import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';

import { inBrowser, isValidBrowser, isValidBrowserOnline, userAgentIsRobot } from '../browser';

describe('inBrowser()', () => {
afterEach(() => {
jest.restoreAllMocks();
vi.restoreAllMocks();
});

it('returns true if window is defined', () => {
expect(inBrowser()).toBe(true);
});
it('returns false if window is undefined', () => {
const windowSpy = jest.spyOn(global, 'window', 'get');
const windowSpy = vi.spyOn(global, 'window', 'get');
// @ts-ignore - Test
windowSpy.mockReturnValue(undefined);
expect(inBrowser()).toBe(false);
Expand All@@ -21,16 +23,23 @@ describe('isValidBrowser', () => {
let webdriverGetter: any;

beforeEach(() => {
userAgentGetter = jest.spyOn(window.navigator, 'userAgent', 'get');
webdriverGetter = jest.spyOn(window.navigator, 'webdriver', 'get');
userAgentGetter = vi.spyOn(window.navigator, 'userAgent', 'get');
// Define webdriver property if it doesn't exist
if (!('webdriver' in window.navigator)) {
Object.defineProperty(window.navigator, 'webdriver', {
configurable: true,
get: () => false,
});
}
webdriverGetter = vi.spyOn(window.navigator, 'webdriver', 'get');
Comment on lines +27 to +34

@coderabbitaicoderabbitaiBotOct 16, 2025

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.

🛠️ Refactor suggestion | 🟠 Major

🧩 Analysis chain

Good defensive coding for JSDOM compatibility.

The code checks if the webdriver property exists on window.navigator before defining it. This suggests that Vitest's JSDOM environment may not include this property by default, unlike Jest's environment.

However, verify whether this defensive check is necessary. If Vitest consistently lacks this property, consider defining it unconditionally in the global test setup (vitest.setup.mts) rather than in individual test files to reduce duplication.

Run the following script to check if this pattern appears in other test files:

🏁 Script executed:

#!/bin/bash# Search for similar defensive property definitions in test files
rg -A 5 "if \(!'.*' in window\.navigator\)" --type ts --glob '**/__tests__/**'

Length of output: 0

I need to try different search approaches since the regex pattern didn't match. Let me search more broadly:


---
🏁 Script executed:
```shell
#!/bin/bash
# Search for webdriver property usage in test files
rg -n "webdriver" --type ts --glob '**/__tests__/**' -B 2 -A 2

Length of output: 9467

🏁 Script executed:

#!/bin/bash# Search for Object.defineProperty usage in test files
rg -n "Object\.defineProperty" --type ts --glob '**/__tests__/**' -B 2 -A 2

Length of output: 39064

🏁 Script executed:

#!/bin/bash# Look for vitest setup files
fd -e mts -e ts --glob '**/vitest*' -o --glob '**/setup*'2>/dev/null | head -20

Length of output: 0

Consider extracting the repeated webdriver property setup into a helper function to avoid duplication.

The defensive webdriver property check appears twice in the same file:

  • Lines 28–33 (in isValidBrowser describe block)
  • Lines 113–118 (in isValidBrowserOnline describe block)

Since the logic is identical and JSDOM appears to consistently lack this property in both test suites, extract this into a shared setup helper to reduce duplication and improve maintainability:

constsetupWebdriverProperty=()=>{if(!('webdriver'inwindow.navigator)){Object.defineProperty(window.navigator,'webdriver',{configurable: true,get: ()=>false,});}};

Then call setupWebdriverProperty() in each beforeEach instead of repeating the full block.

🤖 Prompt for AI Agents
In packages/shared/src/__tests__/browser.spec.ts around lines 27 to 34, the
defensive webdriver property check is duplicated elsewhere; extract the repeated
block into a small helper function (e.g., setupWebdriverProperty) that checks
for 'webdriver' on window.navigator and defines it when missing, then call that
helper from each beforeEach instead of repeating the Object.defineProperty
block; update imports/locals as needed and ensure the vi.spyOn call still
references window.navigator.webdriver after the helper runs.

});

afterEach(() => {
jest.restoreAllMocks();
vi.restoreAllMocks();
});

it('returns false if not in browser', () => {
const windowSpy = jest.spyOn(global, 'window', 'get');
const windowSpy = vi.spyOn(global, 'window', 'get');
// @ts-ignore - Test
windowSpy.mockReturnValue(undefined);

Expand DownExpand Up@@ -99,11 +108,27 @@ describe('isValidBrowserOnline', () => {
let connectionGetter: any;

beforeEach(() => {
userAgentGetter = jest.spyOn(window.navigator, 'userAgent', 'get');
webdriverGetter = jest.spyOn(window.navigator, 'webdriver', 'get');
onLineGetter = jest.spyOn(window.navigator, 'onLine', 'get');
userAgentGetter = vi.spyOn(window.navigator, 'userAgent', 'get');
// Define webdriver property if it doesn't exist
if (!('webdriver' in window.navigator)) {
Object.defineProperty(window.navigator, 'webdriver', {
configurable: true,
get: () => false,
});
}
webdriverGetter = vi.spyOn(window.navigator, 'webdriver', 'get');
onLineGetter = vi.spyOn(window.navigator, 'onLine', 'get');
// Define connection property if it doesn't exist
// @ts-ignore
if (!('connection' in window.navigator)) {
// @ts-ignore
Object.defineProperty(window.navigator, 'connection', {
configurable: true,
get: () => ({ downlink: 10, rtt: 100 }),
});
}
// @ts-ignore
connectionGetter = jest.spyOn(window.navigator, 'connection', 'get');
connectionGetter = vi.spyOn(window.navigator, 'connection', 'get');
});

it('returns TRUE if connection is online, navigator is online, has disabled webdriver, and not a bot', () => {
Expand Down
2 changes: 2 additions & 0 deletions ...rc/__tests__/buildAccountsBaseUrl.test.ts → ...rc/__tests__/buildAccountsBaseUrl.spec.ts
View file
Open in desktop
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
import { expect, test } from 'vitest';

import { buildAccountsBaseUrl } from '../buildAccountsBaseUrl';

test.each([
Expand Down
72 changes: 72 additions & 0 deletions packages/shared/src/__tests__/color.spec.ts
View file
Open in desktop
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,72 @@
import type { Color } from '@clerk/types';
import { describe, expect, it } from 'vitest';

import { colorToSameTypeString, hexStringToRgbaColor, stringToHslaColor, stringToSameTypeColor } from '../color';

describe('stringToHslaColor(color)', function () {
const hsla = { h: 195, s: 1, l: 0.5 };
const cases: Array<[string, Color | null]> = [
['', null],
['transparent', { h: 0, s: 0, l: 0, a: 0 }],
['#00bfff', hsla],
['00bfff', hsla],
['rgb(0, 191, 255)', hsla],
['rgba(0, 191, 255, 0.3)', { ...hsla, a: 0.3 }],
];

it.each(cases)('.stringToHslaColor(%s) => %s', (a, expected) => {
expect(stringToHslaColor(a)).toEqual(expected);
});
});

describe('hexStringToRgbaColor(color)', function () {
const cases: Array<[string, Color | null]> = [
['#00bfff', { r: 0, g: 191, b: 255 }],
['00bfff', { r: 0, g: 191, b: 255 }],
];

it.each(cases)('.hexStringToRgbaColor(%s) => %s', (a, expected) => {
expect(hexStringToRgbaColor(a)).toEqual(expected);
});
});

describe('stringToSameTypeColor(color)', function () {
const cases: Array<[string, Color | null]> = [
['', ''],
['invalid', ''],
['12ff12', '#12ff12'],
['#12ff12', '#12ff12'],
['1ff', '#1ff'],
['transparent', 'transparent'],
['rgb(100,100,100)', { r: 100, g: 100, b: 100, a: undefined }],
['rgba(100,100,100,0.5)', { r: 100, g: 100, b: 100, a: 0.5 }],
['rgb(100,100,100)', { r: 100, g: 100, b: 100, a: undefined }],
['rgba(100,100,100,0.5)', { r: 100, g: 100, b: 100, a: 0.5 }],
Comment on lines +41 to +44

@coderabbitaicoderabbitaiBotOct 17, 2025

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.

⚠️ Potential issue | 🟡 Minor

Remove duplicate test cases.

Lines 43-44 are exact duplicates of lines 41-42 (testing the same rgb/rgba inputs). Remove the redundant entries.

Apply this diff:

 ['rgb(100,100,100)', { r: 100, g: 100, b: 100, a: undefined }],
['rgba(100,100,100,0.5)', { r: 100, g: 100, b: 100, a: 0.5 }],
- ['rgb(100,100,100)', { r: 100, g: 100, b: 100, a: undefined }],- ['rgba(100,100,100,0.5)', { r: 100, g: 100, b: 100, a: 0.5 }],
['hsl(244,66%,33%)', { h: 244, s: 0.66, l: 0.33, a: undefined }],
📝 Committable suggestion

‼️IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
['rgb(100,100,100)',{r: 100,g: 100,b: 100,a: undefined}],
['rgba(100,100,100,0.5)',{r: 100,g: 100,b: 100,a: 0.5}],
['rgb(100,100,100)',{r: 100,g: 100,b: 100,a: undefined}],
['rgba(100,100,100,0.5)',{r: 100,g: 100,b: 100,a: 0.5}],
['rgb(100,100,100)',{r: 100,g: 100,b: 100,a: undefined}],
['rgba(100,100,100,0.5)',{r: 100,g: 100,b: 100,a: 0.5}],
🤖 Prompt for AI Agents
In packages/shared/src/__tests__/color.spec.ts around lines 41 to 44 there are
duplicated test entries for 'rgb(100,100,100)' and 'rgba(100,100,100,0.5)';
remove the redundant lines (the second occurrence on lines 43-44) so each input
case appears only once in the test array.

['hsl(244,66%,33%)', { h: 244, s: 0.66, l: 0.33, a: undefined }],
['hsla(244,66%,33%,0.5)', { h: 244, s: 0.66, l: 0.33, a: 0.5 }],
['hsl(244,66%,33)', ''],
['hsla(244,66%,33,0.5)', ''],
];

it.each(cases)('.stringToSameTypeColor(%s) => %s', (a, expected) => {
expect(stringToSameTypeColor(a)).toEqual(expected);
});
});

describe('colorToSameTypeString(color)', function () {
const cases: Array<[Color, string]> = [
['', ''],
['invalid', ''],
['#12ff12', '#12ff12'],
['#12ff12', '#12ff12'],
['#1ff', '#1ff'],
Comment on lines +57 to +62

@coderabbitaicoderabbitaiBotOct 17, 2025

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.

⚠️ Potential issue | 🟠 Major

Fix type annotation and remove duplicate test case.

Two issues:

  1. Type mismatch: The type annotation Array<[Color, string]> indicates the first element should be a Color object, but lines 58-62 use string literals. If colorToSameTypeString accepts both Color objects and strings, update the type annotation to reflect this (e.g., Array<[Color | string, string]>).

  2. Duplicate test case: Lines 60-61 both test '#12ff12'.

Apply this diff to fix the type annotation and remove the duplicate:

- const cases: Array<[Color, string]> = [+ const cases: Array<[Color | string, string]> = [
['', ''],
['invalid', ''],
['#12ff12', '#12ff12'],
- ['#12ff12', '#12ff12'],
['#1ff', '#1ff'],
📝 Committable suggestion

‼️IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
constcases: Array<[Color,string]>=[
['',''],
['invalid',''],
['#12ff12','#12ff12'],
['#12ff12','#12ff12'],
['#1ff','#1ff'],
constcases: Array<[Color|string,string]>=[
['',''],
['invalid',''],
['#12ff12','#12ff12'],
['#1ff','#1ff'],
🤖 Prompt for AI Agents
In packages/shared/src/__tests__/color.spec.ts around lines 57 to 62, the test
cases array is annotated as Array<[Color, string]> but uses string literals and
contains a duplicate '#12ff12' entry; update the type annotation to Array<[Color
| string, string]> (or Array<[string | Color, string]>) so strings are allowed,
and remove the duplicate '#12ff12' test case so each case is unique.

[{ r: 100, g: 100, b: 100, a: undefined }, 'rgb(100,100,100)'],
[{ r: 100, g: 100, b: 100, a: 0.5 }, 'rgba(100,100,100,0.5)'],
[{ h: 100, s: 0.55, l: 0.33, a: undefined }, 'hsl(100,55%,33%)'],
[{ h: 100, s: 1, l: 1, a: 0.5 }, 'hsla(100,100%,100%,0.5)'],
];

it.each(cases)('.colorToSameTypeString(%s) => %s', (a, expected) => {
expect(colorToSameTypeString(a)).toEqual(expected);
});
});
0 packages/shared/src/__tests__/color.test.ts → ...es/shared/src/__tests__/color.test.ts.bak
View file
Open in desktop
File renamed without changes.
2 changes: 2 additions & 0 deletions packages/shared/src/__tests__/date.test.ts → packages/shared/src/__tests__/date.spec.ts
View file
Open in desktop
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
import { describe, expect, it } from 'vitest';

import type { RelativeDateCase } from '../date';
import { addYears, dateTo12HourTime, differenceInCalendarDays, formatRelative } from '../date';

Expand Down
Loading
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all 
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '

Skip to content
chore(shared): Migrate tests to vitest#7014

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.

 Sign up for GitHub

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

 Merged

nikosdouvlis
merged 3 commits into

from

Oct 17, 2025
 Merged


chore(shared): Migrate tests to vitest
#7014
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


3 commits


Select commit
Hold shift + click to select a range
acad25b

chore(shared): Migrate tests to vitest


nikosdouvlis Oct 16, 2025
e70ddad

fix cicd


nikosdouvlis Oct 16, 2025
5fa2f97

Create breezy-flowers-float.md


nikosdouvlis Oct 16, 2025

File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


2 changes: 2 additions & 0 deletions
.changeset/breezy-flowers-float.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
---
---

9 changes: 0 additions & 9 deletions
packages/shared/customJSDOMEnvironment.ts

View file


Open in desktop


This file was deleted.


28 changes: 0 additions & 28 deletions
packages/shared/jest.config.js

View file


Open in desktop


This file was deleted.


45 changes: 0 additions & 45 deletions
packages/shared/jest.setup.ts

View file


Open in desktop


This file was deleted.


7 changes: 3 additions & 4 deletions
packages/shared/package.json

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -148,10 +148,9 @@
 "lint:attw": "attw --pack . --profile node16",
 "lint:publint": "publint",
 "publish:local": "pnpm yalc push --replace --sig",
 "test": "jest && vitest",
 "test:cache:clear": "jest --clearCache --useStderr",
 "test:ci": "jest --maxWorkers=70%",
 "test:coverage": "jest --collectCoverage && open coverage/lcov-report/index.html"
 "test": "vitest",
 "test:ci": "vitest --maxWorkers=70%",
 "test:coverage": "vitest --collectCoverage && open coverage/lcov-report/index.html"
 },
 "dependencies": {
 "@clerk/types": "workspace:^",
Expand Down

2 changes: 2 additions & 0 deletions
..._tests__/apiUrlFromPublishableKey.test.ts → ..._tests__/apiUrlFromPublishableKey.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
import { describe, expect, test } from 'vitest';

import { apiUrlFromPublishableKey } from '../apiUrlFromPublishableKey';

describe('apiUrlFromPublishableKey', () => {
Expand Down

45 changes: 35 additions & 10 deletions
...ages/shared/src/__tests__/browser.test.ts → ...ages/shared/src/__tests__/browser.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,15 +1,17 @@
import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';

import { inBrowser, isValidBrowser, isValidBrowserOnline, userAgentIsRobot } from '../browser';

describe('inBrowser()', () => {
 afterEach(() => {
jest.restoreAllMocks();
vi.restoreAllMocks();
 });

 it('returns true if window is defined', () => {
 expect(inBrowser()).toBe(true);
 });
 it('returns false if window is undefined', () => {
 const windowSpy = jest.spyOn(global, 'window', 'get');
 const windowSpy = vi.spyOn(global, 'window', 'get');
 // @ts-ignore - Test
 windowSpy.mockReturnValue(undefined);
 expect(inBrowser()).toBe(false);
Expand All@@ -21,16 +23,23 @@ describe('isValidBrowser', () => {
 let webdriverGetter: any;

 beforeEach(() => {
 userAgentGetter = jest.spyOn(window.navigator, 'userAgent', 'get');
 webdriverGetter = jest.spyOn(window.navigator, 'webdriver', 'get');
 userAgentGetter = vi.spyOn(window.navigator, 'userAgent', 'get');
 // Define webdriver property if it doesn't exist
 if (!('webdriver' in window.navigator)) {
 Object.defineProperty(window.navigator, 'webdriver', {
 configurable: true,
 get: () => false,
 });
 }
 webdriverGetter = vi.spyOn(window.navigator, 'webdriver', 'get');

Comment on lines

+27
 to 
+34

@coderabbitaicoderabbitaiBotOct 16, 2025

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.

🛠️ Refactor suggestion | 🟠 Major
🧩 Analysis chain
Good defensive coding for JSDOM compatibility.
The code checks if the webdriver property exists on window.navigator before defining it. This suggests that Vitest's JSDOM environment may not include this property by default, unlike Jest's environment.
However, verify whether this defensive check is necessary. If Vitest consistently lacks this property, consider defining it unconditionally in the global test setup (vitest.setup.mts) rather than in individual test files to reduce duplication.
Run the following script to check if this pattern appears in other test files:
🏁 Script executed:
#!/bin/bash# Search for similar defensive property definitions in test files
rg -A 5 "if \(!'.*' in window\.navigator\)" --type ts --glob '**/__tests__/**'
Length of output: 0
I need to try different search approaches since the regex pattern didn't match. Let me search more broadly:

---
🏁 Script executed:
```shell
#!/bin/bash
# Search for webdriver property usage in test files
rg -n "webdriver" --type ts --glob '**/__tests__/**' -B 2 -A 2

Length of output: 9467
🏁 Script executed:
#!/bin/bash# Search for Object.defineProperty usage in test files
rg -n "Object\.defineProperty" --type ts --glob '**/__tests__/**' -B 2 -A 2
Length of output: 39064
🏁 Script executed:
#!/bin/bash# Look for vitest setup files
fd -e mts -e ts --glob '**/vitest*' -o --glob '**/setup*'2>/dev/null | head -20
Length of output: 0
Consider extracting the repeated webdriver property setup into a helper function to avoid duplication.
The defensive webdriver property check appears twice in the same file:
  • Lines 28–33 (in isValidBrowser describe block)
  • Lines 113–118 (in isValidBrowserOnline describe block)
Since the logic is identical and JSDOM appears to consistently lack this property in both test suites, extract this into a shared setup helper to reduce duplication and improve maintainability:
constsetupWebdriverProperty=()=>{if(!('webdriver'inwindow.navigator)){Object.defineProperty(window.navigator,'webdriver',{configurable: true,get: ()=>false,});}};
Then call setupWebdriverProperty() in each beforeEach instead of repeating the full block.
🤖 Prompt for AI Agents
In packages/shared/src/__tests__/browser.spec.ts around lines 27 to 34, the
defensive webdriver property check is duplicated elsewhere; extract the repeated
block into a small helper function (e.g., setupWebdriverProperty) that checks
for 'webdriver' on window.navigator and defines it when missing, then call that
helper from each beforeEach instead of repeating the Object.defineProperty
block; update imports/locals as needed and ensure the vi.spyOn call still
references window.navigator.webdriver after the helper runs.

 });

 afterEach(() => {
jest.restoreAllMocks();
vi.restoreAllMocks();
 });

 it('returns false if not in browser', () => {
 const windowSpy = jest.spyOn(global, 'window', 'get');
 const windowSpy = vi.spyOn(global, 'window', 'get');
 // @ts-ignore - Test
 windowSpy.mockReturnValue(undefined);

Expand DownExpand Up@@ -99,11 +108,27 @@ describe('isValidBrowserOnline', () => {
 let connectionGetter: any;

 beforeEach(() => {
 userAgentGetter = jest.spyOn(window.navigator, 'userAgent', 'get');
 webdriverGetter = jest.spyOn(window.navigator, 'webdriver', 'get');
 onLineGetter = jest.spyOn(window.navigator, 'onLine', 'get');
 userAgentGetter = vi.spyOn(window.navigator, 'userAgent', 'get');
 // Define webdriver property if it doesn't exist
 if (!('webdriver' in window.navigator)) {
 Object.defineProperty(window.navigator, 'webdriver', {
 configurable: true,
 get: () => false,
 });
 }
 webdriverGetter = vi.spyOn(window.navigator, 'webdriver', 'get');
 onLineGetter = vi.spyOn(window.navigator, 'onLine', 'get');
 // Define connection property if it doesn't exist
 // @ts-ignore
 if (!('connection' in window.navigator)) {
 // @ts-ignore
 Object.defineProperty(window.navigator, 'connection', {
 configurable: true,
 get: () => ({ downlink: 10, rtt: 100 }),
 });
 }
 // @ts-ignore
 connectionGetter = jest.spyOn(window.navigator, 'connection', 'get');
 connectionGetter = vi.spyOn(window.navigator, 'connection', 'get');
 });

 it('returns TRUE if connection is online, navigator is online, has disabled webdriver, and not a bot', () => {
Expand Down

2 changes: 2 additions & 0 deletions
...rc/__tests__/buildAccountsBaseUrl.test.ts → ...rc/__tests__/buildAccountsBaseUrl.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
import { expect, test } from 'vitest';

import { buildAccountsBaseUrl } from '../buildAccountsBaseUrl';

test.each([
Expand Down

72 changes: 72 additions & 0 deletions
packages/shared/src/__tests__/color.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,72 @@
import type { Color } from '@clerk/types';
import { describe, expect, it } from 'vitest';

import { colorToSameTypeString, hexStringToRgbaColor, stringToHslaColor, stringToSameTypeColor } from '../color';

describe('stringToHslaColor(color)', function () {
 const hsla = { h: 195, s: 1, l: 0.5 };
 const cases: Array<[string, Color | null]> = [
 ['', null],
 ['transparent', { h: 0, s: 0, l: 0, a: 0 }],
 ['#00bfff', hsla],
 ['00bfff', hsla],
 ['rgb(0, 191, 255)', hsla],
 ['rgba(0, 191, 255, 0.3)', { ...hsla, a: 0.3 }],
 ];

 it.each(cases)('.stringToHslaColor(%s) => %s', (a, expected) => {
 expect(stringToHslaColor(a)).toEqual(expected);
 });
});

describe('hexStringToRgbaColor(color)', function () {
 const cases: Array<[string, Color | null]> = [
 ['#00bfff', { r: 0, g: 191, b: 255 }],
 ['00bfff', { r: 0, g: 191, b: 255 }],
 ];

 it.each(cases)('.hexStringToRgbaColor(%s) => %s', (a, expected) => {
 expect(hexStringToRgbaColor(a)).toEqual(expected);
 });
});

describe('stringToSameTypeColor(color)', function () {
 const cases: Array<[string, Color | null]> = [
 ['', ''],
 ['invalid', ''],
 ['12ff12', '#12ff12'],
 ['#12ff12', '#12ff12'],
 ['1ff', '#1ff'],
 ['transparent', 'transparent'],
 ['rgb(100,100,100)', { r: 100, g: 100, b: 100, a: undefined }],
 ['rgba(100,100,100,0.5)', { r: 100, g: 100, b: 100, a: 0.5 }],
 ['rgb(100,100,100)', { r: 100, g: 100, b: 100, a: undefined }],
 ['rgba(100,100,100,0.5)', { r: 100, g: 100, b: 100, a: 0.5 }],

Comment on lines

+41
 to 
+44

@coderabbitaicoderabbitaiBotOct 17, 2025

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.

⚠️ Potential issue | 🟡 Minor
Remove duplicate test cases.
Lines 43-44 are exact duplicates of lines 41-42 (testing the same rgb/rgba inputs). Remove the redundant entries.
Apply this diff:
 ['rgb(100,100,100)', { r: 100, g: 100, b: 100, a: undefined }],
['rgba(100,100,100,0.5)', { r: 100, g: 100, b: 100, a: 0.5 }],
- ['rgb(100,100,100)', { r: 100, g: 100, b: 100, a: undefined }],- ['rgba(100,100,100,0.5)', { r: 100, g: 100, b: 100, a: 0.5 }],
['hsl(244,66%,33%)', { h: 244, s: 0.66, l: 0.33, a: undefined }],
📝 Committable suggestion
‼️IMPORTANT

Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

['rgb(100,100,100)',{r: 100,g: 100,b: 100,a: undefined}],
['rgba(100,100,100,0.5)',{r: 100,g: 100,b: 100,a: 0.5}],
['rgb(100,100,100)',{r: 100,g: 100,b: 100,a: undefined}],
['rgba(100,100,100,0.5)',{r: 100,g: 100,b: 100,a: 0.5}],
['rgb(100,100,100)',{r: 100,g: 100,b: 100,a: undefined}],
['rgba(100,100,100,0.5)',{r: 100,g: 100,b: 100,a: 0.5}],
🤖 Prompt for AI Agents
In packages/shared/src/__tests__/color.spec.ts around lines 41 to 44 there are
duplicated test entries for 'rgb(100,100,100)' and 'rgba(100,100,100,0.5)';
remove the redundant lines (the second occurrence on lines 43-44) so each input
case appears only once in the test array.

 ['hsl(244,66%,33%)', { h: 244, s: 0.66, l: 0.33, a: undefined }],
 ['hsla(244,66%,33%,0.5)', { h: 244, s: 0.66, l: 0.33, a: 0.5 }],
 ['hsl(244,66%,33)', ''],
 ['hsla(244,66%,33,0.5)', ''],
 ];

 it.each(cases)('.stringToSameTypeColor(%s) => %s', (a, expected) => {
 expect(stringToSameTypeColor(a)).toEqual(expected);
 });
});

describe('colorToSameTypeString(color)', function () {
 const cases: Array<[Color, string]> = [
 ['', ''],
 ['invalid', ''],
 ['#12ff12', '#12ff12'],
 ['#12ff12', '#12ff12'],
 ['#1ff', '#1ff'],

Comment on lines

+57
 to 
+62

@coderabbitaicoderabbitaiBotOct 17, 2025

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.

⚠️ Potential issue | 🟠 Major
Fix type annotation and remove duplicate test case.
Two issues:
  1. Type mismatch: The type annotation Array<[Color, string]> indicates the first element should be a Color object, but lines 58-62 use string literals. If colorToSameTypeString accepts both Color objects and strings, update the type annotation to reflect this (e.g., Array<[Color | string, string]>).
  2. Duplicate test case: Lines 60-61 both test '#12ff12'.
Apply this diff to fix the type annotation and remove the duplicate:
- const cases: Array<[Color, string]> = [+ const cases: Array<[Color | string, string]> = [
['', ''],
['invalid', ''],
['#12ff12', '#12ff12'],
- ['#12ff12', '#12ff12'],
['#1ff', '#1ff'],
📝 Committable suggestion
‼️IMPORTANT

Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

constcases: Array<[Color,string]>=[
['',''],
['invalid',''],
['#12ff12','#12ff12'],
['#12ff12','#12ff12'],
['#1ff','#1ff'],
constcases: Array<[Color|string,string]>=[
['',''],
['invalid',''],
['#12ff12','#12ff12'],
['#1ff','#1ff'],
🤖 Prompt for AI Agents
In packages/shared/src/__tests__/color.spec.ts around lines 57 to 62, the test
cases array is annotated as Array<[Color, string]> but uses string literals and
contains a duplicate '#12ff12' entry; update the type annotation to Array<[Color
| string, string]> (or Array<[string | Color, string]>) so strings are allowed,
and remove the duplicate '#12ff12' test case so each case is unique.

 [{ r: 100, g: 100, b: 100, a: undefined }, 'rgb(100,100,100)'],
 [{ r: 100, g: 100, b: 100, a: 0.5 }, 'rgba(100,100,100,0.5)'],
 [{ h: 100, s: 0.55, l: 0.33, a: undefined }, 'hsl(100,55%,33%)'],
 [{ h: 100, s: 1, l: 1, a: 0.5 }, 'hsla(100,100%,100%,0.5)'],
 ];

 it.each(cases)('.colorToSameTypeString(%s) => %s', (a, expected) => {
 expect(colorToSameTypeString(a)).toEqual(expected);
 });
});

0
packages/shared/src/__tests__/color.test.ts → ...es/shared/src/__tests__/color.test.ts.bak

View file


Open in desktop


File renamed without changes.


2 changes: 2 additions & 0 deletions
packages/shared/src/__tests__/date.test.ts → packages/shared/src/__tests__/date.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
import { describe, expect, it } from 'vitest';

import type { RelativeDateCase } from '../date';
import { addYears, dateTo12HourTime, differenceInCalendarDays, formatRelative } from '../date';

Expand Down
Loading

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix");
}
} catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
chore(shared): Migrate tests to vitest#7014

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.

 Sign up for GitHub

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

 Merged

nikosdouvlis
merged 3 commits into

from

Oct 17, 2025
 Merged


chore(shared): Migrate tests to vitest
#7014
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


3 commits


Select commit
Hold shift + click to select a range
acad25b

chore(shared): Migrate tests to vitest


nikosdouvlis Oct 16, 2025
e70ddad

fix cicd


nikosdouvlis Oct 16, 2025
5fa2f97

Create breezy-flowers-float.md


nikosdouvlis Oct 16, 2025

File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


2 changes: 2 additions & 0 deletions
.changeset/breezy-flowers-float.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
---
---

9 changes: 0 additions & 9 deletions
packages/shared/customJSDOMEnvironment.ts

View file


Open in desktop


This file was deleted.


28 changes: 0 additions & 28 deletions
packages/shared/jest.config.js

View file


Open in desktop


This file was deleted.


45 changes: 0 additions & 45 deletions
packages/shared/jest.setup.ts

View file


Open in desktop


This file was deleted.


7 changes: 3 additions & 4 deletions
packages/shared/package.json

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -148,10 +148,9 @@
 "lint:attw": "attw --pack . --profile node16",
 "lint:publint": "publint",
 "publish:local": "pnpm yalc push --replace --sig",
 "test": "jest && vitest",
 "test:cache:clear": "jest --clearCache --useStderr",
 "test:ci": "jest --maxWorkers=70%",
 "test:coverage": "jest --collectCoverage && open coverage/lcov-report/index.html"
 "test": "vitest",
 "test:ci": "vitest --maxWorkers=70%",
 "test:coverage": "vitest --collectCoverage && open coverage/lcov-report/index.html"
 },
 "dependencies": {
 "@clerk/types": "workspace:^",
Expand Down

2 changes: 2 additions & 0 deletions
..._tests__/apiUrlFromPublishableKey.test.ts → ..._tests__/apiUrlFromPublishableKey.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
import { describe, expect, test } from 'vitest';

import { apiUrlFromPublishableKey } from '../apiUrlFromPublishableKey';

describe('apiUrlFromPublishableKey', () => {
Expand Down

45 changes: 35 additions & 10 deletions
...ages/shared/src/__tests__/browser.test.ts → ...ages/shared/src/__tests__/browser.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,15 +1,17 @@
import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';

import { inBrowser, isValidBrowser, isValidBrowserOnline, userAgentIsRobot } from '../browser';

describe('inBrowser()', () => {
 afterEach(() => {
jest.restoreAllMocks();
vi.restoreAllMocks();
 });

 it('returns true if window is defined', () => {
 expect(inBrowser()).toBe(true);
 });
 it('returns false if window is undefined', () => {
 const windowSpy = jest.spyOn(global, 'window', 'get');
 const windowSpy = vi.spyOn(global, 'window', 'get');
 // @ts-ignore - Test
 windowSpy.mockReturnValue(undefined);
 expect(inBrowser()).toBe(false);
Expand All@@ -21,16 +23,23 @@ describe('isValidBrowser', () => {
 let webdriverGetter: any;

 beforeEach(() => {
 userAgentGetter = jest.spyOn(window.navigator, 'userAgent', 'get');
 webdriverGetter = jest.spyOn(window.navigator, 'webdriver', 'get');
 userAgentGetter = vi.spyOn(window.navigator, 'userAgent', 'get');
 // Define webdriver property if it doesn't exist
 if (!('webdriver' in window.navigator)) {
 Object.defineProperty(window.navigator, 'webdriver', {
 configurable: true,
 get: () => false,
 });
 }
 webdriverGetter = vi.spyOn(window.navigator, 'webdriver', 'get');

Comment on lines

+27
 to 
+34

@coderabbitaicoderabbitaiBotOct 16, 2025

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.

🛠️ Refactor suggestion | 🟠 Major
🧩 Analysis chain
Good defensive coding for JSDOM compatibility.
The code checks if the webdriver property exists on window.navigator before defining it. This suggests that Vitest's JSDOM environment may not include this property by default, unlike Jest's environment.
However, verify whether this defensive check is necessary. If Vitest consistently lacks this property, consider defining it unconditionally in the global test setup (vitest.setup.mts) rather than in individual test files to reduce duplication.
Run the following script to check if this pattern appears in other test files:
🏁 Script executed:
#!/bin/bash# Search for similar defensive property definitions in test files
rg -A 5 "if \(!'.*' in window\.navigator\)" --type ts --glob '**/__tests__/**'
Length of output: 0
I need to try different search approaches since the regex pattern didn't match. Let me search more broadly:

---
🏁 Script executed:
```shell
#!/bin/bash
# Search for webdriver property usage in test files
rg -n "webdriver" --type ts --glob '**/__tests__/**' -B 2 -A 2

Length of output: 9467
🏁 Script executed:
#!/bin/bash# Search for Object.defineProperty usage in test files
rg -n "Object\.defineProperty" --type ts --glob '**/__tests__/**' -B 2 -A 2
Length of output: 39064
🏁 Script executed:
#!/bin/bash# Look for vitest setup files
fd -e mts -e ts --glob '**/vitest*' -o --glob '**/setup*'2>/dev/null | head -20
Length of output: 0
Consider extracting the repeated webdriver property setup into a helper function to avoid duplication.
The defensive webdriver property check appears twice in the same file:
  • Lines 28–33 (in isValidBrowser describe block)
  • Lines 113–118 (in isValidBrowserOnline describe block)
Since the logic is identical and JSDOM appears to consistently lack this property in both test suites, extract this into a shared setup helper to reduce duplication and improve maintainability:
constsetupWebdriverProperty=()=>{if(!('webdriver'inwindow.navigator)){Object.defineProperty(window.navigator,'webdriver',{configurable: true,get: ()=>false,});}};
Then call setupWebdriverProperty() in each beforeEach instead of repeating the full block.
🤖 Prompt for AI Agents
In packages/shared/src/__tests__/browser.spec.ts around lines 27 to 34, the
defensive webdriver property check is duplicated elsewhere; extract the repeated
block into a small helper function (e.g., setupWebdriverProperty) that checks
for 'webdriver' on window.navigator and defines it when missing, then call that
helper from each beforeEach instead of repeating the Object.defineProperty
block; update imports/locals as needed and ensure the vi.spyOn call still
references window.navigator.webdriver after the helper runs.

 });

 afterEach(() => {
jest.restoreAllMocks();
vi.restoreAllMocks();
 });

 it('returns false if not in browser', () => {
 const windowSpy = jest.spyOn(global, 'window', 'get');
 const windowSpy = vi.spyOn(global, 'window', 'get');
 // @ts-ignore - Test
 windowSpy.mockReturnValue(undefined);

Expand DownExpand Up@@ -99,11 +108,27 @@ describe('isValidBrowserOnline', () => {
 let connectionGetter: any;

 beforeEach(() => {
 userAgentGetter = jest.spyOn(window.navigator, 'userAgent', 'get');
 webdriverGetter = jest.spyOn(window.navigator, 'webdriver', 'get');
 onLineGetter = jest.spyOn(window.navigator, 'onLine', 'get');
 userAgentGetter = vi.spyOn(window.navigator, 'userAgent', 'get');
 // Define webdriver property if it doesn't exist
 if (!('webdriver' in window.navigator)) {
 Object.defineProperty(window.navigator, 'webdriver', {
 configurable: true,
 get: () => false,
 });
 }
 webdriverGetter = vi.spyOn(window.navigator, 'webdriver', 'get');
 onLineGetter = vi.spyOn(window.navigator, 'onLine', 'get');
 // Define connection property if it doesn't exist
 // @ts-ignore
 if (!('connection' in window.navigator)) {
 // @ts-ignore
 Object.defineProperty(window.navigator, 'connection', {
 configurable: true,
 get: () => ({ downlink: 10, rtt: 100 }),
 });
 }
 // @ts-ignore
 connectionGetter = jest.spyOn(window.navigator, 'connection', 'get');
 connectionGetter = vi.spyOn(window.navigator, 'connection', 'get');
 });

 it('returns TRUE if connection is online, navigator is online, has disabled webdriver, and not a bot', () => {
Expand Down

2 changes: 2 additions & 0 deletions
...rc/__tests__/buildAccountsBaseUrl.test.ts → ...rc/__tests__/buildAccountsBaseUrl.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
import { expect, test } from 'vitest';

import { buildAccountsBaseUrl } from '../buildAccountsBaseUrl';

test.each([
Expand Down

72 changes: 72 additions & 0 deletions
packages/shared/src/__tests__/color.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,72 @@
import type { Color } from '@clerk/types';
import { describe, expect, it } from 'vitest';

import { colorToSameTypeString, hexStringToRgbaColor, stringToHslaColor, stringToSameTypeColor } from '../color';

describe('stringToHslaColor(color)', function () {
 const hsla = { h: 195, s: 1, l: 0.5 };
 const cases: Array<[string, Color | null]> = [
 ['', null],
 ['transparent', { h: 0, s: 0, l: 0, a: 0 }],
 ['#00bfff', hsla],
 ['00bfff', hsla],
 ['rgb(0, 191, 255)', hsla],
 ['rgba(0, 191, 255, 0.3)', { ...hsla, a: 0.3 }],
 ];

 it.each(cases)('.stringToHslaColor(%s) => %s', (a, expected) => {
 expect(stringToHslaColor(a)).toEqual(expected);
 });
});

describe('hexStringToRgbaColor(color)', function () {
 const cases: Array<[string, Color | null]> = [
 ['#00bfff', { r: 0, g: 191, b: 255 }],
 ['00bfff', { r: 0, g: 191, b: 255 }],
 ];

 it.each(cases)('.hexStringToRgbaColor(%s) => %s', (a, expected) => {
 expect(hexStringToRgbaColor(a)).toEqual(expected);
 });
});

describe('stringToSameTypeColor(color)', function () {
 const cases: Array<[string, Color | null]> = [
 ['', ''],
 ['invalid', ''],
 ['12ff12', '#12ff12'],
 ['#12ff12', '#12ff12'],
 ['1ff', '#1ff'],
 ['transparent', 'transparent'],
 ['rgb(100,100,100)', { r: 100, g: 100, b: 100, a: undefined }],
 ['rgba(100,100,100,0.5)', { r: 100, g: 100, b: 100, a: 0.5 }],
 ['rgb(100,100,100)', { r: 100, g: 100, b: 100, a: undefined }],
 ['rgba(100,100,100,0.5)', { r: 100, g: 100, b: 100, a: 0.5 }],

Comment on lines

+41
 to 
+44

@coderabbitaicoderabbitaiBotOct 17, 2025

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.

⚠️ Potential issue | 🟡 Minor
Remove duplicate test cases.
Lines 43-44 are exact duplicates of lines 41-42 (testing the same rgb/rgba inputs). Remove the redundant entries.
Apply this diff:
 ['rgb(100,100,100)', { r: 100, g: 100, b: 100, a: undefined }],
['rgba(100,100,100,0.5)', { r: 100, g: 100, b: 100, a: 0.5 }],
- ['rgb(100,100,100)', { r: 100, g: 100, b: 100, a: undefined }],- ['rgba(100,100,100,0.5)', { r: 100, g: 100, b: 100, a: 0.5 }],
['hsl(244,66%,33%)', { h: 244, s: 0.66, l: 0.33, a: undefined }],
📝 Committable suggestion
‼️IMPORTANT

Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

['rgb(100,100,100)',{r: 100,g: 100,b: 100,a: undefined}],
['rgba(100,100,100,0.5)',{r: 100,g: 100,b: 100,a: 0.5}],
['rgb(100,100,100)',{r: 100,g: 100,b: 100,a: undefined}],
['rgba(100,100,100,0.5)',{r: 100,g: 100,b: 100,a: 0.5}],
['rgb(100,100,100)',{r: 100,g: 100,b: 100,a: undefined}],
['rgba(100,100,100,0.5)',{r: 100,g: 100,b: 100,a: 0.5}],
🤖 Prompt for AI Agents
In packages/shared/src/__tests__/color.spec.ts around lines 41 to 44 there are
duplicated test entries for 'rgb(100,100,100)' and 'rgba(100,100,100,0.5)';
remove the redundant lines (the second occurrence on lines 43-44) so each input
case appears only once in the test array.

 ['hsl(244,66%,33%)', { h: 244, s: 0.66, l: 0.33, a: undefined }],
 ['hsla(244,66%,33%,0.5)', { h: 244, s: 0.66, l: 0.33, a: 0.5 }],
 ['hsl(244,66%,33)', ''],
 ['hsla(244,66%,33,0.5)', ''],
 ];

 it.each(cases)('.stringToSameTypeColor(%s) => %s', (a, expected) => {
 expect(stringToSameTypeColor(a)).toEqual(expected);
 });
});

describe('colorToSameTypeString(color)', function () {
 const cases: Array<[Color, string]> = [
 ['', ''],
 ['invalid', ''],
 ['#12ff12', '#12ff12'],
 ['#12ff12', '#12ff12'],
 ['#1ff', '#1ff'],

Comment on lines

+57
 to 
+62

@coderabbitaicoderabbitaiBotOct 17, 2025

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.

⚠️ Potential issue | 🟠 Major
Fix type annotation and remove duplicate test case.
Two issues:
  1. Type mismatch: The type annotation Array<[Color, string]> indicates the first element should be a Color object, but lines 58-62 use string literals. If colorToSameTypeString accepts both Color objects and strings, update the type annotation to reflect this (e.g., Array<[Color | string, string]>).
  2. Duplicate test case: Lines 60-61 both test '#12ff12'.
Apply this diff to fix the type annotation and remove the duplicate:
- const cases: Array<[Color, string]> = [+ const cases: Array<[Color | string, string]> = [
['', ''],
['invalid', ''],
['#12ff12', '#12ff12'],
- ['#12ff12', '#12ff12'],
['#1ff', '#1ff'],
📝 Committable suggestion
‼️IMPORTANT

Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

constcases: Array<[Color,string]>=[
['',''],
['invalid',''],
['#12ff12','#12ff12'],
['#12ff12','#12ff12'],
['#1ff','#1ff'],
constcases: Array<[Color|string,string]>=[
['',''],
['invalid',''],
['#12ff12','#12ff12'],
['#1ff','#1ff'],
🤖 Prompt for AI Agents
In packages/shared/src/__tests__/color.spec.ts around lines 57 to 62, the test
cases array is annotated as Array<[Color, string]> but uses string literals and
contains a duplicate '#12ff12' entry; update the type annotation to Array<[Color
| string, string]> (or Array<[string | Color, string]>) so strings are allowed,
and remove the duplicate '#12ff12' test case so each case is unique.

 [{ r: 100, g: 100, b: 100, a: undefined }, 'rgb(100,100,100)'],
 [{ r: 100, g: 100, b: 100, a: 0.5 }, 'rgba(100,100,100,0.5)'],
 [{ h: 100, s: 0.55, l: 0.33, a: undefined }, 'hsl(100,55%,33%)'],
 [{ h: 100, s: 1, l: 1, a: 0.5 }, 'hsla(100,100%,100%,0.5)'],
 ];

 it.each(cases)('.colorToSameTypeString(%s) => %s', (a, expected) => {
 expect(colorToSameTypeString(a)).toEqual(expected);
 });
});

0
packages/shared/src/__tests__/color.test.ts → ...es/shared/src/__tests__/color.test.ts.bak

View file


Open in desktop


File renamed without changes.


2 changes: 2 additions & 0 deletions
packages/shared/src/__tests__/date.test.ts → packages/shared/src/__tests__/date.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
import { describe, expect, it } from 'vitest';

import type { RelativeDateCase } from '../date';
import { addYears, dateTo12HourTime, differenceInCalendarDays, formatRelative } from '../date';

Expand Down
Loading

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms");
}
} catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
chore(shared): Migrate tests to vitest#7014

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.

 Sign up for GitHub

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

 Merged

nikosdouvlis
merged 3 commits into

from

Oct 17, 2025
 Merged


chore(shared): Migrate tests to vitest
#7014
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


3 commits


Select commit
Hold shift + click to select a range
acad25b

chore(shared): Migrate tests to vitest


nikosdouvlis Oct 16, 2025
e70ddad

fix cicd


nikosdouvlis Oct 16, 2025
5fa2f97

Create breezy-flowers-float.md


nikosdouvlis Oct 16, 2025

File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


2 changes: 2 additions & 0 deletions
.changeset/breezy-flowers-float.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
---
---

9 changes: 0 additions & 9 deletions
packages/shared/customJSDOMEnvironment.ts

View file


Open in desktop


This file was deleted.


28 changes: 0 additions & 28 deletions
packages/shared/jest.config.js

View file


Open in desktop


This file was deleted.


45 changes: 0 additions & 45 deletions
packages/shared/jest.setup.ts

View file


Open in desktop


This file was deleted.


7 changes: 3 additions & 4 deletions
packages/shared/package.json

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -148,10 +148,9 @@
 "lint:attw": "attw --pack . --profile node16",
 "lint:publint": "publint",
 "publish:local": "pnpm yalc push --replace --sig",
 "test": "jest && vitest",
 "test:cache:clear": "jest --clearCache --useStderr",
 "test:ci": "jest --maxWorkers=70%",
 "test:coverage": "jest --collectCoverage && open coverage/lcov-report/index.html"
 "test": "vitest",
 "test:ci": "vitest --maxWorkers=70%",
 "test:coverage": "vitest --collectCoverage && open coverage/lcov-report/index.html"
 },
 "dependencies": {
 "@clerk/types": "workspace:^",
Expand Down

2 changes: 2 additions & 0 deletions
..._tests__/apiUrlFromPublishableKey.test.ts → ..._tests__/apiUrlFromPublishableKey.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
import { describe, expect, test } from 'vitest';

import { apiUrlFromPublishableKey } from '../apiUrlFromPublishableKey';

describe('apiUrlFromPublishableKey', () => {
Expand Down

45 changes: 35 additions & 10 deletions
...ages/shared/src/__tests__/browser.test.ts → ...ages/shared/src/__tests__/browser.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,15 +1,17 @@
import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';

import { inBrowser, isValidBrowser, isValidBrowserOnline, userAgentIsRobot } from '../browser';

describe('inBrowser()', () => {
 afterEach(() => {
jest.restoreAllMocks();
vi.restoreAllMocks();
 });

 it('returns true if window is defined', () => {
 expect(inBrowser()).toBe(true);
 });
 it('returns false if window is undefined', () => {
 const windowSpy = jest.spyOn(global, 'window', 'get');
 const windowSpy = vi.spyOn(global, 'window', 'get');
 // @ts-ignore - Test
 windowSpy.mockReturnValue(undefined);
 expect(inBrowser()).toBe(false);
Expand All@@ -21,16 +23,23 @@ describe('isValidBrowser', () => {
 let webdriverGetter: any;

 beforeEach(() => {
 userAgentGetter = jest.spyOn(window.navigator, 'userAgent', 'get');
 webdriverGetter = jest.spyOn(window.navigator, 'webdriver', 'get');
 userAgentGetter = vi.spyOn(window.navigator, 'userAgent', 'get');
 // Define webdriver property if it doesn't exist
 if (!('webdriver' in window.navigator)) {
 Object.defineProperty(window.navigator, 'webdriver', {
 configurable: true,
 get: () => false,
 });
 }
 webdriverGetter = vi.spyOn(window.navigator, 'webdriver', 'get');

Comment on lines

+27
 to 
+34

@coderabbitaicoderabbitaiBotOct 16, 2025

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.

🛠️ Refactor suggestion | 🟠 Major
🧩 Analysis chain
Good defensive coding for JSDOM compatibility.
The code checks if the webdriver property exists on window.navigator before defining it. This suggests that Vitest's JSDOM environment may not include this property by default, unlike Jest's environment.
However, verify whether this defensive check is necessary. If Vitest consistently lacks this property, consider defining it unconditionally in the global test setup (vitest.setup.mts) rather than in individual test files to reduce duplication.
Run the following script to check if this pattern appears in other test files:
🏁 Script executed:
#!/bin/bash# Search for similar defensive property definitions in test files
rg -A 5 "if \(!'.*' in window\.navigator\)" --type ts --glob '**/__tests__/**'
Length of output: 0
I need to try different search approaches since the regex pattern didn't match. Let me search more broadly:

---
🏁 Script executed:
```shell
#!/bin/bash
# Search for webdriver property usage in test files
rg -n "webdriver" --type ts --glob '**/__tests__/**' -B 2 -A 2

Length of output: 9467
🏁 Script executed:
#!/bin/bash# Search for Object.defineProperty usage in test files
rg -n "Object\.defineProperty" --type ts --glob '**/__tests__/**' -B 2 -A 2
Length of output: 39064
🏁 Script executed:
#!/bin/bash# Look for vitest setup files
fd -e mts -e ts --glob '**/vitest*' -o --glob '**/setup*'2>/dev/null | head -20
Length of output: 0
Consider extracting the repeated webdriver property setup into a helper function to avoid duplication.
The defensive webdriver property check appears twice in the same file:
  • Lines 28–33 (in isValidBrowser describe block)
  • Lines 113–118 (in isValidBrowserOnline describe block)
Since the logic is identical and JSDOM appears to consistently lack this property in both test suites, extract this into a shared setup helper to reduce duplication and improve maintainability:
constsetupWebdriverProperty=()=>{if(!('webdriver'inwindow.navigator)){Object.defineProperty(window.navigator,'webdriver',{configurable: true,get: ()=>false,});}};
Then call setupWebdriverProperty() in each beforeEach instead of repeating the full block.
🤖 Prompt for AI Agents
In packages/shared/src/__tests__/browser.spec.ts around lines 27 to 34, the
defensive webdriver property check is duplicated elsewhere; extract the repeated
block into a small helper function (e.g., setupWebdriverProperty) that checks
for 'webdriver' on window.navigator and defines it when missing, then call that
helper from each beforeEach instead of repeating the Object.defineProperty
block; update imports/locals as needed and ensure the vi.spyOn call still
references window.navigator.webdriver after the helper runs.

 });

 afterEach(() => {
jest.restoreAllMocks();
vi.restoreAllMocks();
 });

 it('returns false if not in browser', () => {
 const windowSpy = jest.spyOn(global, 'window', 'get');
 const windowSpy = vi.spyOn(global, 'window', 'get');
 // @ts-ignore - Test
 windowSpy.mockReturnValue(undefined);

Expand DownExpand Up@@ -99,11 +108,27 @@ describe('isValidBrowserOnline', () => {
 let connectionGetter: any;

 beforeEach(() => {
 userAgentGetter = jest.spyOn(window.navigator, 'userAgent', 'get');
 webdriverGetter = jest.spyOn(window.navigator, 'webdriver', 'get');
 onLineGetter = jest.spyOn(window.navigator, 'onLine', 'get');
 userAgentGetter = vi.spyOn(window.navigator, 'userAgent', 'get');
 // Define webdriver property if it doesn't exist
 if (!('webdriver' in window.navigator)) {
 Object.defineProperty(window.navigator, 'webdriver', {
 configurable: true,
 get: () => false,
 });
 }
 webdriverGetter = vi.spyOn(window.navigator, 'webdriver', 'get');
 onLineGetter = vi.spyOn(window.navigator, 'onLine', 'get');
 // Define connection property if it doesn't exist
 // @ts-ignore
 if (!('connection' in window.navigator)) {
 // @ts-ignore
 Object.defineProperty(window.navigator, 'connection', {
 configurable: true,
 get: () => ({ downlink: 10, rtt: 100 }),
 });
 }
 // @ts-ignore
 connectionGetter = jest.spyOn(window.navigator, 'connection', 'get');
 connectionGetter = vi.spyOn(window.navigator, 'connection', 'get');
 });

 it('returns TRUE if connection is online, navigator is online, has disabled webdriver, and not a bot', () => {
Expand Down

2 changes: 2 additions & 0 deletions
...rc/__tests__/buildAccountsBaseUrl.test.ts → ...rc/__tests__/buildAccountsBaseUrl.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
import { expect, test } from 'vitest';

import { buildAccountsBaseUrl } from '../buildAccountsBaseUrl';

test.each([
Expand Down

72 changes: 72 additions & 0 deletions
packages/shared/src/__tests__/color.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,72 @@
import type { Color } from '@clerk/types';
import { describe, expect, it } from 'vitest';

import { colorToSameTypeString, hexStringToRgbaColor, stringToHslaColor, stringToSameTypeColor } from '../color';

describe('stringToHslaColor(color)', function () {
 const hsla = { h: 195, s: 1, l: 0.5 };
 const cases: Array<[string, Color | null]> = [
 ['', null],
 ['transparent', { h: 0, s: 0, l: 0, a: 0 }],
 ['#00bfff', hsla],
 ['00bfff', hsla],
 ['rgb(0, 191, 255)', hsla],
 ['rgba(0, 191, 255, 0.3)', { ...hsla, a: 0.3 }],
 ];

 it.each(cases)('.stringToHslaColor(%s) => %s', (a, expected) => {
 expect(stringToHslaColor(a)).toEqual(expected);
 });
});

describe('hexStringToRgbaColor(color)', function () {
 const cases: Array<[string, Color | null]> = [
 ['#00bfff', { r: 0, g: 191, b: 255 }],
 ['00bfff', { r: 0, g: 191, b: 255 }],
 ];

 it.each(cases)('.hexStringToRgbaColor(%s) => %s', (a, expected) => {
 expect(hexStringToRgbaColor(a)).toEqual(expected);
 });
});

describe('stringToSameTypeColor(color)', function () {
 const cases: Array<[string, Color | null]> = [
 ['', ''],
 ['invalid', ''],
 ['12ff12', '#12ff12'],
 ['#12ff12', '#12ff12'],
 ['1ff', '#1ff'],
 ['transparent', 'transparent'],
 ['rgb(100,100,100)', { r: 100, g: 100, b: 100, a: undefined }],
 ['rgba(100,100,100,0.5)', { r: 100, g: 100, b: 100, a: 0.5 }],
 ['rgb(100,100,100)', { r: 100, g: 100, b: 100, a: undefined }],
 ['rgba(100,100,100,0.5)', { r: 100, g: 100, b: 100, a: 0.5 }],

Comment on lines

+41
 to 
+44

@coderabbitaicoderabbitaiBotOct 17, 2025

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.

⚠️ Potential issue | 🟡 Minor
Remove duplicate test cases.
Lines 43-44 are exact duplicates of lines 41-42 (testing the same rgb/rgba inputs). Remove the redundant entries.
Apply this diff:
 ['rgb(100,100,100)', { r: 100, g: 100, b: 100, a: undefined }],
['rgba(100,100,100,0.5)', { r: 100, g: 100, b: 100, a: 0.5 }],
- ['rgb(100,100,100)', { r: 100, g: 100, b: 100, a: undefined }],- ['rgba(100,100,100,0.5)', { r: 100, g: 100, b: 100, a: 0.5 }],
['hsl(244,66%,33%)', { h: 244, s: 0.66, l: 0.33, a: undefined }],
📝 Committable suggestion
‼️IMPORTANT

Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

['rgb(100,100,100)',{r: 100,g: 100,b: 100,a: undefined}],
['rgba(100,100,100,0.5)',{r: 100,g: 100,b: 100,a: 0.5}],
['rgb(100,100,100)',{r: 100,g: 100,b: 100,a: undefined}],
['rgba(100,100,100,0.5)',{r: 100,g: 100,b: 100,a: 0.5}],
['rgb(100,100,100)',{r: 100,g: 100,b: 100,a: undefined}],
['rgba(100,100,100,0.5)',{r: 100,g: 100,b: 100,a: 0.5}],
🤖 Prompt for AI Agents
In packages/shared/src/__tests__/color.spec.ts around lines 41 to 44 there are
duplicated test entries for 'rgb(100,100,100)' and 'rgba(100,100,100,0.5)';
remove the redundant lines (the second occurrence on lines 43-44) so each input
case appears only once in the test array.

 ['hsl(244,66%,33%)', { h: 244, s: 0.66, l: 0.33, a: undefined }],
 ['hsla(244,66%,33%,0.5)', { h: 244, s: 0.66, l: 0.33, a: 0.5 }],
 ['hsl(244,66%,33)', ''],
 ['hsla(244,66%,33,0.5)', ''],
 ];

 it.each(cases)('.stringToSameTypeColor(%s) => %s', (a, expected) => {
 expect(stringToSameTypeColor(a)).toEqual(expected);
 });
});

describe('colorToSameTypeString(color)', function () {
 const cases: Array<[Color, string]> = [
 ['', ''],
 ['invalid', ''],
 ['#12ff12', '#12ff12'],
 ['#12ff12', '#12ff12'],
 ['#1ff', '#1ff'],

Comment on lines

+57
 to 
+62

@coderabbitaicoderabbitaiBotOct 17, 2025

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.

⚠️ Potential issue | 🟠 Major
Fix type annotation and remove duplicate test case.
Two issues:
  1. Type mismatch: The type annotation Array<[Color, string]> indicates the first element should be a Color object, but lines 58-62 use string literals. If colorToSameTypeString accepts both Color objects and strings, update the type annotation to reflect this (e.g., Array<[Color | string, string]>).
  2. Duplicate test case: Lines 60-61 both test '#12ff12'.
Apply this diff to fix the type annotation and remove the duplicate:
- const cases: Array<[Color, string]> = [+ const cases: Array<[Color | string, string]> = [
['', ''],
['invalid', ''],
['#12ff12', '#12ff12'],
- ['#12ff12', '#12ff12'],
['#1ff', '#1ff'],
📝 Committable suggestion
‼️IMPORTANT

Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

constcases: Array<[Color,string]>=[
['',''],
['invalid',''],
['#12ff12','#12ff12'],
['#12ff12','#12ff12'],
['#1ff','#1ff'],
constcases: Array<[Color|string,string]>=[
['',''],
['invalid',''],
['#12ff12','#12ff12'],
['#1ff','#1ff'],
🤖 Prompt for AI Agents
In packages/shared/src/__tests__/color.spec.ts around lines 57 to 62, the test
cases array is annotated as Array<[Color, string]> but uses string literals and
contains a duplicate '#12ff12' entry; update the type annotation to Array<[Color
| string, string]> (or Array<[string | Color, string]>) so strings are allowed,
and remove the duplicate '#12ff12' test case so each case is unique.

 [{ r: 100, g: 100, b: 100, a: undefined }, 'rgb(100,100,100)'],
 [{ r: 100, g: 100, b: 100, a: 0.5 }, 'rgba(100,100,100,0.5)'],
 [{ h: 100, s: 0.55, l: 0.33, a: undefined }, 'hsl(100,55%,33%)'],
 [{ h: 100, s: 1, l: 1, a: 0.5 }, 'hsla(100,100%,100%,0.5)'],
 ];

 it.each(cases)('.colorToSameTypeString(%s) => %s', (a, expected) => {
 expect(colorToSameTypeString(a)).toEqual(expected);
 });
});

0
packages/shared/src/__tests__/color.test.ts → ...es/shared/src/__tests__/color.test.ts.bak

View file


Open in desktop


File renamed without changes.


2 changes: 2 additions & 0 deletions
packages/shared/src/__tests__/date.test.ts → packages/shared/src/__tests__/date.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
import { describe, expect, it } from 'vitest';

import type { RelativeDateCase } from '../date';
import { addYears, dateTo12HourTime, differenceInCalendarDays, formatRelative } from '../date';

Expand Down
Loading

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links");
}
} catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); }
})();
(function(){
try {
var __m = "youtube.com";
var __re = new RegExp('^' + "youtube\\.com" + '

Skip to content
chore(shared): Migrate tests to vitest#7014

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.

 Sign up for GitHub

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

 Merged

nikosdouvlis
merged 3 commits into

from

Oct 17, 2025
 Merged


chore(shared): Migrate tests to vitest
#7014
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


3 commits


Select commit
Hold shift + click to select a range
acad25b

chore(shared): Migrate tests to vitest


nikosdouvlis Oct 16, 2025
e70ddad

fix cicd


nikosdouvlis Oct 16, 2025
5fa2f97

Create breezy-flowers-float.md


nikosdouvlis Oct 16, 2025

File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


2 changes: 2 additions & 0 deletions
.changeset/breezy-flowers-float.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
---
---

9 changes: 0 additions & 9 deletions
packages/shared/customJSDOMEnvironment.ts

View file


Open in desktop


This file was deleted.


28 changes: 0 additions & 28 deletions
packages/shared/jest.config.js

View file


Open in desktop


This file was deleted.


45 changes: 0 additions & 45 deletions
packages/shared/jest.setup.ts

View file


Open in desktop


This file was deleted.


7 changes: 3 additions & 4 deletions
packages/shared/package.json

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -148,10 +148,9 @@
 "lint:attw": "attw --pack . --profile node16",
 "lint:publint": "publint",
 "publish:local": "pnpm yalc push --replace --sig",
 "test": "jest && vitest",
 "test:cache:clear": "jest --clearCache --useStderr",
 "test:ci": "jest --maxWorkers=70%",
 "test:coverage": "jest --collectCoverage && open coverage/lcov-report/index.html"
 "test": "vitest",
 "test:ci": "vitest --maxWorkers=70%",
 "test:coverage": "vitest --collectCoverage && open coverage/lcov-report/index.html"
 },
 "dependencies": {
 "@clerk/types": "workspace:^",
Expand Down

2 changes: 2 additions & 0 deletions
..._tests__/apiUrlFromPublishableKey.test.ts → ..._tests__/apiUrlFromPublishableKey.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
import { describe, expect, test } from 'vitest';

import { apiUrlFromPublishableKey } from '../apiUrlFromPublishableKey';

describe('apiUrlFromPublishableKey', () => {
Expand Down

45 changes: 35 additions & 10 deletions
...ages/shared/src/__tests__/browser.test.ts → ...ages/shared/src/__tests__/browser.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,15 +1,17 @@
import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';

import { inBrowser, isValidBrowser, isValidBrowserOnline, userAgentIsRobot } from '../browser';

describe('inBrowser()', () => {
 afterEach(() => {
jest.restoreAllMocks();
vi.restoreAllMocks();
 });

 it('returns true if window is defined', () => {
 expect(inBrowser()).toBe(true);
 });
 it('returns false if window is undefined', () => {
 const windowSpy = jest.spyOn(global, 'window', 'get');
 const windowSpy = vi.spyOn(global, 'window', 'get');
 // @ts-ignore - Test
 windowSpy.mockReturnValue(undefined);
 expect(inBrowser()).toBe(false);
Expand All@@ -21,16 +23,23 @@ describe('isValidBrowser', () => {
 let webdriverGetter: any;

 beforeEach(() => {
 userAgentGetter = jest.spyOn(window.navigator, 'userAgent', 'get');
 webdriverGetter = jest.spyOn(window.navigator, 'webdriver', 'get');
 userAgentGetter = vi.spyOn(window.navigator, 'userAgent', 'get');
 // Define webdriver property if it doesn't exist
 if (!('webdriver' in window.navigator)) {
 Object.defineProperty(window.navigator, 'webdriver', {
 configurable: true,
 get: () => false,
 });
 }
 webdriverGetter = vi.spyOn(window.navigator, 'webdriver', 'get');

Comment on lines

+27
 to 
+34

@coderabbitaicoderabbitaiBotOct 16, 2025

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.

🛠️ Refactor suggestion | 🟠 Major
🧩 Analysis chain
Good defensive coding for JSDOM compatibility.
The code checks if the webdriver property exists on window.navigator before defining it. This suggests that Vitest's JSDOM environment may not include this property by default, unlike Jest's environment.
However, verify whether this defensive check is necessary. If Vitest consistently lacks this property, consider defining it unconditionally in the global test setup (vitest.setup.mts) rather than in individual test files to reduce duplication.
Run the following script to check if this pattern appears in other test files:
🏁 Script executed:
#!/bin/bash# Search for similar defensive property definitions in test files
rg -A 5 "if \(!'.*' in window\.navigator\)" --type ts --glob '**/__tests__/**'
Length of output: 0
I need to try different search approaches since the regex pattern didn't match. Let me search more broadly:

---
🏁 Script executed:
```shell
#!/bin/bash
# Search for webdriver property usage in test files
rg -n "webdriver" --type ts --glob '**/__tests__/**' -B 2 -A 2

Length of output: 9467
🏁 Script executed:
#!/bin/bash# Search for Object.defineProperty usage in test files
rg -n "Object\.defineProperty" --type ts --glob '**/__tests__/**' -B 2 -A 2
Length of output: 39064
🏁 Script executed:
#!/bin/bash# Look for vitest setup files
fd -e mts -e ts --glob '**/vitest*' -o --glob '**/setup*'2>/dev/null | head -20
Length of output: 0
Consider extracting the repeated webdriver property setup into a helper function to avoid duplication.
The defensive webdriver property check appears twice in the same file:
  • Lines 28–33 (in isValidBrowser describe block)
  • Lines 113–118 (in isValidBrowserOnline describe block)
Since the logic is identical and JSDOM appears to consistently lack this property in both test suites, extract this into a shared setup helper to reduce duplication and improve maintainability:
constsetupWebdriverProperty=()=>{if(!('webdriver'inwindow.navigator)){Object.defineProperty(window.navigator,'webdriver',{configurable: true,get: ()=>false,});}};
Then call setupWebdriverProperty() in each beforeEach instead of repeating the full block.
🤖 Prompt for AI Agents
In packages/shared/src/__tests__/browser.spec.ts around lines 27 to 34, the
defensive webdriver property check is duplicated elsewhere; extract the repeated
block into a small helper function (e.g., setupWebdriverProperty) that checks
for 'webdriver' on window.navigator and defines it when missing, then call that
helper from each beforeEach instead of repeating the Object.defineProperty
block; update imports/locals as needed and ensure the vi.spyOn call still
references window.navigator.webdriver after the helper runs.

 });

 afterEach(() => {
jest.restoreAllMocks();
vi.restoreAllMocks();
 });

 it('returns false if not in browser', () => {
 const windowSpy = jest.spyOn(global, 'window', 'get');
 const windowSpy = vi.spyOn(global, 'window', 'get');
 // @ts-ignore - Test
 windowSpy.mockReturnValue(undefined);

Expand DownExpand Up@@ -99,11 +108,27 @@ describe('isValidBrowserOnline', () => {
 let connectionGetter: any;

 beforeEach(() => {
 userAgentGetter = jest.spyOn(window.navigator, 'userAgent', 'get');
 webdriverGetter = jest.spyOn(window.navigator, 'webdriver', 'get');
 onLineGetter = jest.spyOn(window.navigator, 'onLine', 'get');
 userAgentGetter = vi.spyOn(window.navigator, 'userAgent', 'get');
 // Define webdriver property if it doesn't exist
 if (!('webdriver' in window.navigator)) {
 Object.defineProperty(window.navigator, 'webdriver', {
 configurable: true,
 get: () => false,
 });
 }
 webdriverGetter = vi.spyOn(window.navigator, 'webdriver', 'get');
 onLineGetter = vi.spyOn(window.navigator, 'onLine', 'get');
 // Define connection property if it doesn't exist
 // @ts-ignore
 if (!('connection' in window.navigator)) {
 // @ts-ignore
 Object.defineProperty(window.navigator, 'connection', {
 configurable: true,
 get: () => ({ downlink: 10, rtt: 100 }),
 });
 }
 // @ts-ignore
 connectionGetter = jest.spyOn(window.navigator, 'connection', 'get');
 connectionGetter = vi.spyOn(window.navigator, 'connection', 'get');
 });

 it('returns TRUE if connection is online, navigator is online, has disabled webdriver, and not a bot', () => {
Expand Down

2 changes: 2 additions & 0 deletions
...rc/__tests__/buildAccountsBaseUrl.test.ts → ...rc/__tests__/buildAccountsBaseUrl.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
import { expect, test } from 'vitest';

import { buildAccountsBaseUrl } from '../buildAccountsBaseUrl';

test.each([
Expand Down

72 changes: 72 additions & 0 deletions
packages/shared/src/__tests__/color.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,72 @@
import type { Color } from '@clerk/types';
import { describe, expect, it } from 'vitest';

import { colorToSameTypeString, hexStringToRgbaColor, stringToHslaColor, stringToSameTypeColor } from '../color';

describe('stringToHslaColor(color)', function () {
 const hsla = { h: 195, s: 1, l: 0.5 };
 const cases: Array<[string, Color | null]> = [
 ['', null],
 ['transparent', { h: 0, s: 0, l: 0, a: 0 }],
 ['#00bfff', hsla],
 ['00bfff', hsla],
 ['rgb(0, 191, 255)', hsla],
 ['rgba(0, 191, 255, 0.3)', { ...hsla, a: 0.3 }],
 ];

 it.each(cases)('.stringToHslaColor(%s) => %s', (a, expected) => {
 expect(stringToHslaColor(a)).toEqual(expected);
 });
});

describe('hexStringToRgbaColor(color)', function () {
 const cases: Array<[string, Color | null]> = [
 ['#00bfff', { r: 0, g: 191, b: 255 }],
 ['00bfff', { r: 0, g: 191, b: 255 }],
 ];

 it.each(cases)('.hexStringToRgbaColor(%s) => %s', (a, expected) => {
 expect(hexStringToRgbaColor(a)).toEqual(expected);
 });
});

describe('stringToSameTypeColor(color)', function () {
 const cases: Array<[string, Color | null]> = [
 ['', ''],
 ['invalid', ''],
 ['12ff12', '#12ff12'],
 ['#12ff12', '#12ff12'],
 ['1ff', '#1ff'],
 ['transparent', 'transparent'],
 ['rgb(100,100,100)', { r: 100, g: 100, b: 100, a: undefined }],
 ['rgba(100,100,100,0.5)', { r: 100, g: 100, b: 100, a: 0.5 }],
 ['rgb(100,100,100)', { r: 100, g: 100, b: 100, a: undefined }],
 ['rgba(100,100,100,0.5)', { r: 100, g: 100, b: 100, a: 0.5 }],

Comment on lines

+41
 to 
+44

@coderabbitaicoderabbitaiBotOct 17, 2025

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.

⚠️ Potential issue | 🟡 Minor
Remove duplicate test cases.
Lines 43-44 are exact duplicates of lines 41-42 (testing the same rgb/rgba inputs). Remove the redundant entries.
Apply this diff:
 ['rgb(100,100,100)', { r: 100, g: 100, b: 100, a: undefined }],
['rgba(100,100,100,0.5)', { r: 100, g: 100, b: 100, a: 0.5 }],
- ['rgb(100,100,100)', { r: 100, g: 100, b: 100, a: undefined }],- ['rgba(100,100,100,0.5)', { r: 100, g: 100, b: 100, a: 0.5 }],
['hsl(244,66%,33%)', { h: 244, s: 0.66, l: 0.33, a: undefined }],
📝 Committable suggestion
‼️IMPORTANT

Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

['rgb(100,100,100)',{r: 100,g: 100,b: 100,a: undefined}],
['rgba(100,100,100,0.5)',{r: 100,g: 100,b: 100,a: 0.5}],
['rgb(100,100,100)',{r: 100,g: 100,b: 100,a: undefined}],
['rgba(100,100,100,0.5)',{r: 100,g: 100,b: 100,a: 0.5}],
['rgb(100,100,100)',{r: 100,g: 100,b: 100,a: undefined}],
['rgba(100,100,100,0.5)',{r: 100,g: 100,b: 100,a: 0.5}],
🤖 Prompt for AI Agents
In packages/shared/src/__tests__/color.spec.ts around lines 41 to 44 there are
duplicated test entries for 'rgb(100,100,100)' and 'rgba(100,100,100,0.5)';
remove the redundant lines (the second occurrence on lines 43-44) so each input
case appears only once in the test array.

 ['hsl(244,66%,33%)', { h: 244, s: 0.66, l: 0.33, a: undefined }],
 ['hsla(244,66%,33%,0.5)', { h: 244, s: 0.66, l: 0.33, a: 0.5 }],
 ['hsl(244,66%,33)', ''],
 ['hsla(244,66%,33,0.5)', ''],
 ];

 it.each(cases)('.stringToSameTypeColor(%s) => %s', (a, expected) => {
 expect(stringToSameTypeColor(a)).toEqual(expected);
 });
});

describe('colorToSameTypeString(color)', function () {
 const cases: Array<[Color, string]> = [
 ['', ''],
 ['invalid', ''],
 ['#12ff12', '#12ff12'],
 ['#12ff12', '#12ff12'],
 ['#1ff', '#1ff'],

Comment on lines

+57
 to 
+62

@coderabbitaicoderabbitaiBotOct 17, 2025

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.

⚠️ Potential issue | 🟠 Major
Fix type annotation and remove duplicate test case.
Two issues:
  1. Type mismatch: The type annotation Array<[Color, string]> indicates the first element should be a Color object, but lines 58-62 use string literals. If colorToSameTypeString accepts both Color objects and strings, update the type annotation to reflect this (e.g., Array<[Color | string, string]>).
  2. Duplicate test case: Lines 60-61 both test '#12ff12'.
Apply this diff to fix the type annotation and remove the duplicate:
- const cases: Array<[Color, string]> = [+ const cases: Array<[Color | string, string]> = [
['', ''],
['invalid', ''],
['#12ff12', '#12ff12'],
- ['#12ff12', '#12ff12'],
['#1ff', '#1ff'],
📝 Committable suggestion
‼️IMPORTANT

Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

constcases: Array<[Color,string]>=[
['',''],
['invalid',''],
['#12ff12','#12ff12'],
['#12ff12','#12ff12'],
['#1ff','#1ff'],
constcases: Array<[Color|string,string]>=[
['',''],
['invalid',''],
['#12ff12','#12ff12'],
['#1ff','#1ff'],
🤖 Prompt for AI Agents
In packages/shared/src/__tests__/color.spec.ts around lines 57 to 62, the test
cases array is annotated as Array<[Color, string]> but uses string literals and
contains a duplicate '#12ff12' entry; update the type annotation to Array<[Color
| string, string]> (or Array<[string | Color, string]>) so strings are allowed,
and remove the duplicate '#12ff12' test case so each case is unique.

 [{ r: 100, g: 100, b: 100, a: undefined }, 'rgb(100,100,100)'],
 [{ r: 100, g: 100, b: 100, a: 0.5 }, 'rgba(100,100,100,0.5)'],
 [{ h: 100, s: 0.55, l: 0.33, a: undefined }, 'hsl(100,55%,33%)'],
 [{ h: 100, s: 1, l: 1, a: 0.5 }, 'hsla(100,100%,100%,0.5)'],
 ];

 it.each(cases)('.colorToSameTypeString(%s) => %s', (a, expected) => {
 expect(colorToSameTypeString(a)).toEqual(expected);
 });
});

0
packages/shared/src/__tests__/color.test.ts → ...es/shared/src/__tests__/color.test.ts.bak

View file


Open in desktop


File renamed without changes.


2 changes: 2 additions & 0 deletions
packages/shared/src/__tests__/date.test.ts → packages/shared/src/__tests__/date.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
import { describe, expect, it } from 'vitest';

import type { RelativeDateCase } from '../date';
import { addYears, dateTo12HourTime, differenceInCalendarDays, formatRelative } from '../date';

Expand Down
Loading

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default");
}
} catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
chore(shared): Migrate tests to vitest#7014

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.

 Sign up for GitHub

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

 Merged

nikosdouvlis
merged 3 commits into

from

Oct 17, 2025
 Merged


chore(shared): Migrate tests to vitest
#7014
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


3 commits


Select commit
Hold shift + click to select a range
acad25b

chore(shared): Migrate tests to vitest


nikosdouvlis Oct 16, 2025
e70ddad

fix cicd


nikosdouvlis Oct 16, 2025
5fa2f97

Create breezy-flowers-float.md


nikosdouvlis Oct 16, 2025

File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


2 changes: 2 additions & 0 deletions
.changeset/breezy-flowers-float.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
---
---

9 changes: 0 additions & 9 deletions
packages/shared/customJSDOMEnvironment.ts

View file


Open in desktop


This file was deleted.


28 changes: 0 additions & 28 deletions
packages/shared/jest.config.js

View file


Open in desktop


This file was deleted.


45 changes: 0 additions & 45 deletions
packages/shared/jest.setup.ts

View file


Open in desktop


This file was deleted.


7 changes: 3 additions & 4 deletions
packages/shared/package.json

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -148,10 +148,9 @@
 "lint:attw": "attw --pack . --profile node16",
 "lint:publint": "publint",
 "publish:local": "pnpm yalc push --replace --sig",
 "test": "jest && vitest",
 "test:cache:clear": "jest --clearCache --useStderr",
 "test:ci": "jest --maxWorkers=70%",
 "test:coverage": "jest --collectCoverage && open coverage/lcov-report/index.html"
 "test": "vitest",
 "test:ci": "vitest --maxWorkers=70%",
 "test:coverage": "vitest --collectCoverage && open coverage/lcov-report/index.html"
 },
 "dependencies": {
 "@clerk/types": "workspace:^",
Expand Down

2 changes: 2 additions & 0 deletions
..._tests__/apiUrlFromPublishableKey.test.ts → ..._tests__/apiUrlFromPublishableKey.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
import { describe, expect, test } from 'vitest';

import { apiUrlFromPublishableKey } from '../apiUrlFromPublishableKey';

describe('apiUrlFromPublishableKey', () => {
Expand Down

45 changes: 35 additions & 10 deletions
...ages/shared/src/__tests__/browser.test.ts → ...ages/shared/src/__tests__/browser.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,15 +1,17 @@
import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';

import { inBrowser, isValidBrowser, isValidBrowserOnline, userAgentIsRobot } from '../browser';

describe('inBrowser()', () => {
 afterEach(() => {
jest.restoreAllMocks();
vi.restoreAllMocks();
 });

 it('returns true if window is defined', () => {
 expect(inBrowser()).toBe(true);
 });
 it('returns false if window is undefined', () => {
 const windowSpy = jest.spyOn(global, 'window', 'get');
 const windowSpy = vi.spyOn(global, 'window', 'get');
 // @ts-ignore - Test
 windowSpy.mockReturnValue(undefined);
 expect(inBrowser()).toBe(false);
Expand All@@ -21,16 +23,23 @@ describe('isValidBrowser', () => {
 let webdriverGetter: any;

 beforeEach(() => {
 userAgentGetter = jest.spyOn(window.navigator, 'userAgent', 'get');
 webdriverGetter = jest.spyOn(window.navigator, 'webdriver', 'get');
 userAgentGetter = vi.spyOn(window.navigator, 'userAgent', 'get');
 // Define webdriver property if it doesn't exist
 if (!('webdriver' in window.navigator)) {
 Object.defineProperty(window.navigator, 'webdriver', {
 configurable: true,
 get: () => false,
 });
 }
 webdriverGetter = vi.spyOn(window.navigator, 'webdriver', 'get');

Comment on lines

+27
 to 
+34

@coderabbitaicoderabbitaiBotOct 16, 2025

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.

🛠️ Refactor suggestion | 🟠 Major
🧩 Analysis chain
Good defensive coding for JSDOM compatibility.
The code checks if the webdriver property exists on window.navigator before defining it. This suggests that Vitest's JSDOM environment may not include this property by default, unlike Jest's environment.
However, verify whether this defensive check is necessary. If Vitest consistently lacks this property, consider defining it unconditionally in the global test setup (vitest.setup.mts) rather than in individual test files to reduce duplication.
Run the following script to check if this pattern appears in other test files:
🏁 Script executed:
#!/bin/bash# Search for similar defensive property definitions in test files
rg -A 5 "if \(!'.*' in window\.navigator\)" --type ts --glob '**/__tests__/**'
Length of output: 0
I need to try different search approaches since the regex pattern didn't match. Let me search more broadly:

---
🏁 Script executed:
```shell
#!/bin/bash
# Search for webdriver property usage in test files
rg -n "webdriver" --type ts --glob '**/__tests__/**' -B 2 -A 2

Length of output: 9467
🏁 Script executed:
#!/bin/bash# Search for Object.defineProperty usage in test files
rg -n "Object\.defineProperty" --type ts --glob '**/__tests__/**' -B 2 -A 2
Length of output: 39064
🏁 Script executed:
#!/bin/bash# Look for vitest setup files
fd -e mts -e ts --glob '**/vitest*' -o --glob '**/setup*'2>/dev/null | head -20
Length of output: 0
Consider extracting the repeated webdriver property setup into a helper function to avoid duplication.
The defensive webdriver property check appears twice in the same file:
  • Lines 28–33 (in isValidBrowser describe block)
  • Lines 113–118 (in isValidBrowserOnline describe block)
Since the logic is identical and JSDOM appears to consistently lack this property in both test suites, extract this into a shared setup helper to reduce duplication and improve maintainability:
constsetupWebdriverProperty=()=>{if(!('webdriver'inwindow.navigator)){Object.defineProperty(window.navigator,'webdriver',{configurable: true,get: ()=>false,});}};
Then call setupWebdriverProperty() in each beforeEach instead of repeating the full block.
🤖 Prompt for AI Agents
In packages/shared/src/__tests__/browser.spec.ts around lines 27 to 34, the
defensive webdriver property check is duplicated elsewhere; extract the repeated
block into a small helper function (e.g., setupWebdriverProperty) that checks
for 'webdriver' on window.navigator and defines it when missing, then call that
helper from each beforeEach instead of repeating the Object.defineProperty
block; update imports/locals as needed and ensure the vi.spyOn call still
references window.navigator.webdriver after the helper runs.

 });

 afterEach(() => {
jest.restoreAllMocks();
vi.restoreAllMocks();
 });

 it('returns false if not in browser', () => {
 const windowSpy = jest.spyOn(global, 'window', 'get');
 const windowSpy = vi.spyOn(global, 'window', 'get');
 // @ts-ignore - Test
 windowSpy.mockReturnValue(undefined);

Expand DownExpand Up@@ -99,11 +108,27 @@ describe('isValidBrowserOnline', () => {
 let connectionGetter: any;

 beforeEach(() => {
 userAgentGetter = jest.spyOn(window.navigator, 'userAgent', 'get');
 webdriverGetter = jest.spyOn(window.navigator, 'webdriver', 'get');
 onLineGetter = jest.spyOn(window.navigator, 'onLine', 'get');
 userAgentGetter = vi.spyOn(window.navigator, 'userAgent', 'get');
 // Define webdriver property if it doesn't exist
 if (!('webdriver' in window.navigator)) {
 Object.defineProperty(window.navigator, 'webdriver', {
 configurable: true,
 get: () => false,
 });
 }
 webdriverGetter = vi.spyOn(window.navigator, 'webdriver', 'get');
 onLineGetter = vi.spyOn(window.navigator, 'onLine', 'get');
 // Define connection property if it doesn't exist
 // @ts-ignore
 if (!('connection' in window.navigator)) {
 // @ts-ignore
 Object.defineProperty(window.navigator, 'connection', {
 configurable: true,
 get: () => ({ downlink: 10, rtt: 100 }),
 });
 }
 // @ts-ignore
 connectionGetter = jest.spyOn(window.navigator, 'connection', 'get');
 connectionGetter = vi.spyOn(window.navigator, 'connection', 'get');
 });

 it('returns TRUE if connection is online, navigator is online, has disabled webdriver, and not a bot', () => {
Expand Down

2 changes: 2 additions & 0 deletions
...rc/__tests__/buildAccountsBaseUrl.test.ts → ...rc/__tests__/buildAccountsBaseUrl.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
import { expect, test } from 'vitest';

import { buildAccountsBaseUrl } from '../buildAccountsBaseUrl';

test.each([
Expand Down

72 changes: 72 additions & 0 deletions
packages/shared/src/__tests__/color.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,72 @@
import type { Color } from '@clerk/types';
import { describe, expect, it } from 'vitest';

import { colorToSameTypeString, hexStringToRgbaColor, stringToHslaColor, stringToSameTypeColor } from '../color';

describe('stringToHslaColor(color)', function () {
 const hsla = { h: 195, s: 1, l: 0.5 };
 const cases: Array<[string, Color | null]> = [
 ['', null],
 ['transparent', { h: 0, s: 0, l: 0, a: 0 }],
 ['#00bfff', hsla],
 ['00bfff', hsla],
 ['rgb(0, 191, 255)', hsla],
 ['rgba(0, 191, 255, 0.3)', { ...hsla, a: 0.3 }],
 ];

 it.each(cases)('.stringToHslaColor(%s) => %s', (a, expected) => {
 expect(stringToHslaColor(a)).toEqual(expected);
 });
});

describe('hexStringToRgbaColor(color)', function () {
 const cases: Array<[string, Color | null]> = [
 ['#00bfff', { r: 0, g: 191, b: 255 }],
 ['00bfff', { r: 0, g: 191, b: 255 }],
 ];

 it.each(cases)('.hexStringToRgbaColor(%s) => %s', (a, expected) => {
 expect(hexStringToRgbaColor(a)).toEqual(expected);
 });
});

describe('stringToSameTypeColor(color)', function () {
 const cases: Array<[string, Color | null]> = [
 ['', ''],
 ['invalid', ''],
 ['12ff12', '#12ff12'],
 ['#12ff12', '#12ff12'],
 ['1ff', '#1ff'],
 ['transparent', 'transparent'],
 ['rgb(100,100,100)', { r: 100, g: 100, b: 100, a: undefined }],
 ['rgba(100,100,100,0.5)', { r: 100, g: 100, b: 100, a: 0.5 }],
 ['rgb(100,100,100)', { r: 100, g: 100, b: 100, a: undefined }],
 ['rgba(100,100,100,0.5)', { r: 100, g: 100, b: 100, a: 0.5 }],

Comment on lines

+41
 to 
+44

@coderabbitaicoderabbitaiBotOct 17, 2025

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.

⚠️ Potential issue | 🟡 Minor
Remove duplicate test cases.
Lines 43-44 are exact duplicates of lines 41-42 (testing the same rgb/rgba inputs). Remove the redundant entries.
Apply this diff:
 ['rgb(100,100,100)', { r: 100, g: 100, b: 100, a: undefined }],
['rgba(100,100,100,0.5)', { r: 100, g: 100, b: 100, a: 0.5 }],
- ['rgb(100,100,100)', { r: 100, g: 100, b: 100, a: undefined }],- ['rgba(100,100,100,0.5)', { r: 100, g: 100, b: 100, a: 0.5 }],
['hsl(244,66%,33%)', { h: 244, s: 0.66, l: 0.33, a: undefined }],
📝 Committable suggestion
‼️IMPORTANT

Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

['rgb(100,100,100)',{r: 100,g: 100,b: 100,a: undefined}],
['rgba(100,100,100,0.5)',{r: 100,g: 100,b: 100,a: 0.5}],
['rgb(100,100,100)',{r: 100,g: 100,b: 100,a: undefined}],
['rgba(100,100,100,0.5)',{r: 100,g: 100,b: 100,a: 0.5}],
['rgb(100,100,100)',{r: 100,g: 100,b: 100,a: undefined}],
['rgba(100,100,100,0.5)',{r: 100,g: 100,b: 100,a: 0.5}],
🤖 Prompt for AI Agents
In packages/shared/src/__tests__/color.spec.ts around lines 41 to 44 there are
duplicated test entries for 'rgb(100,100,100)' and 'rgba(100,100,100,0.5)';
remove the redundant lines (the second occurrence on lines 43-44) so each input
case appears only once in the test array.

 ['hsl(244,66%,33%)', { h: 244, s: 0.66, l: 0.33, a: undefined }],
 ['hsla(244,66%,33%,0.5)', { h: 244, s: 0.66, l: 0.33, a: 0.5 }],
 ['hsl(244,66%,33)', ''],
 ['hsla(244,66%,33,0.5)', ''],
 ];

 it.each(cases)('.stringToSameTypeColor(%s) => %s', (a, expected) => {
 expect(stringToSameTypeColor(a)).toEqual(expected);
 });
});

describe('colorToSameTypeString(color)', function () {
 const cases: Array<[Color, string]> = [
 ['', ''],
 ['invalid', ''],
 ['#12ff12', '#12ff12'],
 ['#12ff12', '#12ff12'],
 ['#1ff', '#1ff'],

Comment on lines

+57
 to 
+62

@coderabbitaicoderabbitaiBotOct 17, 2025

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.

⚠️ Potential issue | 🟠 Major
Fix type annotation and remove duplicate test case.
Two issues:
  1. Type mismatch: The type annotation Array<[Color, string]> indicates the first element should be a Color object, but lines 58-62 use string literals. If colorToSameTypeString accepts both Color objects and strings, update the type annotation to reflect this (e.g., Array<[Color | string, string]>).
  2. Duplicate test case: Lines 60-61 both test '#12ff12'.
Apply this diff to fix the type annotation and remove the duplicate:
- const cases: Array<[Color, string]> = [+ const cases: Array<[Color | string, string]> = [
['', ''],
['invalid', ''],
['#12ff12', '#12ff12'],
- ['#12ff12', '#12ff12'],
['#1ff', '#1ff'],
📝 Committable suggestion
‼️IMPORTANT

Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

constcases: Array<[Color,string]>=[
['',''],
['invalid',''],
['#12ff12','#12ff12'],
['#12ff12','#12ff12'],
['#1ff','#1ff'],
constcases: Array<[Color|string,string]>=[
['',''],
['invalid',''],
['#12ff12','#12ff12'],
['#1ff','#1ff'],
🤖 Prompt for AI Agents
In packages/shared/src/__tests__/color.spec.ts around lines 57 to 62, the test
cases array is annotated as Array<[Color, string]> but uses string literals and
contains a duplicate '#12ff12' entry; update the type annotation to Array<[Color
| string, string]> (or Array<[string | Color, string]>) so strings are allowed,
and remove the duplicate '#12ff12' test case so each case is unique.

 [{ r: 100, g: 100, b: 100, a: undefined }, 'rgb(100,100,100)'],
 [{ r: 100, g: 100, b: 100, a: 0.5 }, 'rgba(100,100,100,0.5)'],
 [{ h: 100, s: 0.55, l: 0.33, a: undefined }, 'hsl(100,55%,33%)'],
 [{ h: 100, s: 1, l: 1, a: 0.5 }, 'hsla(100,100%,100%,0.5)'],
 ];

 it.each(cases)('.colorToSameTypeString(%s) => %s', (a, expected) => {
 expect(colorToSameTypeString(a)).toEqual(expected);
 });
});

0
packages/shared/src/__tests__/color.test.ts → ...es/shared/src/__tests__/color.test.ts.bak

View file


Open in desktop


File renamed without changes.


2 changes: 2 additions & 0 deletions
packages/shared/src/__tests__/date.test.ts → packages/shared/src/__tests__/date.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
import { describe, expect, it } from 'vitest';

import type { RelativeDateCase } from '../date';
import { addYears, dateTo12HourTime, differenceInCalendarDays, formatRelative } from '../date';

Expand Down
Loading

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers");
}
} catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
chore(shared): Migrate tests to vitest#7014

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.

 Sign up for GitHub

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

 Merged

nikosdouvlis
merged 3 commits into

from

Oct 17, 2025
 Merged


chore(shared): Migrate tests to vitest
#7014
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


3 commits


Select commit
Hold shift + click to select a range
acad25b

chore(shared): Migrate tests to vitest


nikosdouvlis Oct 16, 2025
e70ddad

fix cicd


nikosdouvlis Oct 16, 2025
5fa2f97

Create breezy-flowers-float.md


nikosdouvlis Oct 16, 2025

File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


2 changes: 2 additions & 0 deletions
.changeset/breezy-flowers-float.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
---
---

9 changes: 0 additions & 9 deletions
packages/shared/customJSDOMEnvironment.ts

View file


Open in desktop


This file was deleted.


28 changes: 0 additions & 28 deletions
packages/shared/jest.config.js

View file


Open in desktop


This file was deleted.


45 changes: 0 additions & 45 deletions
packages/shared/jest.setup.ts

View file


Open in desktop


This file was deleted.


7 changes: 3 additions & 4 deletions
packages/shared/package.json

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -148,10 +148,9 @@
 "lint:attw": "attw --pack . --profile node16",
 "lint:publint": "publint",
 "publish:local": "pnpm yalc push --replace --sig",
 "test": "jest && vitest",
 "test:cache:clear": "jest --clearCache --useStderr",
 "test:ci": "jest --maxWorkers=70%",
 "test:coverage": "jest --collectCoverage && open coverage/lcov-report/index.html"
 "test": "vitest",
 "test:ci": "vitest --maxWorkers=70%",
 "test:coverage": "vitest --collectCoverage && open coverage/lcov-report/index.html"
 },
 "dependencies": {
 "@clerk/types": "workspace:^",
Expand Down

2 changes: 2 additions & 0 deletions
..._tests__/apiUrlFromPublishableKey.test.ts → ..._tests__/apiUrlFromPublishableKey.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
import { describe, expect, test } from 'vitest';

import { apiUrlFromPublishableKey } from '../apiUrlFromPublishableKey';

describe('apiUrlFromPublishableKey', () => {
Expand Down

45 changes: 35 additions & 10 deletions
...ages/shared/src/__tests__/browser.test.ts → ...ages/shared/src/__tests__/browser.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,15 +1,17 @@
import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';

import { inBrowser, isValidBrowser, isValidBrowserOnline, userAgentIsRobot } from '../browser';

describe('inBrowser()', () => {
 afterEach(() => {
jest.restoreAllMocks();
vi.restoreAllMocks();
 });

 it('returns true if window is defined', () => {
 expect(inBrowser()).toBe(true);
 });
 it('returns false if window is undefined', () => {
 const windowSpy = jest.spyOn(global, 'window', 'get');
 const windowSpy = vi.spyOn(global, 'window', 'get');
 // @ts-ignore - Test
 windowSpy.mockReturnValue(undefined);
 expect(inBrowser()).toBe(false);
Expand All@@ -21,16 +23,23 @@ describe('isValidBrowser', () => {
 let webdriverGetter: any;

 beforeEach(() => {
 userAgentGetter = jest.spyOn(window.navigator, 'userAgent', 'get');
 webdriverGetter = jest.spyOn(window.navigator, 'webdriver', 'get');
 userAgentGetter = vi.spyOn(window.navigator, 'userAgent', 'get');
 // Define webdriver property if it doesn't exist
 if (!('webdriver' in window.navigator)) {
 Object.defineProperty(window.navigator, 'webdriver', {
 configurable: true,
 get: () => false,
 });
 }
 webdriverGetter = vi.spyOn(window.navigator, 'webdriver', 'get');

Comment on lines

+27
 to 
+34

@coderabbitaicoderabbitaiBotOct 16, 2025

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.

🛠️ Refactor suggestion | 🟠 Major
🧩 Analysis chain
Good defensive coding for JSDOM compatibility.
The code checks if the webdriver property exists on window.navigator before defining it. This suggests that Vitest's JSDOM environment may not include this property by default, unlike Jest's environment.
However, verify whether this defensive check is necessary. If Vitest consistently lacks this property, consider defining it unconditionally in the global test setup (vitest.setup.mts) rather than in individual test files to reduce duplication.
Run the following script to check if this pattern appears in other test files:
🏁 Script executed:
#!/bin/bash# Search for similar defensive property definitions in test files
rg -A 5 "if \(!'.*' in window\.navigator\)" --type ts --glob '**/__tests__/**'
Length of output: 0
I need to try different search approaches since the regex pattern didn't match. Let me search more broadly:

---
🏁 Script executed:
```shell
#!/bin/bash
# Search for webdriver property usage in test files
rg -n "webdriver" --type ts --glob '**/__tests__/**' -B 2 -A 2

Length of output: 9467
🏁 Script executed:
#!/bin/bash# Search for Object.defineProperty usage in test files
rg -n "Object\.defineProperty" --type ts --glob '**/__tests__/**' -B 2 -A 2
Length of output: 39064
🏁 Script executed:
#!/bin/bash# Look for vitest setup files
fd -e mts -e ts --glob '**/vitest*' -o --glob '**/setup*'2>/dev/null | head -20
Length of output: 0
Consider extracting the repeated webdriver property setup into a helper function to avoid duplication.
The defensive webdriver property check appears twice in the same file:
  • Lines 28–33 (in isValidBrowser describe block)
  • Lines 113–118 (in isValidBrowserOnline describe block)
Since the logic is identical and JSDOM appears to consistently lack this property in both test suites, extract this into a shared setup helper to reduce duplication and improve maintainability:
constsetupWebdriverProperty=()=>{if(!('webdriver'inwindow.navigator)){Object.defineProperty(window.navigator,'webdriver',{configurable: true,get: ()=>false,});}};
Then call setupWebdriverProperty() in each beforeEach instead of repeating the full block.
🤖 Prompt for AI Agents
In packages/shared/src/__tests__/browser.spec.ts around lines 27 to 34, the
defensive webdriver property check is duplicated elsewhere; extract the repeated
block into a small helper function (e.g., setupWebdriverProperty) that checks
for 'webdriver' on window.navigator and defines it when missing, then call that
helper from each beforeEach instead of repeating the Object.defineProperty
block; update imports/locals as needed and ensure the vi.spyOn call still
references window.navigator.webdriver after the helper runs.

 });

 afterEach(() => {
jest.restoreAllMocks();
vi.restoreAllMocks();
 });

 it('returns false if not in browser', () => {
 const windowSpy = jest.spyOn(global, 'window', 'get');
 const windowSpy = vi.spyOn(global, 'window', 'get');
 // @ts-ignore - Test
 windowSpy.mockReturnValue(undefined);

Expand DownExpand Up@@ -99,11 +108,27 @@ describe('isValidBrowserOnline', () => {
 let connectionGetter: any;

 beforeEach(() => {
 userAgentGetter = jest.spyOn(window.navigator, 'userAgent', 'get');
 webdriverGetter = jest.spyOn(window.navigator, 'webdriver', 'get');
 onLineGetter = jest.spyOn(window.navigator, 'onLine', 'get');
 userAgentGetter = vi.spyOn(window.navigator, 'userAgent', 'get');
 // Define webdriver property if it doesn't exist
 if (!('webdriver' in window.navigator)) {
 Object.defineProperty(window.navigator, 'webdriver', {
 configurable: true,
 get: () => false,
 });
 }
 webdriverGetter = vi.spyOn(window.navigator, 'webdriver', 'get');
 onLineGetter = vi.spyOn(window.navigator, 'onLine', 'get');
 // Define connection property if it doesn't exist
 // @ts-ignore
 if (!('connection' in window.navigator)) {
 // @ts-ignore
 Object.defineProperty(window.navigator, 'connection', {
 configurable: true,
 get: () => ({ downlink: 10, rtt: 100 }),
 });
 }
 // @ts-ignore
 connectionGetter = jest.spyOn(window.navigator, 'connection', 'get');
 connectionGetter = vi.spyOn(window.navigator, 'connection', 'get');
 });

 it('returns TRUE if connection is online, navigator is online, has disabled webdriver, and not a bot', () => {
Expand Down

2 changes: 2 additions & 0 deletions
...rc/__tests__/buildAccountsBaseUrl.test.ts → ...rc/__tests__/buildAccountsBaseUrl.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
import { expect, test } from 'vitest';

import { buildAccountsBaseUrl } from '../buildAccountsBaseUrl';

test.each([
Expand Down

72 changes: 72 additions & 0 deletions
packages/shared/src/__tests__/color.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,72 @@
import type { Color } from '@clerk/types';
import { describe, expect, it } from 'vitest';

import { colorToSameTypeString, hexStringToRgbaColor, stringToHslaColor, stringToSameTypeColor } from '../color';

describe('stringToHslaColor(color)', function () {
 const hsla = { h: 195, s: 1, l: 0.5 };
 const cases: Array<[string, Color | null]> = [
 ['', null],
 ['transparent', { h: 0, s: 0, l: 0, a: 0 }],
 ['#00bfff', hsla],
 ['00bfff', hsla],
 ['rgb(0, 191, 255)', hsla],
 ['rgba(0, 191, 255, 0.3)', { ...hsla, a: 0.3 }],
 ];

 it.each(cases)('.stringToHslaColor(%s) => %s', (a, expected) => {
 expect(stringToHslaColor(a)).toEqual(expected);
 });
});

describe('hexStringToRgbaColor(color)', function () {
 const cases: Array<[string, Color | null]> = [
 ['#00bfff', { r: 0, g: 191, b: 255 }],
 ['00bfff', { r: 0, g: 191, b: 255 }],
 ];

 it.each(cases)('.hexStringToRgbaColor(%s) => %s', (a, expected) => {
 expect(hexStringToRgbaColor(a)).toEqual(expected);
 });
});

describe('stringToSameTypeColor(color)', function () {
 const cases: Array<[string, Color | null]> = [
 ['', ''],
 ['invalid', ''],
 ['12ff12', '#12ff12'],
 ['#12ff12', '#12ff12'],
 ['1ff', '#1ff'],
 ['transparent', 'transparent'],
 ['rgb(100,100,100)', { r: 100, g: 100, b: 100, a: undefined }],
 ['rgba(100,100,100,0.5)', { r: 100, g: 100, b: 100, a: 0.5 }],
 ['rgb(100,100,100)', { r: 100, g: 100, b: 100, a: undefined }],
 ['rgba(100,100,100,0.5)', { r: 100, g: 100, b: 100, a: 0.5 }],

Comment on lines

+41
 to 
+44

@coderabbitaicoderabbitaiBotOct 17, 2025

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.

⚠️ Potential issue | 🟡 Minor
Remove duplicate test cases.
Lines 43-44 are exact duplicates of lines 41-42 (testing the same rgb/rgba inputs). Remove the redundant entries.
Apply this diff:
 ['rgb(100,100,100)', { r: 100, g: 100, b: 100, a: undefined }],
['rgba(100,100,100,0.5)', { r: 100, g: 100, b: 100, a: 0.5 }],
- ['rgb(100,100,100)', { r: 100, g: 100, b: 100, a: undefined }],- ['rgba(100,100,100,0.5)', { r: 100, g: 100, b: 100, a: 0.5 }],
['hsl(244,66%,33%)', { h: 244, s: 0.66, l: 0.33, a: undefined }],
📝 Committable suggestion
‼️IMPORTANT

Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

['rgb(100,100,100)',{r: 100,g: 100,b: 100,a: undefined}],
['rgba(100,100,100,0.5)',{r: 100,g: 100,b: 100,a: 0.5}],
['rgb(100,100,100)',{r: 100,g: 100,b: 100,a: undefined}],
['rgba(100,100,100,0.5)',{r: 100,g: 100,b: 100,a: 0.5}],
['rgb(100,100,100)',{r: 100,g: 100,b: 100,a: undefined}],
['rgba(100,100,100,0.5)',{r: 100,g: 100,b: 100,a: 0.5}],
🤖 Prompt for AI Agents
In packages/shared/src/__tests__/color.spec.ts around lines 41 to 44 there are
duplicated test entries for 'rgb(100,100,100)' and 'rgba(100,100,100,0.5)';
remove the redundant lines (the second occurrence on lines 43-44) so each input
case appears only once in the test array.

 ['hsl(244,66%,33%)', { h: 244, s: 0.66, l: 0.33, a: undefined }],
 ['hsla(244,66%,33%,0.5)', { h: 244, s: 0.66, l: 0.33, a: 0.5 }],
 ['hsl(244,66%,33)', ''],
 ['hsla(244,66%,33,0.5)', ''],
 ];

 it.each(cases)('.stringToSameTypeColor(%s) => %s', (a, expected) => {
 expect(stringToSameTypeColor(a)).toEqual(expected);
 });
});

describe('colorToSameTypeString(color)', function () {
 const cases: Array<[Color, string]> = [
 ['', ''],
 ['invalid', ''],
 ['#12ff12', '#12ff12'],
 ['#12ff12', '#12ff12'],
 ['#1ff', '#1ff'],

Comment on lines

+57
 to 
+62

@coderabbitaicoderabbitaiBotOct 17, 2025

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.

⚠️ Potential issue | 🟠 Major
Fix type annotation and remove duplicate test case.
Two issues:
  1. Type mismatch: The type annotation Array<[Color, string]> indicates the first element should be a Color object, but lines 58-62 use string literals. If colorToSameTypeString accepts both Color objects and strings, update the type annotation to reflect this (e.g., Array<[Color | string, string]>).
  2. Duplicate test case: Lines 60-61 both test '#12ff12'.
Apply this diff to fix the type annotation and remove the duplicate:
- const cases: Array<[Color, string]> = [+ const cases: Array<[Color | string, string]> = [
['', ''],
['invalid', ''],
['#12ff12', '#12ff12'],
- ['#12ff12', '#12ff12'],
['#1ff', '#1ff'],
📝 Committable suggestion
‼️IMPORTANT

Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

constcases: Array<[Color,string]>=[
['',''],
['invalid',''],
['#12ff12','#12ff12'],
['#12ff12','#12ff12'],
['#1ff','#1ff'],
constcases: Array<[Color|string,string]>=[
['',''],
['invalid',''],
['#12ff12','#12ff12'],
['#1ff','#1ff'],
🤖 Prompt for AI Agents
In packages/shared/src/__tests__/color.spec.ts around lines 57 to 62, the test
cases array is annotated as Array<[Color, string]> but uses string literals and
contains a duplicate '#12ff12' entry; update the type annotation to Array<[Color
| string, string]> (or Array<[string | Color, string]>) so strings are allowed,
and remove the duplicate '#12ff12' test case so each case is unique.

 [{ r: 100, g: 100, b: 100, a: undefined }, 'rgb(100,100,100)'],
 [{ r: 100, g: 100, b: 100, a: 0.5 }, 'rgba(100,100,100,0.5)'],
 [{ h: 100, s: 0.55, l: 0.33, a: undefined }, 'hsl(100,55%,33%)'],
 [{ h: 100, s: 1, l: 1, a: 0.5 }, 'hsla(100,100%,100%,0.5)'],
 ];

 it.each(cases)('.colorToSameTypeString(%s) => %s', (a, expected) => {
 expect(colorToSameTypeString(a)).toEqual(expected);
 });
});

0
packages/shared/src/__tests__/color.test.ts → ...es/shared/src/__tests__/color.test.ts.bak

View file


Open in desktop


File renamed without changes.


2 changes: 2 additions & 0 deletions
packages/shared/src/__tests__/date.test.ts → packages/shared/src/__tests__/date.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
import { describe, expect, it } from 'vitest';

import type { RelativeDateCase } from '../date';
import { addYears, dateTo12HourTime, differenceInCalendarDays, formatRelative } from '../date';

Expand Down
Loading

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode");
}
} catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); }
})();
})();

Skip to content
chore(shared): Migrate tests to vitest#7014

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.

 Sign up for GitHub

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

 Merged

nikosdouvlis
merged 3 commits into

from

Oct 17, 2025
 Merged


chore(shared): Migrate tests to vitest
#7014
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


3 commits


Select commit
Hold shift + click to select a range
acad25b

chore(shared): Migrate tests to vitest


nikosdouvlis Oct 16, 2025
e70ddad

fix cicd


nikosdouvlis Oct 16, 2025
5fa2f97

Create breezy-flowers-float.md


nikosdouvlis Oct 16, 2025

File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


2 changes: 2 additions & 0 deletions
.changeset/breezy-flowers-float.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
---
---

9 changes: 0 additions & 9 deletions
packages/shared/customJSDOMEnvironment.ts

View file


Open in desktop


This file was deleted.


28 changes: 0 additions & 28 deletions
packages/shared/jest.config.js

View file


Open in desktop


This file was deleted.


45 changes: 0 additions & 45 deletions
packages/shared/jest.setup.ts

View file


Open in desktop


This file was deleted.


7 changes: 3 additions & 4 deletions
packages/shared/package.json

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -148,10 +148,9 @@
 "lint:attw": "attw --pack . --profile node16",
 "lint:publint": "publint",
 "publish:local": "pnpm yalc push --replace --sig",
 "test": "jest && vitest",
 "test:cache:clear": "jest --clearCache --useStderr",
 "test:ci": "jest --maxWorkers=70%",
 "test:coverage": "jest --collectCoverage && open coverage/lcov-report/index.html"
 "test": "vitest",
 "test:ci": "vitest --maxWorkers=70%",
 "test:coverage": "vitest --collectCoverage && open coverage/lcov-report/index.html"
 },
 "dependencies": {
 "@clerk/types": "workspace:^",
Expand Down

2 changes: 2 additions & 0 deletions
..._tests__/apiUrlFromPublishableKey.test.ts → ..._tests__/apiUrlFromPublishableKey.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
import { describe, expect, test } from 'vitest';

import { apiUrlFromPublishableKey } from '../apiUrlFromPublishableKey';

describe('apiUrlFromPublishableKey', () => {
Expand Down

45 changes: 35 additions & 10 deletions
...ages/shared/src/__tests__/browser.test.ts → ...ages/shared/src/__tests__/browser.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,15 +1,17 @@
import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';

import { inBrowser, isValidBrowser, isValidBrowserOnline, userAgentIsRobot } from '../browser';

describe('inBrowser()', () => {
 afterEach(() => {
jest.restoreAllMocks();
vi.restoreAllMocks();
 });

 it('returns true if window is defined', () => {
 expect(inBrowser()).toBe(true);
 });
 it('returns false if window is undefined', () => {
 const windowSpy = jest.spyOn(global, 'window', 'get');
 const windowSpy = vi.spyOn(global, 'window', 'get');
 // @ts-ignore - Test
 windowSpy.mockReturnValue(undefined);
 expect(inBrowser()).toBe(false);
Expand All@@ -21,16 +23,23 @@ describe('isValidBrowser', () => {
 let webdriverGetter: any;

 beforeEach(() => {
 userAgentGetter = jest.spyOn(window.navigator, 'userAgent', 'get');
 webdriverGetter = jest.spyOn(window.navigator, 'webdriver', 'get');
 userAgentGetter = vi.spyOn(window.navigator, 'userAgent', 'get');
 // Define webdriver property if it doesn't exist
 if (!('webdriver' in window.navigator)) {
 Object.defineProperty(window.navigator, 'webdriver', {
 configurable: true,
 get: () => false,
 });
 }
 webdriverGetter = vi.spyOn(window.navigator, 'webdriver', 'get');

Comment on lines

+27
 to 
+34

@coderabbitaicoderabbitaiBotOct 16, 2025

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.

🛠️ Refactor suggestion | 🟠 Major
🧩 Analysis chain
Good defensive coding for JSDOM compatibility.
The code checks if the webdriver property exists on window.navigator before defining it. This suggests that Vitest's JSDOM environment may not include this property by default, unlike Jest's environment.
However, verify whether this defensive check is necessary. If Vitest consistently lacks this property, consider defining it unconditionally in the global test setup (vitest.setup.mts) rather than in individual test files to reduce duplication.
Run the following script to check if this pattern appears in other test files:
🏁 Script executed:
#!/bin/bash# Search for similar defensive property definitions in test files
rg -A 5 "if \(!'.*' in window\.navigator\)" --type ts --glob '**/__tests__/**'
Length of output: 0
I need to try different search approaches since the regex pattern didn't match. Let me search more broadly:

---
🏁 Script executed:
```shell
#!/bin/bash
# Search for webdriver property usage in test files
rg -n "webdriver" --type ts --glob '**/__tests__/**' -B 2 -A 2

Length of output: 9467
🏁 Script executed:
#!/bin/bash# Search for Object.defineProperty usage in test files
rg -n "Object\.defineProperty" --type ts --glob '**/__tests__/**' -B 2 -A 2
Length of output: 39064
🏁 Script executed:
#!/bin/bash# Look for vitest setup files
fd -e mts -e ts --glob '**/vitest*' -o --glob '**/setup*'2>/dev/null | head -20
Length of output: 0
Consider extracting the repeated webdriver property setup into a helper function to avoid duplication.
The defensive webdriver property check appears twice in the same file:
  • Lines 28–33 (in isValidBrowser describe block)
  • Lines 113–118 (in isValidBrowserOnline describe block)
Since the logic is identical and JSDOM appears to consistently lack this property in both test suites, extract this into a shared setup helper to reduce duplication and improve maintainability:
constsetupWebdriverProperty=()=>{if(!('webdriver'inwindow.navigator)){Object.defineProperty(window.navigator,'webdriver',{configurable: true,get: ()=>false,});}};
Then call setupWebdriverProperty() in each beforeEach instead of repeating the full block.
🤖 Prompt for AI Agents
In packages/shared/src/__tests__/browser.spec.ts around lines 27 to 34, the
defensive webdriver property check is duplicated elsewhere; extract the repeated
block into a small helper function (e.g., setupWebdriverProperty) that checks
for 'webdriver' on window.navigator and defines it when missing, then call that
helper from each beforeEach instead of repeating the Object.defineProperty
block; update imports/locals as needed and ensure the vi.spyOn call still
references window.navigator.webdriver after the helper runs.

 });

 afterEach(() => {
jest.restoreAllMocks();
vi.restoreAllMocks();
 });

 it('returns false if not in browser', () => {
 const windowSpy = jest.spyOn(global, 'window', 'get');
 const windowSpy = vi.spyOn(global, 'window', 'get');
 // @ts-ignore - Test
 windowSpy.mockReturnValue(undefined);

Expand DownExpand Up@@ -99,11 +108,27 @@ describe('isValidBrowserOnline', () => {
 let connectionGetter: any;

 beforeEach(() => {
 userAgentGetter = jest.spyOn(window.navigator, 'userAgent', 'get');
 webdriverGetter = jest.spyOn(window.navigator, 'webdriver', 'get');
 onLineGetter = jest.spyOn(window.navigator, 'onLine', 'get');
 userAgentGetter = vi.spyOn(window.navigator, 'userAgent', 'get');
 // Define webdriver property if it doesn't exist
 if (!('webdriver' in window.navigator)) {
 Object.defineProperty(window.navigator, 'webdriver', {
 configurable: true,
 get: () => false,
 });
 }
 webdriverGetter = vi.spyOn(window.navigator, 'webdriver', 'get');
 onLineGetter = vi.spyOn(window.navigator, 'onLine', 'get');
 // Define connection property if it doesn't exist
 // @ts-ignore
 if (!('connection' in window.navigator)) {
 // @ts-ignore
 Object.defineProperty(window.navigator, 'connection', {
 configurable: true,
 get: () => ({ downlink: 10, rtt: 100 }),
 });
 }
 // @ts-ignore
 connectionGetter = jest.spyOn(window.navigator, 'connection', 'get');
 connectionGetter = vi.spyOn(window.navigator, 'connection', 'get');
 });

 it('returns TRUE if connection is online, navigator is online, has disabled webdriver, and not a bot', () => {
Expand Down

2 changes: 2 additions & 0 deletions
...rc/__tests__/buildAccountsBaseUrl.test.ts → ...rc/__tests__/buildAccountsBaseUrl.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
import { expect, test } from 'vitest';

import { buildAccountsBaseUrl } from '../buildAccountsBaseUrl';

test.each([
Expand Down

72 changes: 72 additions & 0 deletions
packages/shared/src/__tests__/color.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,72 @@
import type { Color } from '@clerk/types';
import { describe, expect, it } from 'vitest';

import { colorToSameTypeString, hexStringToRgbaColor, stringToHslaColor, stringToSameTypeColor } from '../color';

describe('stringToHslaColor(color)', function () {
 const hsla = { h: 195, s: 1, l: 0.5 };
 const cases: Array<[string, Color | null]> = [
 ['', null],
 ['transparent', { h: 0, s: 0, l: 0, a: 0 }],
 ['#00bfff', hsla],
 ['00bfff', hsla],
 ['rgb(0, 191, 255)', hsla],
 ['rgba(0, 191, 255, 0.3)', { ...hsla, a: 0.3 }],
 ];

 it.each(cases)('.stringToHslaColor(%s) => %s', (a, expected) => {
 expect(stringToHslaColor(a)).toEqual(expected);
 });
});

describe('hexStringToRgbaColor(color)', function () {
 const cases: Array<[string, Color | null]> = [
 ['#00bfff', { r: 0, g: 191, b: 255 }],
 ['00bfff', { r: 0, g: 191, b: 255 }],
 ];

 it.each(cases)('.hexStringToRgbaColor(%s) => %s', (a, expected) => {
 expect(hexStringToRgbaColor(a)).toEqual(expected);
 });
});

describe('stringToSameTypeColor(color)', function () {
 const cases: Array<[string, Color | null]> = [
 ['', ''],
 ['invalid', ''],
 ['12ff12', '#12ff12'],
 ['#12ff12', '#12ff12'],
 ['1ff', '#1ff'],
 ['transparent', 'transparent'],
 ['rgb(100,100,100)', { r: 100, g: 100, b: 100, a: undefined }],
 ['rgba(100,100,100,0.5)', { r: 100, g: 100, b: 100, a: 0.5 }],
 ['rgb(100,100,100)', { r: 100, g: 100, b: 100, a: undefined }],
 ['rgba(100,100,100,0.5)', { r: 100, g: 100, b: 100, a: 0.5 }],

Comment on lines

+41
 to 
+44

@coderabbitaicoderabbitaiBotOct 17, 2025

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.

⚠️ Potential issue | 🟡 Minor
Remove duplicate test cases.
Lines 43-44 are exact duplicates of lines 41-42 (testing the same rgb/rgba inputs). Remove the redundant entries.
Apply this diff:
 ['rgb(100,100,100)', { r: 100, g: 100, b: 100, a: undefined }],
['rgba(100,100,100,0.5)', { r: 100, g: 100, b: 100, a: 0.5 }],
- ['rgb(100,100,100)', { r: 100, g: 100, b: 100, a: undefined }],- ['rgba(100,100,100,0.5)', { r: 100, g: 100, b: 100, a: 0.5 }],
['hsl(244,66%,33%)', { h: 244, s: 0.66, l: 0.33, a: undefined }],
📝 Committable suggestion
‼️IMPORTANT

Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

['rgb(100,100,100)',{r: 100,g: 100,b: 100,a: undefined}],
['rgba(100,100,100,0.5)',{r: 100,g: 100,b: 100,a: 0.5}],
['rgb(100,100,100)',{r: 100,g: 100,b: 100,a: undefined}],
['rgba(100,100,100,0.5)',{r: 100,g: 100,b: 100,a: 0.5}],
['rgb(100,100,100)',{r: 100,g: 100,b: 100,a: undefined}],
['rgba(100,100,100,0.5)',{r: 100,g: 100,b: 100,a: 0.5}],
🤖 Prompt for AI Agents
In packages/shared/src/__tests__/color.spec.ts around lines 41 to 44 there are
duplicated test entries for 'rgb(100,100,100)' and 'rgba(100,100,100,0.5)';
remove the redundant lines (the second occurrence on lines 43-44) so each input
case appears only once in the test array.

 ['hsl(244,66%,33%)', { h: 244, s: 0.66, l: 0.33, a: undefined }],
 ['hsla(244,66%,33%,0.5)', { h: 244, s: 0.66, l: 0.33, a: 0.5 }],
 ['hsl(244,66%,33)', ''],
 ['hsla(244,66%,33,0.5)', ''],
 ];

 it.each(cases)('.stringToSameTypeColor(%s) => %s', (a, expected) => {
 expect(stringToSameTypeColor(a)).toEqual(expected);
 });
});

describe('colorToSameTypeString(color)', function () {
 const cases: Array<[Color, string]> = [
 ['', ''],
 ['invalid', ''],
 ['#12ff12', '#12ff12'],
 ['#12ff12', '#12ff12'],
 ['#1ff', '#1ff'],

Comment on lines

+57
 to 
+62

@coderabbitaicoderabbitaiBotOct 17, 2025

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.

⚠️ Potential issue | 🟠 Major
Fix type annotation and remove duplicate test case.
Two issues:
  1. Type mismatch: The type annotation Array<[Color, string]> indicates the first element should be a Color object, but lines 58-62 use string literals. If colorToSameTypeString accepts both Color objects and strings, update the type annotation to reflect this (e.g., Array<[Color | string, string]>).
  2. Duplicate test case: Lines 60-61 both test '#12ff12'.
Apply this diff to fix the type annotation and remove the duplicate:
- const cases: Array<[Color, string]> = [+ const cases: Array<[Color | string, string]> = [
['', ''],
['invalid', ''],
['#12ff12', '#12ff12'],
- ['#12ff12', '#12ff12'],
['#1ff', '#1ff'],
📝 Committable suggestion
‼️IMPORTANT

Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

constcases: Array<[Color,string]>=[
['',''],
['invalid',''],
['#12ff12','#12ff12'],
['#12ff12','#12ff12'],
['#1ff','#1ff'],
constcases: Array<[Color|string,string]>=[
['',''],
['invalid',''],
['#12ff12','#12ff12'],
['#1ff','#1ff'],
🤖 Prompt for AI Agents
In packages/shared/src/__tests__/color.spec.ts around lines 57 to 62, the test
cases array is annotated as Array<[Color, string]> but uses string literals and
contains a duplicate '#12ff12' entry; update the type annotation to Array<[Color
| string, string]> (or Array<[string | Color, string]>) so strings are allowed,
and remove the duplicate '#12ff12' test case so each case is unique.

 [{ r: 100, g: 100, b: 100, a: undefined }, 'rgb(100,100,100)'],
 [{ r: 100, g: 100, b: 100, a: 0.5 }, 'rgba(100,100,100,0.5)'],
 [{ h: 100, s: 0.55, l: 0.33, a: undefined }, 'hsl(100,55%,33%)'],
 [{ h: 100, s: 1, l: 1, a: 0.5 }, 'hsla(100,100%,100%,0.5)'],
 ];

 it.each(cases)('.colorToSameTypeString(%s) => %s', (a, expected) => {
 expect(colorToSameTypeString(a)).toEqual(expected);
 });
});

0
packages/shared/src/__tests__/color.test.ts → ...es/shared/src/__tests__/color.test.ts.bak

View file


Open in desktop


File renamed without changes.


2 changes: 2 additions & 0 deletions
packages/shared/src/__tests__/date.test.ts → packages/shared/src/__tests__/date.spec.ts

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
import { describe, expect, it } from 'vitest';

import type { RelativeDateCase } from '../date';
import { addYears, dateTo12HourTime, differenceInCalendarDays, formatRelative } from '../date';

Expand Down
Loading