Repository navigation
Move stdlib tests from src/test/ui-fulldeps to library/std/tests #99417
Description
Activity
- addedA-testsuiteArea: The testsuite used to check the correctness of rustcArea: The testsuite used to check the correctness of rustcC-enhancementCategory: An issue proposing an enhancement or a PR with one.Category: An issue proposing an enhancement or a PR with one.A-contributor-roadblockArea: Makes things more difficult for new or seasoned contributors to RustArea: Makes things more difficult for new or seasoned contributors to RustT-libsRelevant to the library team, which will review and decide on the PR/issue.Relevant to the library team, which will review and decide on the PR/issue.E-easyCall for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.
on Apr 9, 2023 Hi, is this no longer an issue?
this is still an issue.
; ls tests/ui/std issue-15149.rs issue-81357-unsound-file-methods.rs stdio-from.rs switch-stdout.rsissue-81357-unsound-file-methods.rs
OK, is the
issue-81357-unsound-file-methods.rsfile also included? I'll try to resolve this issue@rustbot claim
Reacted by jynHi, I managed to move the
switch-stdout.rsfile but I have a problem with thestdio-from.rsfile, I can't see how to use Stdio without going through the command system? Am I right?6 remaining items
@MoskalykA sorry for the delayed response but are you still interested in the other tests that are missing?
sorry for the delayed response but are you still interested in the other tests that are missing?
Yes, as suggested by
petrochenkov, tests from thetests\ui\stdlib-unit-testsfolder should be moved. But wouldlibrary/std/testsas destination be correct?I can look on it but i need to know what am i supposed to do.
@rustbot claim
Is this issue fully complete? what's remaining? I would love to take over in prompt time!
Is this issue fully complete? what's remaining? I would love to take over in prompt time!
This is probably what needs investigating at the moment, i.e. the current status of this issue.
Reacted by Shivanshu Bansal- addedE-needs-investigationCall for participation: This issues needs some investigation to determine current statusCall for participation: This issues needs some investigation to determine current status
on Nov 27, 2024 Sorry I've not been keeping up with the issue.
I have now checked and it seems there are no longer any std tests in the ui-fulldeps directory so I'm closing this as complete.
Reacted by 许杰友 Jieyou Xu (Joe) and Shivanshu Bansal
I think std tests should ideally live in the
library/stddirectory but today we have a few that are insrc/test/ui-fulldeps.Tests I think could be moved: