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

core tests broken by #39875 #43004

Closed
vtjnash opened this issue Nov 8, 2021 · 5 comments · Fixed by #43739
Closed

core tests broken by #39875 #43004

vtjnash opened this issue Nov 8, 2021 · 5 comments · Fixed by #43739
Assignees
Labels
test This change adds or pertains to unit tests

Comments

@vtjnash
Copy link
Member

vtjnash commented Nov 8, 2021

#39875 broke the core tests (outside of CI), though I don't understand how nobody has noticed since March. We thus must fix these tests and the breakage caused.

core                          (2) |         failed at 2021-11-08T15:28:28.538                                                                                                                                                            
Test Failed at /data/vtjnash/julia1/test/core.jl:3521                                                                                                                                                                                    
  Expression: (Vararg{Int, N} where N <: T) where T                                                                                                                                                                                      
    Expected: ErrorException                                                                                                                                                                                                             
  No exception thrown                                                                                                                                                                                                                    
Test Failed at /data/vtjnash/julia1/test/core.jl:3522                                                                                                                                                                                    
  Expression: Vararg{Int, N} where N <: Integer                                                                                                                                                                                          
    Expected: ErrorException                                                                                                                                                                                                             
  No exception thrown                                                                                                                                                                                                                    
Test Failed at /data/vtjnash/julia1/test/core.jl:3523                                                                                                                                                                                    
  Expression: Vararg{Int, N} where N >: Integer                                                                                                                                                                                          
    Expected: ErrorException                                                                                                                                                                                                             
  No exception thrown                                                                                                                                                                                                                    
                                                                                                                                                                                                                                         
@vtjnash vtjnash added backport 1.6 Change should be backported to release-1.6 backport 1.7 test This change adds or pertains to unit tests labels Nov 8, 2021
@martinholters
Copy link
Member

With --depwarn=error:

julia> Vararg{Int, N} where N>:Integer
ERROR: Wrapping `Vararg` directly in UnionAll is deprecated (wrap the tuple instead).

So that's why this has gone unnoticed.

@DilumAluthge
Copy link
Member

One option would be to run these tests in a Julia subprocess that we start with the --depwarn=error flag.

@martinholters
Copy link
Member

That's what happens with e.g. make test anyway, making these tests pass under the usual circumstances. But IIUC it's not what they were intended to test. They were testing for this error:

julia> Vararg{Int, N} where N>:Integer
ERROR: TypeVar in Vararg length must have bounds Union{} and Any

Which went away between 1.5 and 1.6. So I guess we can just delete the tests.

@martinholters
Copy link
Member

Ah, they've already been deleted in 1.6 by d06bab0, the backport of #39875, which itself did not delete these tests but in retrospect, it probably should have.

@staticfloat
Copy link
Member

@JeffBezanson can you confirm that these tests should be deleted?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test This change adds or pertains to unit tests
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants