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

test(parser): add initial tests #22

Merged
merged 4 commits into from Oct 10, 2022
Merged

test(parser): add initial tests #22

merged 4 commits into from Oct 10, 2022

Conversation

raymondmuller
Copy link
Contributor

@raymondmuller raymondmuller commented Oct 8, 2022

Already found an issue when writing these simple tests.

_FAIL packages/parser/test/if-else.test.ts > if / else
AssertionError: expected '5 > x' to deeply equal '5 > state.x'
Test Files 1 failed | 5 passed (6)
Tests 1 failed | 11 passed (12)

  • Expected "5 > state.x"
  • Received "5 > x"_

We miss this during post-processing it seems. Opening an issue for this, so ignore that failing test for now

@netlify
Copy link

netlify bot commented Oct 8, 2022

Deploy Preview for sveltosis ready!

Name Link
🔨 Latest commit 3dc46a1
🔍 Latest deploy log https://app.netlify.com/sites/sveltosis/deploys/63441d7efd51ce000b2e8abb
😎 Deploy Preview https://deploy-preview-22--sveltosis.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@ajanes93 ajanes93 merged commit 07593d2 into main Oct 10, 2022
@ajanes93 ajanes93 deleted the test/add-tests branch October 10, 2022 13:27
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.

None yet

2 participants