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

Design a framework for testing the SDKs #29

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from
Draft

Conversation

Nirmal4G
Copy link
Contributor

@Nirmal4G Nirmal4G commented Jun 26, 2022

Goals

  • Ability to create templated projects for testing.
  • Ability to distinguish between various SDKs used.
  • Ability to gather and process both text and binary logs.
  • Ability to call targets and inject tasks programmatically.
  • Ability to gather properties and items programmatically.
  • Ability to test against multiple versions of MSBuild programmatically and through build-infra.

TBA

Add Skeletal sources based on `Microsoft.NET.TestFramework` design.
@Nirmal4G Nirmal4G changed the title Design a Test Framework for testing the SDKs Design a framework for testing the SDKs Jun 26, 2022
@Nirmal4G Nirmal4G self-assigned this Jun 26, 2022
@Nirmal4G Nirmal4G changed the base branch from develop to master August 21, 2022 06:52
@Nirmal4G Nirmal4G changed the base branch from master to develop August 21, 2022 06:53
@Nirmal4G Nirmal4G force-pushed the feature/tests branch 2 times, most recently from 28636e8 to 764af8b Compare October 7, 2022 13:54
@jzabroski
Copy link

🔥

@jzabroski
Copy link

@Nirmal4G I have asked @rainersigwald and @Forgind to think about integration testing MSBuildLocator. I think that would be a good place to add testing , as I dont see how a SDK that invokes MSBUild task can be fully tested without a locator test.

@Nirmal4G
Copy link
Contributor Author

I'm using MSBuild locator in one of my prototypes. I also want to test against latest 2 major versions. So, In another prototype, I'm using PackageDownload feature but the issue is the lack of support for the GeneratePathProperty attribute.

There are a whole lot of changes after this but I want to settle the API surface first before committing the work that lead into either of the solutions above. So, expect a long delay before I push some actual code.

For now, everything will be a placeholder until I flesh-out the design and layering.

@Nirmal4G
Copy link
Contributor Author

If you have asked the team, then is there an issue/discussion for that? I would very much like to upvote it and drive it further.

@nirin nirin deleted a comment Oct 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants