Skip to content

XFAIL test_runner_parametrized_protocol on python3.8 when getting duplicate output#6837

Merged
yarikoptic merged 1 commit into
datalad:maintfrom
yarikoptic:bf-skip-test
Jul 15, 2022
Merged

XFAIL test_runner_parametrized_protocol on python3.8 when getting duplicate output#6837
yarikoptic merged 1 commit into
datalad:maintfrom
yarikoptic:bf-skip-test

Conversation

@yarikoptic
Copy link
Copy Markdown
Member

This should close (unresolved) but not fix #4921
which seems to happen so far rarely on conda on 3.8 python only. If some brave soul
manages to figure it out -- great. But I would not break our knuckles to see it fixed
since seems to be very particular. I made xfail also very particular (not over entire
test, only for 3.8 and only for the 55 output) so we do not miss any other problem

…licate output

This should close (unresolved) but not fix datalad#4921
which seems to happen so far rarely on conda on 3.8 python only. If some brave soul
manages to figure it out -- great. But I would not break our knuckles to see it fixed
since seems to be very particular.  I made xfail also very particular (not over entire
test, only for 3.8 and only for the 55 output) so we do not miss any other problem
@yarikoptic yarikoptic added the semver-tests Changes only affect tests, no impact on version label Jul 12, 2022
@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 12, 2022

Codecov Report

Merging #6837 (02f1528) into maint (9679312) will increase coverage by 0.96%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##            maint    #6837      +/-   ##
==========================================
+ Coverage   90.25%   91.22%   +0.96%     
==========================================
  Files         354      354              
  Lines       46064    46083      +19     
==========================================
+ Hits        41575    42038     +463     
+ Misses       4489     4045     -444     
Impacted Files Coverage Δ
datalad/runner/tests/test_witless_runner.py 92.99% <33.33%> (-2.41%) ⬇️
datalad/customremotes/tests/test_datalad.py 95.34% <0.00%> (-2.28%) ⬇️
datalad/distribution/create_sibling.py 69.80% <0.00%> (-0.20%) ⬇️
datalad/cli/common_args.py 100.00% <0.00%> (ø)
datalad/tests/test_dochelpers.py 100.00% <0.00%> (ø)
datalad/support/tests/test_network.py 100.00% <0.00%> (ø)
datalad/tests/utils_pytest.py 89.73% <0.00%> (+0.01%) ⬆️
datalad/utils.py 88.17% <0.00%> (+0.01%) ⬆️
datalad/__init__.py 98.00% <0.00%> (+0.04%) ⬆️
datalad/support/tests/test_locking.py 95.87% <0.00%> (+0.04%) ⬆️
... and 15 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5a1c4bd...02f1528. Read the comment docs.

@yarikoptic
Copy link
Copy Markdown
Member Author

any objections/blessings @datalad/developers ?

@yarikoptic
Copy link
Copy Markdown
Member Author

ok, no objections then.

@yarikoptic yarikoptic merged commit e999cb3 into datalad:maint Jul 15, 2022
@github-actions
Copy link
Copy Markdown

🚀 PR was released in 0.17.2 🚀

eq_(res['stdout'], '5')
except AssertionError:
# TODO: remove when dropping support for python3.8
if res['stdout'] == '55' and sys.version_info[:2] == (3, 8):
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@yarikoptic yarikoptic deleted the bf-skip-test branch October 14, 2022 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released semver-tests Changes only affect tests, no impact on version

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant