-
Notifications
You must be signed in to change notification settings - Fork 10.3k
[Helix] Enable Hosting/DefautBuilder/ServerComparison.FunctionalTests #8248
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
Conversation
Currently results in errror:
|
@@ -28,7 +29,8 @@ public async Task InjectedStartup_DefaultApplicationNameIsEntryAssembly(TestVari | |||
{ | |||
var logger = loggerFactory.CreateLogger(nameof(InjectedStartup_DefaultApplicationNameIsEntryAssembly)); | |||
|
|||
var applicationPath = Path.Combine(TestPathUtilities.GetSolutionRootDirectory("Hosting"), "test", "testassets", "IStartupInjectionAssemblyName"); | |||
var baseDirectory = SkipOnHelixAttribute.OnHelix() ? AppContext.BaseDirectory : Path.Combine(TestPathUtilities.GetSolutionRootDirectory("Hosting"), "test", "testassets"); |
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.
Calling the skip attribute as a helper library? 👃
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.
Yeah need a centralized helper for sure, the universe of these tests that need to do different stuff on helix will drive that, its in a shared source file for now, so it was easiest to just expose it there as a starting point. Note: I'm just opening these PRs as starting points for the feature teams to be investigate. These are 'repro' PRs
</PropertyGroup> | ||
|
||
<ItemGroup> | ||
<Content Include="testroot\**\*" CopyToOutputDirectory="PreserveNewest" CopyToPublishDirectory="PreserveNewest" /> | ||
<HelixContent Include="..\testassets\**\*" /> |
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.
I don't think we should compile on helix machines. We should instead try to generalize using pre-published assets.
See #7027
Looks like an ancient draft. @HaoK I'm going to close this on the assumption it isn't worth keeping around right now. Feel free to re-open if I'm wrong :) |
Starting point PR for #8247