dragonfruitnetwork / dragon6-api

Rainbow Six Siege Stats API for .NET
Apache License 2.0
12 stars 6 forks source link

Cleanup unit tests #350

Closed aspriddell closed 1 year ago

aspriddell commented 1 year ago

This consolidates the existing unit tests to use a central account listing for evaluating stats, and forces things like seasonal lookups and modern stats to test on all platforms and test the most extreme scenarios the api will be exposed to.

Also removes the CanBeNull annotations from the Modern stats extensions, because it was supposed to indicate the task result could be null, but is misinterpreted as the Task itself could be null.

Side Note: the account with the username string.Empty is because the account has been banned, and makes it easy to track if the api starts blocking banned user stats (unlikely)