-
Notifications
You must be signed in to change notification settings - Fork 156
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
Add support for Expecto theory tests #1160
Conversation
As it stands, this change alone doesn't pick up the theory tests - The issue seems to be that in I don't know enough about it to say how it should be changed - e.g. just add a edit: or maybe it needs another |
|
||
<PropertyGroup> | ||
<OutputType>Exe</OutputType> | ||
<TargetFramework>netcoreapp3.1</TargetFramework> |
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.
testTheory is a feature of Expecto 10, which only supports .NET 6+, so I updated thise
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.
🎸 Thank you!!
FYI I fixed the AST stuff in ad2e8ac because yeah that stuff can get gnarly. |
Ok, VSCode is highlighting all my theory tests in the editor now with Ionide 7.11 :-) |
Co-authored-by: Richard Webb <[email protected]>
WHAT
Add support for Expecto 'theory' tests
WHY
haf/expecto#445 (comment)
HOW
Added extra cases to the list (to begin with, see other comments)