Skip to content

Commit 701d3fd

Browse files
Emmanouela/sdk 832 add test files in tests directories (clerk#3161)
* chore(chrome-extension): Create __tests__ folder * chore(clerk-js): Create __tests__ folders * chore(fastify): Create __tests__ folder * chore(next-js): Create __tests__ folders * Create mighty-dragons-build.md * Update packages/chrome-extension/src/__tests__/exports.test.ts Co-authored-by: Nikos Douvlis <nikosdouvlis@gmail.com> --------- Co-authored-by: Nikos Douvlis <nikosdouvlis@gmail.com>
1 parent 112ed1e commit 701d3fd

49 files changed

Lines changed: 86 additions & 82 deletions

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.changeset/mighty-dragons-build.md

Lines changed: 4 additions & 0 deletions

packages/chrome-extension/src/__snapshots__/exports.test.ts.snap renamed to packages/chrome-extension/src/__tests__/__snapshots__/exports.test.ts.snap

File renamed without changes.

packages/chrome-extension/src/exports.test.ts renamed to packages/chrome-extension/src/__tests__/exports.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import * as publicExports from './index';
1+
import * as publicExports from '../index';
22

33
describe('public exports', () => {
44
it('should not include a breaking change', () => {

packages/chrome-extension/src/utils/cookies.test.ts renamed to packages/chrome-extension/src/utils/__tests__/cookies.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import browser from 'webextension-polyfill';
22

3-
import { getClientCookie } from './cookies';
3+
import { getClientCookie } from '../cookies';
44

55
type RequiredCookieOpts = 'domain' | 'name' | 'value';
66
type CreateCookieOpts<T extends keyof browser.Cookies.Cookie> = Pick<browser.Cookies.Cookie, T> &

packages/chrome-extension/src/utils/manifest.test.ts renamed to packages/chrome-extension/src/utils/__tests__/manifest.test.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
import browser, { type Manifest } from 'webextension-polyfill';
22

3-
import { missingManifestKeyError, missingValidManifestHostPermission } from './errors';
4-
import type { ValidatedManifest } from './manifest';
5-
import { getValidPossibleManifestHosts, validateHostPermissionExistence, validateManifest } from './manifest';
3+
import { missingManifestKeyError, missingValidManifestHostPermission } from '../errors';
4+
import type { ValidatedManifest } from '../manifest';
5+
import { getValidPossibleManifestHosts, validateHostPermissionExistence, validateManifest } from '../manifest';
66

77
const validClerkManifest = {
88
permissions: ['cookies', 'storage'],

packages/chrome-extension/src/utils/storage.test.ts renamed to packages/chrome-extension/src/utils/__tests__/storage.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import browser from 'webextension-polyfill';
22

3-
import { BrowserStorageCache, MemoryStorageCache } from './storage';
3+
import { BrowserStorageCache, MemoryStorageCache } from '../storage';
44

55
describe('StorageCache', () => {
66
const KEY = 'foo';

packages/clerk-js/src/core/clerk.redirects.test.ts renamed to packages/clerk-js/src/core/__tests__/clerk.redirects.test.ts

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,16 @@
1-
import { Clerk } from './clerk';
2-
import type { DevBrowser } from './devBrowser';
3-
import type { DisplayConfig } from './resources/internal';
4-
import { Client, Environment } from './resources/internal';
1+
import { Clerk } from '../clerk';
2+
import type { DevBrowser } from '../devBrowser';
3+
import type { DisplayConfig } from '../resources/internal';
4+
import { Client, Environment } from '../resources/internal';
55

66
const mockClientFetch = jest.fn();
77
const mockEnvironmentFetch = jest.fn();
88

9-
jest.mock('./resources/Client');
10-
jest.mock('./resources/Environment');
9+
jest.mock('../resources/Client');
10+
jest.mock('../resources/Environment');
1111

1212
// Because Jest, don't ask me why...
13-
jest.mock('./devBrowser', () => ({
13+
jest.mock('../devBrowser', () => ({
1414
createDevBrowser: (): DevBrowser => ({
1515
clear: jest.fn(),
1616
setup: jest.fn(),

packages/clerk-js/src/core/clerk.test.ts renamed to packages/clerk-js/src/core/__tests__/clerk.test.ts

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,23 +1,23 @@
11
import type { ActiveSessionResource, SignInJSON, SignUpJSON, TokenResource } from '@clerk/types';
22
import { waitFor } from '@testing-library/dom';
33

4-
import { mockNativeRuntime } from '../testUtils';
5-
import { Clerk } from './clerk';
6-
import type { DevBrowser } from './devBrowser';
7-
import { eventBus, events } from './events';
8-
import type { DisplayConfig, Organization } from './resources/internal';
9-
import { BaseResource, Client, EmailLinkErrorCode, Environment, SignIn, SignUp } from './resources/internal';
10-
import { SessionCookieService } from './services';
11-
import { mockJwt } from './test/fixtures';
4+
import { mockNativeRuntime } from '../../testUtils';
5+
import { Clerk } from '../clerk';
6+
import type { DevBrowser } from '../devBrowser';
7+
import { eventBus, events } from '../events';
8+
import type { DisplayConfig, Organization } from '../resources/internal';
9+
import { BaseResource, Client, EmailLinkErrorCode, Environment, SignIn, SignUp } from '../resources/internal';
10+
import { SessionCookieService } from '../services';
11+
import { mockJwt } from '../test/fixtures';
1212

1313
const mockClientFetch = jest.fn();
1414
const mockEnvironmentFetch = jest.fn();
1515

16-
jest.mock('./resources/Client');
17-
jest.mock('./resources/Environment');
16+
jest.mock('../resources/Client');
17+
jest.mock('../resources/Environment');
1818

1919
// Because Jest, don't ask me why...
20-
jest.mock('./devBrowser', () => ({
20+
jest.mock('../devBrowser', () => ({
2121
createDevBrowser: (): DevBrowser => ({
2222
clear: jest.fn(),
2323
setup: jest.fn(),

packages/clerk-js/src/core/devBrowser.test.ts renamed to packages/clerk-js/src/core/__tests__/devBrowser.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
1-
import { createDevBrowser } from './devBrowser';
2-
import type { FapiClient } from './fapiClient';
1+
import { createDevBrowser } from '../devBrowser';
2+
import type { FapiClient } from '../fapiClient';
33

44
type RecursivePartial<T> = {
55
[P in keyof T]?: RecursivePartial<T[P]>;

packages/clerk-js/src/core/fapiClient.test.ts renamed to packages/clerk-js/src/core/__tests__/fapiClient.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import type { Clerk } from '@clerk/types';
22

3-
import { createFapiClient } from './fapiClient';
3+
import { createFapiClient } from '../fapiClient';
44

55
const mockedClerkInstance = {
66
frontendApi: 'clerk.example.com',

0 commit comments

Comments
 (0)