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

DATALAD_TESTS_GITCONFIG env variable to provide custom settings for git during tests #7056

Merged
merged 1 commit into from
Sep 26, 2022

Conversation

yarikoptic
Copy link
Member

@yarikoptic yarikoptic commented Sep 22, 2022

Although could be just once in a lifetime bug, we just got into an odd issue with an unlocked file with content being marked as modified https://git-annex.branchable.com/bugs/reports_file___34__modified__34___whenever_it_is_not/#comment-4e61c4c0f493e98f9416e09de4cfb33f which seems to boil down to have a rarely used annex.stalldetection set.

By adding one more env variable for TESTS we can now add such specific setting to be tested in the tests.

TODOs

  • make sure that tests pass -- we do might trigger that issue

@yarikoptic yarikoptic added semver-tests Changes only affect tests, no impact on version CHANGELOG-missing When a PR's description does not contain a changelog item, yet. labels Sep 22, 2022
@github-actions github-actions bot removed the CHANGELOG-missing When a PR's description does not contain a changelog item, yet. label Sep 22, 2022
@joeyh
Copy link

joeyh commented Sep 22, 2022

I'm not sure if you're using git-annex test, but if you are, it also ignores global gitconfigs. I've added a --test-with-git-config option to it that you could use to run the git-annex test suite with a git config set.

@yarikoptic
Copy link
Member Author

it will need to wait for #7057 to get merged to get a kosher travis run.

@joeyh thanks for the note! we do not do git-annex test here but I am adding a matrix run for git annex test with that option in datalad/git-annex#133

…g tests

Although could be just once in a lifetime bug, we just got into an odd
issue with an unlocked file with content being marked as modified
https://git-annex.branchable.com/bugs/reports_file___34__modified__34___whenever_it_is_not/#comment-4e61c4c0f493e98f9416e09de4cfb33f
which seems to boil down to have a rarely used `annex.stalldetection` set.

By adding one more env variable for TESTS we can now add such specific
setting to be tested in the tests.
@yarikoptic
Copy link
Member Author

Travis is happy, so either this PR is not in effect or we just do not have a test which gets file in unlocked mode? not sure if I am to craft one now, so I will just give it to review.

@yarikoptic yarikoptic marked this pull request as ready for review September 23, 2022 19:05
@yarikoptic yarikoptic merged commit 641c9fb into datalad:maint Sep 26, 2022
@yarikoptic yarikoptic deleted the enh-test-gitconfig branch October 14, 2022 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver-tests Changes only affect tests, no impact on version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants