jussi/DotTiled
1
0
Fork 0
mirror of https://github.com/dcronqvist/DotTiled.git synced 2025-05-08 22:16:03 +03:00
Code Issues Projects Releases Packages Wiki Activity Actions

Add test data that fails due to bug from reported issue

Browse source
This commit is contained in:
Daniel Cronqvist 2025-04-28 20:11:19 +02:00
parent aa9d8498e8
commit 81277fdaf1
10 changed files with 325 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

131
src/DotTiled.Tests/TestData/Maps/map-duplicate-object-id-bug/map-duplicate-object-id-bug.cs Normal file
View file

@ -0,0 +1,131 @@
namespace DotTiled.Tests;
public partial class TestData
{
public static Map MapDuplicateObjectIdBug(string ext) => new Map
{
Class = "",
Orientation = MapOrientation.Orthogonal,
Width = 64,
Height = 64,
TileWidth = 16,
TileHeight = 16,
Infinite = true,
ParallaxOriginX = 0,
ParallaxOriginY = 0,
RenderOrder = RenderOrder.RightDown,
CompressionLevel = -1,
BackgroundColor = new TiledColor { R = 0, G = 0, B = 0, A = 0 },
Version = "1.10",
TiledVersion = "1.11.2",
NextLayerID = 2,
NextObjectID = 3,
Tilesets = [
new Tileset
{
FirstGID = 1,
Source = ext == "tmx" ? "tiles.tsx" : "tiles.tsj",
Version = "1.10",
TiledVersion = "1.11.2",
Name = "Tiles",
TileWidth = 16,
TileHeight = 16,
TileCount = 4,
Columns = 2,
Grid = new Grid
{
Orientation = GridOrientation.Orthogonal,
Width = 32,
Height = 32
},
Image = new Image
{
Source = "tiles.png",
Width = 32,
Height = 32,
Format = ImageFormat.Png
}
}
],
Layers = [
new TileLayer
{
ID = 1,
Name = "Tile Layer 1",
Width = ext == "tmx" ? 64 : 16,
Height = ext == "tmx" ? 64 : 16,
Data = new Data
{
Encoding = DataEncoding.Csv,
Chunks = new Optional<Chunk[]>([
new Chunk
{
X = 0,
Y = 0,
Width = 16,
Height = 16,
GlobalTileIDs = [
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,
1,1,1,1,2,1,1,1,1,1,1,1,1,1,1,1,
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1
],
FlippingFlags = GetAllNoneFlippingFlags(16 * 16)
}
])
}
},
new ObjectLayer
{
ID = 3,
Name = "Object Layer 1",
Objects = [
new TileObject
{
ID = 1,
Template = ext == "tmx" ? "template.tx" : "template.tj",
X = 80,
Y = 144,
GID = 4,
Width = 16,
Height = 16,
},
new TileObject
{
ID = 2,
Template = ext == "tmx" ? "template.tx" : "template.tj",
X = 48,
Y = 144,
GID = 4,
Width = 16,
Height = 16,
}
]
}
]
};
private static FlippingFlags[] GetAllNoneFlippingFlags(int count)
{
var flippingFlags = new FlippingFlags[count];
for (int i = 0; i < count; i++)
{
flippingFlags[i] = FlippingFlags.None;
}
return flippingFlags;
}
}

79
src/DotTiled.Tests/TestData/Maps/map-duplicate-object-id-bug/map-duplicate-object-id-bug.tmj Normal file
View file

@ -0,0 +1,79 @@
{ "compressionlevel":-1,
"height":64,
"infinite":true,
"layers":[
{
"chunks":[
{
"data":[1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1,
1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1,
1, 1, 1, 1, 2, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1,
1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1,
1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1,
1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1,
1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1,
1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1,
1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1,
1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1,
1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1,
1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1,
1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1,
1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1,
1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1,
1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1],
"height":16,
"width":16,
"x":0,
"y":0
}],
"height":16,
"id":1,
"name":"Tile Layer 1",
"opacity":1,
"startx":0,
"starty":0,
"type":"tilelayer",
"visible":true,
"width":16,
"x":0,
"y":0
},
{
"draworder":"topdown",
"id":3,
"name":"Object Layer 1",
"objects":[
{
"id":1,
"template":"template.tj",
"x":80,
"y":144
},
{
"id":2,
"template":"template.tj",
"x":48,
"y":144
}],
"opacity":1,
"type":"objectgroup",
"visible":true,
"x":0,
"y":0
}],
"nextlayerid":2,
"nextobjectid":3,
"orientation":"orthogonal",
"renderorder":"right-down",
"tiledversion":"1.11.2",
"tileheight":16,
"tilesets":[
{
"firstgid":1,
"source":"tiles.tsj"
}],
"tilewidth":16,
"type":"map",
"version":"1.10",
"width":64
}

30
src/DotTiled.Tests/TestData/Maps/map-duplicate-object-id-bug/map-duplicate-object-id-bug.tmx Normal file
View file

@ -0,0 +1,30 @@
<?xml version="1.0" encoding="UTF-8"?>
<map version="1.10" tiledversion="1.11.2" orientation="orthogonal" renderorder="right-down" width="64" height="64" tilewidth="16" tileheight="16" infinite="1" nextlayerid="2" nextobjectid="3">
<tileset firstgid="1" source="tiles.tsx"/>
<layer id="1" name="Tile Layer 1" width="64" height="64">
<data encoding="csv">
<chunk x="0" y="0" width="16" height="16">
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,
1,1,1,1,2,1,1,1,1,1,1,1,1,1,1,1,
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1
</chunk>
</data>
</layer>
<objectgroup id="3" name="Object Layer 1">
<object id="1" template="template.tx" x="80" y="144"/>
<object id="2" template="template.tx" x="48" y="144"/>
</objectgroup>
</map>

18
src/DotTiled.Tests/TestData/Maps/map-duplicate-object-id-bug/template.tj Normal file
View file

@ -0,0 +1,18 @@
{ "object":
{
"gid":4,
"height":16,
"id":2,
"name":"",
"rotation":0,
"type":"",
"visible":true,
"width":16
},
"tileset":
{
"firstgid":1,
"source":"tiles.tsj"
},
"type":"template"
}

6
src/DotTiled.Tests/TestData/Maps/map-duplicate-object-id-bug/template.tx Normal file
View file

@ -0,0 +1,6 @@
<?xml version="1.0" encoding="UTF-8"?>
<template>
<tileset firstgid="1" source="tiles.tsx"/>
<object gid="4" width="16" height="16">
</object>
</template>

BIN
src/DotTiled.Tests/TestData/Maps/map-duplicate-object-id-bug/tiles.png Normal file
View file

Binary file not shown.
Side by side

After

Width:  |  Height:  |  Size: 156 B

20
src/DotTiled.Tests/TestData/Maps/map-duplicate-object-id-bug/tiles.tsj Normal file
View file

@ -0,0 +1,20 @@
{ "columns":2,
"grid":
{
"height":32,
"orientation":"orthogonal",
"width":32
},
"image":"tiles.png",
"imageheight":32,
"imagewidth":32,
"margin":0,
"name":"Tiles",
"spacing":0,
"tilecount":4,
"tiledversion":"1.11.2",
"tileheight":16,
"tilewidth":16,
"type":"tileset",
"version":"1.10"
}

5
src/DotTiled.Tests/TestData/Maps/map-duplicate-object-id-bug/tiles.tsx Normal file
View file

@ -0,0 +1,5 @@
<?xml version="1.0" encoding="UTF-8"?>
<tileset version="1.10" tiledversion="1.11.2" name="Tiles" tilewidth="16" tileheight="16" tilecount="4" columns="2">
<grid orientation="orthogonal" width="32" height="32"/>
<image source="tiles.png" width="32" height="32"/>
</tileset>

35
src/DotTiled.Tests/UnitTests/Serialization/LoaderTests.cs
View file

@ -315,4 +315,39 @@ public class LoaderTests
// Assert // Assert
DotTiledAssert.AssertProperties(customClassDefinition.Members, result.Properties); DotTiledAssert.AssertProperties(customClassDefinition.Members, result.Properties);
} }
public static IEnumerable<object[]> Maps => TestData.MapTests;
[Theory]
[MemberData(nameof(Maps))]
public void LoadMap_ValidFilesExternalTilesetsAndTemplatesWithCache_ReturnsMapThatEqualsExpected(
string testDataFile,
Func<string, Map> expectedMap,
IReadOnlyCollection<ICustomTypeDefinition> customTypeDefinitions)
{
// Arrange
string[] fileFormats = [".tmx", ".tmj"];
foreach (var fileFormat in fileFormats)
{
var testDataFileWithFormat = testDataFile + fileFormat;
var resourceReader = Substitute.For<IResourceReader>();
resourceReader.Read(Arg.Any<string>()).Returns(callInfo =>
{
var filePath = callInfo.Arg<string>();
return TestData.GetRawStringFor(filePath);
});
var loader = Loader.DefaultWith(
resourceReader: resourceReader,
customTypeDefinitions: customTypeDefinitions);
// Act
var map = loader.LoadMap(testDataFileWithFormat);
// Assert
Assert.NotNull(map);
DotTiledAssert.AssertMap(expectedMap(fileFormat[1..]), map);
}
}
} }

1
src/DotTiled.Tests/UnitTests/Serialization/TestData.cs
View file

@ -34,6 +34,7 @@ public static partial class TestData
public static IEnumerable<object[]> MapTests => public static IEnumerable<object[]> MapTests =>
[ [
[GetMapPath("default-map"), (string f) => DefaultMap(), Array.Empty<ICustomTypeDefinition>()], [GetMapPath("default-map"), (string f) => DefaultMap(), Array.Empty<ICustomTypeDefinition>()],
[GetMapPath("map-duplicate-object-id-bug"), (string f) => MapDuplicateObjectIdBug(f), Array.Empty<ICustomTypeDefinition>()],
[GetMapPath("map-with-common-props"), (string f) => MapWithCommonProps(), Array.Empty<ICustomTypeDefinition>()], [GetMapPath("map-with-common-props"), (string f) => MapWithCommonProps(), Array.Empty<ICustomTypeDefinition>()],
[GetMapPath("map-with-custom-type-props"), (string f) => MapWithCustomTypeProps(), MapWithCustomTypePropsCustomTypeDefinitions()], [GetMapPath("map-with-custom-type-props"), (string f) => MapWithCustomTypeProps(), MapWithCustomTypePropsCustomTypeDefinitions()],
[GetMapPath("map-with-custom-type-props-without-defs"), (string f) => MapWithCustomTypePropsWithoutDefs(), Array.Empty<ICustomTypeDefinition>()], [GetMapPath("map-with-custom-type-props-without-defs"), (string f) => MapWithCustomTypePropsWithoutDefs(), Array.Empty<ICustomTypeDefinition>()],