fix(authentication): remove test block (#691)

RedHatInsights · Jul 13, 2021 · 7b34b6b · 7b34b6b
1 parent 896b9f8
commit 7b34b6b
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 7 deletions.
diff --git a/src/components/authentication/__tests__/__snapshots__/authentication.test.js.snap b/src/components/authentication/__tests__/__snapshots__/authentication.test.js.snap
@@ -286,7 +286,12 @@ exports[`Authentication Component should return a message on 401 error: 401 erro
 </MessageView>
 `;
 
-exports[`Authentication Component should return a redirect on 418 error: 418 error 1`] = `"418 redirect"`;
+exports[`Authentication Component should return a redirect on 418 error: 418 error 1`] = `
+<withRouter(Redirect)
+  isRedirect={true}
+  route="/optin"
+/>
+`;
 
 exports[`Authentication Component should return a redirect on a specific 403 error and error code: 403 error 1`] = `
 <MessageView
@@ -307,4 +312,9 @@ exports[`Authentication Component should return a redirect on a specific 403 err
 </MessageView>
 `;
 
-exports[`Authentication Component should return a redirect on a specific 403 error and error code: 403 redirect error 1`] = `"403 redirect"`;
+exports[`Authentication Component should return a redirect on a specific 403 error and error code: 403 redirect error 1`] = `
+<withRouter(Redirect)
+  isRedirect={true}
+  route="/optin"
+/>
+`;
diff --git a/src/components/authentication/__tests__/authentication.test.js b/src/components/authentication/__tests__/authentication.test.js
@@ -64,7 +64,7 @@ describe('Authentication Component', () => {
       </Authentication>
     );
 
-    expect(component.html()).toMatchSnapshot('418 error');
+    expect(component).toMatchSnapshot('418 error');
   });
 
   it('should return a redirect on a specific 403 error and error code', () => {
@@ -88,7 +88,7 @@ describe('Authentication Component', () => {
       </Authentication>
     );
 
-    expect(component.html()).toMatchSnapshot('403 redirect error');
+    expect(component).toMatchSnapshot('403 redirect error');
 
     component.setProps({
       session: {

diff --git a/src/components/authentication/authentication.js b/src/components/authentication/authentication.js
@@ -85,9 +85,6 @@ class Authentication extends Component {
       (session.errorCodes && session.errorCodes.includes(rhsmApiTypes.RHSM_API_RESPONSE_ERROR_DATA_CODE_TYPES.OPTIN)) ||
       session.status === 418
     ) {
-      if (helpers.TEST_MODE) {
-        return <React.Fragment>{session.status} redirect</React.Fragment>;
-      }
       return <Redirect isRedirect route={routerHelpers.getErrorRoute.path} />;
     }