Skip to content
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

Test project templates #201

Closed
codito opened this issue Oct 3, 2016 · 4 comments
Closed

Test project templates #201

codito opened this issue Oct 3, 2016 · 4 comments
Assignees
Labels
Milestone

Comments

@codito
Copy link
Contributor

codito commented Oct 3, 2016

(This issue is a discussion follow up to the initial PR for test templates).

Following are the requirements for the test template:

  1. Emit runtimeconfig.*.json on build. This is required for the dotnet vstest and dotnet test3 commands to setup the --additionalProbingPath for dotnet exec so that nuget references are resolved.
  2. Allow P2P reference to all user projects (both class libraries and console apps).
  3. Allow publish of the test project. This will allow user to run tests in a machine which doesn't have a source code enlistment e.g. a CDP flow, where tests are run in a Release Definition (usually acceptance tests on a Unix or Windows machine with her product installed)
  4. Test project shouldn't include a program entry point. The entry point is always the testhost which comes from the test platform (see Microsoft.TestPlatform.TestHost nuget package).

If we choose a Console App based project template, (4) may need work.
If we choose a Class Library based project template, (4) is met, but 1,3 may need work.

Notes
(1) was supported with a special check in older project model (see Microsoft.Dotnet.ProjectModel).
(4) was supported with emitEntryPoint: false in project.json.

@codito
Copy link
Contributor Author

codito commented Oct 3, 2016

@eerhardt @livarcocc @piotrpMSFT @srivatsn thoughts?

@livarcocc
Copy link
Contributor

@codito We discussed this today and we believe that the best approach is to go the console app approach.

@srivatsn srivatsn added the Bug label Oct 12, 2016
@srivatsn srivatsn added this to the 1.0 Preview 5 milestone Oct 12, 2016
@srivatsn
Copy link
Contributor

Since we are going with the console app approach, we should generate the entry point through the targets I think and not require the templates to have a dummy main method.

@srivatsn
Copy link
Contributor

srivatsn commented Nov 3, 2016

This is done.

@srivatsn srivatsn closed this as completed Nov 3, 2016
GangWang01 pushed a commit to GangWang01/sdk that referenced this issue Mar 20, 2023
)

* document all remaining items and properties

* Add first docs on registry auth

* add note to getting started about console apps

* Update docs/ContainerCustomization.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants