Do all filtering of testitems at AST level

[nickrobinson251]

Follow-up to #170 that allows us to remove the FilteredVector and do all filtering of testitems before eval'ing any code, this is possible (after #170) if we special-case the codepath we have to support RAI usage. This adds a bit more RAI-specific code to this package, but (i) speeds up RAI's use-case a lot and (ii) tidies up the filtering story in this package (since we can drop the codepaths that were doing filtering after evaling code).

On the RAI test codebase this speeds up filtering of test-items considerably. Here's how long it takes to filter out all the tests (e.g. with runtests(; name="doesnotexist", tags=[:nosuchtag]))

• on v1.25.1: ~10 seconds
• after PR#170: ~8 seconds
• with this PR: ~1.3 seconds

Closes #41

[Drvi]

People are going to notice this one:) Very nice, Nick!!

I tried it on RAICode and it was much faster! I did run into this one issue when I tried to run a single top-level @test_rel:

julia> runtests("test"; name="constant-folding-old-periods-old-1")
  Activating project at `~/_proj/raicode_master3`
[ Info: Scanning for test items in project `RAICode` at paths: test
┌ Warning: Encountered duplicate @testsetup with name: `LoadJSONSetup`. Replacing...
└ @ ReTestItems ~/.julia/packages/ReTestItems/NieGI/src/ReTestItems.jl:663
ERROR: TaskFailedException

    nested task error: LoadError: AssertionError: `tags` keyword must be passed a collection of `Symbol`s, got: $(Expr(:escape, :([:ring1, :integration]))) (Expr)
    Stacktrace:
      [1] var"@testitem"(__source__::LineNumberNode, __module__::Module, nm::Any, exs::Vararg{Any})
        @ ReTestItems ~/.julia/packages/ReTestItems/NieGI/src/macros.jl:264
...

(I has to comment out the top-level evals first)

[nickrobinson251]

Thanks for pushing us to do this!

one issue when I tried to run a single top-level @test_rel

yeah, this is a bug in RAI's @test_rel that i'll fix when updating to this new ReTestItems.jl release