Split `boa_tester` into crates
jedel1043 opened this issue · comments
Now that temporal_rs
is a thing, we need a way to test it against test262. We could reimplement boa_tester
but it would be better to extract the test extraction part from the test runner, and then reuse that for temporal_rs
.
Hi, I would like to contribute. Could you tell me if I'm going in the right direction?
I'm not sure if I'm understanding use case for this new crate, you want to extract all parsing logic from boa_tester
to another crate so extraction/parsing logic could be reused. I'm not sure if I understand "extraction part" correctly.
@PawelJastrzebski Thank you for the help! Yeah, that was mostly the idea: separate the test262 parser from the main program. Though, I'd recommend also including the git module inside test262_parser
, maybe with some interface to be able to either download the repo or pass a manual path to a valid test262
directory.
Hello, @jedel1043. Could you take a look at PR #3708. It's probably not perfect, but I guess it's step in right direction.
So even if it happen not to close this issue, maybe it could be merged at this stage, as it easier to maintain/merge and review smaller batches of changes.