Fixes #1995 setting the value should also create the formula underneath

Open-Systems-Pharmacology · May 2, 2023 · ae46b73 · ae46b73
1 parent 7bada56
commit ae46b73
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 11 deletions.
diff --git a/src/OSPSuite.Core/Domain/Quantity.cs b/src/OSPSuite.Core/Domain/Quantity.cs
@@ -1,8 +1,6 @@
-using System;
-using OSPSuite.Core.Domain.Formulas;
+using OSPSuite.Core.Domain.Formulas;
 using OSPSuite.Core.Domain.Services;
 using OSPSuite.Core.Domain.UnitSystem;
-using OSPSuite.Utility.Exceptions;
 
 namespace OSPSuite.Core.Domain
 {
@@ -31,9 +29,9 @@ public interface IQuantity : IFormulaUsable, IUsingFormula, IWithValueOrigin
       QuantityType QuantityType { get; set; }
 
       /// <summary>
-      ///   Returns the <see cref="QuantityType"/> as a string
+      ///    Returns the <see cref="QuantityType" /> as a string
       /// </summary>
-      string QuantityTypeAsString { get;  }
+      string QuantityTypeAsString { get; }
 
       /// <summary>
       ///    The value in the displayed unit
@@ -49,6 +47,7 @@ public interface IQuantity : IFormulaUsable, IUsingFormula, IWithValueOrigin
       ///    The value in the displayed unit
       /// </summary>
       (double value, bool success) TryGetValue();
+
       /// <summary>
       ///    Specifies whether negative values are allowed or not for this quantity
       /// </summary>
@@ -73,8 +72,6 @@ public abstract class Quantity : Entity, IQuantity
       /// <inheritdoc />
       public QuantityType QuantityType { get; set; }
 
-
-
       /// <inheritdoc />
       public bool NegativeValuesAllowed { get; set; }
 
@@ -167,8 +164,8 @@ public double ValueInDisplayUnit
          }
 
          return (value, success);
-
       }
+
       public virtual (double value, bool success) TryGetValueInDisplayUnit()
       {
          var (value, success) = TryGetValue();

diff --git a/src/OSPSuite.Core/Domain/WithExtensions.cs b/src/OSPSuite.Core/Domain/WithExtensions.cs
@@ -1,5 +1,6 @@
 using System.Collections.Generic;
 using System.Linq;
+using OSPSuite.Core.Domain.Formulas;
 using OSPSuite.Core.Domain.UnitSystem;
 
 namespace OSPSuite.Core.Domain
@@ -14,7 +15,12 @@ public static T WithDimension<T>(this T withDimension, IDimension dimension) whe
 
       public static T WithValue<T>(this T withValue, double value) where T : IWithValue
       {
-         withValue.Value = value;
+         //A using formula for which no formula is defined? We create a constant formula
+         if (withValue is IUsingFormula usingFormula && usingFormula.Formula == null)
+            usingFormula.Formula = new ConstantFormula(value);
+         else
+            withValue.Value = value;
+
          return withValue;
       }
 

diff --git a/tests/OSPSuite.Core.Tests/Domain/DistributionFormulaFactorySpecs.cs b/tests/OSPSuite.Core.Tests/Domain/DistributionFormulaFactorySpecs.cs
@@ -13,12 +13,16 @@ public abstract class concern_for_DistributionFormulaFactory : ContextSpecificat
    {
       protected IDistributedParameter _distributedParam;
       protected IDimension _dim;
-      protected IObjectBaseFactory _objectBaseFacotry;
+      protected IObjectBaseFactory _objectBaseFactory;
       private Utility.Container.IContainer _container;
 
       protected override void Context()
       {
-         var percentileParam = new Parameter().WithName(Constants.Distribution.PERCENTILE).WithDimension(_dim).WithValue(0.5);
+         var percentileParam = new Parameter()
+            .WithName(Constants.Distribution.PERCENTILE)
+            .WithDimension(_dim)
+            .WithValue(0.5);
+
          _dim = new Dimension(new BaseDimensionRepresentation(), "dimenion", "unit");
          _distributedParam = new DistributedParameter().WithName("P1").WithDimension(_dim);
          _distributedParam.Add(percentileParam);
@@ -105,6 +109,7 @@ protected override void Context()
          _distributedParam.Add(_meanParam);
          _discreteDistribution = sut.CreateDiscreteDistributionFormulaFor(_distributedParam, _meanParam);
          _distributedParam.Formula = _discreteDistribution;
+         _distributedParam.Percentile = 0.5;
       }
 
       protected override void Because()