* Fix MapRenderer integration test usage to properly show output.
Added an ITestContextLike interface that can be used to properly run the integration test infrastructure OUTSIDE A TEST.
* Use System.Test.Json instead of Newtonsoft.Json for MapRenderer
* Fix map renderer JSON output being broken
I love not testing or even reading the surrounding code.
* Fix un-reusable integration instances getting leaked.
The pair state was always getting set to Ready even if the instance was killed, meaning it was getting put back into the pool even if killed.
* Mark map renderer integration instances as destructive to avoid memory leak.
* Fix file specification handling.
Map file specification is now backwards compatible again (loose filename match to search prototypes). It also supports proper direct OS filename arguments. The former is the fallback scenario is extremely important for the map server still.
Cleaned up the way that target map files are passed through the application, so mixed file/prototype specifications are now handled properly (which can be caused by the fallback behavior).
Fixes JSON data export to use the proper user-facing map name. This only works if a prototype ID is specified *or* the legacy file behavior is used.
Restructured MapPainter into an instance that has multiple functions called on it, so not all data has to be passed through a single Paint() call.
Clean up the godawful map/grid detection code. Now we just load both in a single call, because yes you can do that. This relies on LogOrphanedGrids = false in the map loader options, which I think is fine for our purposes.
Improved error handling in much of the program.
* Fix duplicate map names in map renderer output
I'm not sure *what* this output is used for, but I'm sure having it duplicated per grid isn't intentional.
* Make maprenderer command line parsing bail on unknown - options
* Fix incorrect docs for --viewer maprenderer argument
It doesn't change directory layout
* Fix parallax layer specification to not use imgur as a fucking CDN
Files are now copied to a separate folder _parallax, and these files are referenced by the parallax configuration.
Parallax data is only output when instructed to via --parallax.
This will break parallax on current map server builds, but it should be graceful. Also, that's fucking good considering we shouldn't be using imgur links. Purge it.
* Fix incorrect assert in test pair clean return
* Restore other map viewer parallax layers, fix attribution.
* This isn't a valid copyright statement but the validator forces me to enter something here.
* Use new Subs.CVar helper
Removes manual config OnValueChanged calls, removes need to remember to manually unsubscribe.
This both reduces boilerplate and fixes many issues where subscriptions weren't removed on entity system shutdown.
* Fix a bunch of warnings
* More warning fixes
* Use new DateTime serializer to get rid of ISerializationHooks in changelog code.
* Get rid of some more ISerializationHooks for enums
* And a little more
* Apply suggestions from code review
Co-authored-by: 0x6273 <0x40@keemail.me>
---------
Co-authored-by: 0x6273 <0x40@keemail.me>
* Give .props files 2-space indents.
* Move to Central Package Management.
Allows us to store NuGet package versions all in one place. Yay!
* Update NuGet packages and fix code for changes.
Notable:
Changes to ILVerify.
Npgsql doesn't need hacks for inet anymore, now we need hacks to make the old code work with this new reality.
NUnit's analyzers are already complaining and I didn't even update it to 4.x yet.
TerraFX changed to GetLastSystemError so error handling had to be changed.
Buncha APIs have more NRT annotations.
* Remove dotnet-eng NuGet package source.
I genuinely don't know what this was for, and Central Package Management starts throwing warnings about it, so YEET.
* Remove Robust.Physics project.
Never used.
* Remove erroneous NVorbis reference.
Should be VorbisPizza and otherwise wasn't used.
* Sandbox fixes
* Remove unused unit test package references.
Castle.Core and NUnit.ConsoleRunner.
* Update NUnit to 4.0.1
This requires replacing all the old assertion methods because they removed them 🥲
* Oh so that's what dotnet-eng was used for. Yeah ok that makes sense.
* Add Robust.Analyzers.Test
* Update submodule
* commit to re-run CI
