Open
Conversation
ad1f4de to
c412933
Compare
phenning
requested changes
Feb 20, 2026
...affolding/dotnet-scaffold.Tests/AspNet/Integration/Identity/IdentityNet11IntegrationTests.cs
Outdated
Show resolved
Hide resolved
| Assert.True(settings.Overwrite); | ||
| } | ||
|
|
||
| [Fact] |
Member
There was a problem hiding this comment.
what are any of these actually testing, IdentitySettings just consists of getters and setters, there is no logic in them
...affolding/dotnet-scaffold.Tests/AspNet/Integration/Identity/IdentityNet11IntegrationTests.cs
Outdated
Show resolved
Hide resolved
...affolding/dotnet-scaffold.Tests/AspNet/Integration/Identity/IdentityNet10IntegrationTests.cs
Outdated
Show resolved
Hide resolved
Member
Author
|
I am going to update so that the files are actually generated in the different projects after scaffolding and before build, right now it just builds, scaffolds, builds but the files aren't actually added |
631c2a2 to
c028c9f
Compare
8150b39 to
4ed9ebe
Compare
0cfae0d to
fe3906b
Compare
fe3906b to
7584c9b
Compare
phenning
approved these changes
Mar 6, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
For all the integration tests, a project is created and built before the scaffolder runs and then each project is built after the scaffolder and the files' existence is checked. For each scaffolder these tests exist for net 8, net 9, net 10, net 11 projects
adds integration tests for AspNet core identity, net 8, net 9, net 10, net 11
fixes Add Asp Net Core Identity net 8 integration tests #3609 fixes Add Asp Net Core Identity net 9 integration tests #3610 fixes Add Asp Net Core Identity net 10 integration tests #3611 fixes Add Asp Net Core Identity net 11 integration tests #3612
adds integration tests for MVC Area, net 8, net 9, net 10, net 11
fixes Add MVC Area net 8 integration tests #3613 fixes Add MVC Area net 9 integration tests #3614 fixes Add MVC Area net 10 integration tests #3615 fixes Add MVC Area net 11 integration tests #3616
adds integration tests for MVC Controller, net 8, net 9, net 10, net 11
fixes Add MVC Controller net 8 integration tests #3617 fixes Add MVC Controller net 9 integration tests #3618 fixes Add MVC Controller net 10 integration tests #3619 fixes Add MVC Controller net 11 integration tests #3620
adds integration tests for MVC Controller Entity Framework with CRUD, net 8, net 9, net 10, net 11
fixes Add MVC Controller with entity framework CRUD net 8 integration tests #3621 fixes Add MVC Controller with entity framework CRUD net 9 integration tests #3622 fixes Add MVC Controller with entity framework CRUD net 10 integration tests #3623 fixes Add MVC Controller with entity framework CRUD net 11 integration tests #3624
adds integration tests for Razor View Empty net 8, net 9, net 10, net 11
fixes Add Razor View Empty net 8 integration tests #3625 fixes Add Razor View Empty net 9 integration tests #3626 fixes Add Razor View Empty net 10 integration tests #3627 fixes Add Razor View Empty net 11 integration tests #3628
adds integration tests for Razor Views net 8, net 9, net 10, net 11
fixes Add Razor Views net 8 integration tests #3629 fixes Add Razor Views net 9 integration tests #3630 fixes Add Razor Views net 10 integration tests #3631 fixes Add Razor Views net 11 integration tests #3632
adds integration tests for Razor Page Empty net 8, net 9, net 10, net 11
fixes Add Razor Page Empty net 8 integration tests #3633 fixes Add Razor Page Empty net 9 integration tests #3634 fixes Add Razor Page Empty net 10 integration tests #3635 fixes Add Razor Page Empty net 11 integration tests #3636
adds integration tests for Razor Page with Entity Framework net 8, net 9, net 10, net 11
fixes Add Razor Page with Entity Framework 8 integration tests #3637 fixes Add Razor Page with Entity Framework 9 integration tests #3638 fixes Add Razor Page with Entity Framework 10 integration tests #3639 fixes Add Razor Page with Entity Framework 11 integration tests #3640