Skip to content
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

Jump to bottom

[MOB-9505] rename merge parameter #450

Merged
merged 4 commits into from
Sep 17, 2024
Merged

[MOB-9505] rename merge parameter #450

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
8efb811
[MOB-9505] rename merge parameter
darshan-iterable Sep 9, 2024
7d8c810
fix ci failure and resolved comment
darshan-iterable Sep 10, 2024
534e58a
Merge branch 'AUT_main' into bugfix/MOB-9505-rename-merge-parameter
darshan-iterable Sep 13, 2024
370934c
fix: ci failure and merged AUT_main changes
darshan-iterable Sep 16, 2024
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
62 changes: 32 additions & 30 deletions src/anonymousUserTracking/tests/userMergeScenarios.test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -127,7 +127,7 @@ describe('UserMergeScenariosTests', () => {
});

describe('UserMergeScenariosTests with setUserID', () => {
it('criteria not met with merge false with setUserId', async () => {
it('criteria not met with disableEventReplay true with setUserId', async () => {
const { setUserID, logout } = initializeWithConfig({
authToken: '123',
configOptions: { enableAnonTracking: true }
Expand All @@ -142,7 +142,7 @@ describe('UserMergeScenariosTests', () => {
SHARED_PREFS_EVENT_LIST_KEY,
expect.any(String)
);
await setUserID('testuser123', false);
await setUserID('testuser123', true);
const response = await getInAppMessages({
count: 10,
packageName: 'my-lil-website'
Expand All @@ -160,7 +160,7 @@ describe('UserMergeScenariosTests', () => {
expect(mergePostRequestData).toBeUndefined(); // ensure that merge API Do NOT get called
});

it('criteria not met with merge true with setUserId', async () => {
it('criteria not met with disableEventReplay false with setUserId', async () => {
const { setUserID, logout } = initializeWithConfig({
authToken: '123',
configOptions: { enableAnonTracking: true }
Expand All @@ -175,7 +175,7 @@ describe('UserMergeScenariosTests', () => {
SHARED_PREFS_EVENT_LIST_KEY,
expect.any(String)
);
await setUserID('testuser123', true);
await setUserID('testuser123', false);
const response = await getInAppMessages({
count: 10,
packageName: 'my-lil-website'
Expand All @@ -193,7 +193,7 @@ describe('UserMergeScenariosTests', () => {
expect(mergePostRequestData).toBeUndefined(); // ensure that merge API Do NOT get called
});

it('criteria not met with merge default value with setUserId', async () => {
it('criteria not met with disableEventReplay default value with setUserId', async () => {
const { setUserID, logout } = initializeWithConfig({
authToken: '123',
configOptions: { enableAnonTracking: true }
Expand Down Expand Up @@ -226,7 +226,7 @@ describe('UserMergeScenariosTests', () => {
expect(mergePostRequestData).toBeUndefined(); // ensure that merge API Do NOT get called
});

it('criteria is met with merge false with setUserId', async () => {
it('criteria is met with disableEventReplay true with setUserId', async () => {
(localStorage.getItem as jest.Mock).mockImplementation((key) => {
if (key === SHARED_PREFS_EVENT_LIST_KEY) {
return JSON.stringify([eventDataMatched]);
Expand All @@ -247,7 +247,7 @@ describe('UserMergeScenariosTests', () => {
} catch (e) {
console.log('');
}
await setUserID('testuser123', false);
await setUserID('testuser123', true);
expect(localStorageMock.removeItem).toHaveBeenCalledWith(
SHARED_PREF_ANON_USER_ID
);
Expand All @@ -262,7 +262,7 @@ describe('UserMergeScenariosTests', () => {
expect(mergePostRequestData).toBeUndefined(); // ensure that merge API Do NOT get called
});

it('criteria is met with merge true with setUserId', async () => {
it('criteria is met with disableEventReplay false with setUserId', async () => {
(localStorage.getItem as jest.Mock).mockImplementation((key) => {
if (key === SHARED_PREFS_EVENT_LIST_KEY) {
return JSON.stringify([eventDataMatched]);
Expand All @@ -288,7 +288,7 @@ describe('UserMergeScenariosTests', () => {
count: 10,
packageName: 'my-lil-website'
});
await setUserID('testuser123', true);
await setUserID('testuser123', false);
expect(localStorageMock.removeItem).toHaveBeenCalledWith(
SHARED_PREF_ANON_USER_ID
);
Expand All @@ -302,7 +302,7 @@ describe('UserMergeScenariosTests', () => {
jest.runAllTimers();
});

it('criteria is met with merge default with setUserId', async () => {
it('criteria is met with disableEventReplay default with setUserId', async () => {
(localStorage.getItem as jest.Mock).mockImplementation((key) => {
if (key === SHARED_PREFS_EVENT_LIST_KEY) {
return JSON.stringify([eventDataMatched]);
Expand Down Expand Up @@ -343,7 +343,7 @@ describe('UserMergeScenariosTests', () => {
jest.runAllTimers();
});

it('current user identified with setUserId merge false', async () => {
it('current user identified with setUserId disableEventReplay true', async () => {
const { setUserID, logout } = initializeWithConfig({
authToken: '123',
configOptions: { enableAnonTracking: true }
Expand All @@ -363,7 +363,7 @@ describe('UserMergeScenariosTests', () => {
expect(localStorageMock.setItem).not.toHaveBeenCalledWith(
SHARED_PREF_ANON_USER_ID
);
await setUserID('testuseranotheruser', false);
await setUserID('testuseranotheruser', true);
const secondResponse = await getInAppMessages({
count: 10,
packageName: 'my-lil-website'
Expand All @@ -374,7 +374,8 @@ describe('UserMergeScenariosTests', () => {
);
expect(mergePostRequestData).toBeUndefined(); // ensure that merge API Do NOT get called
});
it('current user identified with setUserId merge true', async () => {

it('current user identified with setUserId disableEventReplay false', async () => {
const { setUserID, logout } = initializeWithConfig({
authToken: '123',
configOptions: { enableAnonTracking: true }
Expand All @@ -389,7 +390,7 @@ describe('UserMergeScenariosTests', () => {
expect(localStorageMock.setItem).not.toHaveBeenCalledWith(
SHARED_PREF_ANON_USER_ID
);
await setUserID('testuseranotheruser', true);
await setUserID('testuseranotheruser', false);
const secondResponse = await getInAppMessages({
count: 10,
packageName: 'my-lil-website'
Expand All @@ -400,7 +401,8 @@ describe('UserMergeScenariosTests', () => {
);
expect(mergePostRequestData).toBeDefined(); // ensure that merge API gets called
});
it('current user identified with setUserId merge default', async () => {

it('current user identified with setUserId disableEventReplay default', async () => {
const { setUserID, logout } = initializeWithConfig({
authToken: '123',
configOptions: { enableAnonTracking: true }
Expand All @@ -424,12 +426,12 @@ describe('UserMergeScenariosTests', () => {
const mergePostRequestData = mockRequest.history.post.find(
(req) => req.url === ENDPOINT_MERGE_USER
);
expect(mergePostRequestData).toBeUndefined(); // ensure that merge API Do NOT get called
expect(mergePostRequestData).toBeDefined(); // ensure that merge API gets called
});
});

describe('UserMergeScenariosTests with setEmail', () => {
it('criteria not met with merge false with setEmail', async () => {
it('criteria not met with disableEventReplay true with setEmail', async () => {
const { setEmail, logout } = initializeWithConfig({
authToken: '123',
configOptions: { enableAnonTracking: true }
Expand All @@ -444,7 +446,7 @@ describe('UserMergeScenariosTests', () => {
SHARED_PREFS_EVENT_LIST_KEY,
expect.any(String)
);
await setEmail('[email protected]', false);
await setEmail('[email protected]', true);
const response = await getInAppMessages({
count: 10,
packageName: 'my-lil-website'
Expand All @@ -462,7 +464,7 @@ describe('UserMergeScenariosTests', () => {
expect(mergePostRequestData).toBeUndefined(); // ensure that merge API Do NOT get called
});

it('criteria not met with merge true with setEmail', async () => {
it('criteria not met with disableEventReplay false with setEmail', async () => {
const { setEmail, logout } = initializeWithConfig({
authToken: '123',
configOptions: { enableAnonTracking: true }
Expand All @@ -477,7 +479,7 @@ describe('UserMergeScenariosTests', () => {
SHARED_PREFS_EVENT_LIST_KEY,
expect.any(String)
);
await setEmail('[email protected]', true);
await setEmail('[email protected]', false);
const response = await getInAppMessages({
count: 10,
packageName: 'my-lil-website'
Expand All @@ -495,7 +497,7 @@ describe('UserMergeScenariosTests', () => {
expect(mergePostRequestData).toBeUndefined(); // ensure that merge API Do NOT get called
});

it('criteria not met with merge default value with setEmail', async () => {
it('criteria not met with disableEventReplay default value with setEmail', async () => {
const { setEmail, logout } = initializeWithConfig({
authToken: '123',
configOptions: { enableAnonTracking: true }
Expand Down Expand Up @@ -528,7 +530,7 @@ describe('UserMergeScenariosTests', () => {
expect(mergePostRequestData).toBeUndefined(); // ensure that merge API Do NOT get called
});

it('criteria is met with merge true with setEmail', async () => {
it('criteria is met with disableEventReplay false with setEmail', async () => {
(localStorage.getItem as jest.Mock).mockImplementation((key) => {
if (key === SHARED_PREFS_EVENT_LIST_KEY) {
return JSON.stringify([eventDataMatched]);
Expand All @@ -553,7 +555,7 @@ describe('UserMergeScenariosTests', () => {
count: 10,
packageName: 'my-lil-website'
});
await setEmail('[email protected]', true);
await setEmail('[email protected]', false);
expect(localStorageMock.removeItem).toHaveBeenCalledWith(
SHARED_PREF_ANON_USER_ID
);
Expand All @@ -567,7 +569,7 @@ describe('UserMergeScenariosTests', () => {
jest.runAllTimers();
});

it('criteria is met with merge default with setEmail', async () => {
it('criteria is met with disableEventReplay default with setEmail', async () => {
(localStorage.getItem as jest.Mock).mockImplementation((key) => {
if (key === SHARED_PREFS_EVENT_LIST_KEY) {
return JSON.stringify([eventDataMatched]);
Expand Down Expand Up @@ -608,7 +610,7 @@ describe('UserMergeScenariosTests', () => {
jest.runAllTimers();
});

it('current user identified with setEmail with merge false', async () => {
it('current user identified with setEmail with disableEventReplay true', async () => {
const { setEmail, logout } = initializeWithConfig({
authToken: '123',
configOptions: { enableAnonTracking: true }
Expand All @@ -628,7 +630,7 @@ describe('UserMergeScenariosTests', () => {
expect(localStorageMock.setItem).not.toHaveBeenCalledWith(
SHARED_PREF_ANON_USER_ID
);
await setEmail('[email protected]', false);
await setEmail('[email protected]', true);
const secondResponse = await getInAppMessages({
count: 10,
packageName: 'my-lil-website'
Expand All @@ -642,7 +644,7 @@ describe('UserMergeScenariosTests', () => {
expect(mergePostRequestData).toBeUndefined(); // ensure that merge API Do NOT get called
});

it('current user identified with setEmail merge true', async () => {
it('current user identified with setEmail disableEventReplay false', async () => {
const { setEmail, logout } = initializeWithConfig({
authToken: '123',
configOptions: { enableAnonTracking: true }
Expand All @@ -662,7 +664,7 @@ describe('UserMergeScenariosTests', () => {
expect(localStorageMock.setItem).not.toHaveBeenCalledWith(
SHARED_PREF_ANON_USER_ID
);
await setEmail('[email protected]', true);
await setEmail('[email protected]', false);
const secondResponse = await getInAppMessages({
count: 10,
packageName: 'my-lil-website'
Expand All @@ -676,7 +678,7 @@ describe('UserMergeScenariosTests', () => {
expect(mergePostRequestData).toBeDefined(); // ensure that merge API gets called
});

it('current user identified with setEmail merge default', async () => {
it('current user identified with setEmail disableEventReplay default', async () => {
const { setEmail, logout } = initializeWithConfig({
authToken: '123',
configOptions: { enableAnonTracking: true }
Expand Down Expand Up @@ -707,7 +709,7 @@ describe('UserMergeScenariosTests', () => {
const mergePostRequestData = mockRequest.history.post.find(
(req) => req.url === ENDPOINT_MERGE_USER
);
expect(mergePostRequestData).toBeUndefined(); // ensure that merge API Do NOT get called
expect(mergePostRequestData).toBeDefined(); // ensure that merge API gets called
});
});
});
Loading
Loading