Issue #416: test should be updated to reuse inlined to input violation messages#418
Issue #416: test should be updated to reuse inlined to input violation messages#418Brijeshthummar02 wants to merge 1 commit into
Conversation
0afa9fa to
2698f0a
Compare
this is a goal. |
|
we can upgrade tests step by step to avoid huge PRs |
…t violation messages
2698f0a to
8389108
Compare
So i made a small focused migration by moving violations from
Removed the corresponding Should further migration of remaining test bundles be continued in this PR or handled in separate follow-up PRs? |
For the testing part, i did some migration in this PR so you can check it. Once this get's merged i will migrate all remaining test bundles one by one. |
Issue #416
I updated
AbstractPatchFilterEvaluationTestto support inline violation messages from input files, while keeping backward compatibility with existingexpected.txtfiles.Before proceeding further, should this issue also include migration of existing test input bundles to inline violation comments and removal of corresponding
expected.txtfiles, or is the framework-level support sufficient for this PR?