Removes a bunch of useless skip specs, adds a lot more back in for things that really should get tested. Changes to pending instead of skip because it seems to make more sense and I can't find a good reason for one or the other.
Some files that were not being tested were also not being used; removing them improves test coverage number honestly.
There are also some small bugfixes here and some other notes for future TODOs.
Removes a bunch of useless
skipspecs, adds a lot more back in for things that really should get tested. Changes topendinginstead ofskipbecause it seems to make more sense and I can't find a good reason for one or the other.Some files that were not being tested were also not being used; removing them improves test coverage number honestly.
There are also some small bugfixes here and some other notes for future TODOs.