Revert "Suite clone does not clone fullUrl fixed"

gemini-testing · Mar 16, 2018 · 3c40424 · 3c40424
1 parent 3e12e72
commit 3c40424
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 21 deletions.
diff --git a/lib/runner/browser-runner/index.js b/lib/runner/browser-runner/index.js
@@ -30,7 +30,6 @@ 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,8 +84,7 @@ module.exports = class Suite {
         const clonedSuite = Suite.create(this.name, this.parent);
 
         _.forOwn(this, (value, key) => {
-            let desc = Object.getOwnPropertyDescriptor(this, key);
-            Object.defineProperty(clonedSuite, key, _.cloneDeep(desc));
+            clonedSuite[key] = _.clone(value);
         });
 
         this.children.forEach((child) => clonedSuite.addChild(child.clone()));

diff --git a/test/unit/suite.js b/test/unit/suite.js
@@ -342,26 +342,9 @@ describe('suite', () => {
 
         it('should return cloned suite', () => {
             const suite = createSuite('origin');
-            suite.browsers = ['bro1', 'bro2'];
             const clonedSuite = suite.clone();
 
-            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');
+            assert.notEqual(clonedSuite, suite);
         });
 
         it('should clone nested suites', () => {