Skip to content

Commit eac6edf

Browse files
committed
rename isCmcdResponseKey
1 parent 199d37a commit eac6edf

File tree

4 files changed

+14
-17
lines changed

4 files changed

+14
-17
lines changed

lib/src/cmcd.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ export { groupCmcdHeaders } from './cmcd/groupCmcdHeaders.js';
6363
export { isCmcdCustomKey } from './cmcd/isCmcdCustomKey.js';
6464
export { isCmcdEventKey } from './cmcd/isCmcdEventKey.js';
6565
export { isCmcdRequestKey } from './cmcd/isCmcdRequestKey.js';
66-
export { isCmcdResponseKey } from './cmcd/isCmcdResponseKey.js';
66+
export { isCmcdResponseReceivedKey } from './cmcd/isCmcdResponseReceivedKey.js';
6767
export { isCmcdV1Key } from './cmcd/isCmcdV1Key.js';
6868
export { prepareCmcdData } from './cmcd/prepareCmcdData.js';
6969
export { toCmcdHeaders } from './cmcd/toCmcdHeaders.js';

lib/src/cmcd/isCmcdResponseKey.ts renamed to lib/src/cmcd/isCmcdResponseReceivedKey.ts

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,5 @@
1-
import { CMCD_COMMON_KEYS } from './CMCD_COMMON_KEYS.js';
2-
import { CMCD_REQUEST_KEYS } from './CMCD_REQUEST_KEYS.js';
31
import { CMCD_RESPONSE_KEYS } from './CMCD_RESPONSE_KEYS.js';
42
import type { CmcdResponse } from './CmcdResponse.js';
5-
import { isCmcdCustomKey } from './isCmcdCustomKey.js';
63

74
/**
85
* Check if a key is a valid CMCD response key.
@@ -16,8 +13,8 @@ import { isCmcdCustomKey } from './isCmcdCustomKey.js';
1613
* @beta
1714
*
1815
* @example
19-
* {@includeCode ../../test/cmcd/isCmcdResponseKey.test.ts#example}
16+
* {@includeCode ../../test/cmcd/isCmcdResponseReceivedKey.test.ts#example}
2017
*/
21-
export function isCmcdResponseKey(key: string): key is keyof CmcdResponse {
18+
export function isCmcdResponseReceivedKey(key: string): key is keyof CmcdResponse {
2219
return CMCD_RESPONSE_KEYS.includes(key as any);
2320
}

lib/src/cmcd/prepareCmcdData.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ import type { CmcdFormatterOptions } from './CmcdFormatterOptions.js';
1010
import type { CmcdValue } from './CmcdValue.js';
1111
import { isCmcdEventKey } from './isCmcdEventKey.js';
1212
import { isCmcdRequestKey } from './isCmcdRequestKey.js';
13-
import { isCmcdResponseKey } from './isCmcdResponseKey.js';
13+
import { isCmcdResponseReceivedKey } from './isCmcdResponseReceivedKey.js';
1414
import { isCmcdV1Key } from './isCmcdV1Key.js';
1515

1616
const filterMap = {
@@ -41,7 +41,7 @@ export function prepareCmcdData(obj: Record<string, any>, options: CmcdEncodeOpt
4141

4242
if (!options.events || !options.events.includes('rr')){
4343
const remainingObj = Object.fromEntries(
44-
Object.entries(obj).filter(([key]) => !isCmcdResponseKey(key)),
44+
Object.entries(obj).filter(([key]) => !isCmcdResponseReceivedKey(key)),
4545
);
4646
obj = remainingObj;
4747
}
Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,33 +1,33 @@
1-
import { CMCD_REQUEST_KEYS, CMCD_RESPONSE_KEYS, isCmcdResponseKey } from '@svta/common-media-library/cmcd';
1+
import { CMCD_REQUEST_KEYS, CMCD_RESPONSE_KEYS, isCmcdResponseReceivedKey } from '@svta/common-media-library/cmcd';
22
import { equal } from 'node:assert';
33
import { describe, it } from 'node:test';
44

5-
describe('isCmcdResponseKey', () => {
5+
describe('isCmcdResponseReceivedKey', () => {
66
it('provides a valid example', () => {
77
//#region example
8-
equal(isCmcdResponseKey('url'), true);
9-
equal(isCmcdResponseKey('e'), false);
8+
equal(isCmcdResponseReceivedKey('url'), true);
9+
equal(isCmcdResponseReceivedKey('e'), false);
1010
//#endregion example
1111
});
1212

1313
it('Rejects all request keys', () => {
1414
for (const key of CMCD_REQUEST_KEYS) {
15-
equal(isCmcdResponseKey(key), false);
15+
equal(isCmcdResponseReceivedKey(key), false);
1616
}
1717
});
1818

1919
it('Accepts all response keys', () => {
2020
for (const key of CMCD_RESPONSE_KEYS) {
21-
equal(isCmcdResponseKey(key), true);
21+
equal(isCmcdResponseReceivedKey(key), true);
2222
}
2323
});
2424

2525
it('Rejects custom keys', () => {
26-
equal(isCmcdResponseKey('com.hello.world-foo'), false);
26+
equal(isCmcdResponseReceivedKey('com.hello.world-foo'), false);
2727
});
2828

2929
it('Rejects invalid keys', () => {
30-
equal(isCmcdResponseKey('nrr'), false);
31-
equal(isCmcdResponseKey('e'), false);
30+
equal(isCmcdResponseReceivedKey('nrr'), false);
31+
equal(isCmcdResponseReceivedKey('e'), false);
3232
});
3333
});

0 commit comments

Comments
 (0)