Changes after review

app/assets/javascripts/clusters/components/applications.vue

@@ -345,7 +345,6 @@ export default {
           NOTE: Don't forget to update `clusters.scss`
           min-height for this block and uncomment `application_spec` tests
         -->
-        <!-- Add Jupyter row, all other plumbing is complete -->
       </div>
     </div>
   </section>

app/assets/javascripts/clusters/stores/clusters_store.js

@@ -92,7 +92,11 @@ export default class ClusterStore {
       if (appId === INGRESS) {
         this.state.applications.ingress.externalIp = serverAppEntry.external_ip;
       } else if (appId === JUPYTER) {
-        this.state.applications.jupyter.hostname = serverAppEntry.hostname || this.state.applications.ingress.externalIp ? `jupyter.${this.state.applications.ingress.externalIp}.xip.io` : '';
+        this.state.applications.jupyter.hostname =
+          serverAppEntry.hostname ||
+          (this.state.applications.ingress.externalIp
+            ? `jupyter.${this.state.applications.ingress.externalIp}.xip.io`
+            : '');
       }
     });
   }

app/assets/stylesheets/pages/clusters.scss

@@ -6,7 +6,7 @@
 .cluster-applications-table {
   // Wait for the Vue to kick-in and render the applications block
-  min-height: 500px;
+  min-height: 628px;
 }
 .clusters-dropdown-menu {

spec/javascripts/clusters/components/applications_spec.js

@@ -128,21 +128,37 @@ describe('Applications', () => {
           expect(vm.$el.querySelector('.js-hostname').getAttribute('readonly')).toEqual(null);
         });
-        describe('with ingress & jupyter installed', () => {
-          it('renders readonly input', () => {
-            vm = mountComponent(Applications, {
-              applications: {
-                helm: { title: 'Helm Tiller', status: 'installed' },
-                ingress: { title: 'Ingress', status: 'installed', externalIp: '1.1.1.1' },
-                runner: { title: 'GitLab Runner' },
-                prometheus: { title: 'Prometheus' },
-                jupyter: { title: 'JupyterHub', status: 'installed', hostname: '' },
-              },
-            });
-            expect(vm.$el.querySelector('.js-hostname').getAttribute('readonly')).toEqual('readonly');
+      });
+      describe('with ingress installed without external ip', () => {
+        it('does not render hostname input', () => {
+          vm = mountComponent(Applications, {
+            applications: {
+              helm: { title: 'Helm Tiller', status: 'installed' },
+              ingress: { title: 'Ingress', status: 'installed' },
+              runner: { title: 'GitLab Runner' },
+              prometheus: { title: 'Prometheus' },
+              jupyter: { title: 'JupyterHub', hostname: '', status: 'installable' },
+            },
+          });
+
+          expect(vm.$el.querySelector('.js-hostname')).toBe(null);
+        });
+      });
+      describe('with ingress & jupyter installed', () => {
+        it('renders readonly input', () => {
+          vm = mountComponent(Applications, {
+            applications: {
+              helm: { title: 'Helm Tiller', status: 'installed' },
+              ingress: { title: 'Ingress', status: 'installed', externalIp: '1.1.1.1' },
+              runner: { title: 'GitLab Runner' },
+              prometheus: { title: 'Prometheus' },
+              jupyter: { title: 'JupyterHub', status: 'installed', hostname: '' },
+            },
           });
+
+          expect(vm.$el.querySelector('.js-hostname').getAttribute('readonly')).toEqual('readonly');
         });
       });