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

Failure in pkg tests #20566

Closed
ararslan opened this issue Feb 11, 2017 · 4 comments
Closed

Failure in pkg tests #20566

ararslan opened this issue Feb 11, 2017 · 4 comments
Labels
packages Package management and loading test This change adds or pertains to unit tests

Comments

@ararslan
Copy link
Member

The following error is showing up in CI on all platforms:

Worker 3 failed running test pkg:
Some tests did not pass: 105 passed, 0 failed, 1 errored, 0 broken.pkg: Error During Test
  Got an exception of type LoadError outside of a @test
  LoadError: ArgumentError: argument to Flatten must contain at least one iterator
  start at ./iterators.jl:610
  grow_to! at ./array.jl:445
  intersect at ./pkg/types.jl:44
  push! at ./pkg/types.jl:183
  requirements at ./pkg/query.jl:34
  resolve at ./pkg/entry.jl:481
  edit at ./pkg/entry.jl:30
  #1 at ./task.jl:335
  while loading /tmp/julia/share/julia/test/pkg.jl, in expression starting on line 39

Test Summary:    | Pass  Error  Total
  Overall        |  109      1    110
    libgit2-online |    4             4
    pkg          |  105      1    106
    FAILURE
Error in testset pkg:
Error During Test
  Got an exception of type LoadError outside of a @test
  LoadError: ArgumentError: argument to Flatten must contain at least one iterator
  start at ./iterators.jl:610
  grow_to! at ./array.jl:445
  intersect at ./pkg/types.jl:44
  push! at ./pkg/types.jl:183
  requirements at ./pkg/query.jl:34
  resolve at ./pkg/entry.jl:481
  edit at ./pkg/entry.jl:30
  #1 at ./task.jl:335
  while loading /tmp/julia/share/julia/test/pkg.jl, in expression starting on line 39
ERROR: LoadError: Test run finished with errors
while loading /tmp/julia/share/julia/test/runtests.jl, in expression starting on line 38
@ararslan ararslan added packages Package management and loading test This change adds or pertains to unit tests labels Feb 11, 2017
@tkelman
Copy link
Contributor

tkelman commented Feb 11, 2017

doesn't fail with a 4-day-old master, so cc @carlobaldassi

@tkelman
Copy link
Contributor

tkelman commented Feb 11, 2017

bisects to d054d36 - must be changes in metadata hitting new code paths that weren't causing a problem until now

@carlobaldassi
Copy link
Member

Sorry, not sure how I missed this...

@tkelman
Copy link
Contributor

tkelman commented Feb 11, 2017

thanks for the fix. is it possible for the metadata situation that caused this to change and no longer cause an issue with the old code? if so, a targeted test might be good if possible to write one

carlobaldassi added a commit that referenced this issue Feb 13, 2017
* Enforce VectorSet normalization in constructor (fixes equality
  testing)
* Improve VectorSet normalization code (faster, fewer allocations),
* Test VectorSet intersections (including the empty cases which were
  at the root of issue #20566)
carlobaldassi added a commit that referenced this issue Feb 14, 2017
* Enforce VectorSet normalization in constructor (fixes equality
  testing)
* Improve VectorSet normalization code (faster, fewer allocations),
* Test VectorSet intersections (including the empty cases which were
  at the root of issue #20566)
* Change VectorSet printing to avoid unicode on Windows
tkelman pushed a commit that referenced this issue Feb 14, 2017
* Enforce VectorSet normalization in constructor (fixes equality
  testing)
* Improve VectorSet normalization code (faster, fewer allocations),
* Test VectorSet intersections (including the empty cases which were
  at the root of issue #20566)
* Change VectorSet printing to avoid unicode on Windows
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages Package management and loading test This change adds or pertains to unit tests
Projects
None yet
Development

No branches or pull requests

3 participants