claunia/SabreTools
1
0
Fork 0
mirror of https://github.com/claunia/SabreTools.git synced 2025-12-16 19:14:27 +00:00
Code Issues Packages Projects Releases Wiki Activity

Add CompareTo tests and make changes

Browse Source
This commit is contained in:
Matt Nadareski
2025-01-08 10:32:11 -05:00
parent bdfbb35160
commit 8cf360a747
2 changed files with 71 additions and 33 deletions
Download Patch File Download Diff File Expand all files Collapse all files

54
SabreTools.DatItems.Test/DatItemTests.cs
View File

@@ -128,7 +128,59 @@ namespace SabreTools.DatItems.Test
#region CompareTo #region CompareTo
// TODO: Implement CompareTo tests [Fact]
public void CompareTo_NullOther_Returns1()
{
DatItem self = new Rom();
DatItem? other = null;
int actual = self.CompareTo(other);
Assert.Equal(1, actual);
}
[Fact]
public void CompareTo_DifferentOther_Returns1()
{
DatItem self = new Rom();
self.SetName("name");
DatItem? other = new Disk();
other.SetName("name");
int actual = self.CompareTo(other);
Assert.Equal(1, actual);
}
[Fact]
public void CompareTo_Empty_Returns1()
{
DatItem self = new Rom();
DatItem? other = new Rom();
int actual = self.CompareTo(other);
Assert.Equal(1, actual);
}
[Theory]
[InlineData(null, null, 0)]
[InlineData("name", null, 1)]
[InlineData("name", "other", -1)]
[InlineData(null, "name", -1)]
[InlineData("other", "name", 1)]
[InlineData("name", "name", 0)]
public void CompareTo_NamesOnly(string? selfName, string? otherName, int expected)
{
DatItem self = new Rom();
self.SetName(selfName);
self.SetFieldValue(Models.Metadata.Rom.CRCKey, "DEADBEEF");
DatItem? other = new Rom();
other.SetName(otherName);
other.SetFieldValue(Models.Metadata.Rom.CRCKey, "DEADBEEF");
int actual = self.CompareTo(other);
Assert.Equal(expected, actual);
}
#endregion #endregion

50
SabreTools.DatItems/DatItem.cs
View File

@@ -161,24 +161,17 @@ namespace SabreTools.DatItems
if (other == null) if (other == null)
return 1; return 1;
try // Get the names to avoid changing values
{ string? selfName = GetName();
// Get the names to avoid changing values string? otherName = other.GetName();
string? selfName = GetName();
string? otherName = other.GetName();
// If the names are equal // If the names are equal
if (selfName == otherName) if (selfName == otherName)
return Equals(other) ? 0 : 1; return Equals(other) ? 0 : 1;
// If `otherName` is null, Compare will return > 0 // If `otherName` is null, Compare will return > 0
// If `selfName` is null, Compare will return < 0 // If `selfName` is null, Compare will return < 0
return string.Compare(selfName, otherName, StringComparison.Ordinal); return string.Compare(selfName, otherName, StringComparison.Ordinal);
}
catch
{
return 1;
}
} }
/// <summary> /// <summary>
@@ -558,24 +551,17 @@ namespace SabreTools.DatItems
if (other == null) if (other == null)
return 1; return 1;
try // Get the names to avoid changing values
{ string? selfName = GetName();
// Get the names to avoid changing values string? otherName = other.GetName();
string? selfName = GetName();
string? otherName = other.GetName();
// If the names are equal // If the names are equal
if (selfName == otherName) if (selfName == otherName)
return Equals(other) ? 0 : 1; return Equals(other) ? 0 : 1;
// If `otherName` is null, Compare will return > 0 // If `otherName` is null, Compare will return > 0
// If `selfName` is null, Compare will return < 0 // If `selfName` is null, Compare will return < 0
return string.Compare(selfName, otherName, StringComparison.Ordinal); return string.Compare(selfName, otherName, StringComparison.Ordinal);
}
catch
{
return 1;
}
} }
/// <summary> /// <summary>