From 2b7440524e2ff9d384a7aa61ff148dec2a018a14 Mon Sep 17 00:00:00 2001 From: Matthew Irish Date: Thu, 31 May 2018 16:22:49 -0500 Subject: [PATCH] UI - don't re-throw error when fetching features (#4669) --- ui/app/services/version.js | 2 +- ui/tests/acceptance/enterprise-replication-test.js | 11 +++++------ 2 files changed, 6 insertions(+), 7 deletions(-) diff --git a/ui/app/services/version.js b/ui/app/services/version.js index 8b07823051..5b3a079f7b 100644 --- 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 index 8489bc19d0..d722475dc4 100644 --- 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)'); });