refactor: 💡 confirmations should be enabled by default

addons/core/addon/services/confirm.js

@@ -19,9 +19,11 @@ export default class ConfirmationsService extends Service {
 
   /**
    * When the confirm service is disabled, it always returns resolving promises.
+   * By default the service is enabled but it is useful to disable it for test
+   * purposes.
    * @type {boolean}
    */
-  enabled = config.enableConfirmService;
+  enabled = config.enableConfirmService ?? true;
 
   /**
    * @type {Array}

addons/core/tests/dummy/config/environment.js

@@ -39,7 +39,9 @@ module.exports = function (environment) {
 
     'ember-cli-mirage': {
       directory: '../api/mirage'
-    }
+    },
+
+    enableConfirmService: false
   };
 
   if (environment === "development") {

ui/admin/config/environment.js

@@ -36,7 +36,6 @@ module.exports = function (environment) {
 
     notifyTimeout: 4000,
     sessionPollingTimeoutSeconds: 2.5,
-    enableConfirmService: true,
 
     documentation: {
       baseURL: 'https://boundaryproject.io/help/admin-ui',

ui/desktop/config/environment.js

@@ -70,6 +70,8 @@ module.exports = function (environment) {
 
     // Notification timeout should be 0 for fast tests
     ENV.notifyTimeout = 0;
+    
+    ENV.enableConfirmService = false;
   }
 
   if (environment === 'production') {