Accept seedItemIds as a number, add tests

Author: TarekAlQaddy

spec/src/modules/tracker.js

@@ -4690,6 +4690,35 @@ describe(`ConstructorIO - Tracker${bundledDescriptionSuffix}`, () => {
       expect(tracker.trackRecommendationView(requiredParamsWithSeedItemIds)).to.equal(true);
     });
 
+    it('Should respond with a valid response and convert seedItemIds to an array if it\'s is a number', (done) => {
+      const seedItemIds = 123;
+      const { tracker } = new ConstructorIO({
+        apiKey: testApiKey,
+        fetch: fetchSpy,
+        ...requestQueueOptions,
+      });
+      const requiredParamsWithSeedItemIds = {
+        seedItemIds,
+        ...requiredParameters,
+      };
+
+      tracker.on('success', (responseParams) => {
+        const requestParams = helpers.extractBodyParamsFromFetch(fetchSpy);
+
+        // Request
+        expect(fetchSpy).to.have.been.called;
+        expect(requestParams).to.have.property('seed_item_ids').to.deep.equal(['123']);
+
+        // Response
+        expect(responseParams).to.have.property('method').to.equal('POST');
+        expect(responseParams).to.have.property('message');
+
+        done();
+      });
+
+      expect(tracker.trackRecommendationView(requiredParamsWithSeedItemIds)).to.equal(true);
+    });
+
     it('Should respond with a valid response and convert seedItemIds to an array if it\'s a string', (done) => {
       const seedItemIds = '123';
       const { tracker } = new ConstructorIO({
@@ -4719,8 +4748,8 @@ describe(`ConstructorIO - Tracker${bundledDescriptionSuffix}`, () => {
       expect(tracker.trackRecommendationView(requiredParamsWithSeedItemIds)).to.equal(true);
     });
 
-    it('Should respond with a valid response and omit seed_item_ids if seedItemIds is a number', (done) => {
-      const seedItemIds = 123;
+    it('Should respond with a valid response and omit seed_item_ids if seedItemIds is null', (done) => {
+      const seedItemIds = null;
       const { tracker } = new ConstructorIO({
         apiKey: testApiKey,
         fetch: fetchSpy,

src/modules/tracker.js

@@ -1222,7 +1222,7 @@ class Tracker {
    * @param {string} [parameters.resultId] - Recommendation result identifier (returned in response from Constructor)
    * @param {string} [parameters.section="Products"] - Results section
    * @param {object} [parameters.analyticsTags] - Pass additional analytics data
-   * @param {string[]|string} [parameters.seedItemIds] - Item ID(s) to be used as seed
+   * @param {string[]|string|number} [parameters.seedItemIds] - Item ID(s) to be used as seed
    * @param {object} [networkParameters] - Parameters relevant to the network request
    * @param {number} [networkParameters.timeout] - Request timeout (in milliseconds)
    * @returns {(true|Error)}
@@ -1301,7 +1301,9 @@ class Tracker {
         bodyParams.analytics_tags = analyticsTags;
       }
 
-      if (seedItemIds?.length && typeof seedItemIds === 'string') {
+      if (typeof seedItemIds === 'number') {
+        bodyParams.seed_item_ids = [String(seedItemIds)];
+      } else if (seedItemIds?.length && typeof seedItemIds === 'string') {
         bodyParams.seed_item_ids = [seedItemIds];
       } else if (seedItemIds?.length && Array.isArray(seedItemIds)) {
         bodyParams.seed_item_ids = seedItemIds;