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 ContainsWritable tests

Browse Source
This commit is contained in:
Matt Nadareski
2025-01-09 20:10:56 -05:00
parent dece9c0511
commit 2f5d2fa961
2 changed files with 34 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

30
SabreTools.DatFiles.Test/DatFileTests.cs
View File

@@ -1514,7 +1514,35 @@ namespace SabreTools.DatFiles.Test
#region ContainsWritable
// TODO: Write ContainsWritable tests
[Fact]
public void ContainsWritable_Empty_True()
{
List<DatItem> datItems = [];
DatFile datFile = new Formats.Logiqx(null, deprecated: false);
bool actual = datFile.ContainsWritable(datItems);
Assert.True(actual);
}
[Fact]
public void ContainsWritable_NoWritable_False()
{
List<DatItem> datItems = [new Blank()];
DatFile datFile = new Formats.Logiqx(null, deprecated: false);
bool actual = datFile.ContainsWritable(datItems);
Assert.False(actual);
}
[Fact]
public void ContainsWritable_Writable_True()
{
List<DatItem> datItems = [new Rom()];
DatFile datFile = new Formats.Logiqx(null, deprecated: false);
bool actual = datFile.ContainsWritable(datItems);
Assert.True(actual);
}
#endregion

10
SabreTools.DatFiles/DatFile.cs
View File

@@ -431,15 +431,15 @@ namespace SabreTools.DatFiles
protected internal virtual List<string>? GetMissingRequiredFields(DatItem datItem) => null;
/// <summary>
/// Get if a machine contains any writable items
/// Get if a list contains any writable items
/// </summary>
/// <param name="datItems">DatItems to check</param>
/// <returns>True if the machine contains at least one writable item, false otherwise</returns>
/// <remarks>Empty machines are kept with this</remarks>
/// <returns>True if the list contains at least one writable item, false otherwise</returns>
/// <remarks>Empty list are kept with this</remarks>
protected internal bool ContainsWritable(List<DatItem> datItems)
{
// Empty machines are considered writable
if (datItems == null || datItems.Count == 0)
// Empty list are considered writable
if (datItems.Count == 0)
return true;
foreach (DatItem datItem in datItems)