mirror of
https://github.com/ajisaacs/OpenNest.git
synced 2026-10-07 07:52:11 -04:00
aj
03b3ed47b2
Nest Info edits only the material name, but SaveNestInfo replaced the nest's material with new Material(name), so pressing OK without touching the material silently dropped its grade and density. SaveNestInfo now goes through NestMaterialSelection.Apply in OpenNest.Data: a name with the same SharedListNames.Key as the current material (trimmed, whitespace collapsed, invariant upper case) keeps its grade and density and takes the typed name; any other name gets a name-only material, as before. It always returns a new Material. SharedListNames.Key is the name normalizer the shared customer/material lists will use. Behavioral reds (each restored byte-identically): restoring the old name-only path fails both keep-grade tests; exact-string matching, dropping the upper-casing or the whitespace collapse fails the normalized-name test; returning the current instance fails the copy test. EditNestInfoFormTests cover Load -> Save through the real form; they are cross-compiled here and execute in the windows-desktop CI job.
91 lines
2.6 KiB
C#
91 lines
2.6 KiB
C#
using OpenNest.Data;
|
|
|
|
namespace OpenNest.Tests.Data;
|
|
|
|
public class NestMaterialSelectionTests
|
|
{
|
|
[Fact]
|
|
public void Apply_UnchangedName_KeepsGradeAndDensity()
|
|
{
|
|
var current = new Material("Stainless", "304", 0.289);
|
|
|
|
var result = NestMaterialSelection.Apply(current, "Stainless");
|
|
|
|
Assert.Equal("Stainless", result.Name);
|
|
Assert.Equal("304", result.Grade);
|
|
Assert.Equal(0.289, result.Density);
|
|
}
|
|
|
|
[Fact]
|
|
public void Apply_NameDifferingOnlyInSpacingAndCase_KeepsGradeAndDensityAndTakesTypedName()
|
|
{
|
|
var current = new Material("Cold Rolled Steel", "1008", 0.284);
|
|
|
|
var result = NestMaterialSelection.Apply(current, " cold rolled\tsteel ");
|
|
|
|
Assert.Equal(" cold rolled\tsteel ", result.Name);
|
|
Assert.Equal("1008", result.Grade);
|
|
Assert.Equal(0.284, result.Density);
|
|
}
|
|
|
|
[Fact]
|
|
public void Apply_ChangedName_GetsNameOnly()
|
|
{
|
|
var current = new Material("Stainless", "304", 0.289);
|
|
|
|
var result = NestMaterialSelection.Apply(current, "Aluminum");
|
|
|
|
Assert.Equal("Aluminum", result.Name);
|
|
Assert.True(string.IsNullOrEmpty(result.Grade));
|
|
Assert.Equal(0, result.Density);
|
|
}
|
|
|
|
[Fact]
|
|
public void Apply_ClearedName_GetsEmptyNameOnly()
|
|
{
|
|
var current = new Material("Stainless", "304", 0.289);
|
|
|
|
var result = NestMaterialSelection.Apply(current, "");
|
|
|
|
Assert.Equal("", result.Name);
|
|
Assert.True(string.IsNullOrEmpty(result.Grade));
|
|
Assert.Equal(0, result.Density);
|
|
}
|
|
|
|
[Fact]
|
|
public void Apply_NoCurrentMaterial_GetsNameOnly()
|
|
{
|
|
var result = NestMaterialSelection.Apply(null, "Stainless");
|
|
|
|
Assert.Equal("Stainless", result.Name);
|
|
Assert.True(string.IsNullOrEmpty(result.Grade));
|
|
Assert.Equal(0, result.Density);
|
|
}
|
|
|
|
[Fact]
|
|
public void Apply_ReturnsCopy_LeavingCurrentMaterialUnchanged()
|
|
{
|
|
var current = new Material("Stainless", "304", 0.289);
|
|
|
|
var result = NestMaterialSelection.Apply(current, "stainless");
|
|
result.Grade = "316";
|
|
result.Density = 1;
|
|
|
|
Assert.NotSame(current, result);
|
|
Assert.Equal("Stainless", current.Name);
|
|
Assert.Equal("304", current.Grade);
|
|
Assert.Equal(0.289, current.Density);
|
|
}
|
|
|
|
[Theory]
|
|
[InlineData("Carbon Steel", "CARBON STEEL")]
|
|
[InlineData(" carbon \t steel\n", "CARBON STEEL")]
|
|
[InlineData("", "")]
|
|
[InlineData(" ", "")]
|
|
[InlineData(null, "")]
|
|
public void SharedListNamesKey_TrimsCollapsesWhitespaceAndUppercases(string? name, string expected)
|
|
{
|
|
Assert.Equal(expected, SharedListNames.Key(name));
|
|
}
|
|
}
|