Skip to content
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

Improve snippet testing #524

Merged
merged 13 commits into from
May 5, 2024
Merged

Improve snippet testing #524

merged 13 commits into from
May 5, 2024

Conversation

MateuszKubuszok
Copy link
Member

@MateuszKubuszok MateuszKubuszok commented May 2, 2024

TODO:

  • separate Chimney-specific things from something that could be made a separate library
  • use colors to distinct tests results
  • test errors
  • automate the detection of strategies
    • snippets with pseudocode would contain // this is a pseudocode or similar
    • snippets testing errors should contain // should produce the following error
  • turn each .md file into a separate test suite
  • allow filtering tests with a simple filter (sth like testOnly)

Copy link

codecov bot commented May 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.60%. Comparing base (c64c690) to head (0344a63).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #524   +/-   ##
=======================================
  Coverage   87.60%   87.60%           
=======================================
  Files         136      136           
  Lines        5488     5488           
  Branches      440      440           
=======================================
  Hits         4808     4808           
  Misses        680      680           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@MateuszKubuszok MateuszKubuszok force-pushed the improve-snippet-testing branch from 258b070 to a39f29e Compare May 2, 2024 13:23
@MateuszKubuszok MateuszKubuszok force-pushed the improve-snippet-testing branch from a39f29e to 6a2fd47 Compare May 2, 2024 15:36
@MateuszKubuszok MateuszKubuszok force-pushed the improve-snippet-testing branch from 1ac2152 to d5a2112 Compare May 3, 2024 12:12
@MateuszKubuszok MateuszKubuszok force-pushed the improve-snippet-testing branch from e5fbccd to 6552397 Compare May 3, 2024 23:17
@MateuszKubuszok MateuszKubuszok force-pushed the improve-snippet-testing branch from 65cbfde to 59c63a5 Compare May 5, 2024 16:19
@MateuszKubuszok MateuszKubuszok marked this pull request as ready for review May 5, 2024 16:19
@MateuszKubuszok MateuszKubuszok merged commit 82637f1 into master May 5, 2024
24 checks passed
@MateuszKubuszok MateuszKubuszok deleted the improve-snippet-testing branch May 5, 2024 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant