From 366f14156d3c459bf6600da9f5bc1b10fe30dc72 Mon Sep 17 00:00:00 2001
From: Randall Morey <randall@randallmorey.com>
Date: Sat, 26 Sep 2020 10:45:18 -0400
Subject: [PATCH] =?UTF-8?q?fix:=20=F0=9F=90=9B=20speed=20up=20tests=20by?=
 =?UTF-8?q?=20eliminating=20notification=20timeout?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 ui/core/app/controllers/application.js | 5 +++++
 ui/core/app/templates/application.hbs  | 2 +-
 ui/core/config/environment.js          | 5 +++++
 3 files changed, 11 insertions(+), 1 deletion(-)

diff --git a/ui/core/app/controllers/application.js b/ui/core/app/controllers/application.js
index 565eb7d7a..0eb5226e0 100644
--- a/ui/core/app/controllers/application.js
+++ b/ui/core/app/controllers/application.js
@@ -1,8 +1,13 @@
 import Controller from '@ember/controller';
 import { inject as service } from '@ember/service';
+import config from '../config/environment';
 
 export default class ApplicationController extends Controller {
   // =services
 
   @service session;
+
+  // =attributes
+
+  notifyTimeout = config.notifyTimeout;
 }
diff --git a/ui/core/app/templates/application.hbs b/ui/core/app/templates/application.hbs
index 8e16b5615..30d1302fe 100644
--- a/ui/core/app/templates/application.hbs
+++ b/ui/core/app/templates/application.hbs
@@ -33,7 +33,7 @@
 
 </Rose::Layout::Global>
 
-<EmberNotify @closeAfter={{4000}} as |message close|>
+<EmberNotify @closeAfter={{this.notifyTimeout}} as |message close|>
   <Rose::Notification
     @style={{message.type}}
     @heading={{t (concat 'states.' message.type)}}
diff --git a/ui/core/config/environment.js b/ui/core/config/environment.js
index dfde07a60..6504f571b 100644
--- a/ui/core/config/environment.js
+++ b/ui/core/config/environment.js
@@ -33,6 +33,8 @@ module.exports = function (environment) {
 
     appName: APP_NAME,
 
+    notifyTimeout: 4000,
+
     contentSecurityPolicyHeader: 'Content-Security-Policy',
     contentSecurityPolicyMeta: true,
     contentSecurityPolicy: {
@@ -93,6 +95,9 @@ module.exports = function (environment) {
 
     // CSP breaks test coverage, so it is disabled in this environment
     ENV.contentSecurityPolicyMeta = false;
+
+    // Notification timeout should be 0 for fast tests
+    ENV.notifyTimeout = 0;
   }
 
   if (environment === 'production') {