Changed behavior of masked members in Digestionparams (#756)

* Changed behavior of masked members in Digestionparams * Added unit test * Added unit test
smith-chem-wisc · Jan 11, 2024 · 1dc99fc · 1dc99fc
1 parent 40c4c4b
commit 1dc99fc
Show file tree

Hide file tree

Showing 2 changed files with 33 additions and 3 deletions.
diff --git a/mzLib/Proteomics/ProteolyticDigestion/DigestionParams.cs b/mzLib/Proteomics/ProteolyticDigestion/DigestionParams.cs
@@ -51,9 +51,24 @@ public DigestionParams(string protease = "trypsin", int maxMissedCleavages = 2,
         #region Properties overridden by more generic interface
 
         public Protease Protease { get; private set; }
-        public int MinPeptideLength => MinLength;
-        public int MaxPeptideLength => MaxLength;
-        public int MaxModsForPeptide => MaxMods;
+
+        public int MinPeptideLength
+        {
+            get => MinLength;
+            set => MinLength = value;
+        }
+
+        public int MaxPeptideLength
+        {
+            get => MaxLength;
+            set => MaxLength = value;
+        }
+
+        public int MaxModsForPeptide
+        {
+            get => MaxMods;
+            set => MaxMods = value;
+        }
 
         #endregion
 

diff --git a/mzLib/Test/TestDigestionMotif.cs b/mzLib/Test/TestDigestionMotif.cs
@@ -578,5 +578,20 @@ public static void TestProteolyticDigestion()
             Assert.IsFalse(pwsmsN.Any(x => x.Length < speedySemiN.MinLength));
             Assert.IsFalse(pwsmsC.Any(x => x.Length < speedySemiC.MinLength));
         }
+
+        [Test]
+        public void TestDigestionParamsMaskedProperties()
+        {
+            var digestionParams = new DigestionParams();
+            digestionParams.MinPeptideLength = 1;
+            Assert.That(digestionParams.MinLength, Is.EqualTo(digestionParams.MinPeptideLength));
+
+            digestionParams.MaxPeptideLength = 2;
+            Assert.That(digestionParams.MaxLength, Is.EqualTo(digestionParams.MaxPeptideLength));
+
+
+            digestionParams.MaxModsForPeptide = 3;
+            Assert.That(digestionParams.MaxMods, Is.EqualTo(digestionParams.MaxModsForPeptide));
+        }
     }
 }