test: Don't use GPL module when Base.USE_GPL_LIBS=false

[inkydragon]

The GPL module SparseArrays.CHOLMOD has already been used before the if Base.USE_GPL_LIBS statement.

This results in the error

ERROR: LoadError: UndefVarError: `CHOLMOD` not defined in `SparseArrays`
Stacktrace:

[gitboy16]

Is my understanding correct if I say that tests are failing because of method ambiguities between SparseArrays and LinearAlgebra when Aqua.jl runs?

[ViralBShah]

Should we disable the aqua tests until they run more reliably?

[inkydragon]

disable the aqua tests

Or run Aqua tests in a standalone CI job #537

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.07%. Comparing base (3b30333) to head (4098ca4).
Report is 2 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #535      +/-   ##
==========================================
+ Coverage   84.05%   84.07%   +0.01%     
==========================================
  Files          12       12              
  Lines        9020     9069      +49     
==========================================
+ Hits         7582     7625      +43     
- Misses       1438     1444       +6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ViralBShah]

Is this ok to merge? I assume we need to fix the Aqua tests separately (and those failures are not related to this PR).

[SobhanMP]

I don't think anything can cause failure, but it might be a good to also run the tests with USE_GPL_LIB=false...

[dkarrasch]

I believe the Aqua tests were running reliably. A commit on Julia master introduced method ambiguities that have been resolved meanwhile. They were failing even on the "split Aqua tests" PR. ;-)

[inkydragon]

Apply suggestions.