diff --git a/MetaMorpheus/Test/GuiTests/DeconHostViewModelTest.cs b/MetaMorpheus/Test/GuiTests/DeconHostViewModelTest.cs index 0c83c362e..0e98e3216 100644 --- a/MetaMorpheus/Test/GuiTests/DeconHostViewModelTest.cs +++ b/MetaMorpheus/Test/GuiTests/DeconHostViewModelTest.cs @@ -69,8 +69,8 @@ public void Constructor_WithProvidedParameters_ShouldSetCorrectly_Classic() Assert.That(viewModel.ProductDeconvolutionParameters.MaxAssumedChargeState, Is.EqualTo(initialProductParameters.MaxAssumedChargeState)); Assert.That(initialPrecursorParameters.DeconvolutionType, Is.EqualTo(DeconvolutionType.ClassicDeconvolution)); Assert.That(initialProductParameters.DeconvolutionType, Is.EqualTo(DeconvolutionType.ClassicDeconvolution)); - Assert.That(initialPrecursorParameters, Is.InstanceOf()); - Assert.That(initialProductParameters, Is.InstanceOf()); + Assert.That(viewModel.PrecursorDeconvolutionParameters.Parameters, Is.InstanceOf()); + Assert.That(viewModel.ProductDeconvolutionParameters.Parameters, Is.InstanceOf()); } [Test] @@ -90,8 +90,9 @@ public void Constructor_WithProvidedParameters_ShouldSetCorrectly_IsoDec() Assert.That(viewModel.ProductDeconvolutionParameters.MaxAssumedChargeState, Is.EqualTo(initialProductParameters.MaxAssumedChargeState)); Assert.That(initialPrecursorParameters.DeconvolutionType, Is.EqualTo(DeconvolutionType.IsoDecDeconvolution)); Assert.That(initialProductParameters.DeconvolutionType, Is.EqualTo(DeconvolutionType.IsoDecDeconvolution)); - Assert.That(initialPrecursorParameters, Is.InstanceOf()); - Assert.That(initialProductParameters, Is.InstanceOf()); + Assert.That(viewModel.PrecursorDeconvolutionParameters.Parameters, Is.InstanceOf()); + Assert.That(viewModel.PrecursorDeconvolutionParameters.Parameters, Is.InstanceOf()); + Assert.That(viewModel.PrecursorDeconvolutionParameters.ToString(), Is.EqualTo("IsoDec")); } [Test] @@ -151,7 +152,6 @@ public void ProductDeconvolutionParameters_Setter_ShouldTriggerPropertyChanged() Assert.That(propertyChangedTriggered, Is.True); } - [Test] [NonParallelizable] public void TestDeconHostViewModel_GlobalVariables_Proteoform_Classic() @@ -248,6 +248,21 @@ public void TestDeconHostViewModel_GlobalVariables_Unknown() var deconHostViewModel = new DeconHostViewModel(ClassicPrecursorDeconvolutionParameters, null); }); + Assert.Throws(() => + { + var deconHostViewModel = new DeconHostViewModel(null, ClassicProductDeconvolutionParameters); + }); + + Assert.Throws(() => + { + var deconHostViewModel = new DeconHostViewModel(IsoDecPrecursorDeconvolutionParameters, null); + }); + + Assert.Throws(() => + { + var deconHostViewModel = new DeconHostViewModel(null, IsoDecProductDeconvolutionParameters); + }); + // Revert back to default GlobalVariables.AnalyteType = AnalyteType.Peptide; } @@ -295,4 +310,38 @@ public void TestDisplayDeconSelectionComboBox_SingleOption() // Assert Assert.That(result, Is.False); } + + [Test] + public void SetAllPrecursorMaxChargeState_ShouldUpdateAllPrecursorParams() + { + // Arrange + var viewModel = new DeconHostViewModel(); + int newMaxCharge = 5; + + // Act + viewModel.SetAllPrecursorMaxChargeState(newMaxCharge); + + // Assert + foreach (var precursorParams in viewModel.PrecursorDeconvolutionParametersList) + { + Assert.That(precursorParams.MaxAssumedChargeState, Is.EqualTo(newMaxCharge)); + } + } + + [Test] + public void SetAllProductMaxChargeState_ShouldUpdateAllProductParams() + { + // Arrange + var viewModel = new DeconHostViewModel(); + int newMaxCharge = 5; + + // Act + viewModel.SetAllProductMaxChargeState(newMaxCharge); + + // Assert + foreach (var productParams in viewModel.ProductDeconvolutionParametersList) + { + Assert.That(productParams.MaxAssumedChargeState, Is.EqualTo(newMaxCharge)); + } + } } \ No newline at end of file