tahoe-lafs/trac-2024-07-25

Add integration tests for file get and put variations, especially stdin/stdout #3715

New issue

Closed

opened 2021-05-13 15:03:48 +00:00 by itamarst · 2 comments

itamarst commented

2021-05-13 15:03:48 +00:00

Owner

Unicode vs. bytes is one of the harder parts of the port.
Unit testing stdin/stdout interactions is likely to be quite fragile, given both quite different semantics on Python 2 and 3, and need to emulate things like stdin/stdout.

As such, integration tests of tahoe get and tahoe put with (a) binary files and (b) stdin/stdout usage seems like a good idea, even with nominally good unit test coverage.

1. Unicode vs. bytes is one of the harder parts of the port. 2. Unit testing stdin/stdout interactions is likely to be quite fragile, given both quite different semantics on Python 2 and 3, and need to emulate things like stdin/stdout. As such, integration tests of `tahoe get` and `tahoe put` with (a) binary files and (b) stdin/stdout usage seems like a good idea, even with nominally good unit test coverage.

tahoe-lafs added the

labels 2021-05-13 15:03:48 +00:00

tahoe-lafs added this to the Support Python 3 milestone 2021-05-13 15:03:48 +00:00

itamarst commented

2021-05-21 15:24:56 +00:00

Author

Owner

Python has a pipes module, but it does the stupid thing on Python 3 where it assumes unicode files. So going to do it manually.

Python has a pipes module, but it does the stupid thing on Python 3 where it assumes unicode files. So going to do it manually.

GitHub <noreply@github.com> commented

2021-05-24 13:15:35 +00:00

Author

Owner

In 05a1349/trunk:

Merge pull request #1066 from tahoe-lafs/3715-get-put-integration-tests

CLI get/put integration tests.

Fixes ticket:3715

In [05a1349/trunk](/tahoe-lafs/trac-2024-07-25/commit/05a13499ad7209bd258cf61a88973c59bad6f859): ``` Merge pull request #1066 from tahoe-lafs/3715-get-put-integration-tests CLI get/put integration tests. Fixes ticket:3715 ```

tahoe-lafs added the

label 2021-05-24 13:15:35 +00:00

GitHub <noreply@github.com> closed this issue

2021-05-24 13:15:35 +00:00

Sign in to join this conversation.

No labels

LeastAuthority.com automation

cannot reproduce

code-frontend-cli

code-frontend-ftp-sftp

code-frontend-magic-folder

code-frontend-web

code-peerselection

dev-infrastructure

somebody else's problem

was already fixed

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: tahoe-lafs/trac-2024-07-25#3715

No description provided.