Repository navigation
split test/parallel into subsystem folders #6217
Description
The /parallel/ tests directory is pretty huge, with 1010 tests at the time of writing.
So large, that GitHub has begun truncating it. And, it is pretty hard to scroll though to find something (depending on your editor of choice).
Perhaps we should split it into actual subsystem folders? Does the test runner allow for that?
It would be a bit of a headache again though, like when we moved /simple/ to /parallel/..
cc @nodejs/testing & @nodejs/ctc
Activity
jasnell commentedon Apr 15, 2016
Definite +1 on this, it's getting rather difficult to manage. Not sure if
the test runner supports it but would be easy to test. Definite headache
but worth doing.
On Apr 15, 2016 7:31 AM, "Jeremiah Senkpiel" notifications@github.com
wrote:
evanlucas commentedon Apr 15, 2016
I like it +1 from me
bengl commentedon Apr 15, 2016
I've got this working. Just need to fix the relative URLs in tests. PR coming soon.
indutny commentedon Apr 15, 2016
What is a main difficulty of managing it? (Note: I'm +0 myself so far, so would appreciate to learn some arguments)
trevnorris commentedon Apr 15, 2016
I don't see the need to change the current directory structure besides accommodating github. Am ±0 on whether this is important enough to warrant changing the entire structure again. except in a way that makes backporting tests more annoying.
evanlucas commentedon Apr 15, 2016
oooo that's a good point @trevnorris.
calvinmetcalf commentedon Apr 29, 2016
so this also has problems in some test editors, e.g. atom doesn't do awesome with that folder
Trott commentedon Jul 7, 2017
This issue has been inactive for sufficiently long that it seems like perhaps it should be closed. Feel free to re-open (or leave a comment requesting that it be re-opened) if you disagree. I'm just tidying up and not acting on a super-strong opinion or anything like that.
FWIW, the implementation in #6230 got some mixed reactions.