diff --git a/.editorconfig b/.editorconfig index 71746d2..8e747e9 100644 --- a/.editorconfig +++ b/.editorconfig @@ -235,6 +235,7 @@ dotnet_diagnostic.IDE0004.severity = silent dotnet_diagnostic.IDE0005.severity = error dotnet_diagnostic.IDE0008.severity = silent dotnet_diagnostic.IDE0055.severity = silent +dotnet_diagnostic.IDE0058.severity = silent dotnet_diagnostic.IDE0160.severity = none dotnet_diagnostic.CA1707.severity = silent dotnet_diagnostic.CA1852.severity = none @@ -243,7 +244,7 @@ dotnet_diagnostic.CA1720.severity = silent dotnet_diagnostic.CA1711.severity = silent dotnet_diagnostic.CA1716.severity = silent -[.github/**/*.yml] +[*.(yml|json|js)] charset = utf-8 end_of_line = lf indent_size = 2 diff --git a/.github/pull_request_template/dev_template.md b/.github/pull_request_template/dev_template.md new file mode 100644 index 0000000..128b591 --- /dev/null +++ b/.github/pull_request_template/dev_template.md @@ -0,0 +1,15 @@ +## Description + + +### Type of Change + +- [ ] Bug fix (non-breaking change which fixes an issue) +- [ ] New feature (non-breaking change which adds functionality) +- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected) +- [ ] Documentation update + +## Checklist + +- [ ] Tests have been added/updated to cover new functionality. +- [ ] Documentation has been updated for all new changes (e.g., usage examples, API documentation). + diff --git a/.github/pull_request_template/master_template.md b/.github/pull_request_template/master_template.md new file mode 100644 index 0000000..28ba2d5 --- /dev/null +++ b/.github/pull_request_template/master_template.md @@ -0,0 +1,10 @@ +# Release vX.Y.Z + + + +## Checklist for a release PR + +- [ ] Version compatibility matrix has been updated. +- [ ] Tests have been added/updated to cover new functionality. +- [ ] Documentation has been updated for all new changes (e.g., usage examples, API documentation). + diff --git a/Makefile b/Makefile index 601eda2..f7382f5 100644 --- a/Makefile +++ b/Makefile @@ -2,13 +2,15 @@ test: dotnet build src/DotTiled.sln dotnet test src/DotTiled.sln -docs-serve: +docs-serve: docs/index.md docfx docs/docfx.json --serve -docs-build: - cp README.md docs/index.md +docs-build: docs/index.md docfx docs/docfx.json +docs/index.md: + cp README.md docs/index.md + lint: dotnet format style --verify-no-changes src/DotTiled.sln dotnet format analyzers --verify-no-changes src/DotTiled.sln diff --git a/README.md b/README.md index 8739d03..6d9de13 100644 --- a/README.md +++ b/README.md @@ -16,10 +16,10 @@ Other similar libraries exist, and you may want to consider them for your projec |**Comparison**|**DotTiled**|[TiledLib](https://github.com/Ragath/TiledLib.Net)|[TiledCSPlus](https://github.com/nolemretaWxd/TiledCSPlus)|[TiledSharp](https://github.com/marshallward/TiledSharp)|[TiledCS](https://github.com/TheBoneJarmer/TiledCS)|[TiledNet](https://github.com/napen123/Tiled.Net)| |---------------------------------|:-----------------------:|:--------:|:-----------:|:----------:|:-------:|:------:| -| Actively maintained | ✅ | ✅ | ✅ | ❌ | ❌ | ❌ | +| Actively maintained | ✅ | ✅ | ❌ | ❌ | ❌ | ❌ | | Benchmark (time)* | 1.00 | 1.83 | 2.16 | - | - | - | | Benchmark (memory)* | 1.00 | 1.43 | 2.03 | - | - | - | -| .NET Targets | `net8.0` |`net6.0`
`net7.0`|`netstandard2.1`|`netstandard2.0`|`netstandard2.0`|`net45`| +| .NET Targets | `net8.0` | `net8.0` |`netstandard2.1`|`netstandard2.0`|`netstandard2.0`|`net45`| | Docs |Usage, API,
XML Docs|Usage|Usage, API,
XML Docs|Usage, API|Usage, XML Docs|Usage, XML Docs| | License | MIT | MIT | MIT | Apache-2.0 | MIT | BSD 3-Clause | diff --git a/docs/docfx.json b/docs/docfx.json index 4c955cb..744755c 100644 --- a/docs/docfx.json +++ b/docs/docfx.json @@ -41,7 +41,10 @@ "_appName": "DotTiled", "_appTitle": "DotTiled", "_enableSearch": true, - "pdf": true - } + "pdf": false + }, + "xref": [ + "https://learn.microsoft.com/en-us/dotnet/.xrefmap.json" + ] } } \ No newline at end of file diff --git a/docs/docs/essentials/custom-properties.md b/docs/docs/essentials/custom-properties.md index b18ba7f..9fc19aa 100644 --- a/docs/docs/essentials/custom-properties.md +++ b/docs/docs/essentials/custom-properties.md @@ -66,11 +66,14 @@ Tiled supports a variety of property types, which are represented in the DotTile - `object` - - `string` - -In addition to these primitive property types, [Tiled also supports more complex property types](https://doc.mapeditor.org/en/stable/manual/custom-properties/#custom-types). These custom property types are defined in Tiled according to the linked documentation, and to work with them in DotTiled, you *must* define their equivalences as a . You must then provide a resolving function to a defined type given a custom type name, as it is defined in Tiled. +In addition to these primitive property types, [Tiled also supports more complex property types](https://doc.mapeditor.org/en/stable/manual/custom-properties/#custom-types). These custom property types are defined in Tiled according to the linked documentation, and to work with them in DotTiled, you *must* define their equivalences as a . This is because of how Tiled handles default values for custom property types, and DotTiled needs to know these defaults to be able to populate the properties correctly. ## Custom types -Tiled allows you to define custom property types that can be used in your maps. These custom property types can be of type `class` or `enum`. DotTiled supports custom property types by allowing you to define the equivalent in C# and then providing a custom type resolver function that will return the equivalent definition given a custom type name. +Tiled allows you to define custom property types that can be used in your maps. These custom property types can be of type `class` or `enum`. DotTiled supports custom property types by allowing you to define the equivalent in C#. This section will guide you through how to define custom property types in DotTiled and how to map properties in loaded maps to C# classes or enums. + +> [!NOTE] +> In the future, DotTiled could provide a way to configure the use of custom property types such that they aren't necessary to be defined, given that you have set the `Resolve object types and properties` setting in Tiled. ### Class properties @@ -88,14 +91,39 @@ var monsterSpawnerDefinition = new CustomClassDefinition Name = "MonsterSpawner", UseAs = CustomClassUseAs.All, // Not really validated by DotTiled Members = [ // Make sure that the default values match the Tiled UI - new BoolProperty { Name = "enabled", Value = true }, - new IntProperty { Name = "maxSpawnAmount", Value = 10 }, - new IntProperty { Name = "minSpawnAmount", Value = 0 }, - new StringProperty { Name = "monsterNames", Value = "" } + new BoolProperty { Name = "Enabled", Value = true }, + new IntProperty { Name = "MaxSpawnAmount", Value = 10 }, + new IntProperty { Name = "MinSpawnAmount", Value = 0 }, + new StringProperty { Name = "MonsterNames", Value = "" } ] }; ``` +Luckily, you don't have to manually define these custom class definitions, even though you most definitively can for scenarios that require it. DotTiled provides a way to automatically generate these definitions for you from a C# class. This is done by using the method, or one of its overloads. This method will generate a from a given C# class, and you can then use this definition when loading your maps. + +```csharp +class MonsterSpawner +{ + public bool Enabled { get; set; } = true; + public int MaxSpawnAmount { get; set; } = 10; + public int MinSpawnAmount { get; set; } = 0; + public string MonsterNames { get; set; } = ""; +} + +// ... + +// These are all valid ways to create your custom class definitions from a C# class +// The first two require the class to have a default, parameterless constructor +var monsterSpawnerDefinition1 = CustomClassDefinition.FromClass(); +var monsterSpawnerDefinition2 = CustomClassDefinition.FromClass(typeof(MonsterSpawner)); +var monsterSpawnerDefinition3 = CustomClassDefinition.FromClass(() => new MonsterSpawner +{ + Enabled = false // This will use the property values in the instance created by a factory method as the default values +}); +``` + +The last one is especially useful if you have classes that may not have parameterless constructors, or if you want to provide custom default values for the properties. Finally, the generated custom class definition will be identical to the one defined manually in the first example. + ### Enum properties Tiled also allows you to define custom property types that work as enums. Similarly to `class` properties, you must define the equivalent in DotTiled as a . You can then return the corresponding definition in the resolving function. @@ -110,7 +138,7 @@ The equivalent definition in DotTiled would look like the following: var entityTypeDefinition = new CustomEnumDefinition { Name = "EntityType", - StorageType = CustomEnumStorageType.String, + StorageType = CustomEnumStorageType.Int, ValueAsFlags = false, Values = [ "Bomb", @@ -121,23 +149,9 @@ var entityTypeDefinition = new CustomEnumDefinition }; ``` -### [Future] Automatically map custom property `class` types to C# classes - -In the future, DotTiled will support automatically mapping custom property `class` types to C# classes. This will allow you to define a C# class that matches the structure of the `class` property in Tiled, and DotTiled will automatically map the properties of the `class` property to the properties of the C# class. This will make working with `class` properties much easier and more intuitive. - -The idea is to expand on the interface with a method like `GetMappedProperty(string propertyName)`, where `T` is a class that matches the structure of the `class` property in Tiled. - -This functionality would be accompanied by a way to automatically create a matching given a C# class or enum. Something like this would then be possible: +Similarly to custom class definitions, you can also automatically generate custom enum definitions from C# enums. This is done by using the method, or one of its overloads. This method will generate a from a given C# enum, and you can then use this definition when loading your maps. ```csharp -class MonsterSpawner -{ - public bool Enabled { get; set; } = true; - public int MaxSpawnAmount { get; set; } = 10; - public int MinSpawnAmount { get; set; } = 0; - public string MonsterNames { get; set; } = ""; -} - enum EntityType { Bomb, @@ -146,16 +160,89 @@ enum EntityType Chair } -var monsterSpawnerDefinition = CustomClassDefinition.FromClass(); -var entityTypeDefinition = CustomEnumDefinition.FromEnum(); - // ... -var map = LoadMap(); -var monsterSpawner = map.GetMappedProperty("monsterSpawnerPropertyInMap"); -var entityType = map.GetMappedProperty("entityTypePropertyInMap"); +// These are both valid ways to create your custom enum definitions from a C# enum +var entityTypeDefinition1 = CustomEnumDefinition.FromEnum(); +var entityTypeDefinition2 = CustomEnumDefinition.FromEnum(typeof(EntityType)); ``` -Finally, it might be possible to also make some kind of exporting functionality for . Given a collection of custom type definitions, DotTiled could generate a corresponding `propertytypes.json` file that you then can import into Tiled. This would make it so that you only have to define your custom property types once (in C#) and then import them into Tiled to use them in your maps. +The generated custom enum definition will be identical to the one defined manually in the first example. -Depending on implementation this might become something that can inhibit native AOT compilation due to potential reflection usage. Source generators could be used to mitigate this, but it is not yet clear how this will be implemented. \ No newline at end of file +For enum definitions, the can be used to indicate that the enum should be treated as a flags enum. This will make it so the enum definition will have `ValueAsFlags = true` and the enum values will be treated as flags when working with them in DotTiled. + +## Mapping properties to C# classes or enums + +So far, we have only discussed how to define custom property types in DotTiled, and why they are needed. However, the most important part is how you can map properties inside your maps to their corresponding C# classes or enums. + +The interface has two overloads of the method. These methods allow you to map a collection of properties to a given C# class. Let's look at an example: + +```csharp +// Define a few Tiled compatible custom types +enum EntityType +{ + Player, + Enemy, + Collectible, + Explosive +} + +class EntityData +{ + public EntityType Type { get; set; } = EntityType.Player; + public int Health { get; set; } = 100; + public string Name { get; set; } = "Unnamed Entity"; +} + +var entityTypeDef = CustomEnumDefinition.FromEnum(); +var entityDataDef = CustomClassDefinition.FromClass(); +``` + +The above gives us two custom type definitions that we can supply to our map loader. Given a map that looks like this: + +```xml + + + + + + 1,1,1,1,1, + 0,0,0,0,0, + 0,0,0,0,0, + 0,0,0,0,0, + 0,0,0,0,0 + + + + + + + + + + + + + +``` + +We can see that there is an ellipse object in the map that has the type `EntityData` and it has set the three properties to some value other than their defaults. After having loaded this map, we can map the properties of the object to the `EntityData` class like this: + +```csharp +var map = LoadMap([entityTypeDef, entityDataDef]); // Load the map somehow, using DotTiled.Loader or similar + +// Retrieve the object layer from the map in some way +var objectLayer = map.Layers.Skip(1).First() as ObjectLayer; + +// Retrieve the object from the object layer +var entityObject = objectLayer.Objects.First(); + +// Map the properties of the object to the EntityData class +var entityData = entityObject.MapPropertiesTo(); +``` + +The above snippet will map the properties of the object to the `EntityData` class using reflection based on the property names. The `entityData` object will now have the properties set to the values found in the object in the map. If a property is not found in the object, the default value of the property in the `EntityData` class will be used. It will even map the nested enum to its corresponding value in C#. This will work for several levels of depth as it performs this kind of mapping recursively. only supports mapping to classes that have a default, parameterless constructor. + +### [Future] Exporting custom types + +It might be possible to also make some kind of exporting functionality for . Given a collection of custom type definitions, DotTiled could generate a corresponding `propertytypes.json` file that you then can import into Tiled. This would make it so that you only have to define your custom property types once (in C#) and then import them into Tiled to use them in your maps. \ No newline at end of file diff --git a/docs/docs/essentials/loading-maps.md b/docs/docs/essentials/loading-maps.md index b34b964..d262ec9 100644 --- a/docs/docs/essentials/loading-maps.md +++ b/docs/docs/essentials/loading-maps.md @@ -1,15 +1,92 @@ # Loading maps -Loading maps with DotTiled is straightforward and easy. The class is a representation of a Tiled map, mimicking the structure of a Tiled map file. Map files can either be in the [`.tmx`/XML](https://doc.mapeditor.org/en/stable/reference/tmx-map-format/) or [`.tmj`/json](https://doc.mapeditor.org/en/stable/reference/json-map-format/) format. DotTiled supports **both** formats fully. +Loading maps with DotTiled is very flexible and allows you as a developer to freely choose how you want to load your maps and tilesets. This guide will show you how to customize the loading process to fit your needs. As the tip below suggests, you can also use the quickstart guide if you just want to load maps from the filesystem without any particular customization. -> [!NOTE] -> Using the `.tmj` file format will result in not having the same amount of information as for the `.tmx` format. This is due to the fact that the `.tmj` format does not include the full information that the `.tmx` format does. This is not a problem with DotTiled, but rather a limitation of the `.tmj` format. +> [!TIP] +> For a quick and easy way to load maps from the filesystem, please refer to the [quickstart guide](../quickstart.md). -## External resolution +## File format caveats -Tiled maps may consist of several external files, such as tilesets or object templates. In Tiled map files, they are typically referenced by their path relative to the map file. It would be annoying to have to first load all these external resources before loading a map (which is how some other similar libraries work), so loading a map with DotTiled is designed in a way that you only have to provide a function that resolves these external resources. This way, DotTiled will figure out which external resources are needed and will invoke the corresponding resolver function to load them. +The class is a representation of a Tiled map, mimicking the structure of a Tiled XML map file. Map files can either be in the [`.tmx`/XML](https://doc.mapeditor.org/en/stable/reference/tmx-map-format/) or [`.tmj`/json](https://doc.mapeditor.org/en/stable/reference/json-map-format/) format. DotTiled supports **both** formats fully. -Loading a map, tileset, or template will require you to specify **three** resolver functions. We'll go through each of them below. +> [!WARNING] +> Using the `.tmj` file format will result in (the source image for image layers) not having the same amount of information as for the `.tmx` format. This is due to the fact that the `.tmj` format does not include the full information that the `.tmx` format does. This is not a problem with DotTiled, but rather a limitation of the `.tmj` format. + +## The process of loading a map + +Loading a map with DotTiled is not a complex process, but one that at least demands a basic understanding of how Tiled maps are structured. The process can be broken down into the following flow(-ish) chart: + +```mermaid +flowchart LR + Z{{Loading a map + with DotTiled}} --> A + + subgraph Parsing map + A[(Read map)] --> B(Parse map) + end + + subgraph Parsing tileset + B -.->|References + external tileset| C[(Read tileset)] + C --> D(Parse tileset) --o|Store in map| B + end + + subgraph Parsing template + B -.->|References external + template in object| E[(Read template)] + E --> F(Parse template) --o|Use as template + for object| B + end + + F -.-> |References + external tileset| C + F -.-> |References + external template| E +``` + +As you can see, the process is quite simple. You read the map, parse it, and then read and parse any external tilesets or templates that are referenced in the map. The tilesets and templates are then stored in the map object that is returned to you. + +However, because DotTiled works in the way that it does, you will need to provide a way to resolve these external resources. We'll go through how to do that in the next section. + +## Loading a map with + +When using , external resources like tilesets and templates will be resolved by the loader itself. Since Tiled saves the external resource paths relative to the map file, the loader will automatically resolve these paths and use the provided to read the external resources. Therefore, as long as the external resources are accessible in a "relative path" way using the provided , you don't have to worry about resolving them yourself. + +```xml + + + + + +1,1,0,0,7, +1,1,0,0,7, +0,0,1,0,7, +0,0,0,1,7, +21,21,21,21,1 + + + +``` + +A map like the one above that is loaded by the following code will result in the loader calling `IResourceReader.Read("path/to/tileset.tsx")` to read the external tileset, since it will use the path relative to the map file to resolve the tileset. + +```csharp +var loader = Loader.Default(); +var map = loader.LoadMap("path/to/map.tmx"); +``` + +Additionally, the loader will use an in-memory cache to avoid loading the same tileset or template multiple times. This is done using the that is provided to the loader. If you don't provide a cache, the loader will use the by default. + +## Loading a map manually with + +While it is recommended to use the class to perform the loading of maps and tilesets, you may have certain requirements that necessitate you to load maps in a more manual way. This section will guide you through how to load a map manually without the use of the provided loader. + +### , , and + +are the three classes that you will use to read the map, tileset, and template, respectively. They are designed to be used in a way that you can provide your own resolver functions to load external resources. + +> [!IMPORTANT] +> The resolving functions will get the source path of the external resource as a parameter, *in the exact way it is written in the map file*. You will have to perform your own path resolution to load the external resources. ### `Func` - Tileset resolver @@ -38,17 +115,17 @@ Tileset ResolveTileset(string source) ### `Func` - Template resolver -This function is used to resolve external object templates by their source path. The function should return a instance given the source path of the template. If you just want to load templates from the file system, you can use something very similar to the tileset resolver by replacing with . +This function is used to resolve external object templates by their source path. The function should return a instance given the source path of the template. If you just want to load templates from the file system, you can use something very similar to the example tileset resolver by replacing with . -### `Func` - Custom type resolver +### `Func` - Custom type resolver -This function is used to resolve custom types that are defined in Tiled maps. Please refer to the [custom properties](custom-properties.md) documentation for more information on custom types. The function should return a instance given the custom type's name. +This function is used to resolve custom types that are defined in your Tiled maps. Please refer to the [custom properties](custom-properties.md) documentation for more information on custom types. The function should return a instance given the custom type's name. ## Putting it all together The following classes are the readers that you will need to use to read the map, tileset, and template: , , and . -Here is an example of how you can load a map with DotTiled: +Here is an example of how you can load a map with DotTiled, and is very similar to how the class works: ```csharp string mapPath = "path/to/map.tmx"; diff --git a/docs/docs/essentials/representation-model.md b/docs/docs/essentials/representation-model.md new file mode 100644 index 0000000..88dbdff --- /dev/null +++ b/docs/docs/essentials/representation-model.md @@ -0,0 +1,17 @@ +# Representation model + +Tiled map files contain various types of data, such as tilesets, layers, and object groups. The representation model is a way to represent this data in a structured way. By using the `.tmx` file format as inspiration, the representation model is a collection of classes which mimic the structure of a Tiled map file. + +Certain properties throughout the representation model are marked as *optional* by being either wrapped in a or by having a set default value. + +- Properties that make use of the [required](https://learn.microsoft.com/en-us/dotnet/csharp/language-reference/keywords/required) keyword must be present in the file, otherwise an error will be raised. +- Properties that have default values will use the default value if the property is not present in the file, and are not marked as required or optional since you must not provide a value for them. +- Properties that are wrapped in may or may not be present in the file, and have no default value. + +## Version compatibility + +The representation model is designed to be compatible with the latest version of Tiled. This means that it may not be able to read files from older versions of Tiled, or files that use features that are not yet supported by the representation model. However, here is a table of which versions of Tiled are supported by which versions of DotTiled. + +| Tiled version | Compatible DotTiled version(s) | +|---------------|--------------------------------| +| 1.11 | 0.1.0, 0.2.0 | \ No newline at end of file diff --git a/docs/docs/quickstart.md b/docs/docs/quickstart.md index 8d067eb..c43031b 100644 --- a/docs/docs/quickstart.md +++ b/docs/docs/quickstart.md @@ -6,41 +6,64 @@ Install DotTiled from NuGet: dotnet add package DotTiled ``` -Load a map from file system: +Use the `DotTiled` namespace (if you want). ```csharp -string mapPath = "path/to/map.tmx"; -string mapDirectory = Path.GetDirectoryName(mapPath); - -Tileset ResolveTileset(string source) -{ - string tilesetPath = Path.Combine(mapDirectory, source); - using var tilesetFileReader = new StreamReader(tilesetPath); - var tilesetString = tilesetReader.ReadToEnd(); - using var tilesetReader = new TilesetReader(tilesetString, ResolveTileset, ResolveTemplate, ResolveCustomType); - return tilesetReader.ReadTileset(); -} - -Template ResolveTemplate(string source) -{ - string templatePath = Path.Combine(mapDirectory, source); - using var templateFileReader = new StreamReader(templatePath); - var templateString = templateReader.ReadToEnd(); - using var templateReader = new TemplateReader(templateString, ResolveTileset, ResolveTemplate, ResolveCustomType); - return templateReader.ReadTemplate(); -} - -ICustomTypeDefinition ResolveCustomType(string name) -{ - var allDefinedTypes = [ ... ]; - return allDefinedTypes.FirstOrDefault(type => type.Name == name); -} - -using var mapFileReader = new StreamReader(mapPath); -var mapString = mapFileReader.ReadToEnd(); -using var mapReader = new MapReader(mapString, ResolveTileset, ResolveTemplate, ResolveCustomType); - -var map = mapReader.ReadMap(); +using DotTiled; ``` -If the above looks intimidating, don't worry! DotTiled is designed to be flexible and allow you to load maps from any source, such as a database or a custom file format. The above example is just one way to load a map from a file system. Please look at [Loading Maps](essentials/loading-maps.md) for more information on how to load maps from different sources. \ No newline at end of file +Or fully qualify all `DotTiled` types e.g. `DotTiled.Loader`. + +## Loading a map from the file system + +This will create a loader that will load files from the underlying file system using . It will also be configured to use an in-memory cache to avoid loading the same tileset or template multiple times using . + +```csharp +var loader = Loader.Default(); +var map = loader.LoadMap("path/to/map.tmx"); +``` + +## Loading a map from a different source + +If you want to load resources (maps, tilesets, templates) from a different source than the underlying file system, you can override the that is being used with your own implementation of . + +```csharp +var loader = Loader.DefaultWith( + resourceReader: new MyCustomResourceReader()); +var map = loader.LoadMap("path/to/map.tmx"); +``` + +## Caching resources + +Similarly, you can override the that is being used with your own implementation of . + +```csharp +var loader = Loader.DefaultWith( + resourceReader: new MyCustomResourceReader(), + resourceCache: new MyCustomResourceCache()); +var map = loader.LoadMap("path/to/map.tmx"); +``` + +## Custom types + +If you have custom types in your map, you can provide any `IEnumerable` to the loader. This will allow the loader to deserialize the custom types in your map. + +```csharp +var monsterSpawnerDef = new CustomClassDefinition { ... }; +var chestDef = new CustomClassDefinition +{ + Name = "Chest", + UseAs = CustomClassUseAs.All, + Members = [ + new IntProperty { Name = "coins", Value = 0 }, + new BoolProperty { Name = "locked", Value = true } + ] +}; + +var loader = Loader.DefaultWith( + customTypeDefinitions: [monsterSpawnerDef, chestDef]); +var map = loader.LoadMap("path/to/map.tmx"); + +var chest = map.GetProperty("chest").Value; +var coinsToSpawn = chest.GetProperty("coins").Value; +``` \ No newline at end of file diff --git a/docs/docs/toc.yml b/docs/docs/toc.yml index 6a04547..5b190b8 100644 --- a/docs/docs/toc.yml +++ b/docs/docs/toc.yml @@ -4,4 +4,5 @@ - name: Essentials - href: essentials/loading-maps.md +- href: essentials/representation-model.md - href: essentials/custom-properties.md \ No newline at end of file diff --git a/docs/template/public/main.js b/docs/template/public/main.js new file mode 100644 index 0000000..7ab60f9 --- /dev/null +++ b/docs/template/public/main.js @@ -0,0 +1,9 @@ +export default { + iconLinks: [ + { + icon: 'github', + href: 'https://github.com/dcronqvist/DotTiled', + title: 'GitHub' + } + ] +} \ No newline at end of file diff --git a/src/DotTiled.Tests/Assert/AssertCustomTypes.cs b/src/DotTiled.Tests/Assert/AssertCustomTypes.cs new file mode 100644 index 0000000..19eae5b --- /dev/null +++ b/src/DotTiled.Tests/Assert/AssertCustomTypes.cs @@ -0,0 +1,23 @@ +namespace DotTiled.Tests; + +public static partial class DotTiledAssert +{ + internal static void AssertCustomClassDefinitionEqual(CustomClassDefinition expected, CustomClassDefinition actual) + { + AssertEqual(expected.ID, actual.ID, nameof(CustomClassDefinition.ID)); + AssertEqual(expected.Name, actual.Name, nameof(CustomClassDefinition.Name)); + AssertEqual(expected.Color, actual.Color, nameof(CustomClassDefinition.Color)); + AssertEqual(expected.DrawFill, actual.DrawFill, nameof(CustomClassDefinition.DrawFill)); + AssertEqual(expected.UseAs, actual.UseAs, nameof(CustomClassDefinition.UseAs)); + AssertProperties(expected.Members, actual.Members); + } + + internal static void AssertCustomEnumDefinitionEqual(CustomEnumDefinition expected, CustomEnumDefinition actual) + { + AssertEqual(expected.ID, actual.ID, nameof(CustomEnumDefinition.ID)); + AssertEqual(expected.Name, actual.Name, nameof(CustomEnumDefinition.Name)); + AssertEqual(expected.StorageType, actual.StorageType, nameof(CustomEnumDefinition.StorageType)); + AssertEqual(expected.ValueAsFlags, actual.ValueAsFlags, nameof(CustomEnumDefinition.ValueAsFlags)); + AssertListOrdered(expected.Values, actual.Values, nameof(CustomEnumDefinition.Values)); + } +} diff --git a/src/DotTiled.Tests/Assert/AssertData.cs b/src/DotTiled.Tests/Assert/AssertData.cs index 31ffff2..6e8aaa2 100644 --- a/src/DotTiled.Tests/Assert/AssertData.cs +++ b/src/DotTiled.Tests/Assert/AssertData.cs @@ -2,7 +2,7 @@ namespace DotTiled.Tests; public static partial class DotTiledAssert { - internal static void AssertData(Data? expected, Data? actual) + internal static void AssertData(Data expected, Data actual) { if (expected is null) { @@ -18,14 +18,7 @@ public static partial class DotTiledAssert // Data AssertEqual(expected.GlobalTileIDs, actual.GlobalTileIDs, nameof(Data.GlobalTileIDs)); AssertEqual(expected.FlippingFlags, actual.FlippingFlags, nameof(Data.FlippingFlags)); - - if (expected.Chunks is not null) - { - Assert.NotNull(actual.Chunks); - AssertEqual(expected.Chunks.Length, actual.Chunks.Length, "Chunks.Length"); - for (var i = 0; i < expected.Chunks.Length; i++) - AssertChunk(expected.Chunks[i], actual.Chunks[i]); - } + AssertOptionalsEqual(expected.Chunks, actual.Chunks, nameof(Data.Chunks), (a, b) => AssertListOrdered(a, b, nameof(Chunk), AssertChunk)); } private static void AssertChunk(Chunk expected, Chunk actual) diff --git a/src/DotTiled.Tests/Assert/AssertImage.cs b/src/DotTiled.Tests/Assert/AssertImage.cs index a674faa..6bb5f4e 100644 --- a/src/DotTiled.Tests/Assert/AssertImage.cs +++ b/src/DotTiled.Tests/Assert/AssertImage.cs @@ -2,7 +2,7 @@ namespace DotTiled.Tests; public static partial class DotTiledAssert { - internal static void AssertImage(Image? expected, Image? actual) + internal static void AssertImage(Image expected, Image actual) { if (expected is null) { diff --git a/src/DotTiled.Tests/Assert/AssertLayer.cs b/src/DotTiled.Tests/Assert/AssertLayer.cs index 5432d62..7dc8011 100644 --- a/src/DotTiled.Tests/Assert/AssertLayer.cs +++ b/src/DotTiled.Tests/Assert/AssertLayer.cs @@ -2,7 +2,7 @@ namespace DotTiled.Tests; public static partial class DotTiledAssert { - internal static void AssertLayer(BaseLayer? expected, BaseLayer? actual) + internal static void AssertLayer(BaseLayer expected, BaseLayer actual) { if (expected is null) { diff --git a/src/DotTiled.Tests/Assert/AssertMap.cs b/src/DotTiled.Tests/Assert/AssertMap.cs index 0110f51..bc87002 100644 --- a/src/DotTiled.Tests/Assert/AssertMap.cs +++ b/src/DotTiled.Tests/Assert/AssertMap.cs @@ -5,7 +5,69 @@ namespace DotTiled.Tests; public static partial class DotTiledAssert { - private static void AssertEqual(T expected, T actual, string nameof) + internal static void AssertListOrdered(IList expected, IList actual, string nameof, Action assertEqual = null) + { + if (expected is null) + { + Assert.Null(actual); + return; + } + + Assert.NotNull(actual); + AssertEqual(expected.Count, actual.Count, $"{nameof}.Count"); + + for (var i = 0; i < expected.Count; i++) + { + if (assertEqual is not null) + { + assertEqual(expected[i], actual[i]); + continue; + } + AssertEqual(expected[i], actual[i], $"{nameof}[{i}]"); + } + } + + internal static void AssertOptionalsEqual( + Optional expected, + Optional actual, + string nameof, + Action assertEqual) + { + if (expected is null) + { + Assert.Null(actual); + return; + } + + if (expected.HasValue) + { + Assert.True(actual.HasValue, $"Expected {nameof} to have a value"); + assertEqual(expected.Value, actual.Value); + return; + } + + Assert.False(actual.HasValue, $"Expected {nameof} to not have a value"); + } + + internal static void AssertEqual(Optional expected, Optional actual, string nameof) + { + if (expected is null) + { + Assert.Null(actual); + return; + } + + if (expected.HasValue) + { + Assert.True(actual.HasValue, $"Expected {nameof} to have a value"); + AssertEqual(expected.Value, actual.Value, nameof); + return; + } + + Assert.False(actual.HasValue, $"Expected {nameof} to not have a value"); + } + + internal static void AssertEqual(T expected, T actual, string nameof) { if (expected == null) { diff --git a/src/DotTiled.Tests/Assert/AssertProperties.cs b/src/DotTiled.Tests/Assert/AssertProperties.cs index 843d8d0..9454486 100644 --- a/src/DotTiled.Tests/Assert/AssertProperties.cs +++ b/src/DotTiled.Tests/Assert/AssertProperties.cs @@ -2,7 +2,7 @@ namespace DotTiled.Tests; public static partial class DotTiledAssert { - internal static void AssertProperties(IList? expected, IList? actual) + internal static void AssertProperties(IList expected, IList actual) { if (expected is null) { diff --git a/src/DotTiled.Tests/Assert/AssertTileset.cs b/src/DotTiled.Tests/Assert/AssertTileset.cs index 4646a85..607b542 100644 --- a/src/DotTiled.Tests/Assert/AssertTileset.cs +++ b/src/DotTiled.Tests/Assert/AssertTileset.cs @@ -4,7 +4,6 @@ public static partial class DotTiledAssert { internal static void AssertTileset(Tileset expected, Tileset actual) { - // Attributes AssertEqual(expected.Version, actual.Version, nameof(Tileset.Version)); AssertEqual(expected.TiledVersion, actual.TiledVersion, nameof(Tileset.TiledVersion)); AssertEqual(expected.FirstGID, actual.FirstGID, nameof(Tileset.FirstGID)); @@ -21,29 +20,16 @@ public static partial class DotTiledAssert AssertEqual(expected.RenderSize, actual.RenderSize, nameof(Tileset.RenderSize)); AssertEqual(expected.FillMode, actual.FillMode, nameof(Tileset.FillMode)); - // At most one of - AssertImage(expected.Image, actual.Image); - AssertTileOffset(expected.TileOffset, actual.TileOffset); - AssertGrid(expected.Grid, actual.Grid); + AssertOptionalsEqual(expected.Image, actual.Image, nameof(Tileset.Image), AssertImage); + AssertOptionalsEqual(expected.TileOffset, actual.TileOffset, nameof(Tileset.TileOffset), AssertTileOffset); + AssertOptionalsEqual(expected.Grid, actual.Grid, nameof(Tileset.Grid), AssertGrid); AssertProperties(expected.Properties, actual.Properties); - // TODO: AssertTerrainTypes(actual.TerrainTypes, expected.TerrainTypes); - if (expected.Wangsets is not null) - { - Assert.NotNull(actual.Wangsets); - AssertEqual(expected.Wangsets.Count, actual.Wangsets.Count, "Wangsets.Count"); - for (var i = 0; i < expected.Wangsets.Count; i++) - AssertWangset(expected.Wangsets[i], actual.Wangsets[i]); - } - AssertTransformations(expected.Transformations, actual.Transformations); - - // Any number of - Assert.NotNull(actual.Tiles); - AssertEqual(expected.Tiles.Count, actual.Tiles.Count, "Tiles.Count"); - for (var i = 0; i < expected.Tiles.Count; i++) - AssertTile(expected.Tiles[i], actual.Tiles[i]); + AssertListOrdered(expected.Wangsets, actual.Wangsets, nameof(Tileset.Wangsets), AssertWangset); + AssertOptionalsEqual(expected.Transformations, actual.Transformations, nameof(Tileset.Transformations), AssertTransformations); + AssertListOrdered(expected.Tiles, actual.Tiles, nameof(Tileset.Tiles), AssertTile); } - private static void AssertTileOffset(TileOffset? expected, TileOffset? actual) + private static void AssertTileOffset(TileOffset expected, TileOffset actual) { if (expected is null) { @@ -57,7 +43,7 @@ public static partial class DotTiledAssert AssertEqual(expected.Y, actual.Y, nameof(TileOffset.Y)); } - private static void AssertGrid(Grid? expected, Grid? actual) + private static void AssertGrid(Grid expected, Grid actual) { if (expected is null) { @@ -74,27 +60,17 @@ public static partial class DotTiledAssert private static void AssertWangset(Wangset expected, Wangset actual) { - // Attributes AssertEqual(expected.Name, actual.Name, nameof(Wangset.Name)); AssertEqual(expected.Class, actual.Class, nameof(Wangset.Class)); AssertEqual(expected.Tile, actual.Tile, nameof(Wangset.Tile)); - // At most one of AssertProperties(expected.Properties, actual.Properties); - if (expected.WangColors is not null) - { - Assert.NotNull(actual.WangColors); - AssertEqual(expected.WangColors.Count, actual.WangColors.Count, "WangColors.Count"); - for (var i = 0; i < expected.WangColors.Count; i++) - AssertWangColor(expected.WangColors[i], actual.WangColors[i]); - } - for (var i = 0; i < expected.WangTiles.Count; i++) - AssertWangTile(expected.WangTiles[i], actual.WangTiles[i]); + AssertListOrdered(expected.WangColors, actual.WangColors, nameof(Wangset.WangColors), AssertWangColor); + AssertListOrdered(expected.WangTiles, actual.WangTiles, nameof(Wangset.WangTiles), AssertWangTile); } private static void AssertWangColor(WangColor expected, WangColor actual) { - // Attributes AssertEqual(expected.Name, actual.Name, nameof(WangColor.Name)); AssertEqual(expected.Class, actual.Class, nameof(WangColor.Class)); AssertEqual(expected.Color, actual.Color, nameof(WangColor.Color)); @@ -106,12 +82,11 @@ public static partial class DotTiledAssert private static void AssertWangTile(WangTile expected, WangTile actual) { - // Attributes AssertEqual(expected.TileID, actual.TileID, nameof(WangTile.TileID)); AssertEqual(expected.WangID, actual.WangID, nameof(WangTile.WangID)); } - private static void AssertTransformations(Transformations? expected, Transformations? actual) + private static void AssertTransformations(Transformations expected, Transformations actual) { if (expected is null) { @@ -119,7 +94,6 @@ public static partial class DotTiledAssert return; } - // Attributes Assert.NotNull(actual); AssertEqual(expected.HFlip, actual.HFlip, nameof(Transformations.HFlip)); AssertEqual(expected.VFlip, actual.VFlip, nameof(Transformations.VFlip)); @@ -129,7 +103,6 @@ public static partial class DotTiledAssert private static void AssertTile(Tile expected, Tile actual) { - // Attributes AssertEqual(expected.ID, actual.ID, nameof(Tile.ID)); AssertEqual(expected.Type, actual.Type, nameof(Tile.Type)); AssertEqual(expected.Probability, actual.Probability, nameof(Tile.Probability)); @@ -138,22 +111,14 @@ public static partial class DotTiledAssert AssertEqual(expected.Width, actual.Width, nameof(Tile.Width)); AssertEqual(expected.Height, actual.Height, nameof(Tile.Height)); - // Elements AssertProperties(expected.Properties, actual.Properties); - AssertImage(expected.Image, actual.Image); - AssertLayer((BaseLayer?)expected.ObjectLayer, (BaseLayer?)actual.ObjectLayer); - if (expected.Animation is not null) - { - Assert.NotNull(actual.Animation); - AssertEqual(expected.Animation.Count, actual.Animation.Count, "Animation.Count"); - for (var i = 0; i < expected.Animation.Count; i++) - AssertFrame(expected.Animation[i], actual.Animation[i]); - } + AssertOptionalsEqual(expected.Image, actual.Image, nameof(Tile.Image), AssertImage); + AssertOptionalsEqual(expected.ObjectLayer, actual.ObjectLayer, nameof(Tile.ObjectLayer), (a, b) => AssertLayer((BaseLayer)a, (BaseLayer)b)); + AssertListOrdered(expected.Animation, actual.Animation, nameof(Tile.Animation), AssertFrame); } private static void AssertFrame(Frame expected, Frame actual) { - // Attributes AssertEqual(expected.TileID, actual.TileID, nameof(Frame.TileID)); AssertEqual(expected.Duration, actual.Duration, nameof(Frame.Duration)); } diff --git a/src/DotTiled.Tests/DotTiled.Tests.csproj b/src/DotTiled.Tests/DotTiled.Tests.csproj index faa22d4..552a423 100644 --- a/src/DotTiled.Tests/DotTiled.Tests.csproj +++ b/src/DotTiled.Tests/DotTiled.Tests.csproj @@ -3,7 +3,6 @@ net8.0 enable - enable false true @@ -12,6 +11,7 @@ + @@ -26,7 +26,12 @@ - + + + + + + diff --git a/src/DotTiled.Tests/IntegrationTests/CustomTypes/FromTypeUsedInLoaderTests.cs b/src/DotTiled.Tests/IntegrationTests/CustomTypes/FromTypeUsedInLoaderTests.cs new file mode 100644 index 0000000..dae9464 --- /dev/null +++ b/src/DotTiled.Tests/IntegrationTests/CustomTypes/FromTypeUsedInLoaderTests.cs @@ -0,0 +1,180 @@ +using DotTiled.Serialization; +using NSubstitute; + +namespace DotTiled.Tests; + +public class FromTypeUsedInLoaderTests +{ + private sealed class TestClass + { + public string Name { get; set; } = "John Doe"; + public int Age { get; set; } = 42; + } + + [Fact] + public void LoadMap_MapHasClassAndClassIsDefined_ReturnsCorrectMap() + { + // Arrange + var resourceReader = Substitute.For(); + resourceReader.Read("map.tmx").Returns( + """ + + + + + 0,0,0,0,0, + 0,0,0,0,0, + 0,0,0,0,0, + 0,0,0,0,0, + 0,0,0,0,0 + + + + """); + var classDefinition = CustomClassDefinition.FromClass(); + var loader = Loader.DefaultWith( + resourceReader: resourceReader, + customTypeDefinitions: [classDefinition]); + var expectedMap = new Map + { + Class = "TestClass", + Orientation = MapOrientation.Orthogonal, + Width = 5, + Height = 5, + TileWidth = 32, + TileHeight = 32, + Infinite = false, + ParallaxOriginX = 0, + ParallaxOriginY = 0, + RenderOrder = RenderOrder.RightDown, + CompressionLevel = -1, + BackgroundColor = new Color { R = 0, G = 0, B = 0, A = 0 }, + Version = "1.10", + TiledVersion = "1.11.0", + NextLayerID = 2, + NextObjectID = 1, + Layers = [ + new TileLayer + { + ID = 1, + Name = "Tile Layer 1", + Width = 5, + Height = 5, + Data = new Data + { + Encoding = DataEncoding.Csv, + GlobalTileIDs = new Optional([ + 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0 + ]), + FlippingFlags = new Optional([ + FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, + FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, + FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, + FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, + FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None + ]) + } + } + ], + Properties = [ + new StringProperty { Name = "Name", Value = "John Doe" }, + new IntProperty { Name = "Age", Value = 42 } + ] + }; + + // Act + var result = loader.LoadMap("map.tmx"); + + // Assert + DotTiledAssert.AssertMap(expectedMap, result); + } + + [Fact] + public void LoadMap_MapHasClassAndClassIsDefinedAndFieldIsOverridenFromDefault_ReturnsCorrectMap() + { + // Arrange + var resourceReader = Substitute.For(); + resourceReader.Read("map.tmx").Returns( + """ + + + + + + + + 0,0,0,0,0, + 0,0,0,0,0, + 0,0,0,0,0, + 0,0,0,0,0, + 0,0,0,0,0 + + + + """); + var classDefinition = CustomClassDefinition.FromClass(); + var loader = Loader.DefaultWith( + resourceReader: resourceReader, + customTypeDefinitions: [classDefinition]); + var expectedMap = new Map + { + Class = "TestClass", + Orientation = MapOrientation.Orthogonal, + Width = 5, + Height = 5, + TileWidth = 32, + TileHeight = 32, + Infinite = false, + ParallaxOriginX = 0, + ParallaxOriginY = 0, + RenderOrder = RenderOrder.RightDown, + CompressionLevel = -1, + BackgroundColor = new Color { R = 0, G = 0, B = 0, A = 0 }, + Version = "1.10", + TiledVersion = "1.11.0", + NextLayerID = 2, + NextObjectID = 1, + Layers = [ + new TileLayer + { + ID = 1, + Name = "Tile Layer 1", + Width = 5, + Height = 5, + Data = new Data + { + Encoding = DataEncoding.Csv, + GlobalTileIDs = new Optional([ + 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0 + ]), + FlippingFlags = new Optional([ + FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, + FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, + FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, + FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, + FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None + ]) + } + } + ], + Properties = [ + new StringProperty { Name = "Name", Value = "John Doe" }, + new IntProperty { Name = "Age", Value = 42 } + ] + }; + + // Act + var result = loader.LoadMap("map.tmx"); + + // Assert + DotTiledAssert.AssertMap(expectedMap, result); + } +} diff --git a/src/DotTiled.Tests/Serialization/TestData.cs b/src/DotTiled.Tests/Serialization/TestData.cs deleted file mode 100644 index 7f68c9e..0000000 --- a/src/DotTiled.Tests/Serialization/TestData.cs +++ /dev/null @@ -1,45 +0,0 @@ -using System.Xml; - -namespace DotTiled.Tests; - -public static partial class TestData -{ - public static XmlReader GetXmlReaderFor(string testDataFile) - { - var fullyQualifiedTestDataFile = $"DotTiled.Tests.{ConvertPathToAssemblyResourcePath(testDataFile)}"; - using var stream = typeof(TestData).Assembly.GetManifestResourceStream(fullyQualifiedTestDataFile) - ?? throw new ArgumentException($"Test data file '{fullyQualifiedTestDataFile}' not found"); - - using var stringReader = new StreamReader(stream); - var xml = stringReader.ReadToEnd(); - var xmlStringReader = new StringReader(xml); - return XmlReader.Create(xmlStringReader); - } - - public static string GetRawStringFor(string testDataFile) - { - var fullyQualifiedTestDataFile = $"DotTiled.Tests.{ConvertPathToAssemblyResourcePath(testDataFile)}"; - using var stream = typeof(TestData).Assembly.GetManifestResourceStream(fullyQualifiedTestDataFile) - ?? throw new ArgumentException($"Test data file '{fullyQualifiedTestDataFile}' not found"); - - using var stringReader = new StreamReader(stream); - return stringReader.ReadToEnd(); - } - - private static string ConvertPathToAssemblyResourcePath(string path) => - path.Replace("/", ".").Replace("\\", ".").Replace(" ", "_"); - - public static IEnumerable MapTests => - [ - ["Serialization/TestData/Map/default_map/default-map", (string f) => DefaultMap(), Array.Empty()], - ["Serialization/TestData/Map/map_with_common_props/map-with-common-props", (string f) => MapWithCommonProps(), Array.Empty()], - ["Serialization/TestData/Map/map_with_custom_type_props/map-with-custom-type-props", (string f) => MapWithCustomTypeProps(), MapWithCustomTypePropsCustomTypeDefinitions()], - ["Serialization/TestData/Map/map_with_embedded_tileset/map-with-embedded-tileset", (string f) => MapWithEmbeddedTileset(), Array.Empty()], - ["Serialization/TestData/Map/map_with_external_tileset/map-with-external-tileset", (string f) => MapWithExternalTileset(f), Array.Empty()], - ["Serialization/TestData/Map/map_with_flippingflags/map-with-flippingflags", (string f) => MapWithFlippingFlags(f), Array.Empty()], - ["Serialization/TestData/Map/map_external_tileset_multi/map-external-tileset-multi", (string f) => MapExternalTilesetMulti(f), Array.Empty()], - ["Serialization/TestData/Map/map_external_tileset_wangset/map-external-tileset-wangset", (string f) => MapExternalTilesetWangset(f), Array.Empty()], - ["Serialization/TestData/Map/map_with_many_layers/map-with-many-layers", (string f) => MapWithManyLayers(f), Array.Empty()], - ["Serialization/TestData/Map/map_with_deep_props/map-with-deep-props", (string f) => MapWithDeepProps(), MapWithDeepPropsCustomTypeDefinitions()], - ]; -} diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/default-map/default-map.cs b/src/DotTiled.Tests/TestData/Maps/default-map/default-map.cs similarity index 87% rename from src/DotTiled.Tests/Serialization/TestData/Map/default-map/default-map.cs rename to src/DotTiled.Tests/TestData/Maps/default-map/default-map.cs index eff73d9..8aeae0d 100644 --- a/src/DotTiled.Tests/Serialization/TestData/Map/default-map/default-map.cs +++ b/src/DotTiled.Tests/TestData/Maps/default-map/default-map.cs @@ -11,9 +11,6 @@ public partial class TestData TileWidth = 32, TileHeight = 32, Infinite = false, - HexSideLength = null, - StaggerAxis = null, - StaggerIndex = null, ParallaxOriginX = 0, ParallaxOriginY = 0, RenderOrder = RenderOrder.RightDown, @@ -33,22 +30,20 @@ public partial class TestData Data = new Data { Encoding = DataEncoding.Csv, - Chunks = null, - Compression = null, - GlobalTileIDs = [ + GlobalTileIDs = new Optional([ 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0 - ], - FlippingFlags = [ + ]), + FlippingFlags = new Optional([ FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None - ] + ]) } } ] diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/default-map/default-map.tmj b/src/DotTiled.Tests/TestData/Maps/default-map/default-map.tmj similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/default-map/default-map.tmj rename to src/DotTiled.Tests/TestData/Maps/default-map/default-map.tmj diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/default-map/default-map.tmx b/src/DotTiled.Tests/TestData/Maps/default-map/default-map.tmx similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/default-map/default-map.tmx rename to src/DotTiled.Tests/TestData/Maps/default-map/default-map.tmx diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-external-tileset-multi/map-external-tileset-multi.cs b/src/DotTiled.Tests/TestData/Maps/map-external-tileset-multi/map-external-tileset-multi.cs similarity index 93% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-external-tileset-multi/map-external-tileset-multi.cs rename to src/DotTiled.Tests/TestData/Maps/map-external-tileset-multi/map-external-tileset-multi.cs index 28d6272..2500a4d 100644 --- a/src/DotTiled.Tests/Serialization/TestData/Map/map-external-tileset-multi/map-external-tileset-multi.cs +++ b/src/DotTiled.Tests/TestData/Maps/map-external-tileset-multi/map-external-tileset-multi.cs @@ -13,9 +13,6 @@ public partial class TestData TileWidth = 32, TileHeight = 32, Infinite = false, - HexSideLength = null, - StaggerAxis = null, - StaggerIndex = null, ParallaxOriginX = 0, ParallaxOriginY = 0, RenderOrder = RenderOrder.RightDown, @@ -97,22 +94,20 @@ public partial class TestData Data = new Data { Encoding = DataEncoding.Csv, - Chunks = null, - Compression = null, - GlobalTileIDs = [ + GlobalTileIDs = new Optional([ 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2, 0, 0, 0 - ], - FlippingFlags = [ + ]), + FlippingFlags = new Optional([ FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None - ] + ]) } } ] diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-external-tileset-multi/map-external-tileset-multi.tmj b/src/DotTiled.Tests/TestData/Maps/map-external-tileset-multi/map-external-tileset-multi.tmj similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-external-tileset-multi/map-external-tileset-multi.tmj rename to src/DotTiled.Tests/TestData/Maps/map-external-tileset-multi/map-external-tileset-multi.tmj diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-external-tileset-multi/map-external-tileset-multi.tmx b/src/DotTiled.Tests/TestData/Maps/map-external-tileset-multi/map-external-tileset-multi.tmx similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-external-tileset-multi/map-external-tileset-multi.tmx rename to src/DotTiled.Tests/TestData/Maps/map-external-tileset-multi/map-external-tileset-multi.tmx diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-external-tileset-multi/multi-tileset.tsj b/src/DotTiled.Tests/TestData/Maps/map-external-tileset-multi/multi-tileset.tsj similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-external-tileset-multi/multi-tileset.tsj rename to src/DotTiled.Tests/TestData/Maps/map-external-tileset-multi/multi-tileset.tsj diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-external-tileset-multi/multi-tileset.tsx b/src/DotTiled.Tests/TestData/Maps/map-external-tileset-multi/multi-tileset.tsx similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-external-tileset-multi/multi-tileset.tsx rename to src/DotTiled.Tests/TestData/Maps/map-external-tileset-multi/multi-tileset.tsx diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-external-tileset-multi/tileset.png b/src/DotTiled.Tests/TestData/Maps/map-external-tileset-multi/tileset.png similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-external-tileset-multi/tileset.png rename to src/DotTiled.Tests/TestData/Maps/map-external-tileset-multi/tileset.png diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-external-tileset-wangset/map-external-tileset-wangset.cs b/src/DotTiled.Tests/TestData/Maps/map-external-tileset-wangset/map-external-tileset-wangset.cs similarity index 61% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-external-tileset-wangset/map-external-tileset-wangset.cs rename to src/DotTiled.Tests/TestData/Maps/map-external-tileset-wangset/map-external-tileset-wangset.cs index 9aaa7d7..756c59f 100644 --- a/src/DotTiled.Tests/Serialization/TestData/Map/map-external-tileset-wangset/map-external-tileset-wangset.cs +++ b/src/DotTiled.Tests/TestData/Maps/map-external-tileset-wangset/map-external-tileset-wangset.cs @@ -13,9 +13,6 @@ public partial class TestData TileWidth = 24, TileHeight = 24, Infinite = false, - HexSideLength = null, - StaggerAxis = null, - StaggerIndex = null, ParallaxOriginX = 0, ParallaxOriginY = 0, RenderOrder = RenderOrder.RightDown, @@ -56,7 +53,59 @@ public partial class TestData Source = "tileset.png", Width = 256, Height = 96, - } + }, + Wangsets = [ + new Wangset + { + Name = "test-terrain", + Tile = -1, + WangColors = [ + new WangColor + { + Name = "Water", + Color = Color.Parse("#ff0000", CultureInfo.InvariantCulture), + Tile = 0, + Probability = 1 + }, + new WangColor + { + Name = "Grass", + Color = Color.Parse("#00ff00", CultureInfo.InvariantCulture), + Tile = -1, + Probability = 1 + }, + new WangColor + { + Name = "Stone", + Color = Color.Parse("#0000ff", CultureInfo.InvariantCulture), + Tile = 29, + Probability = 1 + } + ], + WangTiles = [ + new WangTile + { + TileID = 0, + WangID = [1,1,0,0,0,1,1,1] + }, + new WangTile + { + TileID = 1, + WangID = [1,1,1,1,0,0,0,1] + }, + new WangTile + { + TileID = 10, + WangID = [0,0,0,1,1,1,1,1] + }, + new WangTile + { + TileID = 11, + WangID = [0,1,1,1,1,1,0,0] + } + ] + } + ] } ], Layers = [ @@ -69,22 +118,20 @@ public partial class TestData Data = new Data { Encoding = DataEncoding.Csv, - Chunks = null, - Compression = null, - GlobalTileIDs = [ + GlobalTileIDs = new Optional([ 2, 2, 12, 11, 0, 1, 12, 1, 11, 0, 2, 1, 0, 1, 0, 12, 11, 12, 2, 0, 0, 0, 0, 0, 0 - ], - FlippingFlags = [ + ]), + FlippingFlags = new Optional([ FlippingFlags.FlippedHorizontally, FlippingFlags.None, FlippingFlags.FlippedHorizontally, FlippingFlags.FlippedHorizontally, FlippingFlags.None, FlippingFlags.FlippedVertically, FlippingFlags.None, FlippingFlags.None, FlippingFlags.FlippedVertically | FlippingFlags.FlippedHorizontally, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.FlippedVertically | FlippingFlags.FlippedHorizontally, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.FlippedHorizontally, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None - ] + ]) } } ] diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-external-tileset-wangset/map-external-tileset-wangset.tmj b/src/DotTiled.Tests/TestData/Maps/map-external-tileset-wangset/map-external-tileset-wangset.tmj similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-external-tileset-wangset/map-external-tileset-wangset.tmj rename to src/DotTiled.Tests/TestData/Maps/map-external-tileset-wangset/map-external-tileset-wangset.tmj diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-external-tileset-wangset/map-external-tileset-wangset.tmx b/src/DotTiled.Tests/TestData/Maps/map-external-tileset-wangset/map-external-tileset-wangset.tmx similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-external-tileset-wangset/map-external-tileset-wangset.tmx rename to src/DotTiled.Tests/TestData/Maps/map-external-tileset-wangset/map-external-tileset-wangset.tmx diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-external-tileset-wangset/tileset.png b/src/DotTiled.Tests/TestData/Maps/map-external-tileset-wangset/tileset.png similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-external-tileset-wangset/tileset.png rename to src/DotTiled.Tests/TestData/Maps/map-external-tileset-wangset/tileset.png diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-external-tileset-wangset/wangset-tileset.tsj b/src/DotTiled.Tests/TestData/Maps/map-external-tileset-wangset/wangset-tileset.tsj similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-external-tileset-wangset/wangset-tileset.tsj rename to src/DotTiled.Tests/TestData/Maps/map-external-tileset-wangset/wangset-tileset.tsj diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-external-tileset-wangset/wangset-tileset.tsx b/src/DotTiled.Tests/TestData/Maps/map-external-tileset-wangset/wangset-tileset.tsx similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-external-tileset-wangset/wangset-tileset.tsx rename to src/DotTiled.Tests/TestData/Maps/map-external-tileset-wangset/wangset-tileset.tsx diff --git a/src/DotTiled.Tests/TestData/Maps/map-with-class-and-props/map-with-class-and-props.cs b/src/DotTiled.Tests/TestData/Maps/map-with-class-and-props/map-with-class-and-props.cs new file mode 100644 index 0000000..8cbd6bd --- /dev/null +++ b/src/DotTiled.Tests/TestData/Maps/map-with-class-and-props/map-with-class-and-props.cs @@ -0,0 +1,83 @@ +namespace DotTiled.Tests; + +public partial class TestData +{ + public static Map MapWithClassAndProps() => new Map + { + Class = "TestClass", + Orientation = MapOrientation.Orthogonal, + Width = 5, + Height = 5, + TileWidth = 32, + TileHeight = 32, + Infinite = false, + ParallaxOriginX = 0, + ParallaxOriginY = 0, + RenderOrder = RenderOrder.RightDown, + CompressionLevel = -1, + BackgroundColor = new Color { R = 0, G = 0, B = 0, A = 0 }, + Version = "1.10", + TiledVersion = "1.11.0", + NextLayerID = 2, + NextObjectID = 1, + Layers = [ + new TileLayer + { + ID = 1, + Name = "Tile Layer 1", + Width = 5, + Height = 5, + Data = new Data + { + Encoding = DataEncoding.Csv, + GlobalTileIDs = new Optional([ + 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0 + ]), + FlippingFlags = new Optional([ + FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, + FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, + FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, + FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, + FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None + ]) + } + } + ], + Properties = [ + new BoolProperty + { + Name = "classbool", + Value = true + }, + new StringProperty + { + Name = "classstring", + Value = "I am not default value" + } + ] + }; + + public static IReadOnlyCollection MapWithClassAndPropsCustomTypeDefinitions() => [ + new CustomClassDefinition + { + Name = "TestClass", + UseAs = CustomClassUseAs.Map, + Members = [ + new BoolProperty + { + Name = "classbool", + Value = true + }, + new StringProperty + { + Name = "classstring", + Value = "Hello there default value" + } + ] + }, + ]; +} diff --git a/src/DotTiled.Tests/TestData/Maps/map-with-class-and-props/map-with-class-and-props.tmj b/src/DotTiled.Tests/TestData/Maps/map-with-class-and-props/map-with-class-and-props.tmj new file mode 100644 index 0000000..d60afe3 --- /dev/null +++ b/src/DotTiled.Tests/TestData/Maps/map-with-class-and-props/map-with-class-and-props.tmj @@ -0,0 +1,39 @@ +{ "class":"TestClass", + "compressionlevel":-1, + "height":5, + "infinite":false, + "layers":[ + { + "data":[0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0], + "height":5, + "id":1, + "name":"Tile Layer 1", + "opacity":1, + "type":"tilelayer", + "visible":true, + "width":5, + "x":0, + "y":0 + }], + "nextlayerid":2, + "nextobjectid":1, + "orientation":"orthogonal", + "properties":[ + { + "name":"classstring", + "type":"string", + "value":"I am not default value" + }], + "renderorder":"right-down", + "tiledversion":"1.11.0", + "tileheight":32, + "tilesets":[], + "tilewidth":32, + "type":"map", + "version":"1.10", + "width":5 +} \ No newline at end of file diff --git a/src/DotTiled.Tests/TestData/Maps/map-with-class-and-props/map-with-class-and-props.tmx b/src/DotTiled.Tests/TestData/Maps/map-with-class-and-props/map-with-class-and-props.tmx new file mode 100644 index 0000000..4da805d --- /dev/null +++ b/src/DotTiled.Tests/TestData/Maps/map-with-class-and-props/map-with-class-and-props.tmx @@ -0,0 +1,15 @@ + + + + + + + +0,0,0,0,0, +0,0,0,0,0, +0,0,0,0,0, +0,0,0,0,0, +0,0,0,0,0 + + + diff --git a/src/DotTiled.Tests/TestData/Maps/map-with-class/map-with-class.cs b/src/DotTiled.Tests/TestData/Maps/map-with-class/map-with-class.cs new file mode 100644 index 0000000..ef98d02 --- /dev/null +++ b/src/DotTiled.Tests/TestData/Maps/map-with-class/map-with-class.cs @@ -0,0 +1,83 @@ +namespace DotTiled.Tests; + +public partial class TestData +{ + public static Map MapWithClass() => new Map + { + Class = "TestClass", + Orientation = MapOrientation.Orthogonal, + Width = 5, + Height = 5, + TileWidth = 32, + TileHeight = 32, + Infinite = false, + ParallaxOriginX = 0, + ParallaxOriginY = 0, + RenderOrder = RenderOrder.RightDown, + CompressionLevel = -1, + BackgroundColor = new Color { R = 0, G = 0, B = 0, A = 0 }, + Version = "1.10", + TiledVersion = "1.11.0", + NextLayerID = 2, + NextObjectID = 1, + Layers = [ + new TileLayer + { + ID = 1, + Name = "Tile Layer 1", + Width = 5, + Height = 5, + Data = new Data + { + Encoding = DataEncoding.Csv, + GlobalTileIDs = new Optional([ + 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0 + ]), + FlippingFlags = new Optional([ + FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, + FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, + FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, + FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, + FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None + ]) + } + } + ], + Properties = [ + new BoolProperty + { + Name = "classbool", + Value = true + }, + new StringProperty + { + Name = "classstring", + Value = "Hello there default value" + } + ] + }; + + public static IReadOnlyCollection MapWithClassCustomTypeDefinitions() => [ + new CustomClassDefinition + { + Name = "TestClass", + UseAs = CustomClassUseAs.Map, + Members = [ + new BoolProperty + { + Name = "classbool", + Value = true + }, + new StringProperty + { + Name = "classstring", + Value = "Hello there default value" + } + ] + }, + ]; +} diff --git a/src/DotTiled.Tests/TestData/Maps/map-with-class/map-with-class.tmj b/src/DotTiled.Tests/TestData/Maps/map-with-class/map-with-class.tmj new file mode 100644 index 0000000..480e3bb --- /dev/null +++ b/src/DotTiled.Tests/TestData/Maps/map-with-class/map-with-class.tmj @@ -0,0 +1,33 @@ +{ "class":"TestClass", + "compressionlevel":-1, + "height":5, + "infinite":false, + "layers":[ + { + "data":[0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0], + "height":5, + "id":1, + "name":"Tile Layer 1", + "opacity":1, + "type":"tilelayer", + "visible":true, + "width":5, + "x":0, + "y":0 + }], + "nextlayerid":2, + "nextobjectid":1, + "orientation":"orthogonal", + "renderorder":"right-down", + "tiledversion":"1.11.0", + "tileheight":32, + "tilesets":[], + "tilewidth":32, + "type":"map", + "version":"1.10", + "width":5 +} \ No newline at end of file diff --git a/src/DotTiled.Tests/TestData/Maps/map-with-class/map-with-class.tmx b/src/DotTiled.Tests/TestData/Maps/map-with-class/map-with-class.tmx new file mode 100644 index 0000000..b3382eb --- /dev/null +++ b/src/DotTiled.Tests/TestData/Maps/map-with-class/map-with-class.tmx @@ -0,0 +1,12 @@ + + + + +0,0,0,0,0, +0,0,0,0,0, +0,0,0,0,0, +0,0,0,0,0, +0,0,0,0,0 + + + diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-common-props/map-with-common-props.cs b/src/DotTiled.Tests/TestData/Maps/map-with-common-props/map-with-common-props.cs similarity index 90% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-with-common-props/map-with-common-props.cs rename to src/DotTiled.Tests/TestData/Maps/map-with-common-props/map-with-common-props.cs index fdedbf8..b2dbc7a 100644 --- a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-common-props/map-with-common-props.cs +++ b/src/DotTiled.Tests/TestData/Maps/map-with-common-props/map-with-common-props.cs @@ -13,9 +13,6 @@ public partial class TestData TileWidth = 32, TileHeight = 16, Infinite = false, - HexSideLength = null, - StaggerAxis = null, - StaggerIndex = null, ParallaxOriginX = 0, ParallaxOriginY = 0, RenderOrder = RenderOrder.RightDown, @@ -35,22 +32,20 @@ public partial class TestData Data = new Data { Encoding = DataEncoding.Csv, - Chunks = null, - Compression = null, - GlobalTileIDs = [ + GlobalTileIDs = new Optional([ 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0 - ], - FlippingFlags = [ + ]), + FlippingFlags = new Optional([ FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None - ] + ]) } } ], diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-common-props/map-with-common-props.tmj b/src/DotTiled.Tests/TestData/Maps/map-with-common-props/map-with-common-props.tmj similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-with-common-props/map-with-common-props.tmj rename to src/DotTiled.Tests/TestData/Maps/map-with-common-props/map-with-common-props.tmj diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-common-props/map-with-common-props.tmx b/src/DotTiled.Tests/TestData/Maps/map-with-common-props/map-with-common-props.tmx similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-with-common-props/map-with-common-props.tmx rename to src/DotTiled.Tests/TestData/Maps/map-with-common-props/map-with-common-props.tmx diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-custom-type-props/map-with-custom-type-props.cs b/src/DotTiled.Tests/TestData/Maps/map-with-custom-type-props/map-with-custom-type-props.cs similarity index 96% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-with-custom-type-props/map-with-custom-type-props.cs rename to src/DotTiled.Tests/TestData/Maps/map-with-custom-type-props/map-with-custom-type-props.cs index 56759d4..f77638b 100644 --- a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-custom-type-props/map-with-custom-type-props.cs +++ b/src/DotTiled.Tests/TestData/Maps/map-with-custom-type-props/map-with-custom-type-props.cs @@ -13,9 +13,6 @@ public partial class TestData TileWidth = 32, TileHeight = 32, Infinite = false, - HexSideLength = null, - StaggerAxis = null, - StaggerIndex = null, ParallaxOriginX = 0, ParallaxOriginY = 0, RenderOrder = RenderOrder.RightDown, @@ -35,22 +32,20 @@ public partial class TestData Data = new Data { Encoding = DataEncoding.Csv, - Chunks = null, - Compression = null, - GlobalTileIDs = [ + GlobalTileIDs = new Optional([ 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0 - ], - FlippingFlags = [ + ]), + FlippingFlags = new Optional([ FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None - ] + ]) } } ], diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-custom-type-props/map-with-custom-type-props.tmj b/src/DotTiled.Tests/TestData/Maps/map-with-custom-type-props/map-with-custom-type-props.tmj similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-with-custom-type-props/map-with-custom-type-props.tmj rename to src/DotTiled.Tests/TestData/Maps/map-with-custom-type-props/map-with-custom-type-props.tmj diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-custom-type-props/map-with-custom-type-props.tmx b/src/DotTiled.Tests/TestData/Maps/map-with-custom-type-props/map-with-custom-type-props.tmx similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-with-custom-type-props/map-with-custom-type-props.tmx rename to src/DotTiled.Tests/TestData/Maps/map-with-custom-type-props/map-with-custom-type-props.tmx diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-custom-type-props/propertytypes.json b/src/DotTiled.Tests/TestData/Maps/map-with-custom-type-props/propertytypes.json similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-with-custom-type-props/propertytypes.json rename to src/DotTiled.Tests/TestData/Maps/map-with-custom-type-props/propertytypes.json diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-deep-props/map-with-deep-props.cs b/src/DotTiled.Tests/TestData/Maps/map-with-deep-props/map-with-deep-props.cs similarity index 95% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-with-deep-props/map-with-deep-props.cs rename to src/DotTiled.Tests/TestData/Maps/map-with-deep-props/map-with-deep-props.cs index 90aac6c..b67548a 100644 --- a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-deep-props/map-with-deep-props.cs +++ b/src/DotTiled.Tests/TestData/Maps/map-with-deep-props/map-with-deep-props.cs @@ -13,9 +13,6 @@ public partial class TestData TileWidth = 32, TileHeight = 32, Infinite = false, - HexSideLength = null, - StaggerAxis = null, - StaggerIndex = null, ParallaxOriginX = 0, ParallaxOriginY = 0, RenderOrder = RenderOrder.RightDown, @@ -35,22 +32,20 @@ public partial class TestData Data = new Data { Encoding = DataEncoding.Csv, - Chunks = null, - Compression = null, - GlobalTileIDs = [ + GlobalTileIDs = new Optional([ 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0 - ], - FlippingFlags = [ + ]), + FlippingFlags = new Optional([ FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None - ] + ]) } } ], diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-deep-props/map-with-deep-props.tmj b/src/DotTiled.Tests/TestData/Maps/map-with-deep-props/map-with-deep-props.tmj similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-with-deep-props/map-with-deep-props.tmj rename to src/DotTiled.Tests/TestData/Maps/map-with-deep-props/map-with-deep-props.tmj diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-deep-props/map-with-deep-props.tmx b/src/DotTiled.Tests/TestData/Maps/map-with-deep-props/map-with-deep-props.tmx similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-with-deep-props/map-with-deep-props.tmx rename to src/DotTiled.Tests/TestData/Maps/map-with-deep-props/map-with-deep-props.tmx diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-embedded-tileset/map-with-embedded-tileset.cs b/src/DotTiled.Tests/TestData/Maps/map-with-embedded-tileset/map-with-embedded-tileset.cs similarity index 89% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-with-embedded-tileset/map-with-embedded-tileset.cs rename to src/DotTiled.Tests/TestData/Maps/map-with-embedded-tileset/map-with-embedded-tileset.cs index fb3c95f..0056fd1 100644 --- a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-embedded-tileset/map-with-embedded-tileset.cs +++ b/src/DotTiled.Tests/TestData/Maps/map-with-embedded-tileset/map-with-embedded-tileset.cs @@ -13,9 +13,6 @@ public partial class TestData TileWidth = 32, TileHeight = 32, Infinite = false, - HexSideLength = null, - StaggerAxis = null, - StaggerIndex = null, ParallaxOriginX = 0, ParallaxOriginY = 0, RenderOrder = RenderOrder.RightDown, @@ -28,6 +25,8 @@ public partial class TestData Tilesets = [ new Tileset { + Version = "1.10", + TiledVersion = "1.11.0", FirstGID = 1, Name = "tileset", TileWidth = 32, @@ -53,22 +52,20 @@ public partial class TestData Data = new Data { Encoding = DataEncoding.Csv, - Chunks = null, - Compression = null, - GlobalTileIDs = [ + GlobalTileIDs = new Optional([ 1, 1, 0, 0, 7, 1, 1, 0, 0, 7, 0, 0, 0, 0, 7, 9, 10, 0, 0, 7, 17, 18, 0, 0, 0 - ], - FlippingFlags = [ + ]), + FlippingFlags = new Optional([ FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None - ] + ]) } } ] diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-embedded-tileset/map-with-embedded-tileset.tmj b/src/DotTiled.Tests/TestData/Maps/map-with-embedded-tileset/map-with-embedded-tileset.tmj similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-with-embedded-tileset/map-with-embedded-tileset.tmj rename to src/DotTiled.Tests/TestData/Maps/map-with-embedded-tileset/map-with-embedded-tileset.tmj diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-embedded-tileset/map-with-embedded-tileset.tmx b/src/DotTiled.Tests/TestData/Maps/map-with-embedded-tileset/map-with-embedded-tileset.tmx similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-with-embedded-tileset/map-with-embedded-tileset.tmx rename to src/DotTiled.Tests/TestData/Maps/map-with-embedded-tileset/map-with-embedded-tileset.tmx diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-embedded-tileset/tileset.png b/src/DotTiled.Tests/TestData/Maps/map-with-embedded-tileset/tileset.png similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-with-embedded-tileset/tileset.png rename to src/DotTiled.Tests/TestData/Maps/map-with-embedded-tileset/tileset.png diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-external-tileset/map-with-external-tileset.cs b/src/DotTiled.Tests/TestData/Maps/map-with-external-tileset/map-with-external-tileset.cs similarity index 90% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-with-external-tileset/map-with-external-tileset.cs rename to src/DotTiled.Tests/TestData/Maps/map-with-external-tileset/map-with-external-tileset.cs index 10c4d67..15b9873 100644 --- a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-external-tileset/map-with-external-tileset.cs +++ b/src/DotTiled.Tests/TestData/Maps/map-with-external-tileset/map-with-external-tileset.cs @@ -13,9 +13,6 @@ public partial class TestData TileWidth = 32, TileHeight = 32, Infinite = false, - HexSideLength = null, - StaggerAxis = null, - StaggerIndex = null, ParallaxOriginX = 0, ParallaxOriginY = 0, RenderOrder = RenderOrder.RightDown, @@ -56,22 +53,20 @@ public partial class TestData Data = new Data { Encoding = DataEncoding.Csv, - Chunks = null, - Compression = null, - GlobalTileIDs = [ + GlobalTileIDs = new Optional([ 1, 1, 0, 0, 7, 1, 1, 0, 0, 7, 0, 0, 1, 0, 7, 0, 0, 0, 1, 7, 21, 21, 21, 21, 1 - ], - FlippingFlags = [ + ]), + FlippingFlags = new Optional([ FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None - ] + ]) } } ] diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-external-tileset/map-with-external-tileset.tmj b/src/DotTiled.Tests/TestData/Maps/map-with-external-tileset/map-with-external-tileset.tmj similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-with-external-tileset/map-with-external-tileset.tmj rename to src/DotTiled.Tests/TestData/Maps/map-with-external-tileset/map-with-external-tileset.tmj diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-external-tileset/map-with-external-tileset.tmx b/src/DotTiled.Tests/TestData/Maps/map-with-external-tileset/map-with-external-tileset.tmx similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-with-external-tileset/map-with-external-tileset.tmx rename to src/DotTiled.Tests/TestData/Maps/map-with-external-tileset/map-with-external-tileset.tmx diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-external-tileset/tileset.png b/src/DotTiled.Tests/TestData/Maps/map-with-external-tileset/tileset.png similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-with-external-tileset/tileset.png rename to src/DotTiled.Tests/TestData/Maps/map-with-external-tileset/tileset.png diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-external-tileset/tileset.tsj b/src/DotTiled.Tests/TestData/Maps/map-with-external-tileset/tileset.tsj similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-with-external-tileset/tileset.tsj rename to src/DotTiled.Tests/TestData/Maps/map-with-external-tileset/tileset.tsj diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-external-tileset/tileset.tsx b/src/DotTiled.Tests/TestData/Maps/map-with-external-tileset/tileset.tsx similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-with-external-tileset/tileset.tsx rename to src/DotTiled.Tests/TestData/Maps/map-with-external-tileset/tileset.tsx diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-flippingflags/map-with-flippingflags.cs b/src/DotTiled.Tests/TestData/Maps/map-with-flippingflags/map-with-flippingflags.cs similarity index 91% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-with-flippingflags/map-with-flippingflags.cs rename to src/DotTiled.Tests/TestData/Maps/map-with-flippingflags/map-with-flippingflags.cs index 4e181c4..0777e3f 100644 --- a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-flippingflags/map-with-flippingflags.cs +++ b/src/DotTiled.Tests/TestData/Maps/map-with-flippingflags/map-with-flippingflags.cs @@ -13,9 +13,6 @@ public partial class TestData TileWidth = 32, TileHeight = 32, Infinite = false, - HexSideLength = null, - StaggerAxis = null, - StaggerIndex = null, ParallaxOriginX = 0, ParallaxOriginY = 0, RenderOrder = RenderOrder.RightDown, @@ -56,22 +53,20 @@ public partial class TestData Data = new Data { Encoding = DataEncoding.Csv, - Chunks = null, - Compression = null, - GlobalTileIDs = [ + GlobalTileIDs = new Optional([ 1, 1, 0, 0, 7, 1, 1, 0, 0, 7, 0, 0, 1, 0, 7, 0, 0, 0, 1, 7, 21, 21, 21, 21, 1 - ], - FlippingFlags = [ + ]), + FlippingFlags = new Optional([ FlippingFlags.None, FlippingFlags.FlippedDiagonally | FlippingFlags.FlippedHorizontally, FlippingFlags.None, FlippingFlags.None, FlippingFlags.FlippedVertically, FlippingFlags.FlippedDiagonally | FlippingFlags.FlippedVertically, FlippingFlags.FlippedVertically | FlippingFlags.FlippedHorizontally, FlippingFlags.None, FlippingFlags.None, FlippingFlags.FlippedVertically, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.FlippedVertically, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.FlippedVertically, FlippingFlags.FlippedHorizontally, FlippingFlags.FlippedHorizontally, FlippingFlags.FlippedHorizontally, FlippingFlags.FlippedHorizontally, FlippingFlags.None - ] + ]) } } ] diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-flippingflags/map-with-flippingflags.tmj b/src/DotTiled.Tests/TestData/Maps/map-with-flippingflags/map-with-flippingflags.tmj similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-with-flippingflags/map-with-flippingflags.tmj rename to src/DotTiled.Tests/TestData/Maps/map-with-flippingflags/map-with-flippingflags.tmj diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-flippingflags/map-with-flippingflags.tmx b/src/DotTiled.Tests/TestData/Maps/map-with-flippingflags/map-with-flippingflags.tmx similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-with-flippingflags/map-with-flippingflags.tmx rename to src/DotTiled.Tests/TestData/Maps/map-with-flippingflags/map-with-flippingflags.tmx diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-flippingflags/tileset.png b/src/DotTiled.Tests/TestData/Maps/map-with-flippingflags/tileset.png similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-with-flippingflags/tileset.png rename to src/DotTiled.Tests/TestData/Maps/map-with-flippingflags/tileset.png diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-flippingflags/tileset.tsj b/src/DotTiled.Tests/TestData/Maps/map-with-flippingflags/tileset.tsj similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-with-flippingflags/tileset.tsj rename to src/DotTiled.Tests/TestData/Maps/map-with-flippingflags/tileset.tsj diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-flippingflags/tileset.tsx b/src/DotTiled.Tests/TestData/Maps/map-with-flippingflags/tileset.tsx similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-with-flippingflags/tileset.tsx rename to src/DotTiled.Tests/TestData/Maps/map-with-flippingflags/tileset.tsx diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-many-layers/map-with-many-layers.cs b/src/DotTiled.Tests/TestData/Maps/map-with-many-layers/map-with-many-layers.cs similarity index 91% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-with-many-layers/map-with-many-layers.cs rename to src/DotTiled.Tests/TestData/Maps/map-with-many-layers/map-with-many-layers.cs index 64974bc..1b06285 100644 --- a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-many-layers/map-with-many-layers.cs +++ b/src/DotTiled.Tests/TestData/Maps/map-with-many-layers/map-with-many-layers.cs @@ -13,9 +13,6 @@ public partial class TestData TileWidth = 32, TileHeight = 32, Infinite = false, - HexSideLength = null, - StaggerAxis = null, - StaggerIndex = null, ParallaxOriginX = 0, ParallaxOriginY = 0, RenderOrder = RenderOrder.RightDown, @@ -124,22 +121,20 @@ public partial class TestData Data = new Data { Encoding = DataEncoding.Csv, - Chunks = null, - Compression = null, - GlobalTileIDs = [ + GlobalTileIDs = new Optional([ 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0 - ], - FlippingFlags = [ + ]), + FlippingFlags = new Optional([ FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None - ] + ]) } }, new TileLayer @@ -151,22 +146,20 @@ public partial class TestData Data = new Data { Encoding = DataEncoding.Csv, - Chunks = null, - Compression = null, - GlobalTileIDs = [ + GlobalTileIDs = new Optional([ 0, 15, 15, 0, 0, 0, 15, 15, 0, 0, 0, 15, 15, 15, 0, 15, 15, 15, 0, 0, 0, 0, 0, 0, 0 - ], - FlippingFlags = [ + ]), + FlippingFlags = new Optional([ FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None - ] + ]) } } ] @@ -193,22 +186,20 @@ public partial class TestData Data = new Data { Encoding = DataEncoding.Csv, - Chunks = null, - Compression = null, - GlobalTileIDs = [ + GlobalTileIDs = new Optional([ 1, 1, 1, 1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0 - ], - FlippingFlags = [ + ]), + FlippingFlags = new Optional([ FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None, FlippingFlags.None - ] + ]) } } ] diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-many-layers/map-with-many-layers.tmj b/src/DotTiled.Tests/TestData/Maps/map-with-many-layers/map-with-many-layers.tmj similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-with-many-layers/map-with-many-layers.tmj rename to src/DotTiled.Tests/TestData/Maps/map-with-many-layers/map-with-many-layers.tmj diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-many-layers/map-with-many-layers.tmx b/src/DotTiled.Tests/TestData/Maps/map-with-many-layers/map-with-many-layers.tmx similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-with-many-layers/map-with-many-layers.tmx rename to src/DotTiled.Tests/TestData/Maps/map-with-many-layers/map-with-many-layers.tmx diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-many-layers/poly.tj b/src/DotTiled.Tests/TestData/Maps/map-with-many-layers/poly.tj similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-with-many-layers/poly.tj rename to src/DotTiled.Tests/TestData/Maps/map-with-many-layers/poly.tj diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-many-layers/poly.tx b/src/DotTiled.Tests/TestData/Maps/map-with-many-layers/poly.tx similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-with-many-layers/poly.tx rename to src/DotTiled.Tests/TestData/Maps/map-with-many-layers/poly.tx diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-many-layers/tileset.png b/src/DotTiled.Tests/TestData/Maps/map-with-many-layers/tileset.png similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-with-many-layers/tileset.png rename to src/DotTiled.Tests/TestData/Maps/map-with-many-layers/tileset.png diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-many-layers/tileset.tsj b/src/DotTiled.Tests/TestData/Maps/map-with-many-layers/tileset.tsj similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-with-many-layers/tileset.tsj rename to src/DotTiled.Tests/TestData/Maps/map-with-many-layers/tileset.tsj diff --git a/src/DotTiled.Tests/Serialization/TestData/Map/map-with-many-layers/tileset.tsx b/src/DotTiled.Tests/TestData/Maps/map-with-many-layers/tileset.tsx similarity index 100% rename from src/DotTiled.Tests/Serialization/TestData/Map/map-with-many-layers/tileset.tsx rename to src/DotTiled.Tests/TestData/Maps/map-with-many-layers/tileset.tsx diff --git a/src/DotTiled.Tests/UnitTests/OptionalTests.cs b/src/DotTiled.Tests/UnitTests/OptionalTests.cs new file mode 100644 index 0000000..5390a64 --- /dev/null +++ b/src/DotTiled.Tests/UnitTests/OptionalTests.cs @@ -0,0 +1,317 @@ +namespace DotTiled.Tests; + +public class OptionalTests +{ + // Constructor Tests + + [Fact] + public void Constructor_WithNonNullValue_ShouldSetHasValueToTrue() + { + // Arrange + string expectedValue = "test"; + + // Act + var optional = new Optional(expectedValue); + + // Assert + Assert.True(optional.HasValue); + Assert.Equal(expectedValue, optional.Value); + } + + [Fact] + public void Constructor_WithNullValue_ShouldSetHasValueToTrue() + { + // Arrange + string expectedValue = null; + + // Act + var optional = new Optional(expectedValue); + + // Assert + Assert.True(optional.HasValue); + Assert.Null(optional.Value); + } + + [Fact] + public void DefaultConstructor_ShouldSetHasValueToFalse() + { + // Arrange & Act + var optional = new Optional(); + + // Assert + Assert.False(optional.HasValue); + _ = Assert.Throws(() => optional.Value); + } + + // Implicit Conversion Tests + + [Fact] + public void ImplicitConversion_FromValueToOptional_ShouldSetHasValueToTrue() + { + // Arrange + int expectedValue = 5; + + // Act + Optional optional = expectedValue; + + // Assert + Assert.True(optional.HasValue); + Assert.Equal(expectedValue, optional.Value); + } + + [Fact] + public void ImplicitConversion_FromOptionalToValue_ShouldReturnValue_WhenHasValueIsTrue() + { + // Arrange + int expectedValue = 10; + var optional = new Optional(expectedValue); + + // Act + int value = optional; + + // Assert + Assert.Equal(expectedValue, value); + } + + [Fact] + public void ImplicitConversion_FromOptionalToValue_ShouldThrowException_WhenHasValueIsFalse() + { + // Arrange + var optional = new Optional(); + + // Act & Assert + _ = Assert.Throws(() => { int value = optional; }); + } + + // ToString Method Tests + + [Fact] + public void ToString_WithValue_ShouldReturnValueToString() + { + // Arrange + int expectedValue = 42; + var optional = new Optional(expectedValue); + + // Act + string result = optional.ToString(); + + // Assert +#pragma warning disable CA1305 // Specify IFormatProvider + Assert.Equal(expectedValue.ToString(), result); +#pragma warning restore CA1305 // Specify IFormatProvider + } + + [Fact] + public void ToString_WithoutValue_ShouldReturnEmpty() + { + // Arrange & Act + var optional = new Optional(); + string result = optional.ToString(); + + // Assert + Assert.Equal("Empty", result); + } + + // Equality Tests + + [Fact] + public void Equals_WithSameValue_ShouldReturnTrue() + { + // Arrange + var optional1 = new Optional(10); + var optional2 = new Optional(10); + + // Act + bool areEqual = optional1.Equals(optional2); + + // Assert + Assert.True(areEqual); + } + + [Fact] + public void Equals_WithDifferentValues_ShouldReturnFalse() + { + // Arrange + var optional1 = new Optional(10); + var optional2 = new Optional(20); + + // Act + bool areEqual = optional1.Equals(optional2); + + // Assert + Assert.False(areEqual); + } + + [Fact] + public void Equals_WithNull_ShouldReturnFalse() + { + // Arrange + var optional = new Optional("test"); + + // Act + bool areEqual = optional.Equals(null); + + // Assert + Assert.False(areEqual); + } + + [Fact] + public void Equals_WithEmptyOptional_ShouldReturnTrue() + { + // Arrange + var optional1 = new Optional(); + var optional2 = new Optional(); + + // Act + bool areEqual = optional1.Equals(optional2); + + // Assert + Assert.True(areEqual); + } + + [Fact] + public void Equals_WithSameReferenceTypeValue_ShouldReturnTrue() + { + // Arrange + var value = new object(); + var optional1 = new Optional(value); + var optional2 = new Optional(value); + + // Act + bool areEqual = optional1.Equals(optional2); + + // Assert + Assert.True(areEqual); + } + + [Fact] + public void Equals_WithDifferentReferenceTypeValues_ShouldReturnFalse() + { + // Arrange + var optional1 = new Optional(new object()); + var optional2 = new Optional(new object()); + + // Act + bool areEqual = optional1.Equals(optional2); + + // Assert + Assert.False(areEqual); + } + + // GetHashCode Method Tests + + [Fact] + public void GetHashCode_WithValue_ShouldReturnValueHashCode() + { + // Arrange + int value = 42; + var optional = new Optional(value); + + // Act + int hashCode = optional.GetHashCode(); + + // Assert + Assert.Equal(value.GetHashCode(), hashCode); + } + + [Fact] + public void GetHashCode_WithoutValue_ShouldReturnZero() + { + // Arrange & Act + var optional = new Optional(); + int hashCode = optional.GetHashCode(); + + // Assert + Assert.Equal(0, hashCode); + } + + // Exception Tests + + [Fact] + public void Value_WhenHasValueIsFalse_ShouldThrowInvalidOperationException() + { + // Arrange + var optional = new Optional(); + + // Act & Assert + _ = Assert.Throws(() => optional.Value); + } + + [Fact] + public void ImplicitConversion_WhenHasValueIsFalse_ShouldThrowInvalidOperationException() + { + // Arrange + var optional = new Optional(); + + // Act & Assert + _ = Assert.Throws(() => { int value = optional; }); + } + + // Edge Cases + + [Fact] + public void EmptyOptionalEquality_ShouldReturnTrue() + { + // Arrange + var optional1 = new Optional(); + var optional2 = new Optional(); + + // Act + bool areEqual = optional1.Equals(optional2); + + // Assert + Assert.True(areEqual); + } + + // Special Scenarios + + public struct CustomStruct + { + public int X { get; set; } + } + + [Fact] + public void OptionalWithCustomStruct_ShouldBehaveCorrectly() + { + // Arrange + var structValue = new CustomStruct { X = 10 }; + var optional = new Optional(structValue); + + // Act + CustomStruct value = optional.Value; + + // Assert + Assert.True(optional.HasValue); + Assert.Equal(structValue, value); + } + + [Fact] + public void OptionalWithNullableInt_ShouldBehaveCorrectly() + { + // Arrange + int? nullableValue = 5; + var optional = new Optional(nullableValue); + + // Act + int? value = optional.Value; + + // Assert + Assert.True(optional.HasValue); + Assert.Equal(nullableValue, value); + } + + [Fact] + public void OptionalWithNullNullableInt_ShouldBehaveCorrectly() + { + // Arrange + int? nullableValue = null; + var optional = new Optional(nullableValue); + + // Act + int? value = optional.Value; + + // Assert + Assert.True(optional.HasValue); + Assert.Null(value); + } +} diff --git a/src/DotTiled.Tests/UnitTests/Properties/CustomTypes/CustomClassDefinitionTests.cs b/src/DotTiled.Tests/UnitTests/Properties/CustomTypes/CustomClassDefinitionTests.cs new file mode 100644 index 0000000..7e920e3 --- /dev/null +++ b/src/DotTiled.Tests/UnitTests/Properties/CustomTypes/CustomClassDefinitionTests.cs @@ -0,0 +1,122 @@ +namespace DotTiled.Tests; + +public class CustomClassDefinitionTests +{ + private sealed class TestClass1 + { + public string Name { get; set; } = "John Doe"; + public int Age { get; set; } = 42; + } + + private static CustomClassDefinition ExpectedTestClass1Definition => new CustomClassDefinition + { + Name = "TestClass1", + UseAs = CustomClassUseAs.All, + Members = new List + { + new StringProperty { Name = "Name", Value = "John Doe" }, + new IntProperty { Name = "Age", Value = 42 } + } + }; + + private sealed class TestClass2WithNestedClass + { + public string Name { get; set; } = "John Doe"; + public int Age { get; set; } = 42; + public TestClass1 Nested { get; set; } = new TestClass1(); + } + + private static CustomClassDefinition ExpectedTestClass2WithNestedClassDefinition => new CustomClassDefinition + { + Name = "TestClass2WithNestedClass", + UseAs = CustomClassUseAs.All, + Members = [ + new StringProperty { Name = "Name", Value = "John Doe" }, + new IntProperty { Name = "Age", Value = 42 }, + new ClassProperty + { + Name = "Nested", + PropertyType = "TestClass1", + Value = [] + } + ] + }; + + private sealed class TestClass3WithOverridenNestedClass + { + public string Name { get; set; } = "John Doe"; + public int Age { get; set; } = 42; + public TestClass1 Nested { get; set; } = new TestClass1 + { + Name = "Jane Doe" + }; + } + + private static CustomClassDefinition ExpectedTestClass3WithOverridenNestedClassDefinition => new CustomClassDefinition + { + Name = "TestClass3WithOverridenNestedClass", + UseAs = CustomClassUseAs.All, + Members = [ + new StringProperty { Name = "Name", Value = "John Doe" }, + new IntProperty { Name = "Age", Value = 42 }, + new ClassProperty + { + Name = "Nested", + PropertyType = "TestClass1", + Value = [ + new StringProperty { Name = "Name", Value = "Jane Doe" }, + ] + } + ] + }; + + private static IEnumerable<(Type, CustomClassDefinition)> GetCustomClassDefinitionTestData() + { + yield return (typeof(TestClass1), ExpectedTestClass1Definition); + yield return (typeof(TestClass2WithNestedClass), ExpectedTestClass2WithNestedClassDefinition); + yield return (typeof(TestClass3WithOverridenNestedClass), ExpectedTestClass3WithOverridenNestedClassDefinition); + } + + public static IEnumerable CustomClassDefinitionTestData => + GetCustomClassDefinitionTestData().Select(data => new object[] { data.Item1, data.Item2 }); + [Theory] + [MemberData(nameof(CustomClassDefinitionTestData))] + public void FromClass_Type_WhenTypeIsCustomClass_ReturnsCustomClassDefinition(Type type, CustomClassDefinition expected) + { + // Arrange & Act + var result = CustomClassDefinition.FromClass(type); + + // Assert + DotTiledAssert.AssertCustomClassDefinitionEqual(expected, result); + } + + [Fact] + public void FromClass_Type_WhenTypeIsNull_ThrowsArgumentNullException() + { + // Arrange + Type type = null; + + // Act & Assert + Assert.Throws(() => CustomClassDefinition.FromClass(type)); + } + + [Fact] + public void FromClass_Type_WhenTypeIsString_ThrowsArgumentException() + { + // Arrange + Type type = typeof(string); + + // Act & Assert + Assert.Throws(() => CustomClassDefinition.FromClass(type)); + } + + [Fact] + public void FromClass_Type_WhenTypeIsNotClass_ThrowsArgumentException() + { + // Arrange + Type type = typeof(int); + + // Act & Assert + Assert.Throws(() => CustomClassDefinition.FromClass(type)); + } +} diff --git a/src/DotTiled.Tests/UnitTests/Properties/CustomTypes/CustomEnumDefinitionTests.cs b/src/DotTiled.Tests/UnitTests/Properties/CustomTypes/CustomEnumDefinitionTests.cs new file mode 100644 index 0000000..c342f77 --- /dev/null +++ b/src/DotTiled.Tests/UnitTests/Properties/CustomTypes/CustomEnumDefinitionTests.cs @@ -0,0 +1,101 @@ +namespace DotTiled.Tests; + +public class CustomEnumDefinitionTests +{ + [Fact] + public void FromEnum_Type_WhenTypeIsNotEnum_ThrowsArgumentException() + { + // Arrange + var type = typeof(string); + + // Act & Assert + Assert.Throws(() => CustomEnumDefinition.FromEnum(type)); + } + + private enum TestEnum1 { Value1, Value2, Value3 } + + [Fact] + public void FromEnum_Type_WhenTypeIsEnum_ReturnsCustomEnumDefinition() + { + // Arrange + var type = typeof(TestEnum1); + var expected = new CustomEnumDefinition + { + ID = 0, + Name = "TestEnum1", + StorageType = CustomEnumStorageType.Int, + Values = ["Value1", "Value2", "Value3"], + ValueAsFlags = false + }; + + // Act + var result = CustomEnumDefinition.FromEnum(type); + + // Assert + DotTiledAssert.AssertCustomEnumDefinitionEqual(expected, result); + } + + [Flags] + private enum TestEnum2 { Value1, Value2, Value3 } + + [Fact] + public void FromEnum_Type_WhenEnumIsFlags_ReturnsCustomEnumDefinition() + { + // Arrange + var type = typeof(TestEnum2); + var expected = new CustomEnumDefinition + { + ID = 0, + Name = "TestEnum2", + StorageType = CustomEnumStorageType.Int, + Values = ["Value1", "Value2", "Value3"], + ValueAsFlags = true + }; + + // Act + var result = CustomEnumDefinition.FromEnum(type); + + // Assert + DotTiledAssert.AssertCustomEnumDefinitionEqual(expected, result); + } + + [Fact] + public void FromEnum_T_WhenTypeIsEnum_ReturnsCustomEnumDefinition() + { + // Arrange + var expected = new CustomEnumDefinition + { + ID = 0, + Name = "TestEnum1", + StorageType = CustomEnumStorageType.Int, + Values = ["Value1", "Value2", "Value3"], + ValueAsFlags = false + }; + + // Act + var result = CustomEnumDefinition.FromEnum(); + + // Assert + DotTiledAssert.AssertCustomEnumDefinitionEqual(expected, result); + } + + [Fact] + public void FromEnum_T_WhenEnumIsFlags_ReturnsCustomEnumDefinition() + { + // Arrange + var expected = new CustomEnumDefinition + { + ID = 0, + Name = "TestEnum2", + StorageType = CustomEnumStorageType.Int, + Values = ["Value1", "Value2", "Value3"], + ValueAsFlags = true + }; + + // Act + var result = CustomEnumDefinition.FromEnum(); + + // Assert + DotTiledAssert.AssertCustomEnumDefinitionEqual(expected, result); + } +} diff --git a/src/DotTiled.Tests/UnitTests/Properties/HasPropertiesBaseTests.cs b/src/DotTiled.Tests/UnitTests/Properties/HasPropertiesBaseTests.cs new file mode 100644 index 0000000..4f9ab0f --- /dev/null +++ b/src/DotTiled.Tests/UnitTests/Properties/HasPropertiesBaseTests.cs @@ -0,0 +1,300 @@ +using System.Globalization; + +namespace DotTiled.Tests; + +public class HasPropertiesBaseTests +{ + private sealed class TestHasProperties(IList props) : HasPropertiesBase + { + public override IList GetProperties() => props; + } + + [Fact] + public void TryGetProperty_PropertyNotFound_ReturnsFalseAndOutIsNull() + { + // Arrange + var hasProperties = new TestHasProperties([]); + + // Act + var result = hasProperties.TryGetProperty("Test", out var property); + + // Assert + Assert.False(result); + Assert.Null(property); + } + + [Fact] + public void TryGetProperty_PropertyFound_ReturnsTrueAndOutIsProperty() + { + // Arrange + List props = [new BoolProperty { Name = "Test", Value = true }]; + var hasProperties = new TestHasProperties(props); + + // Act + var result = hasProperties.TryGetProperty("Test", out var property); + + // Assert + Assert.True(result); + Assert.NotNull(property); + Assert.Equal("Test", property.Name); + Assert.True(property.Value); + } + + [Fact] + public void GetProperty_PropertyNotFound_ThrowsKeyNotFoundException() + { + // Arrange + var hasProperties = new TestHasProperties([]); + + // Act & Assert + _ = Assert.Throws(() => hasProperties.GetProperty("Test")); + } + + [Fact] + public void GetProperty_PropertyFound_ReturnsProperty() + { + // Arrange + List props = [new BoolProperty { Name = "Test", Value = true }]; + var hasProperties = new TestHasProperties(props); + + // Act + var property = hasProperties.GetProperty("Test"); + + // Assert + Assert.NotNull(property); + Assert.Equal("Test", property.Name); + Assert.True(property.Value); + } + + [Fact] + public void GetProperty_PropertyIsWrongType_ThrowsInvalidCastException() + { + // Arrange + List props = [new BoolProperty { Name = "Test", Value = true }]; + var hasProperties = new TestHasProperties(props); + + // Act & Assert + _ = Assert.Throws(() => hasProperties.GetProperty("Test")); + } + + private sealed class MapTo + { + public bool MapToBool { get; set; } = false; + public Color MapToColor { get; set; } = Color.Parse("#00000000", CultureInfo.InvariantCulture); + public float MapToFloat { get; set; } = 0.0f; + public string MapToFile { get; set; } = ""; + public int MapToInt { get; set; } = 0; + public int MapToObject { get; set; } = 0; + public string MapToString { get; set; } = ""; + } + + [Fact] + public void MapPropertiesTo_NestedPropertyNotFound_ThrowsKeyNotFoundException() + { + // Arrange + List props = [ + new ClassProperty { + Name = "ClassInObject", + PropertyType = "MapTo", + Value = [ + new StringProperty { Name = "PropertyThatDoesNotExistInMapTo", Value = "Test" } + ], + } + ]; + var hasProperties = new TestHasProperties(props); + + // Act & Assert + _ = Assert.Throws(() => hasProperties.GetProperty("ClassInObject").MapPropertiesTo()); + } + + [Fact] + public void MapPropertiesTo_NestedPropertyIsNotClassProperty_ThrowsInvalidCastException() + { + // Arrange + List props = [ + new StringProperty { Name = "ClassInObject", Value = "Test" } + ]; + var hasProperties = new TestHasProperties(props); + + // Act & Assert + _ = Assert.Throws(() => hasProperties.GetProperty("ClassInObject").MapPropertiesTo()); + } + + [Fact] + public void MapPropertiesTo_NestedAllBasicValidProperties_ReturnsMappedProperty() + { + // Arrange + List props = [ + new ClassProperty { + Name = "ClassInObject", + PropertyType = "MapTo", + Value = [ + new BoolProperty { Name = "MapToBool", Value = true }, + new ColorProperty { Name = "MapToColor", Value = Color.Parse("#FF0000FF", CultureInfo.InvariantCulture) }, + new FloatProperty { Name = "MapToFloat", Value = 1.0f }, + new StringProperty { Name = "MapToFile", Value = "Test" }, + new IntProperty { Name = "MapToInt", Value = 1 }, + new IntProperty { Name = "MapToObject", Value = 1 }, + new StringProperty { Name = "MapToString", Value = "Test" }, + ], + } + ]; + var hasProperties = new TestHasProperties(props); + + // Act + var mappedProperty = hasProperties.GetProperty("ClassInObject").MapPropertiesTo(); + + // Assert + Assert.True(mappedProperty.MapToBool); + Assert.Equal(Color.Parse("#FF0000FF", CultureInfo.InvariantCulture), mappedProperty.MapToColor); + Assert.Equal(1.0f, mappedProperty.MapToFloat); + Assert.Equal("Test", mappedProperty.MapToFile); + Assert.Equal(1, mappedProperty.MapToInt); + Assert.Equal(1, mappedProperty.MapToObject); + Assert.Equal("Test", mappedProperty.MapToString); + } + + private sealed class NestedMapTo + { + public string NestedMapToString { get; set; } = ""; + public MapTo MapToInNested { get; set; } = new MapTo(); + } + + [Fact] + public void MapPropertiesTo_NestedNestedMapTo_ReturnsMappedProperty() + { + // Arrange + List props = [ + new ClassProperty { + Name = "ClassInObject", + PropertyType = "NestedMapTo", + Value = [ + new StringProperty { Name = "NestedMapToString", Value = "Test" }, + new ClassProperty { + Name = "MapToInNested", + PropertyType = "MapTo", + Value = [ + new BoolProperty { Name = "MapToBool", Value = true }, + new ColorProperty { Name = "MapToColor", Value = Color.Parse("#FF0000FF", CultureInfo.InvariantCulture) }, + new FloatProperty { Name = "MapToFloat", Value = 1.0f }, + new StringProperty { Name = "MapToFile", Value = "Test" }, + new IntProperty { Name = "MapToInt", Value = 1 }, + new IntProperty { Name = "MapToObject", Value = 1 }, + new StringProperty { Name = "MapToString", Value = "Test" }, + ], + }, + ], + } + ]; + var hasProperties = new TestHasProperties(props); + + // Act + var mappedProperty = hasProperties.GetProperty("ClassInObject").MapPropertiesTo(); + + // Assert + Assert.Equal("Test", mappedProperty.NestedMapToString); + Assert.True(mappedProperty.MapToInNested.MapToBool); + Assert.Equal(Color.Parse("#FF0000FF", CultureInfo.InvariantCulture), mappedProperty.MapToInNested.MapToColor); + Assert.Equal(1.0f, mappedProperty.MapToInNested.MapToFloat); + Assert.Equal("Test", mappedProperty.MapToInNested.MapToFile); + Assert.Equal(1, mappedProperty.MapToInNested.MapToInt); + Assert.Equal(1, mappedProperty.MapToInNested.MapToObject); + Assert.Equal("Test", mappedProperty.MapToInNested.MapToString); + } + + private enum TestEnum + { + TestValue1, + TestValue2, + TestValue3 + } + + private sealed class EnumMapTo + { + public TestEnum EnumMapToEnum { get; set; } = TestEnum.TestValue1; + } + + [Fact] + public void MapPropertiesTo_NestedEnumProperty_ReturnsMappedProperty() + { + // Arrange + List props = [ + new ClassProperty { + Name = "ClassInObject", + PropertyType = "EnumMapTo", + Value = [ + new EnumProperty { Name = "EnumMapToEnum", PropertyType = "TestEnum", Value = new HashSet { "TestValue1" } }, + ], + } + ]; + var hasProperties = new TestHasProperties(props); + + // Act + var mappedProperty = hasProperties.GetProperty("ClassInObject").MapPropertiesTo(); + + // Assert + Assert.Equal(TestEnum.TestValue1, mappedProperty.EnumMapToEnum); + } + + private enum TestEnumWithFlags + { + TestValue1 = 1, + TestValue2 = 2, + TestValue3 = 4 + } + + private sealed class EnumWithFlagsMapTo + { + public TestEnumWithFlags EnumWithFlagsMapToEnum { get; set; } = TestEnumWithFlags.TestValue1; + } + + [Fact] + public void MapPropertiesTo_NestedEnumWithFlagsProperty_ReturnsMappedProperty() + { + // Arrange + List props = [ + new ClassProperty { + Name = "ClassInObject", + PropertyType = "EnumWithFlagsMapTo", + Value = [ + new EnumProperty { Name = "EnumWithFlagsMapToEnum", PropertyType = "TestEnumWithFlags", Value = new HashSet { "TestValue1", "TestValue2" } }, + ], + } + ]; + var hasProperties = new TestHasProperties(props); + + // Act + var mappedProperty = hasProperties.GetProperty("ClassInObject").MapPropertiesTo(); + + // Assert + Assert.Equal(TestEnumWithFlags.TestValue1 | TestEnumWithFlags.TestValue2, mappedProperty.EnumWithFlagsMapToEnum); + } + + [Fact] + public void MapPropertiesTo_WithProperties_ReturnsMappedProperty() + { + // Arrange + List props = [ + new BoolProperty { Name = "MapToBool", Value = true }, + new ColorProperty { Name = "MapToColor", Value = Color.Parse("#FF0000FF", CultureInfo.InvariantCulture) }, + new FloatProperty { Name = "MapToFloat", Value = 1.0f }, + new StringProperty { Name = "MapToFile", Value = "Test" }, + new IntProperty { Name = "MapToInt", Value = 1 }, + new IntProperty { Name = "MapToObject", Value = 1 }, + new StringProperty { Name = "MapToString", Value = "Test" }, + ]; + var hasProperties = new TestHasProperties(props); + + // Act + var mappedProperty = hasProperties.MapPropertiesTo(); + + // Assert + Assert.True(mappedProperty.MapToBool); + Assert.Equal(Color.Parse("#FF0000FF", CultureInfo.InvariantCulture), mappedProperty.MapToColor); + Assert.Equal(1.0f, mappedProperty.MapToFloat); + Assert.Equal("Test", mappedProperty.MapToFile); + Assert.Equal(1, mappedProperty.MapToInt); + Assert.Equal(1, mappedProperty.MapToObject); + Assert.Equal("Test", mappedProperty.MapToString); + } +} diff --git a/src/DotTiled.Tests/UnitTests/Serialization/DefaultResourceCacheTests.cs b/src/DotTiled.Tests/UnitTests/Serialization/DefaultResourceCacheTests.cs new file mode 100644 index 0000000..6108833 --- /dev/null +++ b/src/DotTiled.Tests/UnitTests/Serialization/DefaultResourceCacheTests.cs @@ -0,0 +1,78 @@ +using DotTiled.Serialization; + +namespace DotTiled.Tests; + +public class DefaultResourceCacheTests +{ + [Fact] + public void GetTemplate_TemplateDoesNotExist_ReturnsEmptyOptional() + { + // Arrange + var cache = new DefaultResourceCache(); + var path = "template.tsx"; + + // Act + var result = cache.GetTemplate(path); + + // Assert + Assert.False(result.HasValue); + } + + [Fact] + public void GetTemplate_TemplateHasBeenInserted_ReturnsTemplate() + { + // Arrange + var cache = new DefaultResourceCache(); + var path = "template.tsx"; + var template = new Template + { + Object = new EllipseObject { } + }; + + // Act + cache.InsertTemplate(path, template); + var result = cache.GetTemplate(path); + + // Assert + Assert.True(result.HasValue); + Assert.Same(template, result.Value); + } + + [Fact] + public void GetTileset_TilesetDoesNotExist_ReturnsEmptyOptional() + { + // Arrange + var cache = new DefaultResourceCache(); + var path = "tileset.tsx"; + + // Act + var result = cache.GetTileset(path); + + // Assert + Assert.False(result.HasValue); + } + + [Fact] + public void GetTileset_TilesetHasBeenInserted_ReturnsTileset() + { + // Arrange + var cache = new DefaultResourceCache(); + var path = "tileset.tsx"; + var tileset = new Tileset + { + Name = "Tileset", + TileWidth = 32, + TileHeight = 32, + TileCount = 1, + Columns = 1 + }; + + // Act + cache.InsertTileset(path, tileset); + var result = cache.GetTileset(path); + + // Assert + Assert.True(result.HasValue); + Assert.Same(tileset, result.Value); + } +} diff --git a/src/DotTiled.Tests/UnitTests/Serialization/LoaderTests.cs b/src/DotTiled.Tests/UnitTests/Serialization/LoaderTests.cs new file mode 100644 index 0000000..d54ab9f --- /dev/null +++ b/src/DotTiled.Tests/UnitTests/Serialization/LoaderTests.cs @@ -0,0 +1,315 @@ +using System.Numerics; +using DotTiled.Serialization; +using NSubstitute; + +namespace DotTiled.Tests; + +public class LoaderTests +{ + [Fact] + public void LoadMap_Always_ReadsFromResourceReader() + { + // Arrange + var resourceReader = Substitute.For(); + resourceReader.Read("map.tmx").Returns( + """ + + + + + 0,0,0,0,0, + 0,0,0,0,0, + 0,0,0,0,0, + 0,0,0,0,0, + 0,0,0,0,0 + + + + """); + + var resourceCache = Substitute.For(); + var customTypeDefinitions = Enumerable.Empty(); + var loader = new Loader(resourceReader, resourceCache, customTypeDefinitions); + + // Act + loader.LoadMap("map.tmx"); + + // Assert + resourceReader.Received(1).Read("map.tmx"); + } + + [Fact] + public void LoadMap_MapReferencesExternalTileset_ReadsTilesetFromResourceReaderAndAttemptsToRetrieveFromCache() + { + // Arrange + var resourceReader = Substitute.For(); + resourceReader.Read("map.tmx").Returns( + """ + + + + + + 0,0,0,0,0, + 0,0,0,0,0, + 0,0,0,0,0, + 0,0,0,0,0, + 0,0,0,0,0 + + + + """); + + resourceReader.Read("tileset.tsx").Returns( + """ + + + + + + + """); + + var resourceCache = Substitute.For(); + resourceCache.GetTileset(Arg.Any()).Returns(Optional.Empty); + resourceCache.GetTemplate(Arg.Any()).Returns(Optional