Merge pull request #241 from DFE-Digital/revert-231-temp-disable-send…

…ing-data-to-crm Revert "Temporarily disable sending data to the CRM"
DFE-Digital · Sep 7, 2020 · 0012096 · 0012096
2 parents e28213e + 095d899
commit 0012096
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/GetIntoTeachingApi/Jobs/UpsertCandidateJob.cs b/GetIntoTeachingApi/Jobs/UpsertCandidateJob.cs
@@ -57,8 +57,7 @@ public void Run(Candidate candidate, PerformContext context)
 
         private void SaveCandidate(Candidate candidate)
         {
-            // TODO: temporarily disabled to avoid collecting PII in a UR session.
-            // _crm.Save(candidate);
+            _crm.Save(candidate);
         }
 
         private IEnumerable<TeachingEventRegistration> ClearTeachingEventRegistrations(Candidate candidate)

diff --git a/GetIntoTeachingApiTests/Jobs/UpsertCandidateJobTests.cs b/GetIntoTeachingApiTests/Jobs/UpsertCandidateJobTests.cs
@@ -33,7 +33,6 @@ public UpsertCandidateJobTests()
                 _mockContext.Object, _mockLogger.Object);
         }
 
-        /* TODO: temporarily disabled to avoid collecting PII in a UR session.
         [Fact]
         public void Run_OnSuccess_SavesCandidate()
         {
@@ -59,7 +58,7 @@ public void Run_WithTeachingEventRegistrationsOnSuccess_SavesTeachingEventRegist
 
             _mockCrm.Verify(mock => mock.Save(registration), Times.Once);
             registration.CandidateId.Should().Be(candidateId);
-        }*/
+        }
 
         [Fact]
         public void Run_OnFailure_EmailsCandidate()