UI - don't re-throw error when fetching features (#4669)

hashicorp · May 31, 2018 · 2b74405 · 2b74405
1 parent 86d4b32
commit 2b74405
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 7 deletions.
diff --git a/ui/app/services/version.js b/ui/app/services/version.js
@@ -57,7 +57,7 @@ export default Service.extend({
       this.setFeatures(response);
       return;
     } catch (err) {
-      throw err;
+      // if we fail here, we're likely in DR Secondary mode and don't need to worry about it
     }
   }).keepLatest(),
 

diff --git a/ui/tests/acceptance/enterprise-replication-test.js b/ui/tests/acceptance/enterprise-replication-test.js
@@ -103,12 +103,11 @@ test('replication', function(assert) {
       `/vault/replication/performance/secondaries`,
       'redirects to the secondaries page'
     );
-    assert
-      .dom('[data-test-flash-message-body]:contains(The performance mount filter)')
-      .hasText(
-        `The performance mount filter config for the secondary ${secondaryName} was successfully deleted.`,
-        'renders success flash upon deletion'
-      );
+    assert.equal(
+      find('[data-test-flash-message-body]:contains(The performance mount filter)').text().trim(),
+      `The performance mount filter config for the secondary ${secondaryName} was successfully deleted.`,
+      'renders success flash upon deletion'
+    );
     click('[data-test-flash-message-body]:contains(The performance mount filter)');
   });