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

Test error with Python3.10 #33

Closed
tillea opened this issue Jan 2, 2022 · 2 comments · Fixed by #62
Closed

Test error with Python3.10 #33

tillea opened this issue Jan 2, 2022 · 2 comments · Fixed by #62
Labels

Comments

@tillea
Copy link

tillea commented Jan 2, 2022

Hi,
there is a bug report against the Debian packaged version 1.5.0. I've upgraded the Debian packaging to 1.6.0 but the issue persists:

  File "/build/annexremote-1.6.0/.pybuild/cpython3_3.10_annexremote/build/tests/test_GitAnnexRequestMessages.py", line 305, in TestExport_MissingName
    self.assertEqual(utils.last_buffer_line(self.output), "ERROR do_EXPORT() missing 1 required positional argument: 'name'")
AssertionError: "ERROR Protocol.do_EXPORT() missing 1 required positional argument: 'name'" != "ERROR do_EXPORT() missing 1 required positional argument: 'name'"
- ERROR Protocol.do_EXPORT() missing 1 required positional argument: 'name'
?       ---------
+ ERROR do_EXPORT() missing 1 required positional argument: 'name'


======================================================================
FAIL: TestRemoveexportdirectory_MissingDirectory (test_GitAnnexRequestMessages.TestGitAnnexRequestMessagesExporttree)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/build/annexremote-1.6.0/.pybuild/cpython3_3.10_annexremote/build/tests/test_GitAnnexRequestMessages.py", line 423, in TestRemoveexportdirectory_MissingDirectory
    self.assertEqual(utils.last_buffer_line(self.output), "ERROR do_REMOVEEXPORTDIRECTORY() missing 1 required positional argument: 'name'")
AssertionError: "ERROR Protocol.do_REMOVEEXPORTDIRECTORY() missi[37 chars]ame'" != "ERROR do_REMOVEEXPORTDIRECTORY() missing 1 requ[28 chars]ame'"
- ERROR Protocol.do_REMOVEEXPORTDIRECTORY() missing 1 required positional argument: 'name'
?       ---------
+ ERROR do_REMOVEEXPORTDIRECTORY() missing 1 required positional argument: 'name'


----------------------------------------------------------------------
Ran 134 tests in 0.199s

FAILED (failures=2)

Kind regards, Andreas.

@Lykos153
Copy link
Owner

Lykos153 commented Jan 3, 2022

#29 would fix this but also break compatibility with all versions <3.10. I hope I find the time to look into this soon.

@github-actions
Copy link

🚀 Issue was released in v1.6.2 🚀

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

Successfully merging a pull request may close this issue.

2 participants