diff options
author | Mathieu Suen <mathieu.suen@sonarsource.com> | 2020-04-27 15:00:44 +0200 |
---|---|---|
committer | sonartech <sonartech@sonarsource.com> | 2020-05-01 20:03:42 +0000 |
commit | 5f0c85650faf2514b1c74fe6389a80574a415e39 (patch) | |
tree | 877af88b96443146ad75f5d8b1d48be7669d4930 | |
parent | 01a703dd8baa53f114195ba035aca1fabda90801 (diff) | |
download | sonarqube-5f0c85650faf2514b1c74fe6389a80574a415e39.tar.gz sonarqube-5f0c85650faf2514b1c74fe6389a80574a415e39.zip |
SONAR-13202 Fix homepage icon checked for main branch.
3 files changed, 54 insertions, 2 deletions
diff --git a/server/sonar-web/src/main/js/app/components/nav/component/HeaderMeta.tsx b/server/sonar-web/src/main/js/app/components/nav/component/HeaderMeta.tsx index ab8275d6b5e..5706d62927c 100644 --- a/server/sonar-web/src/main/js/app/components/nav/component/HeaderMeta.tsx +++ b/server/sonar-web/src/main/js/app/components/nav/component/HeaderMeta.tsx @@ -90,6 +90,8 @@ export function HeaderMeta(props: HeaderMetaProps) { export function getCurrentPage(component: T.Component, branchLike: BranchLike | undefined) { let currentPage: T.HomePage | undefined; + const branch = isBranch(branchLike) && !branchLike.isMain ? branchLike.name : undefined; + switch (component.qualifier) { case ComponentQualifier.Portfolio: case ComponentQualifier.SubPortfolio: @@ -99,7 +101,7 @@ export function getCurrentPage(component: T.Component, branchLike: BranchLike | currentPage = { type: 'APPLICATION', component: component.key, - branch: isBranch(branchLike) ? branchLike.name : undefined + branch }; break; case ComponentQualifier.Project: @@ -107,7 +109,7 @@ export function getCurrentPage(component: T.Component, branchLike: BranchLike | currentPage = { type: 'PROJECT', component: component.key, - branch: isBranch(branchLike) ? branchLike.name : undefined + branch }; break; } diff --git a/server/sonar-web/src/main/js/app/components/nav/component/__tests__/HeaderMeta-test.tsx b/server/sonar-web/src/main/js/app/components/nav/component/__tests__/HeaderMeta-test.tsx index 871ba889cee..b918bdacb38 100644 --- a/server/sonar-web/src/main/js/app/components/nav/component/__tests__/HeaderMeta-test.tsx +++ b/server/sonar-web/src/main/js/app/components/nav/component/__tests__/HeaderMeta-test.tsx @@ -30,6 +30,13 @@ it('should render correctly for a branch', () => { expect(wrapper).toMatchSnapshot(); }); +it('should render correctly for a main project branch', () => { + const wrapper = shallowRender({ + branchLike: mockBranch({ isMain: true }) + }); + expect(wrapper).toMatchSnapshot(); +}); + it('should render correctly for a portfolio', () => { const wrapper = shallowRender({ component: mockComponent({ key: 'foo', qualifier: ComponentQualifier.Portfolio }) diff --git a/server/sonar-web/src/main/js/app/components/nav/component/__tests__/__snapshots__/HeaderMeta-test.tsx.snap b/server/sonar-web/src/main/js/app/components/nav/component/__tests__/__snapshots__/HeaderMeta-test.tsx.snap index 60e7f6c630c..4ad191b493c 100644 --- a/server/sonar-web/src/main/js/app/components/nav/component/__tests__/__snapshots__/HeaderMeta-test.tsx.snap +++ b/server/sonar-web/src/main/js/app/components/nav/component/__tests__/__snapshots__/HeaderMeta-test.tsx.snap @@ -43,6 +43,49 @@ exports[`should render correctly for a branch 1`] = ` </Fragment> `; +exports[`should render correctly for a main project branch 1`] = ` +<Fragment> + <div + className="display-flex-center flex-0 small" + > + <span + className="header-meta-warnings" + > + <ComponentNavWarnings + warnings={ + Array [ + "ERROR_1", + "ERROR_2", + ] + } + /> + </span> + <span + className="spacer-left nowrap note" + > + <DateTimeFormatter + date="2017-01-02T00:00:00.000Z" + /> + </span> + <span + className="spacer-left nowrap note" + > + version 0.0.1 + </span> + <Connect(HomePageSelect) + className="spacer-left" + currentPage={ + Object { + "branch": undefined, + "component": "my-project", + "type": "PROJECT", + } + } + /> + </div> +</Fragment> +`; + exports[`should render correctly for a portfolio 1`] = ` <Fragment> <div |