rafaqz / Interfaces.jl

Macros to define and implement interfaces, to ensure they are checked and correct.
MIT License
72 stars 4 forks source link

Move test objects back to the `@implements` macro #35

Closed rafaqz closed 8 months ago

rafaqz commented 8 months ago

Moving the data to the tests was a mistake. We lose all high level approaches to testing.

The context for this is the BaseInterfaces.jl tests are getting ridiculous - its hard to track which interfaces are actually tested.

It should be literally just:

Interfaces.test(BaseInterfaces)

Then we look through the implements method table and test everything in there defined by BaseInterfaces.jl

But that requires having test data attached to implementations.

(Note we should still be able to pass objects to the tests, we just shouldn't have to)