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 SetOneRomPerGame tests, make fixes

Browse Source
This commit is contained in:
Matt Nadareski
2025-01-13 23:05:56 -05:00
parent 7160d34b07
commit da835ecc8f
2 changed files with 102 additions and 20 deletions
Download Patch File Download Diff File Expand all files Collapse all files

76
SabreTools.DatFiles.Test/DatFileTests.Filtering.cs
View File

@@ -62,7 +62,81 @@ namespace SabreTools.DatFiles.Test
#region SetOneRomPerGame #region SetOneRomPerGame
// TODO: Implement SetOneRomPerGame tests [Fact]
public void SetOneRomPerGame_Items()
{
Source source = new Source(0, source: null);
Machine machine = new Machine();
machine.SetFieldValue(Models.Metadata.Machine.NameKey, "machine");
DatItem rom = new Rom();
rom.SetName("rom.bin");
rom.SetFieldValue(DatItem.MachineKey, machine);
rom.SetFieldValue(DatItem.SourceKey, source);
DatItem disk = new Disk();
disk.SetName("disk");
disk.SetFieldValue(DatItem.MachineKey, machine);
disk.SetFieldValue(DatItem.SourceKey, source);
DatFile datFile = new Logiqx(datFile: null, deprecated: false);
datFile.AddItem(rom, statsOnly: false);
datFile.AddItem(disk, statsOnly: false);
datFile.BucketBy(ItemKey.Machine, DedupeType.None);
datFile.SetOneRomPerGame();
var actualDatItems = datFile.GetItemsForBucket("machine");
Assert.Equal(2, actualDatItems.Count);
DatItem actualRom = Assert.Single(actualDatItems.FindAll(i => i is Rom));
Machine? actualRomMachine = actualRom.GetFieldValue<Machine>(DatItem.MachineKey);
Assert.NotNull(actualRomMachine);
Assert.Equal("machine/rom", actualRomMachine.GetStringFieldValue(Models.Metadata.Machine.NameKey));
DatItem actualDisk = Assert.Single(actualDatItems.FindAll(i => i is Disk));
Machine? actualDiskMachine = actualDisk.GetFieldValue<Machine>(DatItem.MachineKey);
Assert.NotNull(actualDiskMachine);
Assert.Equal("machine/disk", actualDiskMachine.GetStringFieldValue(Models.Metadata.Machine.NameKey));
}
[Fact]
public void SetOneRomPerGame_ItemsDB()
{
Source source = new Source(0, source: null);
Machine machine = new Machine();
machine.SetFieldValue(Models.Metadata.Machine.NameKey, "machine");
DatItem rom = new Rom();
rom.SetName("rom.bin");
DatItem disk = new Disk();
disk.SetName("disk");
DatFile datFile = new Logiqx(datFile: null, deprecated: false);
long sourceIndex = datFile.AddSourceDB(source);
long machineIndex = datFile.AddMachineDB(machine);
_ = datFile.AddItemDB(rom, machineIndex, sourceIndex, statsOnly: false);
_ = datFile.AddItemDB(disk, machineIndex, sourceIndex, statsOnly: false);
datFile.BucketBy(ItemKey.Machine, DedupeType.None);
datFile.SetOneRomPerGame();
var actualDatItems = datFile.GetItemsForBucketDB("machine");
Assert.Equal(2, actualDatItems.Count);
var actualRom = Assert.Single(actualDatItems, i => i.Value is Rom);
var actualRomMachine = datFile.ItemsDB.GetMachineForItem(actualRom.Key);
Assert.NotNull(actualRomMachine.Value);
Assert.Equal("machine/rom", actualRomMachine.Value.GetStringFieldValue(Models.Metadata.Machine.NameKey));
var actualDisk = Assert.Single(actualDatItems, i => i.Value is Disk);
var actualDiskMachine = datFile.ItemsDB.GetMachineForItem(actualDisk.Key);
Assert.NotNull(actualDiskMachine.Value);
Assert.Equal("machine/disk", actualDiskMachine.Value.GetStringFieldValue(Models.Metadata.Machine.NameKey));
}
#endregion #endregion

46
SabreTools.DatFiles/DatFile.Filtering.cs
View File

