-
Notifications
You must be signed in to change notification settings - Fork 369
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
Cover TemplateSamples with tests #5794
Cover TemplateSamples with tests #5794
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.
Looks really great!
First application of the new testing FW - so no option to get proper inspiration (this code is going to be the new inspiration!) - good job!
I have few minor coments to make this even more shiny
...soft.TemplateEngine.Authoring.TemplateVerifier.IntegrationTests/TemplateEngineSamplesTest.cs
Outdated
Show resolved
Hide resolved
...soft.TemplateEngine.Authoring.TemplateVerifier.IntegrationTests/TemplateEngineSamplesTest.cs
Show resolved
Hide resolved
...soft.TemplateEngine.Authoring.TemplateVerifier.IntegrationTests/TemplateEngineSamplesTest.cs
Outdated
Show resolved
Hide resolved
...soft.TemplateEngine.Authoring.TemplateVerifier.IntegrationTests/TemplateEngineSamplesTest.cs
Outdated
Show resolved
Hide resolved
…ub.com/YuliiaKovalova/templating_change_conditional_sntx into dev/ykovalova/cover_samples_with_tests
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.
Couple of comments
...IntegrationTests/Microsoft.TemplateEngine.Authoring.TemplateVerifier.IntegrationTests.csproj
Outdated
Show resolved
Hide resolved
...soft.TemplateEngine.Authoring.TemplateVerifier.IntegrationTests/TemplateEngineSamplesTest.cs
Outdated
Show resolved
Hide resolved
...soft.TemplateEngine.Authoring.TemplateVerifier.IntegrationTests/TemplateEngineSamplesTest.cs
Outdated
Show resolved
Hide resolved
...soft.TemplateEngine.Authoring.TemplateVerifier.IntegrationTests/TemplateEngineSamplesTest.cs
Outdated
Show resolved
Hide resolved
...soft.TemplateEngine.Authoring.TemplateVerifier.IntegrationTests/TemplateEngineSamplesTest.cs
Show resolved
Hide resolved
...tionTests/Snapshots/TemplateEngineSamplesProjectTest.sample02._.verified/sample02/Program.cs
Outdated
Show resolved
Hide resolved
...soft.TemplateEngine.Authoring.TemplateVerifier.IntegrationTests/TemplateEngineSamplesTest.cs
Outdated
Show resolved
Hide resolved
I have addressed all your comments. Please take a look |
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.
Looks good
...soft.TemplateEngine.Authoring.TemplateVerifier.IntegrationTests/TemplateEngineSamplesTest.cs
Outdated
Show resolved
Hide resolved
...soft.TemplateEngine.Authoring.TemplateVerifier.IntegrationTests/TemplateEngineSamplesTest.cs
Outdated
Show resolved
Hide resolved
...soft.TemplateEngine.Authoring.TemplateVerifier.IntegrationTests/TemplateEngineSamplesTest.cs
Outdated
Show resolved
Hide resolved
Co-authored-by: Vlada Shubina <[email protected]>
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.
Perfect!
Related to #2522 (comment)