Fix slider localization test (#15136)

DynamoDS · Apr 16, 2024 · dd7f87d · dd7f87d
1 parent fa01cb7
commit dd7f87d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/Libraries/CoreNodeModelsWpf/SliderViewModel.cs b/src/Libraries/CoreNodeModelsWpf/SliderViewModel.cs
@@ -63,7 +63,8 @@ public T Value
                 if (value.CompareTo(model.Min) == -1)
                     model.Min = value;
 
-                var stepValueString = model.Step.ToString();
+                double.TryParse(model.Step.ToString(), out double stepValue);
+                var stepValueString = stepValue.ToString(null, CultureInfo.InvariantCulture);
                 var decimalPoints = 0;
                 if (stepValueString.Contains('.'))
                 {
@@ -73,8 +74,8 @@ public T Value
                 if (value is IFormattable formattableval)
                 {
                     var invariantString  = formattableval.ToString(null,CultureInfo.InvariantCulture);
-                    var sliderValue = Math.Round(decimal.Parse(invariantString), decimalPoints);
-                    model.UpdateValue(new Dynamo.Graph.UpdateValueParams(nameof(Value), sliderValue.ToString()));
+                    var sliderValue = Math.Round(decimal.Parse(invariantString, CultureInfo.InvariantCulture), decimalPoints);
+                    model.UpdateValue(new Dynamo.Graph.UpdateValueParams(nameof(Value), sliderValue.ToString(CultureInfo.InvariantCulture)));
                 }
                 else
                 {

diff --git a/test/DynamoCoreWpfTests/SliderViewModelTests.cs b/test/DynamoCoreWpfTests/SliderViewModelTests.cs
@@ -140,7 +140,6 @@ public void SliderViewModel_ValueTest()
         /// modify the value.
         /// </summary>
         [Test]
-        [Category("Failure")]
         public void SliderViewModel_ValueTest_Localized()
         {
             //change current thread culture to German.