Add tests #3

Merged

boranseckin merged 61 commits from test into dev

2023-06-21 21:25:46 -04:00

boranseckin commented

2023-04-28 12:43:13 -04:00

(Migrated from github.com)

Implement tests from munificent/craftinginterpreters.

This PR includes:

minor bug fixes that were revealed during testing
a new testing infrastructure that supports both valid and erroneous code
a bunch of tests (obviously)

Implement tests from [munificent/craftinginterpreters](https://github.com/munificent/craftinginterpreters/tree/master/test). This PR includes: - minor bug fixes that were revealed during testing - a new testing infrastructure that supports both valid and erroneous code - a bunch of tests (obviously)

codecov[bot] commented

2023-04-28 12:57:52 -04:00

(Migrated from github.com)

Codecov Report

❗ No coverage uploaded for pull request base (dev@062b0a8). Click here to learn what that means.
Patch has no changes to coverable lines.

❗ Current head fd330bc differs from pull request most recent head 216f5c2. Consider uploading reports for the commit 216f5c2 to get more accurate results

Additional details and impacted files

@@          Coverage Diff           @@
##             dev       #3   +/-   ##
======================================
  Coverage       ?   68.64%           
======================================
  Files          ?       16           
  Lines          ?     1311           
  Branches       ?        0           
======================================
  Hits           ?      900           
  Misses         ?      411           
  Partials       ?        0

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

## [Codecov](https://app.codecov.io/gh/boranseckin/rocks/pull/3?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Boran+Seckin) Report > :exclamation: No coverage uploaded for pull request base (`dev@062b0a8`). [Click here to learn what that means](https://docs.codecov.io/docs/error-reference?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Boran+Seckin#section-missing-base-commit). > Patch has no changes to coverable lines. > :exclamation: Current head fd330bc differs from pull request most recent head 216f5c2. Consider uploading reports for the commit 216f5c2 to get more accurate results <details><summary>Additional details and impacted files</summary> ```diff @@ Coverage Diff @@ ## dev #3 +/- ## ====================================== Coverage ? 68.64% ====================================== Files ? 16 Lines ? 1311 Branches ? 0 ====================================== Hits ? 900 Misses ? 411 Partials ? 0 ``` </details> [:umbrella: View full report in Codecov by Sentry](https://app.codecov.io/gh/boranseckin/rocks/pull/3?src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Boran+Seckin). :loudspeaker: Do you have feedback about the report comment? [Let us know in this issue](https://about.codecov.io/codecov-pr-comment-feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Boran+Seckin).