CliMA / CliMA/ClimaCore.jl

`prevent_leaky_tests = false` results in failure

Open
#1,826 0 comments 0 reactions 0 assignees View on GitHub
bug CI
Dominant language
Julia
Stars
117
Forks
19
Avg merge
3d 4h
Merged PRs (30d)
41

Description

Unfortunately, we can't yet run with `prevent_leaky_tests = false` (which could be a decent bit faster) because we occasionally define global constants, which clash with local variables:

```julia
Spaces - serial CPU DSS: Error During Test at none:1
Got exception outside of a @test
LoadError: cannot declare Main.device constant; it already has a value
Stacktrace:
[1] top-level scope
@ ~/Dropbox/Caltech/work/dev/CliMA/ClimaCore.jl/test/Spaces/ddss1.jl:21
[2] include(fname::String)
@ Base.MainInclude ./client.jl:489
...
```

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by examining test/Spaces/ddss1.jl around line 21 and reproduce the failure with prevent_leaky_tests = false. Trace the conflicting Main.device declaration and identify the test behavior needed for the suite to run without constant/local-variable clashes. Done means the affected test path completes successfully with that setting disabled.

Written by the indexing model from the issue text.

Assessment

Domain
testing-qa
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.