Skip to content

Commit

Permalink
[wasm] Switch off HybridGlobalization lane on v8 on CI (#101671)
Browse files Browse the repository at this point in the history
* Stop testing on v8.

* Try to block by scenario,

* Limit scenarios.

This reverts commit 38f64f7.
  • Loading branch information
ilonatommy authored Apr 30, 2024
1 parent 3fac575 commit 2ad47cd
Show file tree
Hide file tree
Showing 3 changed files with 16 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -197,7 +197,6 @@ jobs:
isWasmOnlyBuild: ${{ parameters.isWasmOnlyBuild }}
alwaysRun: true
scenarios:
- WasmTestOnV8
- WasmTestOnChrome
- WasmTestOnFirefox
- WasmTestOnNodeJS
Expand All @@ -215,7 +214,6 @@ jobs:
isWasmOnlyBuild: ${{ parameters.isWasmOnlyBuild }}
alwaysRun: true
scenarios:
- WasmTestOnV8
- WasmTestOnChrome
- WasmTestOnNodeJS

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,14 @@
<AllowUnsafeBlocks>true</AllowUnsafeBlocks>
<TestRuntime>true</TestRuntime>
<HybridGlobalization>true</HybridGlobalization>
</PropertyGroup>
<PropertyGroup Condition="'$(TargetOS)' == 'browser'">
<!-- This doesn't run on V8 because https://github.com/dotnet/runtime/pull/101671 -->
<Scenario>WasmTestOnChrome</Scenario>
<TestArchiveTestsRoot>$(TestArchiveRoot)browserornodejs/</TestArchiveTestsRoot>
<TestArchiveTestsDir>$(TestArchiveTestsRoot)$(OSPlatformConfig)/</TestArchiveTestsDir>
<DefineConstants>$(DefineConstants);TARGET_BROWSER</DefineConstants>
<XunitShowProgress>true</XunitShowProgress>
</PropertyGroup>
<ItemGroup>
<Compile Include="..\System\Globalization\CalendarTestBase.cs" />
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,14 @@
<IncludeRemoteExecutor>true</IncludeRemoteExecutor>
<HybridGlobalization>true</HybridGlobalization>
</PropertyGroup>
<PropertyGroup Condition="'$(TargetOS)' == 'browser'">
<!-- This doesn't run on V8 because https://github.com/dotnet/runtime/pull/101671 -->
<Scenario>WasmTestOnChrome</Scenario>
<TestArchiveTestsRoot>$(TestArchiveRoot)browserornodejs/</TestArchiveTestsRoot>
<TestArchiveTestsDir>$(TestArchiveTestsRoot)$(OSPlatformConfig)/</TestArchiveTestsDir>
<DefineConstants>$(DefineConstants);TARGET_BROWSER</DefineConstants>
<XunitShowProgress>true</XunitShowProgress>
</PropertyGroup>
<ItemGroup>
<Compile Include="..\CompareInfo\CompareInfoTestsBase.cs" />
<Compile Include="..\System\Globalization\TextInfoTests.cs" />
Expand Down

0 comments on commit 2ad47cd

Please sign in to comment.