TD-5154 Adds always show description validation

Author: kevwhitt-hee

DigitalLearningSolutions.Data/Models/Frameworks/Import/CompetencyTableRow.cs

@@ -1,7 +1,6 @@
 namespace DigitalLearningSolutions.Data.Models.Frameworks.Import
 {
     using ClosedXML.Excel;
-    using Org.BouncyCastle.Asn1.X509;
 
     public enum RowStatus
     {
@@ -12,7 +11,8 @@ public enum RowStatus
         CompetencyUpdated,
         CompetencyGroupInserted,
         CompetencyGroupUpdated,
-        CompetencyGroupAndCompetencyUpdated
+        CompetencyGroupAndCompetencyUpdated,
+        InvalidAlwaysShowDescription
     }
     public class CompetencyTableRow : BulkCompetency
     {
@@ -30,10 +30,13 @@ public CompetencyTableRow(IXLTable table, IXLRangeRow row)
             Competency = FindFieldValue("Competency");
             CompetencyDescription = FindFieldValue("CompetencyDescription");
             GroupDescription = FindFieldValue("GroupDescription");
+            AlwaysShowDescriptionRaw = FindFieldValue("AlwaysShowDescription");
+            AlwaysShowDescription = bool.TryParse(AlwaysShowDescriptionRaw, out var hasPrn) ? hasPrn : (bool?)null;
             FlagsCsv = FindFieldValue("FlagsCSV");
             RowStatus = RowStatus.NotYetProcessed;
         }
         public int RowNumber { get; set; }
+        public string? AlwaysShowDescriptionRaw { get; set; }
         public ImportCompetenciesResult.ErrorReason? Error { get; set; }
         public RowStatus RowStatus { get; set; }
         public bool Validate()
@@ -50,6 +53,10 @@ public bool Validate()
             {
                 Error = ImportCompetenciesResult.ErrorReason.TooLongCompetencyName;
             }
+            else if (!string.IsNullOrWhiteSpace(AlwaysShowDescriptionRaw) && !bool.TryParse(AlwaysShowDescriptionRaw, out _))
+            {
+                Error = ImportCompetenciesResult.ErrorReason.InvalidAlwaysShowDescription;
+            }
 
             return !Error.HasValue;
         }

DigitalLearningSolutions.Web/ViewModels/Frameworks/ImportCompetenciesPreProcessViewModel.cs

@@ -43,7 +43,7 @@ private static string MapReasonToErrorMessage(ImportCompetenciesResult.ErrorReas
                 ImportCompetenciesResult.ErrorReason.TooLongCompetencyName =>
                     "Competency must be 255 characters or less.",
                 ImportCompetenciesResult.ErrorReason.InvalidAlwaysShowDescription =>
-                    "Always show description is invalid. The Always show description  field must contain 'TRUE' or 'FALSE'",
+                    "Always show description is invalid. The Always show description field must contain 'TRUE' or 'FALSE'",
                 _ => "Unspecified error.",
             };
         }