Skip to content
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
27 commits
Select commit Hold shift + click to select a range
7015e7e
sln builds and tests pass locally
JoshLozensky Aug 5, 2025
023fa4f
add the TargetNetNext var to relevant files and cleanup unneeded depe…
JoshLozensky Aug 5, 2025
035662b
address PR feedback and set targetnetnext as env var in ps scripts
JoshLozensky Aug 6, 2025
a1438ab
set env vars in ps scripts and readd net10 if statements
JoshLozensky Aug 7, 2025
577e2aa
Update runTests.ps1
JoshLozensky Aug 8, 2025
c050abb
Merge branch 'dev' into lozensky/TargetNet10Preview
JoshLozensky Aug 11, 2025
5d4335a
comment out TargetNetNext in targets files
JoshLozensky Aug 11, 2025
1572f8d
SuppressTfmSupportBuildErrors for NET 6 since microsoft.test.sdk no l…
JoshLozensky Aug 12, 2025
63e16ef
removing framework eol warning from benchmarks since net6 is eol
JoshLozensky Aug 12, 2025
ff25183
Update benchmarks to ignore net 6 eol
JoshLozensky Aug 12, 2025
a51216a
try readding System libraries to avoid vulnerable versions
JoshLozensky Aug 12, 2025
66c61d0
try adding package refs to avoid vulnerable versions
JoshLozensky Aug 12, 2025
a5d74f7
move package refs to come after project refs
JoshLozensky Aug 12, 2025
8e5e523
change useglobaljson to false so includePreviewVersions can be set
JoshLozensky Aug 12, 2025
0bd2812
move options to inputs
JoshLozensky Aug 12, 2025
eaf5c51
remove unneeded changes
JoshLozensky Aug 13, 2025
2227b89
block NU1510
JoshLozensky Aug 13, 2025
f768175
try rolling back global.json changes
JoshLozensky Aug 13, 2025
eaff530
put global.json changes back
JoshLozensky Aug 13, 2025
de15e83
minor edits
JoshLozensky Aug 19, 2025
54e1598
Update with PR feedback
JoshLozensky Aug 20, 2025
b7e3b07
Merge branch 'dev' into lozensky/TargetNet10Preview
JoshLozensky Aug 20, 2025
8279646
suppress sdk preview warning
JoshLozensky Aug 20, 2025
55c1ae9
remove unneeded supppression
JoshLozensky Aug 20, 2025
bd8eb1a
Suppress NU1510/5104
JoshLozensky Aug 22, 2025
2b8c1fd
remove unneeded suppression
JoshLozensky Aug 26, 2025
85b85ae
standardize usage of preprocessor directives
JoshLozensky Aug 26, 2025
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
set env vars in ps scripts and readd net10 if statements
  • Loading branch information
JoshLozensky committed Aug 7, 2025
commit a1438ab863f2893c44b01f3d6b57c417aa19ecc0
2 changes: 1 addition & 1 deletion .github/workflows/dotnetcore.yml
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ jobs:
regedit /s .\build\strongNameBypass.reg

- name: Run the tests
run: dotnet test Wilson.sln --collect:"XPlat Code Coverage" --settings:./build/CodeCoverage.runsettings --property:TargetNetNext=True
run: dotnet test Wilson.sln --collect:"XPlat Code Coverage" --settings:./build/CodeCoverage.runsettings

- name: Create code coverage report
run: |
Expand Down
4 changes: 2 additions & 2 deletions build.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -55,8 +55,8 @@ if ($env:VSINSTALLDIR)
$msbuildDir = $env:VSINSTALLDIR+"\MSBuild\Current\Bin";
}
}
env:
TargetNetNext: True

$env:TargetNetNext = "True"

WriteSectionHeader("build.ps1 - parameters");
Write-Host "buildType: " $buildType;
Expand Down
3 changes: 1 addition & 2 deletions runTests.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -30,8 +30,7 @@ function WriteSectionFooter($sectionName)

################################################# Functions ############################################################

env:
TargetNetNext: True
$env:TargetNetNext = "True"

