Added Test cases

Author: sunil-lakshman

index.d.ts

@@ -300,47 +300,3 @@ export class Taxonomy extends Query {
     below(key: string, value: string, levels?: number): Query;
     equalAndBelow(key: string, value: string, levels?: number): Query;
 }
-
-export class Variants extends Query {
-    constructor();
-
-    entry_uid: string;
-    content_type_uid: string;
-    _query: object;
-    provider: any;
-    cachePolicy: number;
-    queryCachePolicy: number;
-
-    only(field_uid: string): this;
-    only(field_uids: string[]): this;
-    only(reference_field_uid:string, field_uid: string): this;
-    only(reference_field_uid:string, field_uids: string[]): this;
-
-    except(field_uid: string): this;
-    except(field_uids: string[]): this;
-    except(reference_field_uid:string, field_uid: string): this;
-    except(reference_field_uid:string, field_uids: string[]): this;
-
-    setCacheProvider(provider: object): this;
-    setCachePolicy(policy: number): this;
-    includeReference(val: string[]): this;
-    includeReference(...val: string[]): this;
-    language(language_code: string): this;
-    addQuery(key: string, value: string): this;
-    includeEmbeddedItems(): this;
-    includeFallback(): this;
-    /**
-     * @deprecated since version 3.3.0
-     */
-    includeSchema(): this;
-    includeReferenceContentTypeUID(): this;
-    includeContentType(): this;
-    /**
-     * @deprecated since version 3.3.0
-     */
-    includeOwner(): this;
-    toJSON(): this;
-    addParam(key: string, value: any): this;
-    fetch(fetchOptions?: object): Promise<any>;
-}
-

src/core/modules/entry.js

@@ -1,5 +1,4 @@
 import * as Utils from "../lib/utils";
-import Variants from "./variants";
 
 /**
  * @class 
@@ -389,7 +388,11 @@ export default class Entry {
      * @instance 
      */
     Variants(variant_headers) {
-        this.headers['x-cs-variant-uid'] = variant_headers;
+        if (Array.isArray(variant_headers) && variant_headers.length > 0) {
+            this.headers['x-cs-variant-uid'] = variant_headers.join(',')
+        }else{
+            this.headers['x-cs-variant-uid'] = variant_headers;
+        }
         return this;
      }
 

src/core/modules/query.js

@@ -1,6 +1,5 @@
 import * as Utils from '../lib/utils.js';
 import Entry from './entry';
-import Variants from './variants.js';
 
 const _extend = {
     compare: function(type) {
@@ -838,9 +837,11 @@ export default class Query extends Entry {
      * @instance 
      */
     Variants(variant_headers) {
-       this.headers['x-cs-variant-uid'] = variant_headers;
-       return this;
+        if (Array.isArray(variant_headers) && variant_headers.length > 0) {
+            this.headers['x-cs-variant-uid'] = variant_headers.join(',')
+        }else{
+            this.headers['x-cs-variant-uid'] = variant_headers;
+        }
+        return this;
     }
-
-
 }

src/core/stack.js

@@ -1,7 +1,6 @@
 import config from '../../config';
 import * as Utils from './lib/utils';
 import Entry from './modules/entry';
-import Variants from './modules/variants';
 import Assets from './modules/assets';
 import Query from './modules/query';
 import Taxonomy from './modules/taxonomy';

test/entry/find.js

@@ -1658,4 +1658,20 @@ test('CT Taxonomies Query: Get Entries With Taxonomy Terms Parent and Excluding
             assert.fail("CT Taxonomies Query: Get Entries With Taxonomy Terms Parent and Excluding the term itself ($above, level)");
             assert.end();
         })
-})
+})
+
+test('Variants in entry', function (t) {
+    let Query = Stack.ContentType('source').Query();
+    Query
+        .Variants('variant_entry_1', 'variant_entry_2')
+        .toJSON()
+        .find()
+        .then(entries => {
+            assert.ok(entries[0].length, 'Variant entries present in the resultset');
+            assert.end();
+        }, err => {
+            console.error("error :", err);
+            assert.fail("Variant Entries are not present in the CT");
+            assert.end();
+        })
+});