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
Fixes #21048: Improve rudderc testing system #4289
Fixes #21048: Improve rudderc testing system #4289
Conversation
don't merge for now |
PR updated with a new commit |
@@ -0,0 +1,7 @@ | |||
@format=0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure about the naming of this one.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we can split the refactoring and the improvement of the tests
@format=0 | ||
resource x() { | ||
let varname = { | ||
"key1": "value1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing a nested struct test if supported
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same here
👍 It is a hundred times more readable this way! |
PR updated with a new commit |
I finished porting everything as cargo tests:
|
I still have random tests failures in techniques I'll investigate. |
PR updated with a new commit |
PR updated with a new commit |
PR updated with a new commit |
PR updated with a new commit |
PR updated with a new commit |
PR updated with a new commit |
PR updated with a new commit |
PR updated with a new commit |
OK, squash merging this PR |
ae3770d
to
7d0a0f2
Compare
https://issues.rudder.io/issues/21048