Commit 6028d44a authored by Nicolò Maria Mezzopera's avatar Nicolò Maria Mezzopera

Merge branch 'test-report-link-fix' into 'master'

Test report link fix

See merge request gitlab-org/gitlab!36220
parents 0aad74c8 2310a80d
...@@ -114,7 +114,12 @@ export default { ...@@ -114,7 +114,12 @@ export default {
class="mr-widget-section grouped-security-reports mr-report" class="mr-widget-section grouped-security-reports mr-report"
> >
<template v-if="showViewFullReport" #actionButtons> <template v-if="showViewFullReport" #actionButtons>
<gl-button :href="testTabURL" icon="external-link" data-testid="group-test-reports-full-link"> <gl-button
:href="testTabURL"
icon="external-link"
data-testid="group-test-reports-full-link"
class="gl-mr-3"
>
{{ s__('ciReport|View full report') }} {{ s__('ciReport|View full report') }}
</gl-button> </gl-button>
</template> </template>
......
...@@ -384,7 +384,7 @@ export default { ...@@ -384,7 +384,7 @@ export default {
v-if="mr.testResultsPath" v-if="mr.testResultsPath"
class="js-reports-container" class="js-reports-container"
:endpoint="mr.testResultsPath" :endpoint="mr.testResultsPath"
:pipeline-path="mr.mergeRequestAddCiConfigPath" :pipeline-path="mr.pipeline.path"
/> />
<terraform-plan v-if="mr.terraformReportsPath" :endpoint="mr.terraformReportsPath" /> <terraform-plan v-if="mr.terraformReportsPath" :endpoint="mr.terraformReportsPath" />
......
---
title: Fix for test report link in MR widget
merge_request:
author:
type: fixed
...@@ -458,7 +458,7 @@ export default { ...@@ -458,7 +458,7 @@ export default {
v-if="mr.testResultsPath" v-if="mr.testResultsPath"
class="js-reports-container" class="js-reports-container"
:endpoint="mr.testResultsPath" :endpoint="mr.testResultsPath"
:pipeline-path="mr.mergeRequestAddCiConfigPath" :pipeline-path="mr.pipeline.path"
/> />
<terraform-plan v-if="mr.terraformReportsPath" :endpoint="mr.terraformReportsPath" /> <terraform-plan v-if="mr.terraformReportsPath" :endpoint="mr.terraformReportsPath" />
......
...@@ -91,6 +91,7 @@ describe('Grouped test reports app', () => { ...@@ -91,6 +91,7 @@ describe('Grouped test reports app', () => {
const fullTestReportLink = findFullTestReportLink(); const fullTestReportLink = findFullTestReportLink();
expect(fullTestReportLink.exists()).toBe(true); expect(fullTestReportLink.exists()).toBe(true);
expect(pipelinePath).not.toBe('');
expect(fullTestReportLink.attributes('href')).toBe(`${pipelinePath}/test_report`); expect(fullTestReportLink.attributes('href')).toBe(`${pipelinePath}/test_report`);
}); });
}); });
......
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