Commit 7e7dc5e5 authored by mfluharty's avatar mfluharty

Add context to full code quality report

In the full code quality report on the pipeline page
Add an explanatory/context sentence as a subheading
parent 8c6d3c42
......@@ -41,6 +41,9 @@ export default {
return this.checkReportStatus(this.isLoadingCodequality, this.loadingCodequalityFailed);
},
},
i18n: {
subHeading: s__('ciReport|This report contains all Code Quality issues in the source branch.'),
},
methods: {
...mapActions(['setPage']),
translateText(type) {
......@@ -70,7 +73,9 @@ export default {
:has-issues="hasCodequalityIssues"
:component="$options.componentNames.CodequalityIssueBody"
class="codequality-report"
/>
>
<template v-if="hasCodequalityIssues" #sub-heading>{{ $options.i18n.subHeading }}</template>
</report-section>
<pagination-links
:change="setPage"
:page-info="pageInfo"
......
......@@ -68,7 +68,10 @@ describe('Codequality report app', () => {
const expectedIssueTotal = parsedIssues.length;
expect(findWarningIcon().exists()).toBe(true);
expect(findStatus().text()).toBe(`Found ${expectedIssueTotal} code quality issues`);
expect(findStatus().text()).toContain(`Found ${expectedIssueTotal} code quality issues`);
expect(findStatus().text()).toContain(
`This report contains all Code Quality issues in the source branch.`,
);
expect(wrapper.findAll('.report-block-list-issue')).toHaveLength(expectedIssueTotal);
});
......
......@@ -32508,6 +32508,9 @@ msgstr ""
msgid "ciReport|There was an error reverting the dismissal. Please try again."
msgstr ""
msgid "ciReport|This report contains all Code Quality issues in the source branch."
msgstr ""
msgid "ciReport|Used by %{packagesString}"
msgid_plural "ciReport|Used by %{packagesString}, and %{lastPackage}"
msgstr[0] ""
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment