Merge pull request #8 from rafaqz/noprecompile

dont test in precompile, its too slow

README.md

@@ -14,8 +14,7 @@ specifically:
 - Traits: All `@implements` declarations produce compile-time traits that can be
   checked by other packages - for the whole interface and all of it's optional
   components.
-- Tests: `@implements` declarations are automatically tested againts the interfaces
-  and subtypes they define, during precompilation.
+- Tests: `@implements` declarations should be tested in package tests.
 - Docs: interface documentation can be inserted into trait documentation.
 
 __Note: the syntax here is likely to change over 2022 as we work out the best ways to define interfaces__

src/implements.jl

@@ -84,8 +84,7 @@ function _implements_inner(interface, objtype, test_objects; show=false)
         Interfaces.optional_keys(::Type{<:$interfacetype}, ::Type{<:$objtype}) = $optional_keys
         # Define the object to be used in interface tests
         Interfaces.test_objects(::Type{<:$interfacetype}, ::Type{<:$objtype}) = Interfaces.TestObjectWrapper($test_objects)
-        # Run tests during precompilation
-        Interfaces.test($interface, $objtype; show=$show)
+        nothing
     end |> esc
 end
 

src/test.jl

@@ -34,14 +34,12 @@ function test(T::Type{<:Interface}, O::Type, objs::TestObjectWrapper;
             _showresults(mandatory_results, "Mandatory components")
             _showresults(optional_results, "Optional components")
         end
-        println()
         return all(_bool(mandatory_results)) && all(_bool(optional_results))
     else
         allcomponents = merge(components(T)...)
         optional = NamedTuple{_as_tuple(keys)}(allcomponents)
         results = _test(optional, objs)
         show && _showresults(results, "Specified components")
-        println()
         return all(_bool(results))
     end
 end