Skip to content

test(expo): Isolate native token bootstrap in sync tests #9039

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
wobsoriano merged 1 commit into from
Jun 30, 2026
+5 −15
Merged

test(expo): Isolate native token bootstrap in sync tests #9039

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
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/stable-expo-native-sync-tests.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
---
---
18 changes: 3 additions & 15 deletions packages/expo/src/provider/__tests__/ClerkProvider.nativeClientSync.test.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -123,7 +123,7 @@ describe('ClerkProvider native client sync', () => {
vi.clearAllMocks();
mocks.nativeClientEvent = null;
mocks.configure.mockResolvedValue(undefined);
mocks.getClientToken.mockResolvedValue('native-client-token');
mocks.getClientToken.mockResolvedValue(null);
mocks.syncClientStateFromJs.mockResolvedValue(undefined);
mocks.tokenCache.getToken.mockResolvedValue('client-token');
mocks.tokenCache.saveToken.mockResolvedValue(undefined);
Expand Down Expand Up @@ -184,7 +184,6 @@ describe('ClerkProvider native client sync', () => {

test('reloads JS resources after native emits a device token change', async () => {
mocks.tokenCache.getToken.mockResolvedValue(null);
mocks.getClientToken.mockResolvedValue(null);

const { rerender } = render(
<ClerkProvider
Expand Down Expand Up @@ -223,6 +222,8 @@ describe('ClerkProvider native client sync', () => {
});

test('reloads JS resources after native clears the device token', async () => {
mocks.tokenCache.getToken.mockResolvedValue(null);

const { rerender } = render(
<ClerkProvider
publishableKey='pk_test_123'
Expand All @@ -233,9 +234,6 @@ describe('ClerkProvider native client sync', () => {
await waitFor(() => {
expect(mocks.configure).toHaveBeenCalled();
});
await waitFor(() => {
expect(mocks.clerkInstance.__internal_reloadInitialResources).toHaveBeenCalled();
});

mocks.clerkInstance.__internal_reloadInitialResources.mockClear();
mocks.tokenCache.saveToken.mockClear();
Expand Down Expand Up @@ -264,8 +262,6 @@ describe('ClerkProvider native client sync', () => {
});

test('reloads JS resources after a native client-only change without rewriting the token cache', async () => {
mocks.getClientToken.mockResolvedValue(null);

const { rerender } = render(
<ClerkProvider
publishableKey='pk_test_123'
Expand Down Expand Up @@ -304,8 +300,6 @@ describe('ClerkProvider native client sync', () => {
});

test('does not bounce a JS client listener event while applying a native client change', async () => {
mocks.getClientToken.mockResolvedValue(null);

const { rerender } = render(
<ClerkProvider
publishableKey='pk_test_123'
Expand Down Expand Up @@ -345,7 +339,6 @@ describe('ClerkProvider native client sync', () => {

test('keeps token cache notifications suppressed across overlapping native token writes', async () => {
mocks.tokenCache.getToken.mockResolvedValue(null);
mocks.getClientToken.mockResolvedValue(null);

const firstSave = deferred();
const secondSave = deferred();
Expand Down Expand Up @@ -448,7 +441,6 @@ describe('ClerkProvider native client sync', () => {
fetch: vi.fn().mockResolvedValue(refreshedClient),
};
mocks.clerkInstance.session = activeSession;
mocks.getClientToken.mockResolvedValue(null);

const { rerender } = render(
<ClerkProvider
Expand Down Expand Up @@ -512,7 +504,6 @@ describe('ClerkProvider native client sync', () => {
mocks.clerkInstance.session = session;
return Promise.resolve();
});
mocks.getClientToken.mockResolvedValue(null);

const { rerender } = render(
<ClerkProvider
Expand Down Expand Up @@ -572,7 +563,6 @@ describe('ClerkProvider native client sync', () => {
}),
};
mocks.clerkInstance.session = removedSession;
mocks.getClientToken.mockResolvedValue(null);

const { rerender } = render(
<ClerkProvider
Expand Down Expand Up @@ -646,7 +636,6 @@ describe('ClerkProvider native client sync', () => {
mocks.clerkInstance.session = session;
return Promise.resolve();
});
mocks.getClientToken.mockResolvedValue(null);

render(
<ClerkProvider
Expand Down Expand Up @@ -1044,7 +1033,6 @@ describe('ClerkProvider native client sync', () => {

test('ignores native client events that echo a JS-originated sync', async () => {
mocks.tokenCache.getToken.mockResolvedValue(null);
mocks.getClientToken.mockResolvedValue(null);

const { rerender } = render(
<ClerkProvider
Expand Down
Loading