@@ -433,8 +433,8 @@ namespace SabreTools.DatFiles
private static void SetOneRomPerGameImpl(DatItem datItem) private static void SetOneRomPerGameImpl(DatItem datItem)
{ {
// If the item name is null // If the item name is null
string? machineName = datItem.GetName(); string? itemName = datItem.GetName();
if (machineName == null) if (itemName == null)
return; return;
// Get the current machine // Get the current machine
@@ -448,19 +448,23 @@ namespace SabreTools.DatFiles
// Reassign the item to the new machine // Reassign the item to the new machine
datItem.SetFieldValue<Machine>(DatItem.MachineKey, machine); datItem.SetFieldValue<Machine>(DatItem.MachineKey, machine);
// Remove extensions from Rom items // Remove extensions from File and Rom items
if (datItem is Rom) if (datItem is DatItems.Formats.File || datItem is Rom)
{ {
string[] splitname = machineName.Split('.'); string[] splitname = itemName.Split('.');
machineName = machine.GetStringFieldValue(Models.Metadata.Machine.NameKey) itemName = machine.GetStringFieldValue(Models.Metadata.Machine.NameKey)
+ $"/{string.Join(".", splitname, 0, splitname.Length > 1 ? splitname.Length - 1 : 1)}"; + $"/{string.Join(".", splitname, 0, splitname.Length > 1 ? splitname.Length - 1 : 1)}";
} }
else
{
itemName = machine.GetStringFieldValue(Models.Metadata.Machine.NameKey) + $"/{itemName}";
}
// Strip off "Default" prefix only for ORPG // Strip off "Default" prefix only for ORPG
if (machineName.StartsWith("Default")) if (itemName.StartsWith("Default"))
machineName = machineName.Substring("Default".Length + 1); itemName = itemName.Substring("Default".Length + 1);
machine.SetFieldValue<string?>(Models.Metadata.Machine.NameKey, machineName); machine.SetFieldValue<string?>(Models.Metadata.Machine.NameKey, itemName);
datItem.SetName(Path.GetFileName(datItem.GetName())); datItem.SetName(Path.GetFileName(datItem.GetName()));
} }
@@ -506,8 +510,8 @@ namespace SabreTools.DatFiles
private void SetOneRomPerGameImplDB(KeyValuePair<long, DatItem> datItem) private void SetOneRomPerGameImplDB(KeyValuePair<long, DatItem> datItem)
{ {
// If the item name is null // If the item name is null
string? machineName = datItem.Value.GetName(); string? itemName = datItem.Value.GetName();
if (datItem.Key < 0 || machineName == null) if (datItem.Key < 0 || itemName == null)
return; return;
// Get the current machine // Get the current machine
@@ -522,21 +526,25 @@ namespace SabreTools.DatFiles
return; return;
// Reassign the item to the new machine // Reassign the item to the new machine
ItemsDB._itemToMachineMapping[machine.Key] = newMachineIndex; ItemsDB._itemToMachineMapping[datItem.Key] = newMachineIndex;
// Remove extensions from Rom items // Remove extensions from File and Rom items
if (datItem.Value is Rom) if (datItem.Value is DatItems.Formats.File || datItem.Value is Rom)
{ {
string[] splitname = machineName.Split('.'); string[] splitname = itemName.Split('.');
machineName = machine.Value.GetStringFieldValue(Models.Metadata.Machine.NameKey) itemName = machine.Value.GetStringFieldValue(Models.Metadata.Machine.NameKey)
+ $"/{string.Join(".", splitname, 0, splitname.Length > 1 ? splitname.Length - 1 : 1)}"; + $"/{string.Join(".", splitname, 0, splitname.Length > 1 ? splitname.Length - 1 : 1)}";
} }
else
{
itemName = machine.Value.GetStringFieldValue(Models.Metadata.Machine.NameKey) + $"/{itemName}";
}
// Strip off "Default" prefix only for ORPG // Strip off "Default" prefix only for ORPG
if (machineName.StartsWith("Default")) if (itemName.StartsWith("Default"))
machineName = machineName.Substring("Default".Length + 1); itemName = itemName.Substring("Default".Length + 1);
machine.Value.SetFieldValue<string?>(Models.Metadata.Machine.NameKey, machineName); machine.Value.SetFieldValue<string?>(Models.Metadata.Machine.NameKey, itemName);
datItem.Value.SetName(Path.GetFileName(datItem.Value.GetName())); datItem.Value.SetName(Path.GetFileName(datItem.Value.GetName()));
} }