Always show "email issue" button on group/project issues list refactor

Show the "email issue" button even when there are no issues in the
issues list.

The button will not show when the user is signed out, due
to the check in new_issuable_address.

Behind feature flag `vue_issues_list` defaulted off.

app/assets/javascripts/issues/list/components/issues_list_app.vue

@@ -260,6 +260,9 @@ export default {
     showCsvButtons() {
       return this.isProject && this.isSignedIn;
     },
+    showIssuableByEmail() {
+      return this.initialEmail && this.isSignedIn;
+    },
     showNewIssueDropdown() {
       return !this.isProject && this.hasAnyProjects;
     },
@@ -624,8 +627,9 @@ export default {
 </script>
 
 <template>
-  <div v-if="hasAnyIssues">
+  <div>
     <issuable-list
+      v-if="hasAnyIssues"
       :namespace="fullPath"
       recent-searches-storage-key="issues"
       :search-input-placeholder="$options.i18n.searchPlaceholder"
@@ -768,10 +772,7 @@ export default {
       </template>
     </issuable-list>
 
-    <issuable-by-email v-if="initialEmail" class="gl-text-center gl-pt-5 gl-pb-7" />
-  </div>
-
-  <div v-else-if="isSignedIn">
+    <template v-else-if="isSignedIn">
       <gl-empty-state
         :description="$options.i18n.noIssuesSignedInDescription"
         :title="$options.i18n.noIssuesSignedInTitle"
@@ -804,7 +805,7 @@ export default {
       <p class="gl-text-center gl-text-gray-500">
         {{ $options.i18n.jiraIntegrationSecondaryMessage }}
       </p>
-  </div>
+    </template>
 
     <gl-empty-state
       v-else
@@ -814,4 +815,7 @@ export default {
       :primary-button-text="$options.i18n.noIssuesSignedOutButtonText"
       :primary-button-link="signInPath"
     />
+
+    <issuable-by-email v-if="showIssuableByEmail" class="gl-text-center gl-pt-5 gl-pb-7" />
+  </div>
 </template>

spec/frontend/issues/list/components/issues_list_app_spec.js

@@ -452,13 +452,26 @@ describe('CE IssuesListApp component', () => {
   });
 
   describe('IssuableByEmail component', () => {
-    describe.each([true, false])(`when issue creation by email is enabled=%s`, (enabled) => {
-      it(`${enabled ? 'renders' : 'does not render'}`, () => {
-        wrapper = mountComponent({ provide: { initialEmail: enabled } });
+    describe.each`
+      initialEmail | hasAnyIssues | isSignedIn | exists
+      ${false}     | ${false}     | ${false}   | ${false}
+      ${false}     | ${true}      | ${false}   | ${false}
+      ${false}     | ${false}     | ${true}    | ${false}
+      ${false}     | ${true}      | ${true}    | ${false}
+      ${true}      | ${false}     | ${false}   | ${false}
+      ${true}      | ${true}      | ${false}   | ${false}
+      ${true}      | ${false}     | ${true}    | ${true}
+      ${true}      | ${true}      | ${true}    | ${true}
+    `(
+      `when issue creation by email is enabled=$initialEmail`,
+      ({ initialEmail, hasAnyIssues, isSignedIn, exists }) => {
+        it(`${initialEmail ? 'renders' : 'does not render'}`, () => {
+          wrapper = mountComponent({ provide: { initialEmail, hasAnyIssues, isSignedIn } });
 
-        expect(findIssuableByEmail().exists()).toBe(enabled);
-      });
+          expect(findIssuableByEmail().exists()).toBe(exists);
         });
+      },
+    );
   });
 
   describe('empty states', () => {