iBridges integration tests #34
Conversation
|
The uppercase Metadata keys com from this line: It was once good practice to capitalise keys, however times have changed and I think we can drop that ;) |
|
@chStaiger Thanks! I have moved from the python script to using pytest for our integration test. It might be a while for them to be all done. ;) |
|
Let me know if I can be of any help! |
So far I have added session/resources/meta tests
qubixes
force-pushed
the
ibridges-integration-tests
branch
from
802781e to
e320c14
Compare
|
@chStaiger I think for this PR, this is enough. Expansion of the integration tests can be done in another PR I think. |
|
|
||
| def test_pam_password(session, config, irods_env): | ||
| if not config["can_write_pam_pass"]: | ||
| pytest.xfail("This iRods server cannot write pam passwords.") |
There was a problem hiding this comment.
I think this test should also work. It might have something to do with the write-rights in the docker image. I will make a note and take a look. For this PR we cna leave it like this.
There was a problem hiding this comment.
I don't think it's a simple write issue though, since I'm also writing quite a few files, without problems. Maybe it's just in the root folder, or the directory is not made?
| from ibridges.utils.path import IrodsPath | ||
|
|
||
|
|
||
| def _get_digest(obj_or_file): |
See the file under docker/README.md on how to run these tests (locally).
See the file under docker/README.md on how to run these tests (locally).
Almost working, just some problems with the key that becomes uppercase for which I need @chStaiger!