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

Failing testing of datalad "core" #705

Closed
yarikoptic opened this issue May 21, 2024 · 2 comments
Closed

Failing testing of datalad "core" #705

yarikoptic opened this issue May 21, 2024 · 2 comments

Comments

@yarikoptic
Copy link
Member

e.g. https://github.com/datalad/datalad/actions/runs/9163271335/job/25191987511#step:12:2179

patches/tests/test_push.py::test_nested_pushclone_cycle_allplatforms create(ok): /tmp/datalad_temp_test_nested_pushclone_cycle_allplatformsyh646c0g/super (dataset)
Warning:  Failed to import vcr, no cassettes will be available: AttributeError(module 'urllib3.connectionpool' has no attribute 'VerifiedHTTPSConnection') 
add(ok): sub (dataset)
add(ok): .gitmodules (file)
save(ok): . (dataset)
create(ok): sub (dataset)
action summary:
  add (ok: 2)
  create (ok: 1)
  save (ok: 1)
add(ok): file2.txt (file)
save(ok): sub (dataset)
add(ok): sub (dataset)
add(ok): .gitmodules (file)
add(ok): file1.txt (file)
save(ok): . (dataset)
action summary:
  add (ok: 4)
  save (ok: 2)
[INFO] Start checking pre-existing sibling configuration Dataset(/tmp/datalad_temp_test_nested_pushclone_cycle_allplatformsyh646c0g/super) 
Warning:  Failed to import vcr, no cassettes will be available: AttributeError(module 'urllib3.connectionpool' has no attribute 'VerifiedHTTPSConnection') 
[INFO] Finished checking pre-existing sibling configuration Dataset(/tmp/datalad_temp_test_nested_pushclone_cycle_allplatformsyh646c0g/super) 
[INFO] Creating a new RIA store at /tmp/datalad_temp_test_nested_pushclone_cycle_allplatformsnyqfdzre 
Error:  AssertionError 
....
=========================== short test summary info ============================
FAILED patches/tests/test_push.py::test_nested_pushclone_cycle_allplatforms - datalad.runner.exception.CommandError: CommandError: 'datalad create-sibling-ria --recursive -s store ria+file:///tmp/datalad_temp_test_nested_pushclone_cycle_allplatformsnyqfdzre --new-store-ok' failed with exitcode 1
====== 1 failed, 409 passed, 30 skipped, 17 warnings in 790.82s (0:13:10) ======
@mih
Copy link
Member

mih commented May 22, 2024

This should be fixed in 1.4.1 and with #696 in particular.

@christian-monch
Copy link
Contributor

The problem seems to be that the datalad-next patches are partially used and partially not used.

The class CreateSiblingRia stems from datalad-core, i.e. from datalad.distributed.create_sibling_ria. But the function _ensure_version in ria_utils is the function ria_utils__ensure_version from a datalad-next patch, i.e. from datalad_next.patches.ria_utils.

Running tests with log-level 8 reveals this fact in the assert message.

[Level 8] AssertionError [main.py:_run_with_exception_handler:190,exec.py:call_from_parser:107,base.py:_execute_command_:873,utils.py:_process_results:319,create_sibling_ria.py:__call__:428,ria_utils.py:create_store:177,ria_utils.py:ria_utils__ensure_version:48] 

The test passes for me if the environment variable DATALAD_EXTENSIONS_LOAD is set to next. That should be done, when testing the datalad-next-extension.

I opened PR datalad #7605, which sets DATALAD_EXTENSIONS_LOAD=next in the datalad-next extension tests

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants