Commit cb1ea667 authored by Jose Ivan Vargas's avatar Jose Ivan Vargas

Merge branch '46540-fix-vulnerability-scanners-graphql' into 'master'

Fix typos in vulnerability scanners GraphQL and should typo in files

See merge request gitlab-org/gitlab!50552
parents cf4abe87 5ac25eb5
......@@ -107,7 +107,7 @@ export default {
},
computed: {
/**
* Determines if we shoud render the ref info section based
* Determines if we should render the ref info section based
*/
shouldShowRefInfo() {
return this.showRefInfo && (this.commitRef || this.mergeRequestRef);
......
......@@ -11,8 +11,6 @@ import Filters from './first_class_vulnerability_filters.vue';
import CsvExportButton from './csv_export_button.vue';
import { vulnerabilitiesSeverityCountScopes } from '../constants';
export const BANNER_COOKIE_KEY = 'hide_vulnerabilities_introduction_banner';
export default {
components: {
AutoFixUserCallout,
......@@ -43,11 +41,11 @@ export default {
},
},
data() {
const shoudShowAutoFixUserCallout =
const shouldShowAutoFixUserCallout =
this.glFeatures.securityAutoFix && !Cookies.get('auto_fix_user_callout_dismissed');
return {
filters: {},
shoudShowAutoFixUserCallout,
shouldShowAutoFixUserCallout,
};
},
methods: {
......@@ -56,7 +54,7 @@ export default {
},
handleAutoFixUserCalloutClose() {
Cookies.set('auto_fix_user_callout_dismissed', 'true');
this.shoudShowAutoFixUserCallout = false;
this.shouldShowAutoFixUserCallout = false;
},
},
vulnerabilitiesSeverityCountScopes,
......@@ -67,7 +65,7 @@ export default {
<div>
<template v-if="pipeline.id">
<auto-fix-user-callout
v-if="shoudShowAutoFixUserCallout"
v-if="shouldShowAutoFixUserCallout"
:help-page-path="autoFixDocumentation"
@close="handleAutoFixUserCalloutClose"
/>
......
#import "./vulnerability_scanner.fragment.graphql"
query projectSpecificScanners($fullpath: ID!) {
project(fullPath: $fullpath) {
query projectSpecificScanners($fullPath: ID!) {
project(fullPath: $fullPath) {
vulnerabilityScanners {
nodes {
...VulnerabilityScanner
......
......@@ -52,11 +52,11 @@ describe('Dependency Location component', () => {
});
});
it('shoud render the popover', () => {
it('should render the popover', () => {
expect(findPopover().exists()).toBe(true);
});
it('shoud have the complete path', () => {
it('should have the complete path', () => {
expect(trimText(findPopover().text())).toBe(
'swell 1.2 / emmajsq 10.11 / zeb 12.1 / post 2.5 / core 1.0 There may be multiple paths',
);
......
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