-
Notifications
You must be signed in to change notification settings - Fork 72
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
Convert parser tests to suite format, part 3 #1284
Conversation
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.
One test change in parse-toplevel, otherwise good!
error: Malformed monomophize attribute | ||
--> test:3:0 | ||
| | ||
3 | (monomorphize 5) |
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.
Error message: This should probably be either "unexpected trailing form", pointing at only 5, or something monomorphize specific like "monomorphize takes no arguments"
error: Unknown repr attribute | ||
--> test:3:6 | ||
| | ||
3 | (repr :hello) |
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.
(repr :hello)
should be added, right? /s
|
||
(package coalton-unit-tests) | ||
|
||
(monomorphize) |
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.
(monomorphize)
(declare x UFix)
(define x 2)
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.
done
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.
you can take off (:disable)
:)
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.
got it!
2ab00cc
to
a1237ca
Compare
- convert and, attribute, body, break, cond, continue, declare, let, toplevel
Merge individual test cases into per-function files according to model established in #1274, #1278
Several test cases which parse but do not compile are disabled pending review according to comments in #1272, #1274, #1278