🚧 (mon-pix): wip

1024pix · Jul 4, 2024 · f9cfb4c · f9cfb4c
1 parent 231c583
commit f9cfb4c
Show file tree

Hide file tree

Showing 8 changed files with 80 additions and 45 deletions.
diff --git a/mon-pix/app/components/certification-instructions/steps.js b/mon-pix/app/components/certification-instructions/steps.js
@@ -43,20 +43,27 @@ export default class Steps extends Component {
   }
 
   @action
-  nextStep() {
+  async nextStep() {
     if (this.pageId < this.pageCount) {
       this.pageId = this.pageId + 1;
     }
 
     if (this.isConfirmationCheckboxChecked) {
-      this.router.transitionTo('authenticated.certifications.start', this.args.candidateId, {
-        queryParams: {
-          isConfirmationCheckboxChecked: this.isConfirmationCheckboxChecked,
-        },
-      });
+      await this.submit();
+
+      this.router.transitionTo('authenticated.certifications.start', this.args.candidate.id);
     }
   }
 
+  @action
+  async submit() {
+    await this.args.candidate.save({
+      adapterOptions: {
+        hasSeenCertificationInstructions: true,
+      },
+    });
+  }
+
   @action
   enableNextButton(checked) {
     this.isConfirmationCheckboxChecked = checked;

diff --git a/mon-pix/app/models/certification-candidate.js b/mon-pix/app/models/certification-candidate.js
@@ -5,6 +5,7 @@ export default class CertificationCandidate extends Model {
   @attr('string') firstName;
   @attr('string') lastName;
   @attr('date-only') birthdate;
+  @attr('boolean') hasSeenCertificationInstructions;
 
   // references
   @attr('number') sessionId;

diff --git a/mon-pix/app/routes/authenticated/certifications/information.js b/mon-pix/app/routes/authenticated/certifications/information.js
@@ -3,8 +3,15 @@ import { service } from '@ember/service';
 
 export default class InformationRoute extends Route {
   @service store;
+  @service router;
 
   async model(params) {
-    return this.store.peekRecord('certification-candidate', params.certification_candidate_id);
+    const certificationCandidate = this.store.peekRecord('certification-candidate', params.certification_candidate_id);
+
+    if (!certificationCandidate) {
+      this.router.replaceWith('authenticated.certifications.join');
+    }
+
+    return certificationCandidate;
   }
 }
diff --git a/mon-pix/app/routes/authenticated/certifications/start.js b/mon-pix/app/routes/authenticated/certifications/start.js
@@ -5,20 +5,22 @@ export default class StartRoute extends Route {
   @service store;
   @service router;
   @service featureToggles;
-  hasSeenCertificationInstructions = false;
-
-  beforeModel(transition) {
-    this.hasSeenCertificationInstructions = transition.to.queryParams.isConfirmationCheckboxChecked;
-  }
 
   async model(params) {
     const certificationCandidateSubscription = await this.store.findRecord(
       'certification-candidate-subscription',
       params.certification_candidate_id,
     );
 
+    const certificationCandidate = await this.store.peekRecord(
+      'certification-candidate',
+      params.certification_candidate_id,
+    );
+
+    const hasSeenCertificationInstructions = certificationCandidate?.hasSeenCertificationInstructions;
+
     if (
-      !this.hasSeenCertificationInstructions &&
+      !hasSeenCertificationInstructions &&
       certificationCandidateSubscription.isSessionVersion3 &&
       this.featureToggles.featureToggles.areV3InfoScreensEnabled
     ) {

diff --git a/mon-pix/app/templates/authenticated/certifications/information.hbs b/mon-pix/app/templates/authenticated/certifications/information.hbs
@@ -6,6 +6,6 @@
     <h1 class="instructions-header__title">{{t "pages.certification-instructions.title"}}</h1>
   </header>
   <PixBlock @shadow="heavy" class="instructions-step">
-    <CertificationInstructions::Steps @candidateId={{this.model.id}} />
+    <CertificationInstructions::Steps @candidate={{this.model}} />
   </PixBlock>
 </main>
diff --git a/mon-pix/tests/unit/adapters/certification-candidate_test.js b/mon-pix/tests/unit/adapters/certification-candidate_test.js
@@ -40,7 +40,7 @@ module('Unit | Adapters | certification-candidate', function (hooks) {
         const url = await adapter.urlForUpdateRecord(456, 'certification-candidate', options);
 
         // then
-        assert.true(url.endsWith('/certification-candidate/456/validate-certification-instructions'));
+        assert.true(url.endsWith('/certification-candidates/456/validate-certification-instructions'));
       });
     });
   });

diff --git a/mon-pix/tests/unit/components/certification-instructions/steps_test.js b/mon-pix/tests/unit/components/certification-instructions/steps_test.js
@@ -27,7 +27,7 @@ module('Unit | Component | certification-instruction | steps', function (hooks)
 
     module('when pageId equal pageCount', function () {
       module('when confirmation checkbox is checked', function () {
-        test('should redirect to certificartion starter', async function (assert) {
+        test.only('should redirect to certificartion starter', async function (assert) {
           // given
           const component = createGlimmerComponent('certification-instructions/steps');
 
@@ -38,21 +38,20 @@ module('Unit | Component | certification-instruction | steps', function (hooks)
           };
           component.isConfirmationCheckboxChecked = true;
           const transitionToStub = sinon.stub();
+          const saveStub = sinon.stub();
+          saveStub.resolves();
           component.router = {
             transitionTo: transitionToStub,
           };
+          component.args.candidate = {
+            save: saveStub,
+          };
 
           // when
           await component.nextStep();
 
           // then
-          assert.ok(
-            transitionToStub.calledWith('authenticated.certifications.start', 123, {
-              queryParams: {
-                isConfirmationCheckboxChecked: true,
-              },
-            }),
-          );
+          assert.ok(transitionToStub.calledWith('authenticated.certifications.start', 123));
         });
       });
     });

diff --git a/mon-pix/tests/unit/routes/authenticated/certifications/start_test.js b/mon-pix/tests/unit/routes/authenticated/certifications/start_test.js
@@ -11,12 +11,15 @@ module('Unit | Route | Certification | Start', function (hooks) {
       module('when hasSeenCertificationInstructions is false', function () {
         test('should redirect to certification information page', async function (assert) {
           // given
-          const certificationCandidateId = 'certification-candidate-id';
-          const params = { certification_candidate_id: certificationCandidateId };
           const store = this.owner.lookup('service:store');
+          const certificationCandidate = store.createRecord('certification-candidate', {
+            sessionId: 1234,
+            hasSeenCertificationInstructions: false,
+          });
+          const params = { certification_candidate_id: certificationCandidate.id };
 
           const certificationCandidateSubscription = store.createRecord('certification-candidate-subscription', {
-            id: certificationCandidateId,
+            id: certificationCandidate.id,
             sessionId: 1234,
             sessionVersion: 3,
           });
@@ -27,13 +30,13 @@ module('Unit | Route | Certification | Start', function (hooks) {
           });
 
           const findRecordStub = sinon.stub().returns(certificationCandidateSubscription);
-          const storeStub = Service.create({ findRecord: findRecordStub });
+          const peekRecordStub = sinon.stub().returns(certificationCandidate);
+          const storeStub = Service.create({ findRecord: findRecordStub, peekRecord: peekRecordStub });
 
           const route = this.owner.lookup('route:authenticated/certifications.start');
           route.set('store', storeStub);
           route.set('featureToggles', featureToggles);
           route.router = { replaceWith: sinon.stub() };
-          route.hasSeenCertificationInstructions = false;
 
           // when
           await route.model(params);
@@ -42,7 +45,7 @@ module('Unit | Route | Certification | Start', function (hooks) {
           sinon.assert.calledWithExactly(
             route.router.replaceWith,
             'authenticated.certifications.information',
-            certificationCandidateId,
+            certificationCandidate.id,
           );
           assert.ok(true);
         });
@@ -51,12 +54,15 @@ module('Unit | Route | Certification | Start', function (hooks) {
       module('when hasSeenCertificationInstructions is true', function () {
         test('should not redirect to certification information page', async function (assert) {
           // given
-          const certificationCandidateId = 'certification-candidate-id';
-          const params = { certification_candidate_id: certificationCandidateId };
           const store = this.owner.lookup('service:store');
+          const certificationCandidate = store.createRecord('certification-candidate', {
+            sessionId: 1234,
+            hasSeenCertificationInstructions: true,
+          });
+          const params = { certification_candidate_id: certificationCandidate.id };
 
           const certificationCandidateSubscription = store.createRecord('certification-candidate-subscription', {
-            id: certificationCandidateId,
+            id: certificationCandidate.id,
             sessionId: 1234,
             sessionVersion: 3,
           });
@@ -67,7 +73,8 @@ module('Unit | Route | Certification | Start', function (hooks) {
           });
 
           const findRecordStub = sinon.stub().returns(certificationCandidateSubscription);
-          const storeStub = Service.create({ findRecord: findRecordStub });
+          const peekRecordStub = sinon.stub().returns(certificationCandidate);
+          const storeStub = Service.create({ findRecord: findRecordStub, peekRecord: peekRecordStub });
 
           const route = this.owner.lookup('route:authenticated/certifications.start');
           route.set('store', storeStub);
@@ -88,12 +95,15 @@ module('Unit | Route | Certification | Start', function (hooks) {
     module('when session is V3 and toggle is not enabled', function () {
       test('should not redirect to certification information page', async function (assert) {
         // given
-        const certificationCandidateId = 'certification-candidate-id';
-        const params = { certification_candidate_id: certificationCandidateId };
         const store = this.owner.lookup('service:store');
+        const certificationCandidate = store.createRecord('certification-candidate', {
+          sessionId: 1234,
+          hasSeenCertificationInstructions: false,
+        });
+        const params = { certification_candidate_id: certificationCandidate.id };
 
         const certificationCandidateSubscription = store.createRecord('certification-candidate-subscription', {
-          id: certificationCandidateId,
+          id: certificationCandidate.id,
           sessionId: 1234,
           sessionVersion: 3,
         });
@@ -104,7 +114,8 @@ module('Unit | Route | Certification | Start', function (hooks) {
         });
 
         const findRecordStub = sinon.stub().returns(certificationCandidateSubscription);
-        const storeStub = Service.create({ findRecord: findRecordStub });
+        const peekRecordStub = sinon.stub().returns(certificationCandidate);
+        const storeStub = Service.create({ findRecord: findRecordStub, peekRecord: peekRecordStub });
 
         const route = this.owner.lookup('route:authenticated/certifications.start');
         route.set('store', storeStub);
@@ -123,12 +134,15 @@ module('Unit | Route | Certification | Start', function (hooks) {
     module('when session is not V3 and toggle is enabled', function () {
       test('should not redirect to certification information page', async function (assert) {
         // given
-        const certificationCandidateId = 'certification-candidate-id';
-        const params = { certification_candidate_id: certificationCandidateId };
         const store = this.owner.lookup('service:store');
+        const certificationCandidate = store.createRecord('certification-candidate', {
+          sessionId: 1234,
+          hasSeenCertificationInstructions: false,
+        });
+        const params = { certification_candidate_id: certificationCandidate.id };
 
         const certificationCandidateSubscription = store.createRecord('certification-candidate-subscription', {
-          id: certificationCandidateId,
+          id: certificationCandidate.id,
           sessionId: 1234,
           sessionVersion: 2,
         });
@@ -139,7 +153,8 @@ module('Unit | Route | Certification | Start', function (hooks) {
         });
 
         const findRecordStub = sinon.stub().returns(certificationCandidateSubscription);
-        const storeStub = Service.create({ findRecord: findRecordStub });
+        const peekRecordStub = sinon.stub().returns(certificationCandidate);
+        const storeStub = Service.create({ findRecord: findRecordStub, peekRecord: peekRecordStub });
 
         const route = this.owner.lookup('route:authenticated/certifications.start');
         route.set('store', storeStub);
@@ -158,12 +173,15 @@ module('Unit | Route | Certification | Start', function (hooks) {
     module('when session is not V3 and toggle is not enabled', function () {
       test('should not redirect to certification information page', async function (assert) {
         // given
-        const certificationCandidateId = 'certification-candidate-id';
-        const params = { certification_candidate_id: certificationCandidateId };
         const store = this.owner.lookup('service:store');
+        const certificationCandidate = store.createRecord('certification-candidate', {
+          sessionId: 1234,
+          hasSeenCertificationInstructions: false,
+        });
+        const params = { certification_candidate_id: certificationCandidate.id };
 
         const certificationCandidateSubscription = store.createRecord('certification-candidate-subscription', {
-          id: certificationCandidateId,
+          id: certificationCandidate.id,
           sessionId: 1234,
           sessionVersion: 2,
         });
@@ -174,7 +192,8 @@ module('Unit | Route | Certification | Start', function (hooks) {
         });
 
         const findRecordStub = sinon.stub().returns(certificationCandidateSubscription);
-        const storeStub = Service.create({ findRecord: findRecordStub });
+        const peekRecordStub = sinon.stub().returns(certificationCandidate);
+        const storeStub = Service.create({ findRecord: findRecordStub, peekRecord: peekRecordStub });
 
         const route = this.owner.lookup('route:authenticated/certifications.start');
         route.set('store', storeStub);