SciML / SciML/FunctionProperties.jl

QA: Aqua/JET findings marked @test_broken pending fix

Open
#54 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Julia
Stars
10
Forks
3
Avg merge
7h 48m
Merged PRs (30d)
4

Description

The grouped-tests QA group surfaced the following genuine Aqua/JET findings. They are currently wrapped as @test_broken in test/qa/qa.jl so the QA group runs green in CI; they should be fixed and un-broken.

Aqua (Aqua.test_all)

  • deps_compat (extras): FunctionProperties does not declare a [compat] entry for the Pkg test extra. (Aqua.test_all(...; deps_compat=false) is used so the remaining Aqua sub-checks still run; the deps/compat check is @test_broken.)

JET (JET.test_package)

  • 1 possible error found: Cassette.m is not defined: Cassette.m in overdub(ctx::Cassette.Context, ::typeof(nameof), args::Vararg) (Cassette/src/context.jl:286). The JET.test_package assertion is @test_broken.

All passing Aqua sub-checks remain enforced. These two checks should be fixed and the @test_broken markers removed (they will auto-flag via Test as "Unexpectedly Pass" once fixed).

Ignore until reviewed by @ChrisRackauckas.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start in test/qa/qa.jl and inspect the @test_broken cases for Aqua.test_all and JET.test_package. Review the reported Pkg test-extra compatibility finding and the Cassette.m error at Cassette/src/context.jl:286, then run the grouped-tests QA checks. Done means both findings are fixed, the markers are removed, and all QA checks pass.

Written by the indexing model from the issue text.

Assessment

Tech stack
julia
Domain
testing
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.