Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Quieter tests #49093

Merged
merged 3 commits into from
Mar 24, 2023
Merged

Quieter tests #49093

merged 3 commits into from
Mar 24, 2023

Conversation

IanButterworth
Copy link
Member

@IanButterworth IanButterworth commented Mar 22, 2023

  • Quiets intentional depwarns by catching and testing for them
From worker 5:	┌ Warning: `old_return_args` is deprecated, use `new_return_args` instead.
From worker 5:	│   caller = macro expansion at Test.jl:478 [inlined]
From worker 5:	└ @ Core /cache/build/default-amdci5-0/julialang/julia-master/julia-4d8c22f7f6/share/julia/stdlib/v1.10/Test/src/Test.jl:478
From worker 5:	┌ Warning: `old_return_args` is deprecated, use `new_return_args` instead.
From worker 5:	│   caller = macro expansion at Test.jl:478 [inlined]
From worker 5:	└ @ Core /cache/build/default-amdci5-0/julialang/julia-master/julia-4d8c22f7f6/share/julia/stdlib/v1.10/Test/src/Test.jl:478
  • Groups the threaded tests into a single report per nthreads tested

  • Catches what seem to be intentional unused type variable warnings

From worker 11:	WARNING: method definition for ambig_with_bounds at /cache/build/default-amdci5-0/julialang/julia-master/julia-4d8c22f7f6/share/julia/test/ambiguous.jl:49 declares type variable S but does not use it.
From worker 11:	WARNING: method definition for ambig_with_bounds at /cache/build/default-amdci5-0/julialang/julia-master/julia-4d8c22f7f6/share/julia/test/ambiguous.jl:50 declares type variable S but does not use it.
From worker 11:	WARNING: method definition for f11407 at /cache/build/default-amdci5-0/julialang/julia-master/julia-4d8c22f7f6/share/julia/test/ambiguous.jl:388 declares type variable V but does not use it.
From worker 11:	WARNING: method definition for f11407 at /cache/build/default-amdci5-0/julialang/julia-master/julia-4d8c22f7f6/share/julia/test/ambiguous.jl:388 declares type variable K but does not use it.
From worker 11:	WARNING: method definition for stripType at /cache/build/default-amdci5-0/julialang/julia-master/julia-4d8c22f7f6/share/julia/test/ambiguous.jl:403 declares type variable T but does not use it.
From worker 13:	WARNING: method definition for bad_tvars at /cache/build/default-amdci5-0/julialang/julia-master/julia-4d8c22f7f6/share/julia/test/core.jl:5270 declares type variable T but does not use it.
From worker 13:	WARNING: method definition for bad_tvars2 at /cache/build/default-amdci5-0/julialang/julia-master/julia-4d8c22f7f6/share/julia/test/core.jl:5273 declares type variable T but does not use it.

@fredrikekre
Copy link
Member

Failures look related:

ERROR: LoadError: syntax: "module" expression not at top level
in expression starting at 
/cache/build/default-amdci5-7/julialang/julia-master/julia-10cb68f4a3/share/julia/test/threads_exec.jl:30

@IanButterworth IanButterworth merged commit 7d32099 into JuliaLang:master Mar 24, 2023
@IanButterworth IanButterworth deleted the ib/quieting branch March 24, 2023 15:45
Xnartharax pushed a commit to Xnartharax/julia that referenced this pull request Apr 19, 2023
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.

2 participants