Skip to content

Commit 76ebe0d

Browse files
Anderson, BenAnderson, Ben
Anderson, Ben
authored and
Anderson, Ben
committed
- expose full user id config (including storage) to user id modules (prebid#5803
- removing TODO from test
1 parent 30f7505 commit 76ebe0d

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

modules/fabrickIdSystem.js

+3-2
Original file line numberDiff line numberDiff line change
@@ -35,13 +35,14 @@ export const fabrickIdSubmodule = {
3535
/**
3636
* performs action to obtain id and return a value in the callback's response argument
3737
* @function getId
38-
* @param {SubmoduleParams} [configParams]
38+
* @param {SubmoduleConfig} [config]
3939
* @param {ConsentData}
4040
* @param {Object} cacheIdObj - existing id, if any consentData]
4141
* @returns {IdResponse|undefined}
4242
*/
43-
getId(configParams, consentData, cacheIdObj) {
43+
getId(config, consentData, cacheIdObj) {
4444
try {
45+
const configParams = (config && config.params) || {};
4546
if (window.fabrickMod1) {
4647
window.fabrickMod1(configParams, consentData, cacheIdObj);
4748
}

test/spec/modules/fabrickIdSystem_spec.js

-1
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,6 @@ describe('Fabrick ID System', function() {
8989
request.respond(
9090
200,
9191
responseHeader,
92-
// TODO - actually check the value
9392
JSON.stringify({})
9493
);
9594
expect(callBackSpy.calledOnce).to.be.true;

0 commit comments

Comments
 (0)