WriteSectionHeader("runTests.ps1");
Write-Host "buildType: " $buildType;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -39,9 +39,7 @@ public static class IdentityModelTelemetryUtil
"ID_NET8_0";
#elif NET9_0
"ID_NET9_0";
#else
// We can't use NET10_0 here until after NET 10's release or Visual Studio will crash if NET 10 is not installed.
// In the tests we check to make sure the version running is actually NET 10.
#elif NET10_0
"ID_NET10_0";
#endif

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
<PackageId>Microsoft.IdentityModel.LoggingExtensions</PackageId>
<PackageTags>.NET;Windows;Authentication;Identity;Extensions;Logging</PackageTags>
<TargetFrameworks>netstandard2.0;net9.0</TargetFrameworks>
<TargetFrameworks Condition="'$(TargetNetNext)'== 'True'">$(TargetFrameworks);</TargetFrameworks>
<TargetFrameworks Condition="'$(TargetNetNext)'== 'True'">$(TargetFrameworks);net10.0</TargetFrameworks>
<Nullable>enable</Nullable>
</PropertyGroup>

Expand Down
3 changes: 1 addition & 2 deletions test.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -24,8 +24,7 @@ function WriteSectionFooter($sectionName)

################################################# Functions ############################################################

env:
TargetNetNext: True
$env:TargetNetNext = "True"

WriteSectionHeader("runTests.ps1");
Write-Host "buildType: " $buildType;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -288,19 +288,7 @@ public void OidcCreateAuthenticationRequestUrl(string testId, OpenIdConnectMessa
#elif NET9_0
if (!message.SkuTelemetryValue.Equals("ID_NET9_0"))
context.Diffs.Add($"{message.SkuTelemetryValue} != ID_NET9_0");
#else // we can't use elif NET10_0 or Visual Studio will crash if NET 10 is not installed
//TODO: Use elif NET10_0 after NET 10's release
System.Diagnostics.Process process = new System.Diagnostics.Process();
process.StartInfo.FileName = "dotnet.exe";
process.StartInfo.Arguments = "--version";
process.StartInfo.UseShellExecute = false;
process.StartInfo.RedirectStandardOutput = true;
process.Start();

string version = process.StandardOutput.ReadToEnd()?.TrimEnd();

process.WaitForExit();
Assert.StartsWith("10.", version, StringComparison.OrdinalIgnoreCase);
#elif NET10_0
if (!message.SkuTelemetryValue.Equals("ID_NET10_0"))
context.Diffs.Add($"{message.SkuTelemetryValue} != ID_NET10_0");
#endif
Expand Down Expand Up @@ -560,10 +548,7 @@ public void OidcCreateLogoutRequestUrl(string testId, OpenIdConnectMessage messa
TestUtilities.WriteHeader("OidcCreateLogoutRequestUrl - " + testId, true);

var context = new CompareContext();
#if NET462
if (!message.SkuTelemetryValue.Equals("ID_NET462"))
context.Diffs.Add($"{message.SkuTelemetryValue} != ID_NET462");
#elif NET472
#if NET472
if (!message.SkuTelemetryValue.Equals("ID_NET472"))
context.Diffs.Add($"{message.SkuTelemetryValue} != ID_NET472");
#elif NET6_0
Expand All @@ -575,19 +560,7 @@ public void OidcCreateLogoutRequestUrl(string testId, OpenIdConnectMessage messa
#elif NET9_0
if (!message.SkuTelemetryValue.Equals("ID_NET9_0"))
context.Diffs.Add($"{message.SkuTelemetryValue} != ID_NET9_0");
#else // we can't use elif NET10_0 or Visual Studio will crash if NET 10 is not installed
//TODO: Use elif NET10_0 after NET 10's release
System.Diagnostics.Process process = new System.Diagnostics.Process();
process.StartInfo.FileName = "dotnet.exe";
process.StartInfo.Arguments = "--version";
process.StartInfo.UseShellExecute = false;
process.StartInfo.RedirectStandardOutput = true;
process.Start();

string version = process.StandardOutput.ReadToEnd()?.TrimEnd();

process.WaitForExit();
Assert.StartsWith("10.", version, StringComparison.OrdinalIgnoreCase);
#elif NET10_0
if (!message.SkuTelemetryValue.Equals("ID_NET10_0"))
context.Diffs.Add($"{message.SkuTelemetryValue} != ID_NET10_0");
#endif
Expand Down
Loading