CatchTheTornado / CatchTheTornado/askql
.ask tests should be run by default
- Dominant language
- TypeScript
- Stars
- 387
- Forks
- 30
- PR merge metrics
- No merged PRs in 30d
Description
@km4 wrote on [Discord channel](https://discord.com/channels/688128418914500838/704117328735895662/721036181147615284):
> if create test and write "foobarbaz" in test in *.ask file the test passed :stuck_out_tongue:
The reason is that the .ask test is _parsed only_. It will be run only if an additional *.test.result.ts file with an expected result value is provided.
I also found it confusing and vote for running the .ask programs every time, no matter whether *.test.result.ts exists or not.
The existence of *.test.result.ts would just result in comparing the result value.
Contributor guide
Research direction
Locate the test runner and the handling for *.ask files, then compare it with the optional *.test.result.ts path described in the issue. Run an existing .ask test without a result file to confirm the current behavior. Done means every .ask program executes by default, while *.test.result.ts remains optional and only adds result comparison.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- testing
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100