Remove suitability answer related properties

BookingsApi/BookingsApi.AcceptanceTests/Steps/ParticipantsSteps.cs

@@ -163,14 +163,5 @@ public void ThenIndividualDetailsShouldBeUpdated(string participant)
             model.TelephoneNumber.Should().Be(updateParticipantRequest.TelephoneNumber);
             model.LinkedParticipants.Count.Should().Be(participant == "Individual" ? 1 : 0);
         }
-
-        [Given(@"I have an update participant suitability answers with a valid user '(.*)'")]
-        public void GivenIHaveAnUpdateParticipantSuitabilityAnswersWithAValidUser(string role)
-        {
-            var participantId = _context.TestData.ParticipantsResponses.FirstOrDefault(x => x.UserRoleName.Equals(role)).Id;
-            var updateParticipantRequest = UpdateSuitabilityAnswersRequest.BuildRequest();
-            _context.TestData.Answers = updateParticipantRequest;
-            _context.Request = _context.Put(UpdateSuitabilityAnswers(_context.TestData.Hearing.Id, participantId), updateParticipantRequest);
-        }
     }
 }

BookingsApi/BookingsApi.AcceptanceTests/Steps/PersonsSteps.cs

@@ -92,34 +92,6 @@ public void ThenPersonsDetailsShouldBeRetrieved()
             actual.Username.Should().Be(expected.Username);
         }
 
-        [Then(@"suitability answers for '(.*)' should be updated")]
-        public void ThenSuitabilityAnswersForShouldBeUpdated(string participant)
-        {
-            var username = _context.TestData.ParticipantsResponses
-                .FirstOrDefault(x => x.UserRoleName.Equals(participant)).Username;
-            var participantId = _context.TestData.ParticipantsResponses
-                .FirstOrDefault(x => x.UserRoleName.Equals(participant)).Id;
-            _context.Request = _context.Get(GetPersonSuitabilityAnswers(username));
-            _context.Response = _context.Client().Execute(_context.Request);
-            _context.Response.StatusCode.Should().Be(HttpStatusCode.OK);
-            var model =
-                RequestHelper.Deserialise<List<PersonSuitabilityAnswerResponse>>(
-                    _context.Response.Content);
-            var expectedResult = _context.TestData.Answers;
-
-            expectedResult[0].Key.Should().Be(model[0].Answers[0].Key);
-            expectedResult[0].Answer.Should().Be(model[0].Answers[0].Answer);
-            expectedResult[0].ExtendedAnswer.Should().Be(model[0].Answers[0].ExtendedAnswer);
-            expectedResult[1].Key.Should().Be(model[0].Answers[1].Key);
-            expectedResult[1].Answer.Should().Be(model[0].Answers[1].Answer);
-            expectedResult[1].ExtendedAnswer.Should().Be(model[0].Answers[1].ExtendedAnswer);
-
-            model[0].HearingId.Should().Be(_context.TestData.Hearing.Id);
-            model[0].ParticipantId.Should().Be(participantId);
-            model[0].UpdatedAt.Should().BeAfter(DateTime.UtcNow.AddMinutes(-2));
-            model[0].QuestionnaireNotRequired.Should().BeFalse();
-        }
-
         [Then(@"a list of hearings for deletion is (.*)")]
         public void ThenAListOfHearingsForDeletionIs(int expectedNumOfHearings)
         {

BookingsApi/BookingsApi.Contract/V1/Requests/BookNewHearingRequest.cs

@@ -67,6 +67,7 @@ public BookNewHearingRequest()
         /// <summary>
         /// QuestionnaireNotRequired
         /// </summary>
+        [Obsolete("This property is no longer used.")]
         public bool QuestionnaireNotRequired { get; set; }
 
         /// <summary>

BookingsApi/BookingsApi.Contract/V1/Requests/UpdateHearingRequest.cs

@@ -1,5 +1,6 @@
 using System;
 using System.Collections.Generic;
+using System.ComponentModel;
 
 namespace BookingsApi.Contract.V1.Requests
 {
@@ -43,6 +44,8 @@ public class UpdateHearingRequest
         /// <summary>
         /// QuestionnaireNotRequired
         /// </summary>
+        [Obsolete("This property is no longer used.")]
+        [DefaultValue(false)]
         public bool? QuestionnaireNotRequired { get; set; }
 
         /// <summary>

BookingsApi/BookingsApi.Contract/V1/Responses/BookingsHearingResponse.cs

@@ -103,6 +103,7 @@ public DateTime HearingDate
         /// <summary>
         /// QuestionnaireNotRequired
         /// </summary>
+        [Obsolete("This property is no longer used.")]
         public bool QuestionnaireNotRequired { get; set; }
 
         /// <summary>

BookingsApi/BookingsApi.DAL/Commands/CreateVideoHearingCommand.cs

@@ -8,8 +8,8 @@ public class CreateVideoHearingCommand : ICommand
     {
         public CreateVideoHearingCommand(CaseType caseType, HearingType hearingType, DateTime scheduledDateTime,
             int scheduledDuration, HearingVenue venue, List<NewParticipant> participants, List<Case> cases,
-            bool questionnaireNotRequired, bool audioRecordingRequired, List<NewEndpoint> endpoints,
-            List<LinkedParticipantDto> linkedParticipants, bool isMultiDayFirstHearing)
+            bool audioRecordingRequired, List<NewEndpoint> endpoints, List<LinkedParticipantDto> linkedParticipants,
+            bool isMultiDayFirstHearing)
         {
             CaseType = caseType;
             HearingType = hearingType;
@@ -18,7 +18,6 @@ public CreateVideoHearingCommand(CaseType caseType, HearingType hearingType, Dat
             Venue = venue;
             Participants = participants;
             Cases = cases;
-            QuestionnaireNotRequired = questionnaireNotRequired;
             AudioRecordingRequired = audioRecordingRequired;
             Endpoints = endpoints;
             LinkedParticipants = linkedParticipants;
@@ -36,7 +35,6 @@ public CreateVideoHearingCommand(CaseType caseType, HearingType hearingType, Dat
         public string HearingRoomName { get; set; }
         public string OtherInformation { get; set; }
         public string CreatedBy { get; set; }
-        public bool QuestionnaireNotRequired { get; set; }
         public bool AudioRecordingRequired { get; set; }
         public List<NewEndpoint> Endpoints { get; }
         public string CancelReason { get; set; }
@@ -61,8 +59,7 @@ public async Task Handle(CreateVideoHearingCommand command)
         {
             var videoHearing = new VideoHearing(command.CaseType, command.HearingType, command.ScheduledDateTime,
                 command.ScheduledDuration, command.Venue, command.HearingRoomName,
-                command.OtherInformation, command.CreatedBy, command.QuestionnaireNotRequired,
-                command.AudioRecordingRequired, command.CancelReason);
+                command.OtherInformation, command.CreatedBy, command.AudioRecordingRequired, command.CancelReason);
 
             // Ideally, the domain object would implement the clone method and so this change is a work around.
             videoHearing.IsFirstDayOfMultiDayHearing = command.IsMultiDayFirstHearing;

BookingsApi/BookingsApi.DAL/Commands/UpdateHearingCommand.cs

@@ -12,12 +12,11 @@ public class UpdateHearingCommand : ICommand
         public string OtherInformation { get; set; }
         public string UpdatedBy { get; set; }
         public List<Case> Cases { get; set; }
-        public bool QuestionnaireNotRequired { get; set; }
         public bool AudioRecordingRequired { get; set; }
 
         public UpdateHearingCommand(Guid hearingId, DateTime scheduledDateTime, int scheduledDuration,
             HearingVenue hearingVenue, string hearingRoomName, string otherInformation, string updatedBy,
-            List<Case> cases, bool questionnaireNotRequired, bool audioRecordingRequired)
+            List<Case> cases, bool audioRecordingRequired)
         {
             HearingId = hearingId;
             ScheduledDateTime = scheduledDateTime;
@@ -27,7 +26,6 @@ public UpdateHearingCommand(Guid hearingId, DateTime scheduledDateTime, int sche
             OtherInformation = otherInformation;
             UpdatedBy = updatedBy;
             Cases = cases;
-            QuestionnaireNotRequired = questionnaireNotRequired;
             AudioRecordingRequired = audioRecordingRequired;
         }
     }
@@ -60,7 +58,7 @@ public async Task Handle(UpdateHearingCommand command)
 
             hearing.UpdateHearingDetails(command.HearingVenue, command.ScheduledDateTime,
                 command.ScheduledDuration, command.HearingRoomName, command.OtherInformation,
-                command.UpdatedBy, command.Cases, command.QuestionnaireNotRequired, command.AudioRecordingRequired);
+                command.UpdatedBy, command.Cases, command.AudioRecordingRequired);
 
             if (command.ScheduledDateTime != oldScheduledDateTime)
             {

BookingsApi/BookingsApi.DAL/Helper/CloneHearingToCommandMapper.cs

@@ -52,8 +52,8 @@ public static CreateVideoHearingCommand CloneToCommand(Hearing hearing, DateTime
 
             var duration = 480;
             var command = new CreateVideoHearingCommand(hearing.CaseType, hearing.HearingType, newDate,
-                duration, hearing.HearingVenue, participants, cases, true,
-                hearing.AudioRecordingRequired, newEndpoints, linkedParticipantDtos, false)
+                duration, hearing.HearingVenue, participants, cases, hearing.AudioRecordingRequired, newEndpoints,
+                linkedParticipantDtos, false)
             {
                 HearingRoomName = hearing.HearingRoomName,
                 OtherInformation = hearing.OtherInformation,

BookingsApi/BookingsApi.DAL/Mappings/HearingMap.cs

@@ -22,7 +22,7 @@ public void Configure(EntityTypeBuilder<Hearing> builder)
             builder.Property(x => x.CreatedDate).HasConversion(v => v, v => DateTime.SpecifyKind(v, DateTimeKind.Utc));
             builder.Property(x => x.UpdatedDate).HasConversion(v => v, v => DateTime.SpecifyKind(v, DateTimeKind.Utc));
             builder.Property(x => x.Status).HasColumnName("HearingStatusId");
-            builder.Property(x => x.QuestionnaireNotRequired);
+            builder.Property<bool>("QuestionnaireNotRequired").HasDefaultValue(false);
             builder.Property(x => x.CancelReason).HasMaxLength(255);
             builder.Property(x => x.SourceId);
             builder.Ignore(x => x.IsFirstDayOfMultiDayHearing);