Skip to content

Move test_density_interface to a Test extension - #39

Open
timholy wants to merge 1 commit into
masterfrom
teh/test-ext
Open

timholy wants to merge 1 commit into
masterfrom
teh/test-ext

Conversation

@timholy

@timholy timholy commented Oct 15, 2026

Copy link
Copy Markdown
Member

Loading Test accounted for nearly all of DensityInterface's load time. test_density_interface is now defined in DensityInterfaceTestExt, which loads when Test is loaded. Calling it without Test loaded throws a MethodError whose message tells the user to run using Test.

Extensions require Julia 1.9, so the minimum supported version is now Julia 1.10 (LTS), and CI tests LTS instead of 1.0.

Fixes #38

Assisted-by: Claude Opus 5.5 noreply@anthropic.com

Loading Test accounted for nearly all of DensityInterface's load time.
`test_density_interface` is now defined in DensityInterfaceTestExt, which
loads when Test is loaded. Calling it without Test loaded throws a
MethodError whose message tells the user to run `using Test`.

Extensions require Julia 1.9, so the minimum supported version is now
Julia 1.10 (LTS), and CI tests LTS instead of 1.0.

Fixes #38

Assisted-by: Claude Opus 5.5 <noreply@anthropic.com>
@timholy

timholy commented Oct 15, 2026

Copy link
Copy Markdown
Member Author

Breaking (minimally so), but I didn't bump the version because I wasn't sure whether it should be 0.5 or 1.0.

@codecov

codecov Bot commented Oct 15, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (4c4f066) to head (0c0f5a8).

Additional details and impacted files
@@            Coverage Diff            @@
##            master       #39   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            2         3    +1     
  Lines           60        64    +4     
=========================================
+ Hits            60        64    +4     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Test dependency accounts for almost all the load time

1 participant