-
-
Notifications
You must be signed in to change notification settings - Fork 111
feat: add test project to verify incremental source generators #4592
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
feat: add test project to verify incremental source generators #4592
Conversation
cab8871 to
48bb006
Compare
SummaryAdds a new test project to verify incremental source generator behavior for the MethodAssertionGenerator. Critical Issues1. VSTest Dependency Violation (BLOCKING) Location: Fix: Replace xunit with TUnit for the test framework. Look at existing test projects like 2. Missing Snapshot Test Updates While these are incremental behavior tests (not output tests), if any test failures produce 3. Target Framework Inconsistency Location: Fix: Import <Import Project="../TestProject.props" />Then remove the explicit Suggestions
Previous Review StatusUnable to retrieve previous review comments due to API scope limitations. Verdict |
48bb006 to
8eeeb4f
Compare
SelfRegisteredExtensions.cs(15, 9): [CS0234] The type or namespace name 'Framework' does not exist in the namespace 'TUnit.Engine' (are you missing an assembly reference?)
Tried switching to TUnit, not sure if they run was going to see if CI liked it Updated the .sln file by blindly accepting everything, hopefully it's fine |
8eeeb4f to
7acfb9a
Compare
7acfb9a to
d45d9f5
Compare
|
Can't get it to run with TUnit, I'm absolutely missing something here, but at least it runs 🤷 |
I've made the methods a little more generic so It can be used with different generators.
Follow up to #4563