CatchTheTornado / CatchTheTornado/askql

.ask tests should be run by default

Open
#166 15 comments 0 reactions 0 assignees View on GitHub
CI/TDD/DevOps enhancement
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.