fix: suite clone does not clone fullUrl

Dmitriy-kiselyov · Dmitriy-kiselyov · commit 4d9bc62911cf · 2018-03-14T01:02:07.000+03:00
diff --git a/lib/runner/browser-runner/index.js b/lib/runner/browser-runner/index.js
@@ -30,6 +30,7 @@ module.exports = class BrowserRunner extends Runner {
             .map((suite) => {
                 if (suite.hasOwnProperty('url')) {
                     Object.defineProperty(suite, 'fullUrl', {
+                        enumerable: true,
                         get: () => this._mkFullUrl(suite.url)
                     });
                 }
diff --git a/lib/suite.js b/lib/suite.js
@@ -84,7 +84,8 @@ module.exports = class Suite {
         const clonedSuite = Suite.create(this.name, this.parent);
 
         _.forOwn(this, (value, key) => {
-            clonedSuite[key] = _.clone(value);
+            let desc = Object.getOwnPropertyDescriptor(this, key);
+            Object.defineProperty(clonedSuite, key, _.cloneDeep(desc));
         });
 
         this.children.forEach((child) => clonedSuite.addChild(child.clone()));
diff --git a/test/unit/suite.js b/test/unit/suite.js
@@ -342,9 +342,26 @@ describe('suite', () => {
 
         it('should return cloned suite', () => {
             const suite = createSuite('origin');
+            suite.browsers = ['bro1', 'bro2'];
             const clonedSuite = suite.clone();
 
-            assert.notEqual(clonedSuite, suite);
+            assert.deepEqual(clonedSuite, suite);
+            assert.notEqual(clonedSuite, suite); //not equal by link
+            assert.notEqual(clonedSuite.browsers, suite.browsers); //not equal by link
+        });
+
+        it('should clone all enumerable properties', () => {
+            const suite = createSuite('origin');
+            let value = 'some/path';
+            Object.defineProperty(suite, 'fullUrl', {
+                enumerable: true,
+                get: () => value
+            });
+            const clonedSuite = suite.clone();
+
+            assert.equal(clonedSuite.fullUrl, 'some/path');
+            value = 'another/path';
+            assert.equal(clonedSuite.fullUrl, 'another/path');
         });
 
         it('should clone nested suites', () => {

Original file line number	Diff line number	Diff line change
`@@ -30,6 +30,7 @@ module.exports = class BrowserRunner extends Runner {`
`30`	`30`	`.map((suite) => {`
`31`	`31`	`if (suite.hasOwnProperty('url')) {`
`32`	`32`	`Object.defineProperty(suite, 'fullUrl', {`
	`33`	`+ enumerable: true,`
`33`	`34`	`get: () => this._mkFullUrl(suite.url)`
`34`	`35`	`});`
`35`	`36`	`}`