Fix yaml snapshot specification with data tests #11156
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Resolves #11110
Problem
We were writing the compiled version of the snapshot node to the yaml file name, so that when generic tests were written out the "directory" was a file. In addition multiple snapshots nodes would have been written to the same output file.
Solution
Construct the yaml snapshot compiled path including the "name" + ".sql" of the snapshot node.
Checklist