Eids uid string only (prebid#5047)

* added support for pubcommon, digitrust, id5id

* added support for IdentityLink

* changed the source for id5

* added unit test cases

* changed source param for identityLink

* the user-id in eid should be string only

* updated conversant bidder spec for eid-uid to be string

modules/userId/eids.js

@@ -91,7 +91,7 @@ function createEidObject(userIdData, subModuleKey) {
  let eid = {};
  eid.source = conf['source'];
  const value = utils.isFn(conf['getValue']) ? conf['getValue'](userIdData) : userIdData;
- if (value) {
+ if (utils.isStr(value)) {
  const uid = { id: value, atype: conf['atype'] };
  // getUidExt
  if (utils.isFn(conf['getUidExt'])) {

test/spec/modules/conversantBidAdapter_spec.js

@@ -470,14 +470,14 @@ describe('Conversant adapter tests', function() {
 
  // add pubcid to every entry
  requests.forEach((unit) => {
- Object.assign(unit, {userId: {pubcid: 112233, tdid: 223344, idl_env: 334455}});
+ Object.assign(unit, {userId: {pubcid: '112233', tdid: '223344', idl_env: '334455'}});
  Object.assign(unit, {userIdAsEids: createEidsArray(unit.userId)});
  });
  // construct http post payload
  const payload = spec.buildRequests(requests).data;
  expect(payload).to.have.deep.nested.property('user.ext.eids', [
- {source: 'adserver.org', uids: [{id: 223344, atype: 1, ext: {rtiPartner: 'TDID'}}]},
- {source: 'liveramp.com', uids: [{id: 334455, atype: 1}]}
+ {source: 'adserver.org', uids: [{id: '223344', atype: 1, ext: {rtiPartner: 'TDID'}}]},
+ {source: 'liveramp.com', uids: [{id: '334455', atype: 1}]}
  ]);
  });
  });

test/spec/modules/eids_spec.js

@@ -1,8 +1,8 @@
 import {createEidsArray} from 'modules/userId/eids.js';
 import {expect} from 'chai';
 
-//  Note: In unit tets cases for bidders, call the createEidsArray function over userId object that is used for calling fetchBids
-// this way the request will stay consistent and unit test cases will not need lots of changes.
+//  Note: In unit tets cases for bidders, call the createEidsArray function over userId object that is used for calling fetchBids
+// this way the request will stay consistent and unit test cases will not need lots of changes.
 
 describe('eids array generation for known sub-modules', function() {
  it('pubCommonId', function() {
@@ -158,12 +158,24 @@ describe('Negative case', function() {
  expect(newEids.length).to.equal(0);
  });
 
- it('eids array generation for known sub-module with undefined value', function() {
+ it('eids array generation for known sub-module with non-string value', function() {
  // pubCommonId
- const userId = {
+ let userId = {
  pubcid: undefined
  };
- const newEids = createEidsArray(userId);
+ let newEids = createEidsArray(userId);
+ expect(newEids.length).to.equal(0);
+ userId.pubcid = 123;
+ newEids = createEidsArray(userId);
+ expect(newEids.length).to.equal(0);
+ userId.pubcid = [];
+ newEids = createEidsArray(userId);
+ expect(newEids.length).to.equal(0);
+ userId.pubcid = {};
+ newEids = createEidsArray(userId);
+ expect(newEids.length).to.equal(0);
+ userId.pubcid = null;
+ newEids = createEidsArray(userId);
  expect(newEids.length).to.equal(0);
  });
 });