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

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(),
 

ui/tests/acceptance/enterprise-replication-test.js

@@ -103,9 +103,8 @@ 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(
+    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'
     );