Skip to content

Conversation

@DilumAluthge
Copy link
Member

@DilumAluthge DilumAluthge commented Apr 27, 2021

There is code in Pkg.jl that assumes that the following tuples always have the same length:

  • Base.project_names
  • Base.manifest_names
  • Base.preferences_names
  • Artifacts.artifact_names

So we should have tests to make sure that this always stays the case.

@DilumAluthge DilumAluthge added the test This change adds or pertains to unit tests label Apr 27, 2021
@DilumAluthge DilumAluthge changed the title Add tests to make sure that project_names, manifest_names, and preferences_names are in sync Add tests to make sure that Base.project_names, Base.manifest_names, Base.preferences_names, and Artifacts.artifact_names always have the same length Apr 27, 2021
…s`, `Base.preferences_names`, and `Artifacts.artifact_names` always have the same length
@DilumAluthge DilumAluthge force-pushed the dpa/add-test-manifest-names branch from b76f135 to 4abb6a3 Compare April 27, 2021 03:49
Copy link
Member

@staticfloat staticfloat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems fine, but would be good to add a comment to these tests stating why we're asserting that they are the same length.

@staticfloat staticfloat merged commit 3389940 into master Apr 29, 2021
@staticfloat staticfloat deleted the dpa/add-test-manifest-names branch April 29, 2021 05:00
ElOceanografo pushed a commit to ElOceanografo/julia that referenced this pull request May 4, 2021
…s`, `Base.preferences_names`, and `Artifacts.artifact_names` always have the same length (JuliaLang#40618)

* Add tests to make sure that `Base.project_names`, `Base.manifest_names`, `Base.preferences_names`, and `Artifacts.artifact_names` always have the same length

* Add a comment explaining why we have these tests
jarlebring pushed a commit to jarlebring/julia that referenced this pull request May 4, 2021
…s`, `Base.preferences_names`, and `Artifacts.artifact_names` always have the same length (JuliaLang#40618)

* Add tests to make sure that `Base.project_names`, `Base.manifest_names`, `Base.preferences_names`, and `Artifacts.artifact_names` always have the same length

* Add a comment explaining why we have these tests
antoine-levitt pushed a commit to antoine-levitt/julia that referenced this pull request May 9, 2021
…s`, `Base.preferences_names`, and `Artifacts.artifact_names` always have the same length (JuliaLang#40618)

* Add tests to make sure that `Base.project_names`, `Base.manifest_names`, `Base.preferences_names`, and `Artifacts.artifact_names` always have the same length

* Add a comment explaining why we have these tests
johanmon pushed a commit to johanmon/julia that referenced this pull request Jul 5, 2021
…s`, `Base.preferences_names`, and `Artifacts.artifact_names` always have the same length (JuliaLang#40618)

* Add tests to make sure that `Base.project_names`, `Base.manifest_names`, `Base.preferences_names`, and `Artifacts.artifact_names` always have the same length

* Add a comment explaining why we have these tests
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 this pull request may close these issues.

3 participants