Skip to content

Commit c93fa27

Browse files
Address PR Comments
1 parent 2b371ee commit c93fa27

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

test/jest/cookieSyncManager.spec.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@ const pixelSettings: PixelConfiguration = {
1717

1818
describe.only('CookieSyncManager', () => {
1919
describe('#attemptCookieSync', () => {
20+
// https://go.mparticle.com/work/SQDSDKS-6915
2021
it('should perform a cookie sync with defaults', () => {
2122
const mockMPInstance = ({
2223
_Store: {
@@ -157,7 +158,7 @@ describe.only('CookieSyncManager', () => {
157158
);
158159
});
159160

160-
// QUESTION: What is the purpose of this code path?
161+
// https://go.mparticle.com/work/SQDSDKS-6915
161162
it('should call performCookieSync with mpid if previousMpid and mpid do not match', () => {
162163
const mockMPInstance = ({
163164
_Store: {
@@ -186,7 +187,7 @@ describe.only('CookieSyncManager', () => {
186187
);
187188
});
188189

189-
it('should call performCookieSync with mpid and csd if previousMpid and mpid do not match', () => {
190+
it('should include mpid AND csd when calling performCookieSync if previousMpid and mpid do not match', () => {
190191
const mockMPInstance = ({
191192
_Store: {
192193
webviewBridgeEnabled: false,

0 commit comments

Comments
 (0)