[APITEST-766] Changed packages to an object

examples/collection-v2.json

@@ -43,15 +43,15 @@
             "script": {
                 "type": "text/javascript",
                 "exec": "console.log(\"hello\");",
-                "packages": [{ "id":"script-package-1", "name":"package1"}]
+                "packages": { "package1": {"id":"script-package-1"}}
             }
         },
         {
             "listen": "prerequest",
             "script": {
                 "type": "text/javascript",
                 "exec": "console.log(\"hello\");",
-                "packages": [{ "id":"script-package-1", "name":"package1"}]
+                "packages": { "package1": {"id":"script-package-1"}}
             }
         }
     ],

lib/collection/script.js

@@ -49,7 +49,7 @@ _.assign(Script.prototype, /** @lends Script.prototype */ {
      * @param {String} [options.type] Script type
      * @param {String} [options.src] Script source url
      * @param {String[]|String} [options.exec] Script to execute
-     * @param {Object[]} [options.packages] Packages required by the script
+     * @param {Object} [options.packages] Packages required by the script
      */
     update: function (options) {
         // no splitting is being done here, as string scripts are split right before assignment below anyway
@@ -64,11 +64,7 @@ _.assign(Script.prototype, /** @lends Script.prototype */ {
          */
         this.type = options.type || 'text/javascript';
 
-        /**
-         * @augments {Script.prototype}
-         * @type {Array<{ id: string, name: string}>}
-         */
-        this.packages = options.packages || [];
+        this.packages = options.packages || {};
 
         _.has(options, 'src') && (
 

test/unit/collection.test.js

@@ -26,7 +26,7 @@ describe('Collection', function () {
                             id: 'my-script-1',
                             type: 'text/javascript',
                             exec: ['console.log("This doesn\'t matter");'],
-                            packages: []
+                            packages: {}
                         }
                     }]
                 },
@@ -180,7 +180,7 @@ describe('Collection', function () {
                     id: 'test-script-1',
                     type: 'text/javascript',
                     exec: ['console.log("Random");'],
-                    packages: []
+                    packages: {}
                 }
             });
             expect(collectionJSON.event[1]).to.have.property('listen', 'prerequest');
@@ -221,12 +221,7 @@ describe('Collection', function () {
                             exec: [
                                 'console.log("bcoz I am batman!");'
                             ],
-                            packages: [
-                                {
-                                    id: 'my-package-1',
-                                    name: 'superman'
-                                }
-                            ]
+                            packages: { superman: { id: 'script-apckage-1' } }
                         }
                     }],
                     item: [{
@@ -289,7 +284,7 @@ describe('Collection', function () {
                         exec: [
                             'console.log("bcoz I am batman!");'
                         ],
-                        packages: [{ id: 'my-package-1', name: 'superman' }]
+                        packages: { superman: { id: 'script-apckage-1' } }
                     }
                 }],
                 item: [{

test/unit/event-list.test.js

@@ -37,7 +37,7 @@ describe('EventList', function () {
                         id: 'test-script-1',
                         type: 'text/javascript',
                         exec: 'console.log("hello");',
-                        packages: [{ id: 'script-package-1', name: 'package1' }]
+                        packages: { package1: { id: 'script-package-1' } }
                     }
                 }]),
                 eventListJSON;
@@ -49,7 +49,7 @@ describe('EventList', function () {
                     id: 'test-script-1',
                     type: 'text/javascript',
                     exec: ['console.log("hello");'],
-                    packages: [{ id: 'script-package-1', name: 'package1' }]
+                    packages: { package1: { id: 'script-apckage-1' } }
                 }
             }]);
 
@@ -67,7 +67,7 @@ describe('EventList', function () {
                     id: 'test-script-1',
                     type: 'text/javascript',
                     exec: ['console.log("hello");'],
-                    packages: [{ id: 'script-package-1', name: 'package1' }]
+                    packages: { package1: { id: 'script-apckage-1' } }
                 }
             });
 

test/unit/event.test.js

@@ -12,7 +12,7 @@ describe('Event', function () {
         script: {
             type: 'text/javascript',
             exec: 'console.log("hello");',
-            packages: []
+            packages: {}
         }
     };
 
@@ -53,7 +53,7 @@ describe('Event', function () {
                     expect(postmanEvent).to.have.property('script').that.is.an('object');
                     expect(postmanEvent.script).to.have.property('type', 'text/javascript');
                     expect(postmanEvent.script).to.have.property('exec').that.is.an('array');
-                    expect(postmanEvent.script).to.have.property('packages').that.is.an('array');
+                    expect(postmanEvent.script).to.have.property('packages').that.is.an('object');
                 });
             });
 
@@ -123,7 +123,7 @@ describe('Event', function () {
             expect(eventJSON.script).to.deep.include({
                 type: 'text/javascript',
                 exec: ['console.log("hello");'],
-                packages: []
+                packages: {}
             });
         });
 
@@ -136,7 +136,7 @@ describe('Event', function () {
             expect(beforeJSON.script).to.deep.include({
                 type: 'text/javascript',
                 exec: ['console.log("hello");'],
-                packages: []
+                packages: {}
             });
 
             event.update({ script: { id: 'my-new-script' } });

test/unit/item-group.test.js

@@ -28,7 +28,7 @@ describe('ItemGroup', function () {
                             id: 'my-script-1',
                             type: 'text/javascript',
                             exec: ['console.log("This doesn\'t matter");'],
-                            packages: [{ id: 'my-package-1', name: 'package1' }]
+                            packages: { package1: { id: 'script-apckage-1' } }
                         }
                     }],
                     protocolProfileBehavior: {
@@ -529,7 +529,7 @@ describe('ItemGroup', function () {
                             id: 'my-test-script',
                             type: 'text/javascript',
                             exec: ['console.log("hello there!");'],
-                            packages: [{ id: 'my-package-1', name: 'package1' }]
+                            packages: { package1: { id: 'script-apckage-1' } }
                         }
                     }],
                     item: [{

test/unit/item.test.js

@@ -19,7 +19,7 @@ describe('Item', function () {
                             id: 'my-script-1',
                             type: 'text/javascript',
                             exec: ['console.log("This doesn\'t matter");'],
-                            packages: [{ id: 'script-package-1', name: 'package1' }]
+                            packages: { package1: { id: 'script-apckage-1' } }
                         }
                     }],
                     request: {

test/unit/script.test.js

@@ -90,7 +90,7 @@ describe('Script', function () {
             expect(scriptJSON).to.deep.include({
                 type: 'text/javascript',
                 exec: ['console.log("This is a line of test script code");'],
-                packages: []
+                packages: {}
             });
         });
 

types/index.d.ts

@@ -2158,18 +2158,18 @@ declare module "postman-collection" {
          * @param [options.type] - Script type
          * @param [options.src] - Script source url
          * @param [options.exec] - Script to execute
-         * @param [options.packages] - List of packages to be preloaded
+         * @param [options.packages] - packages to be preloaded
          */
         update(options?: {
             type?: string;
             src?: string;
             exec?: string[] | string;
-            packages?: Object[];
+            packages?: Object;
         }): void;
         type: string;
         src: Url;
         exec: string[];
-        packages: Object[];
+        packages: Object;
         /**
          * Check whether an object is an instance of ItemGroup.
          * @param obj - -