Skip to content

Commit b16dd92

Browse files
authored
[wasm] Enable tests disabled by preloading & compression endpoint issue (#48663)
1 parent 98eef97 commit b16dd92

File tree

4 files changed

+9
-16
lines changed

4 files changed

+9
-16
lines changed

test/Microsoft.NET.Sdk.BlazorWebAssembly.Tests/BlazorWasmStaticWebAssetsIntegrationTest.cs

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,7 @@ namespace Microsoft.NET.Sdk.BlazorWebAssembly.Tests
99
{
1010
public class BlazorWasmStaticWebAssetsIntegrationTest(ITestOutputHelper log) : BlazorWasmBaselineTests(log, GenerateBaselines)
1111
{
12-
// [RequiresMSBuildVersionFact("17.12", Reason = "Needs System.Text.Json 8.0.5")]
13-
[Fact(Skip = "https://github.com/dotnet/runtime/issues/114915")]
12+
[RequiresMSBuildVersionFact("17.12", Reason = "Needs System.Text.Json 8.0.5")]
1413
public void StaticWebAssets_BuildMinimal_Works()
1514
{
1615
// Arrange
@@ -52,8 +51,7 @@ public void StaticWebAssets_BuildMinimal_Works()
5251
intermediateOutputPath);
5352
}
5453

55-
// [RequiresMSBuildVersionFact("17.12", Reason = "Needs System.Text.Json 8.0.5")]
56-
[Fact(Skip = "https://github.com/dotnet/runtime/issues/114915")]
54+
[RequiresMSBuildVersionFact("17.12", Reason = "Needs System.Text.Json 8.0.5")]
5755
public void StaticWebAssets_PublishMinimal_Works()
5856
{
5957
// Arrange

test/Microsoft.NET.Sdk.BlazorWebAssembly.Tests/WasmBuildIntegrationTest.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -562,7 +562,7 @@ public void Build_Hosted_Works()
562562
new FileInfo(Path.Combine(buildOutputDirectory, "wwwroot", "_framework", "_bin", "blazorwasm.wasm")).Should().NotExist();
563563
}
564564

565-
[Fact(Skip = "https://github.com/dotnet/runtime/issues/114915")]
565+
[Fact]
566566
public void Build_SatelliteAssembliesAreCopiedToBuildOutput()
567567
{
568568
// Arrange

test/Microsoft.NET.Sdk.BlazorWebAssembly.Tests/WasmJsModulesIntegrationTests.cs

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,7 @@ namespace Microsoft.NET.Sdk.BlazorWebAssembly.Tests
1111
{
1212
public class WasmJsModulesIntegrationTests(ITestOutputHelper log) : BlazorWasmBaselineTests(log, GenerateBaselines)
1313
{
14-
//[RequiresMSBuildVersionFact("17.12", Reason = "Needs System.Text.Json 8.0.5")]
15-
[Fact(Skip = "https://github.com/dotnet/runtime/issues/114915")]
14+
[RequiresMSBuildVersionFact("17.12", Reason = "Needs System.Text.Json 8.0.5")]
1615
public void Build_DoesNotGenerateManifestJson_IncludesJSModulesOnBlazorBootJsonManifest()
1716
{
1817
// Arrange
@@ -96,8 +95,7 @@ public void JSModules_ManifestIncludesModuleTargetPaths()
9695
new FileInfo(Path.Combine(outputPath, "wwwroot", "blazorhosted.modules.json")).Should().NotExist();
9796
}
9897

99-
//[RequiresMSBuildVersionFact("17.12", Reason = "Needs System.Text.Json 8.0.5")]
100-
[Fact(Skip = "https://github.com/dotnet/runtime/issues/114915")]
98+
[RequiresMSBuildVersionFact("17.12", Reason = "Needs System.Text.Json 8.0.5")]
10199
public void Publish_DoesNotGenerateManifestJson_IncludesJSModulesOnBlazorBootJsonManifest()
102100
{
103101
// Arrange
@@ -144,8 +142,7 @@ public void Publish_DoesNotGenerateManifestJson_IncludesJSModulesOnBlazorBootJso
144142
intermediateOutputPath);
145143
}
146144

147-
//[RequiresMSBuildVersionFact("17.12", Reason = "Needs System.Text.Json 8.0.5")]
148-
[Fact(Skip = "https://github.com/dotnet/runtime/issues/114915")]
145+
[RequiresMSBuildVersionFact("17.12", Reason = "Needs System.Text.Json 8.0.5")]
149146
public void JsModules_CanHaveDifferentBuildAndPublishModules()
150147
{
151148
// Arrange
@@ -217,7 +214,7 @@ static JsonElement GetLibraryInitializer(string path)
217214
}
218215
}
219216

220-
[Fact(Skip = "https://github.com/dotnet/runtime/issues/114915")]
217+
[Fact]
221218
public void JsModules_CanCustomizeBlazorInitialization()
222219
{
223220
// Arrange

test/Microsoft.NET.Sdk.Razor.Tests/StaticWebAssetEndpointsIntegrationTest.cs

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -409,8 +409,7 @@ public void Publish_CreatesEndpointsForAssets_BuildAndPublish_Assets()
409409
AssertManifest(publishManifest, LoadPublishManifest());
410410
}
411411

412-
//[RequiresMSBuildVersionFact("17.12", Reason = "Needs System.Text.Json 8.0.5")]
413-
[Fact(Skip = "https://github.com/dotnet/runtime/issues/114915")]
412+
[RequiresMSBuildVersionFact("17.12", Reason = "Needs System.Text.Json 8.0.5")]
414413
public void Build_EndpointManifest_ContainsEndpoints()
415414
{
416415
// Arrange
@@ -463,8 +462,7 @@ public void BuildHosted_EndpointManifest_ContainsEndpoints()
463462
VerifyEndpointsCollection(buildOutputDirectory, "blazorhosted", readFromDevManifest: true);
464463
}
465464

466-
//[RequiresMSBuildVersionFact("17.12", Reason = "Needs System.Text.Json 8.0.5")]
467-
[Fact(Skip = "https://github.com/dotnet/runtime/issues/114915")]
465+
[RequiresMSBuildVersionFact("17.12", Reason = "Needs System.Text.Json 8.0.5")]
468466
public void Publish_EndpointManifestContainsEndpoints()
469467
{
470468
// Arrange

0 commit comments

Comments
 (0)