From 22dc7db6119b061efe8ed4a856a8e7809876b8e6 Mon Sep 17 00:00:00 2001
From: Phil Renaud <phil@riotindustries.com>
Date: Fri, 1 Nov 2024 21:57:10 -0400
Subject: [PATCH] unit test condition moved to stubbable acceptance test

---
 ui/app/styles/core/navbar.scss                |  2 +-
 .../templates/components/region-switcher.hbs  |  2 +-
 ui/tests/acceptance/regions-test.js           |  6 +++--
 ui/tests/pages/layout.js                      |  5 ++++
 .../unit/components/region-switcher-test.js   | 25 -------------------
 5 files changed, 11 insertions(+), 29 deletions(-)
 delete mode 100644 ui/tests/unit/components/region-switcher-test.js

diff --git a/ui/app/styles/core/navbar.scss b/ui/app/styles/core/navbar.scss
index cff29ac5b37..a26dee23ef4 100644
--- a/ui/app/styles/core/navbar.scss
+++ b/ui/app/styles/core/navbar.scss
@@ -142,7 +142,7 @@ $secondaryNavbarHeight: 4.5rem;
     display: flex;
     align-items: center;
 
-    &.is-region {
+    &.single-region {
       display: block;
       padding: 0;
       font-size: 1em;
diff --git a/ui/app/templates/components/region-switcher.hbs b/ui/app/templates/components/region-switcher.hbs
index 1d592cbdc8a..9d9817295b8 100644
--- a/ui/app/templates/components/region-switcher.hbs
+++ b/ui/app/templates/components/region-switcher.hbs
@@ -22,7 +22,7 @@
     </PowerSelect>
   </span>
 {{else if this.system.hasNonDefaultRegion}}
-  <div class="navbar-item is-region">
+  <div class="navbar-item single-region">
     <span>Region: </span>{{this.system.activeRegion}}
   </div>
 {{/if}}
diff --git a/ui/tests/acceptance/regions-test.js b/ui/tests/acceptance/regions-test.js
index 14b90619e18..a17529fd3b9 100644
--- a/ui/tests/acceptance/regions-test.js
+++ b/ui/tests/acceptance/regions-test.js
@@ -36,21 +36,23 @@ module('Acceptance | regions (only one)', function (hooks) {
     await a11yAudit(assert);
   });
 
-  test('when there is only one region, the region switcher is not shown in the nav bar and the region is not in the page title', async function (assert) {
+  test('when there is only one region, and it is the default one, the region switcher is not shown in the nav bar and the region is not in the page title', async function (assert) {
     server.create('region', { id: 'global' });
 
     await JobsList.visit();
 
     assert.notOk(Layout.navbar.regionSwitcher.isPresent, 'No region switcher');
+    assert.notOk(Layout.navbar.singleRegion.isPresent, 'No single region');
     assert.ok(document.title.includes('Jobs'));
   });
 
-  test('when the only region is not named "global", the region switcher still is not shown', async function (assert) {
+  test('when the only region is not named "global", the region switcher still is not shown, but the single region name is', async function (assert) {
     server.create('region', { id: 'some-region' });
 
     await JobsList.visit();
 
     assert.notOk(Layout.navbar.regionSwitcher.isPresent, 'No region switcher');
+    assert.ok(Layout.navbar.singleRegion.isPresent, 'Single region');
   });
 
   test('pages do not include the region query param', async function (assert) {
diff --git a/ui/tests/pages/layout.js b/ui/tests/pages/layout.js
index 3fbca6063f8..d521bd382a0 100644
--- a/ui/tests/pages/layout.js
+++ b/ui/tests/pages/layout.js
@@ -28,6 +28,11 @@ export default create({
       }),
     },
 
+    singleRegion: {
+      scope: '.single-region',
+      text: text(),
+    },
+
     search: {
       scope: '[data-test-search-parent]',
 
diff --git a/ui/tests/unit/components/region-switcher-test.js b/ui/tests/unit/components/region-switcher-test.js
deleted file mode 100644
index 15a67b847ee..00000000000
--- a/ui/tests/unit/components/region-switcher-test.js
+++ /dev/null
@@ -1,25 +0,0 @@
-/**
- * Copyright (c) HashiCorp, Inc.
- * SPDX-License-Identifier: BUSL-1.1
- */
-
-import { module, test } from 'qunit';
-import { setupRenderingTest } from 'ember-qunit';
-import { render } from '@ember/test-helpers';
-import { hbs } from 'ember-cli-htmlbars';
-
-module('Unit | Component | region-switcher', function (hooks) {
-  setupRenderingTest(hooks);
-
-  test('displays single region', async function (assert) {
-    const system = {
-      shouldShowRegions: false,
-    };
-
-    this.set('system', system);
-
-    await render(hbs`<RegionSwitcher/>`);
-
-    assert.dom('.is-region').exists();
-  });
-});