-
-
Notifications
You must be signed in to change notification settings - Fork 18
Add test suite #9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
I have not used Typescript much, so excuse me if this is incorrect. But I feel we should stick to
|
@tejasbubane yes, I'm fine with
As long as we don't use |
Are fixtures used in any way right now? If yes where? If not whats the plan with them? |
Yes, these are the first 500 submissions to the exercise on the track. They can be used locally to run:
I use it to manually check if the right response is generated for an analyzer:
The plan is to incorporate them in the tests to automatically do that. Basically, as per above, a test could tag a sub-set with "exercise x, y and z should have output a, b, c" |
@depsir I see you've removed your comment. If you don't mind I'd still like to respond to it because it might help other wondering the same thing:
Since babel 7 and TypeScript 3+, Microsoft has been working to help people integrate their TypeScript tooling with their JavaScript tooling, among other things enabling you to "compile" typescript via Babel (7+), essentially by stripping away the type information nodes. This allows us to rely on babel for transpilation ánd typescript for type tooling instead of:
Facebook no longer recommends |
* Add smoke and snapshot test for two-fer Work towards #9 * Add helpers for smoke and snapshot tests * Add docs for smoke and snapshot tests
There is now a document on 🌫 smoke tests and 📸 snapshot tests. |
Fixed by #75 |
I prefer using something like ava because it's nice for our needs (runs everything randomly and concurrently by default, and requires us to compile it first, which is the first step of the code).
We should:
Tracking progress:
two-fer
resistor-color
resistor-color-duo
gigasecond
The text was updated successfully, but these errors were encountered: