Override functionality for Artifacts seems to be untested

[KristofferC]

Looking at hhttps://app.codecov.io/gh/JuliaLang/julia/blob/master/stdlib%2FArtifacts%2Fsrc%2FArtifacts.jl#L121 it seems the override functionality (Overrides.toml) is not tested. Would be good to have that done.

[u7581147]

Hi @KristofferC, I'd love to work on this issue and add tests for Artifacts.jl. Could you assign this issue to me? Thanks.

(u7581147 W-10)

[KristofferC]

We generally don't assign issues to people to prevent "cookie licking" (https://www.redhat.com/en/blog/dont-lick-cookie). You can just start working on it anyway if you wish :)

[u7581147]

Thanks, Kristoffer. I'll try my best to complete it but cannot guarantee anything.

[u7581147]

Hi @KristofferC, I added tests for the artifacts_hash() in Artifacts.jl in #51815. Am I doing it the right way?
Of course, I'm happy to add tests for other functions if it's on the right track.

[u7581147]

Hi @KristofferC, following your comment in #51815, I added tests for load_overrides() in Artifacts.jl in #51833.

Does it do the right thing?

Thanks.

[u7581147]

It basically tests in three steps:

1. Create an Overrides.toml file.
2. Test load_overrides() using the created file.
3. Delete any created file and directory, so that the system behaves like the test never happened.