Test and support switching versions from 'release'

Author: kategengler

app/controllers/project-version.js

@@ -147,6 +147,7 @@ export default class ProjectVersionController extends Controller {
         project,
         targetVersion: ver.id,
         currentVersion: this.projectService.version,
+        currentUrlVersion: this.projectService.getUrlVersion(),
         currentURL,
         currentAnchor: window.location.hash,
       }),
@@ -158,6 +159,7 @@ export function findEndingRoute({
   project,
   targetVersion,
   currentVersion,
+  currentUrlVersion,
   currentURL,
   currentAnchor,
 }) {
@@ -167,7 +169,7 @@ export function findEndingRoute({
   if (shouldGoToVersionIndex(project, targetVersion, currentVersion)) {
     return `/${project}/${projectVersionID}`;
   } else {
-    return `${currentURL.replace(getCompactVersion(currentVersion), projectVersionID)}${currentAnchor}`;
+    return `${currentURL.replace(currentUrlVersion, projectVersionID)}${currentAnchor}`;
   }
 }
 

tests/acceptance/switch-versions-test.js

@@ -38,6 +38,23 @@ module('Acceptance | version navigation', function (hooks) {
     );
   });
 
+  test('switching versions from release', async function (assert) {
+    await visit('/ember/release/modules/@glimmer%2Ftracking');
+
+    assert.equal(
+      currentURL(),
+      '/ember/release/modules/@glimmer%2Ftracking',
+      'navigated to release',
+    );
+    await selectChoose('.ember-power-select-trigger', '6.4');
+
+    assert.equal(
+      currentURL(),
+      '/ember/6.4/modules/@glimmer%2Ftracking',
+      'navigated to v6.4 class',
+    );
+  });
+
   test('switching namespace versions less than 2.16 should retain namespace page', async function (assert) {
     await visit('/ember/2.7/namespaces/Ember');
     await waitForSettled();

tests/unit/controllers/project-version-test.js

@@ -53,6 +53,7 @@ module('Unit | Controller | project version', function (hooks) {
         project: 'ember',
         targetVersion: '6.4.0',
         currentVersion: '6.5.0',
+        currentUrlVersion: '6.5',
         currentURL: '/ember/6.5/classes/Component',
         currentAnchor: '#didInsertElement',
       });
@@ -66,6 +67,7 @@ module('Unit | Controller | project version', function (hooks) {
         project: 'ember',
         targetVersion: '6.4.0',
         currentVersion: '6.5.0',
+        currentUrlVersion: '6.5',
         currentURL: '/ember/6.5/modules/%40ember%2Fapplication',
         currentAnchor: '#classes',
       });
@@ -81,6 +83,7 @@ module('Unit | Controller | project version', function (hooks) {
         project: 'ember',
         targetVersion: '2.15.0',
         currentVersion: '2.16.0',
+        currentUrlVersion: '2.16',
         currentURL: '/ember/2.16/classes/Component',
         currentAnchor: '#didInsertElement',
       });
@@ -91,6 +94,7 @@ module('Unit | Controller | project version', function (hooks) {
         project: 'ember',
         targetVersion: '2.16.0',
         currentVersion: '2.15.0',
+        currentUrlVersion: '2.15',
         currentURL: '/ember/2.15/classes/Component',
         currentAnchor: '#didInsertElement',
       });
@@ -103,6 +107,7 @@ module('Unit | Controller | project version', function (hooks) {
         project: 'ember-data',
         targetVersion: '3.28.0',
         currentVersion: '4.0.0',
+        currentUrlVersion: '4.0',
         currentURL: '/ember-data/4.0/classes/Adapter',
         currentAnchor: '',
       });
@@ -113,6 +118,7 @@ module('Unit | Controller | project version', function (hooks) {
         project: 'ember-data',
         targetVersion: '4.0.0',
         currentVersion: '3.28.0',
+        currentUrlVersion: '3.28',
         currentURL: '/ember-data/3.28/classes/DS.Adapter',
         currentAnchor: '',
       });