Oops, I discovered a few lines I forgot to change :( So, another PR for the same thing, this just fixes a few tests. While technically not wrong, I think using the getter for the artefact_store is the right way. There are a few assignments to the artefact_store in the tests left over, I think that's ok (since they are tests), in general we would not want a user to be able to re-assign the artefact store of a config, so I did not implement a setter for that reason.
Oops, I discovered a few lines I forgot to change :( So, another PR for the same thing, this just fixes a few tests. While technically not wrong, I think using the getter for the artefact_store is the right way. There are a few assignments to the artefact_store in the tests left over, I think that's ok (since they are tests), in general we would not want a user to be able to re-assign the artefact store of a config, so I did not implement a setter for that reason.