Merge branch 'dont-show-bot-docs-link-in-popover-if-no-url' into 'master'

Do not show bot docs link in popover if website_url is missing

See merge request gitlab-org/gitlab!81089

app/assets/javascripts/vue_shared/components/user_popover/user_popover.vue

@@ -116,7 +116,7 @@ export default {
           <div v-if="statusHtml" class="gl-mb-2" data-testid="user-popover-status">
             <span v-safe-html:[$options.safeHtmlConfig]="statusHtml"></span>
           </div>
-          <div v-if="user.bot" class="gl-text-blue-500">
+          <div v-if="user.bot && user.websiteUrl" class="gl-text-blue-500">
             <gl-icon name="question" />
             <gl-link data-testid="user-popover-bot-docs-link" :href="user.websiteUrl">
               <gl-sprintf :message="__('Learn more about %{username}')">

spec/frontend/vue_shared/components/user_popover/user_popover_spec.js

@@ -271,6 +271,12 @@ describe('User Popover Component', () => {
       expect(securityBotDocsLink.text()).toBe('Learn more about GitLab Security Bot');
     });
 
+    it("does not show a link to the bot's documentation if there is no website_url", () => {
+      createWrapper({ user: { ...SECURITY_BOT_USER, websiteUrl: null } });
+      const securityBotDocsLink = findSecurityBotDocsLink();
+      expect(securityBotDocsLink.exists()).toBe(false);
+    });
+
     it("doesn't escape user's name", () => {
       createWrapper({ user: { ...SECURITY_BOT_USER, name: '%<>\';"' } });
       const securityBotDocsLink = findSecurityBotDocsLink();