search

NuGet / Insights

Gather insights about public NuGet.org package data
Apache License 2.0
24 stars 7 forks source link

[WIP] Move several tests to use Verify snapshot library #106

Closed joelverhagen closed 1 month ago

joelverhagen commented 9 months ago

This does not use the object serialization in Verify and instead asserts the existing test data strings as-is. Table entities and wide entities are serialized to JSON then handed off to Verify. CSVs are left as a giant CSV string then handed off to Verify. JSON and CSV are removed as "text" file types in Verify's text file look-up table so that existing encoding decisions are preserved. This makes the existing data files just move to another part of the source tree without any diff, reducing the change of regression.

erdembayar commented 2 months ago

@joelverhagen Is it something you're going to work anytime soon?

joelverhagen commented 1 month ago

No, there were some issues with ergonomic on multiple assertion blocks in the same test (Arrange - Act - Assert - Act - Assert pattern). Verify works best with a single Verify call. https://github.com/VerifyTests/Verify/issues/1081