Windows: tests skipped in P3 (fake sysfs, backslash robot paths)
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 48/100
Research direction
Start with tests/acquire/test_volumes_linux.py and tests/acquire/test_transfer.py::test_robot_path_with_backslash_or_colon_is_skipped, then inspect the fake sysfs fixtures and fake SFTP server setup. Compare how the tests create paths on Windows with the Linux and macOS runs. Done means the affected tests run on Windows while still checking the intended parser and path behavior.
Written by the indexing model from the issue text.
Description
P3 (IgniteRobotics/flashpoint-private#15) added a windows-latest CI job. These tests are skipped on win32 because they can't run there as written. Each still runs on Linux and macOS.
tests/acquire/test_volumes_linux.py: 6 tests that use the fake/systree. Linux sysfs device names contain:(e.g.0000:00:14.0,6:0:0:0), which NTFS can't create. The Linux parser itself is pure and tested on fixtures. Possible fix: map:in the fake tree, or inject a path-resolver into the parser.tests/acquire/test_transfer.py::test_robot_path_with_backslash_or_colon_is_skipped: the fake robot serves a host temp directory, and Windows can't create files whose names contain\or:. Theinbox_pathrule is still unit-tested on every OS. Possible fix: give the fake SFTP server a virtual filesystem.
🤖 Generated with Claude Code
- Dominant language
- Python
- Stars
- 0
- Forks
- 1
- Avg merge
- 18h 37m
- Merged PRs (30d)
- 15
Getting set up
This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from IgniteRobotics/flashpoint
-
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
IgniteRobotics/flashpoint#45 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
IgniteRobotics/flashpoint#44 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
IgniteRobotics/flashpoint#43 ·
Maintainers usually reply within 1 day
-
bug good first issue
Difficulty 1/5 Under an hour Newbie friendliness 85/100
IgniteRobotics/flashpoint#12 ·
Maintainers usually reply within 1 day
-
tools/update-owlet-manifest.py ignores --help and starts downloadingMay be free again @yoveneim claimed this 11 days ago, and no pull request is open. Openbug good first issue
Difficulty 2/5 1-3 hours Newbie friendliness 83/100
IgniteRobotics/flashpoint#11 · 1 comment ·
Maintainers usually reply within 1 day
All issues in IgniteRobotics/flashpoint
Similar issues
-
bump patch
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
autowarefoundation/autoware_carla_scenario#1321 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
Maintainers usually reply within 1 day
-
Difficulty 1/5 1-3 hours Newbie friendliness 78/100
QuantConnect/Documentation#2838 ·
Maintainers usually reply within 1 day
-
get_all_subclasses must deduplicate by class identityPossibly taken @Gambit-Checkmate claimed this today. Open
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
Maintainers usually reply within 15 days
-
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
awslabs/route53-dynamic-dns-with-lambda#67 · 1 comment ·