fix: conditional tests

villander · Nov 4, 2023 · 4092f05 · 4092f05
1 parent 30da677
commit 4092f05
Show file tree

Hide file tree

Showing 3 changed files with 43 additions and 32 deletions.
diff --git a/ember-engines-router-service/src/services/engine-router-service.js b/ember-engines-router-service/src/services/engine-router-service.js
@@ -10,6 +10,9 @@ import { namespaceEngineRouteName } from '../utils/namespace-engine-route-name';
 import { getRootOwner } from '../utils/root-owner';
 import { resemblesURL } from '../utils/resembles-url';
 
+const warningMessage =
+  'Refresh method is not available in ember-source below v4.1';
+
 export default class EngineRouterService extends Service.extend(Evented) {
   constructor(...args) {
     super(...args);
@@ -77,7 +80,7 @@ export default class EngineRouterService extends Service.extend(Evented) {
         ...args
       );
     } else {
-      assert('Refresh method is not available in ember-source below v4.1');
+      assert(warningMessage);
     }
   }
 
@@ -88,7 +91,7 @@ export default class EngineRouterService extends Service.extend(Evented) {
         ...args
       );
     } else {
-      assert('Refresh method is not available in ember-source below v4.1');
+      assert(warningMessage);
     }
   }
 

diff --git a/test-app/package.json b/test-app/package.json
@@ -37,6 +37,7 @@
     "@ember/optional-features": "^2.0.0",
     "@ember/string": "^3.1.1",
     "@ember/test-helpers": "^2.9.3",
+    "@embroider/macros": "^1.13.2",
     "@embroider/test-setup": "^2.0.2",
     "@glimmer/component": "^1.1.2",
     "@glimmer/tracking": "^1.1.2",

diff --git a/test-app/tests/acceptance/routeable-engine-demo-refresh-test.js b/test-app/tests/acceptance/routeable-engine-demo-refresh-test.js
@@ -1,43 +1,50 @@
 import { module, test } from 'qunit';
 import { setupApplicationTest } from 'ember-qunit';
 import { visit, find, click } from '@ember/test-helpers';
+import { macroCondition, dependencySatisfies } from '@embroider/macros';
 
-module('Acceptance | Engine Router Service | Refresh Method', function (hooks) {
-  setupApplicationTest(hooks);
+if (macroCondition(dependencySatisfies('ember-source', '>= 4.1.0'))) {
+  module(
+    'Acceptance | Engine Router Service | Refresh Method',
+    function (hooks) {
+      setupApplicationTest(hooks);
 
-  test('refresh without params triggers refresh with current route', async function (assert) {
-    await visit('/routable-engine-demo/ember-blog/new');
+      test('refresh without params triggers refresh with current route', async function (assert) {
+        await visit('/routable-engine-demo/ember-blog/new');
 
-    let counter = await find('.route-refresh-counter').textContent;
-    await click('.refresh');
+        let counter = await find('.route-refresh-counter').textContent;
 
-    counter = parseInt(counter, 10);
-    counter = ++counter;
-    counter = counter.toString();
-    assert.dom('.route-refresh-counter').hasText(counter);
-  });
+        await click('.refresh');
 
-  test('refresh with params triggers refresh on provided route', async function (assert) {
-    await visit('/routable-engine-demo/ember-blog/new');
+        counter = parseInt(counter, 10);
+        counter = ++counter;
+        counter = counter.toString();
+        assert.dom('.route-refresh-counter').hasText(counter);
+      });
 
-    let counter = await find('.route-refresh-counter').textContent;
-    await click('.refresh-route');
+      test('refresh with params triggers refresh on provided route', async function (assert) {
+        await visit('/routable-engine-demo/ember-blog/new');
 
-    counter = parseInt(counter, 10);
-    counter = ++counter;
-    counter = counter.toString();
-    assert.dom('.route-refresh-counter').hasText(counter);
-  });
+        let counter = await find('.route-refresh-counter').textContent;
+        await click('.refresh-route');
 
-  test('refresh external route', async function (assert) {
-    await visit('/routable-engine-demo/ember-blog/new');
+        counter = parseInt(counter, 10);
+        counter = ++counter;
+        counter = counter.toString();
+        assert.dom('.route-refresh-counter').hasText(counter);
+      });
 
-    let counter = await find('.route-refresh-counter').textContent;
-    await click('.refresh-external');
+      test('refresh external route', async function (assert) {
+        await visit('/routable-engine-demo/ember-blog/new');
 
-    counter = parseInt(counter, 10);
-    counter = ++counter;
-    counter = counter.toString();
-    assert.dom('.global-refresh-counter').hasText(counter);
-  });
-});
+        let counter = await find('.route-refresh-counter').textContent;
+        await click('.refresh-external');
+
+        counter = parseInt(counter, 10);
+        counter = ++counter;
+        counter = counter.toString();
+        assert.dom('.global-refresh-counter').hasText(counter);
+      });
+    }
+  );
+}