fix: in-component navigation guards should not redirect

kiva · Dec 3, 2021 · 4346689 · 4346689
1 parent e712d7a
commit 4346689
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 14 deletions.
diff --git a/src/pages/LoginAndRegister/RegisterSocial.vue b/src/pages/LoginAndRegister/RegisterSocial.vue
@@ -127,22 +127,17 @@ export default {
 		}
 		return validations;
 	},
-	beforeRouteEnter(to, from, next) {
-		// Redirect to error page if query parameters are missing
-		const { state, terms, names } = to.query ?? {};
-		if (!state || !(terms || names)) {
-			next('/error');
-		} else {
-			next();
-		}
-	},
 	created() {
 		if (this.$route.query.terms) {
 			this.needsTerms = true;
 		}
 		if (this.$route.query.names) {
 			this.needsNames = true;
 		}
+		// Support legacy behavior of this page, which was to show the terms checkbox only
+		if (!this.$route.query.terms && !this.$route.query.names) {
+			this.needsTerms = true;
+		}
 	},
 	methods: {
 		postRegisterSocialForm() {

diff --git a/src/router/routes.js b/src/router/routes.js
@@ -220,7 +220,19 @@ export default [
 		}),
 	},
 	{ path: '/protocol', component: () => import('@/pages/Protocol') },
-	{ path: '/register/social', component: () => import('@/pages/LoginAndRegister/RegisterSocial') },
+	{
+		path: '/register/social',
+		component: () => import('@/pages/LoginAndRegister/RegisterSocial'),
+		beforeEnter(to, from, next) {
+			// Redirect to error page if state parameter is missing
+			const { state } = to.query ?? {};
+			if (!state) {
+				next('/error');
+			} else {
+				next();
+			}
+		},
+	},
 	{ path: '/register/guest', component: () => import('@/pages/LoginAndRegister/GuestAccountClaim') },
 	{
 		path: '/register/guest-redirect',

diff --git a/test/unit/specs/pages/LoginAndRegister/RegisterSocial.spec.js b/test/unit/specs/pages/LoginAndRegister/RegisterSocial.spec.js
@@ -1,17 +1,19 @@
-import RegisterSocial from '@/pages/LoginAndRegister/RegisterSocial';
+import routes from '@/router/routes';
+
+const beforeEnterGuard = routes.find(route => route.path === '/register/social').beforeEnter;
 
 // Test that RegisterSocial.beforeRouteEnter redirects to the error page for route object `to`
 const testRedirectToError = to => {
 	const next = jest.fn();
-	RegisterSocial.beforeRouteEnter(to, {}, next);
+	beforeEnterGuard(to, {}, next);
 	expect(next.mock.calls.length).toBe(1);
 	expect(next.mock.calls[0][0]).toBe('/error');
 };
 
 // Test that RegisterSocial.beforeRouteEnter does not redirect for route object `to`
 const testNoRedirect = to => {
 	const next = jest.fn();
-	RegisterSocial.beforeRouteEnter(to, {}, next);
+	beforeEnterGuard(to, {}, next);
 	expect(next.mock.calls.length).toBe(1);
 	expect(next.mock.calls[0][0]).not.toBeDefined();
 };
@@ -23,8 +25,8 @@ describe('RegisterSocial page', () => {
 		testRedirectToError({ query: { names: 1 } });
 		testRedirectToError({ query: { terms: 1 } });
 		testRedirectToError({ query: { names: 1, terms: 1 } });
-		testRedirectToError({ query: { state: 'abc' } });
 
+		testNoRedirect({ query: { state: 'abc' } });
 		testNoRedirect({ query: { state: 'abc', names: 1 } });
 		testNoRedirect({ query: { state: 'abc', terms: 1 } });
 		testNoRedirect({ query: { state: 'abc', names: 1, terms: 1 } });