Merge pull request #409 from gemini-testing/FEI-23991.copy_muted_test

fix: copy link for skipped test

Author: mb1te

lib/static/components/section/section-browser.js

@@ -85,6 +85,7 @@ class SectionBrowser extends Component {
                     <BrowserTitle
                         title={title}
                         browserId={browser.id}
+                        browserName={browser.name}
                         lastResultId={lastResult.id}
                         handler={this.onToggleSection}
                     />

lib/static/components/section/title/browser.js

@@ -14,6 +14,7 @@ class BrowserTitle extends Component {
     static propTypes = {
         title: PropTypes.node.isRequired,
         browserId: PropTypes.string.isRequired,
+        browserName: PropTypes.string.isRequired,
         lastResultId: PropTypes.string.isRequired,
         handler: PropTypes.func.isRequired,
         // from store
@@ -30,10 +31,10 @@ class BrowserTitle extends Component {
     }
 
     _getTestUrl() {
-        const {title, testName, retryIndex} = this.props;
+        const {browserName, testName, retryIndex} = this.props;
 
         return appendQuery(window.location.href, {
-            browser: title,
+            browser: browserName,
             testNameFilter: testName,
             strictMatchFilter: true,
             retryIndex,

test/unit/lib/static/components/section/section-browser.js

@@ -154,4 +154,30 @@ describe('<SectionBrowser/>', () => {
             assert.calledOnceWith(actionsStub.toggleBrowserSection, {browserId: 'yabro-1', shouldBeOpened: true});
         });
     });
+
+    describe('<ClipboardButton/>', () => {
+        let BrowserTitle;
+
+        beforeEach(() => {
+            BrowserTitle = sandbox.stub().returns(null);
+            SectionBrowser = proxyquire('lib/static/components/section/section-browser', {
+                './title/browser': {default: BrowserTitle}
+            }).default;
+        });
+
+        it('should render "BrowserTitle" with "browserName" for correctly working clipboard button', () => {
+            const browsersById = mkBrowser({id: 'yabro', name: 'yabro', resultIds: ['res']});
+            const browsersStateById = {'yabro': {shouldBeShown: true, shouldBeOpened: false}};
+            const resultsById = mkResult({id: 'res', status: SUCCESS});
+            const tree = mkStateTree({browsersById, browsersStateById, resultsById});
+
+            mkSectionBrowserComponent({browserId: 'yabro'}, {tree});
+
+            assert.calledOnceWith(BrowserTitle, {
+                browserId: 'yabro', browserName: 'yabro', handler: sinon.match.any,
+                lastResultId: 'res', title: 'yabro'
+            });
+        });
+    });
 });
+

test/unit/lib/static/components/section/title/browser.js

@@ -0,0 +1,81 @@
+import React from 'react';
+import {defaults} from 'lodash';
+import proxyquire from 'proxyquire';
+import {mkConnectedComponent} from 'test/unit/lib/static/components/utils';
+import {mkBrowser, mkResult, mkStateTree} from 'test/unit/lib/static/state-utils';
+import {SKIPPED} from 'lib/constants/test-statuses';
+import actionNames from 'lib/static/modules/action-names';
+import viewModes from 'lib/constants/view-modes';
+import {EXPAND_ALL} from 'lib/constants/expand-modes';
+
+describe('<BrowserTitle/>', () => {
+    const sandbox = sinon.sandbox.create();
+    let BrowserTitle, actionsStub, queryParams;
+
+    const mkBrowserTitleComponent = (props = {}, initialState = {}) => {
+        props = defaults(props, {
+            title: 'default_title',
+            browserId: 'default_bro',
+            browserName: 'default_name',
+            lastResultId: 'default_res',
+            handler: () => {}
+        });
+        initialState = defaults(initialState, {
+            tree: mkStateTree()
+        });
+
+        return mkConnectedComponent(<BrowserTitle {...props} />, {initialState});
+    };
+
+    beforeEach(() => {
+        actionsStub = {
+            copyTestLink: sandbox.stub().returns({type: actionNames.COPY_TEST_LINK})
+        };
+
+        queryParams = {
+            appendQuery: sandbox.stub().returns(null)
+        };
+
+        BrowserTitle = proxyquire('lib/static/components/section/title/browser', {
+            '../../../modules/actions': actionsStub,
+            '../../../modules/query-params': queryParams
+        }).default;
+    });
+
+    describe('<ClipboardButton/>', () => {
+        it('should call action "onCopyTestLink" on click', () => {
+            const browsersById = mkBrowser({id: 'yabro', name: 'yabro', resultIds: ['default_res']});
+            const resultsById = mkResult({id: 'default_res', status: SKIPPED, skipReason: 'some-reason'});
+            const tree = mkStateTree({browsersById, resultsById});
+
+            const component = mkBrowserTitleComponent({browserId: 'yabro'}, {tree});
+            component.find('ClipboardButton').simulate('click');
+
+            assert.calledOnce(actionsStub.copyTestLink);
+            assert.calledWithExactly(actionsStub.copyTestLink);
+        });
+
+        it('should call "appendQuery" with correct arguments', () => {
+            const browsersById = mkBrowser({id: 'yabro', name: 'yabro', parentId: 'test'});
+            const resultsById = mkResult({id: 'default_res', status: SKIPPED, skipReason: 'some-reason'});
+            const tree = mkStateTree({browsersById, resultsById});
+
+            const component = mkBrowserTitleComponent({browserId: 'yabro', browserName: 'yabro'}, {tree});
+
+            // call prop to simulate click due to multiple requires in ClipboardButton
+            component.find('ClipboardButton').prop('option-text')();
+
+            assert.calledOnce(queryParams.appendQuery);
+            assert.calledWithExactly(queryParams.appendQuery, 'about:blank', {
+                browser: 'yabro',
+                testNameFilter: 'test',
+                strictMatchFilter: true,
+                retryIndex: 0,
+                viewModes: viewModes.ALL,
+                expand: EXPAND_ALL,
+                groupByError: false
+            });
+        });
+    });
+});
+