Skip to content

Commit 2bd9abc

Browse files
committed
Exclude live BOM tests from PR integration
1 parent 0c3b495 commit 2bd9abc

2 files changed

Lines changed: 41 additions & 14 deletions

File tree

‎.github/workflows/Integration.yml‎

Lines changed: 28 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,31 @@ on:
66

77
jobs:
88
CI:
9-
uses: HydrologicEngineeringCenter/dotnet-workflows/.github/workflows/integration.yml@main
10-
with:
11-
dotnet-version: '10.0.x'
12-
run-tests: true
9+
# Preserve the check name used by the previous reusable workflow.
10+
name: CI / CI
11+
runs-on: windows-latest
12+
timeout-minutes: 90
13+
14+
steps:
15+
- name: Checkout
16+
uses: actions/checkout@v4
17+
with:
18+
fetch-depth: 0
19+
20+
- name: Setup .NET SDKs
21+
uses: actions/setup-dotnet@v4
22+
with:
23+
dotnet-version: |
24+
8.0.x
25+
9.0.x
26+
10.0.x
27+
28+
- name: Build
29+
run: dotnet build -c Release
30+
31+
- name: Test
32+
env:
33+
VSTEST_CONNECTION_TIMEOUT: '600'
34+
# BOM's live WDP backend intermittently returns HTTP 500 DatasourceError.
35+
# Keep offline BOM validation and all other tests in the PR gate.
36+
run: dotnet test -c Release --no-build --filter "TestCategory!=BOMIntegration"

‎Test_Numerics/Data/Time Series/Test_TimeSeriesDownload.cs‎

Lines changed: 13 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1750,10 +1750,13 @@ await TimeSeriesDownload.FromGHCN(GHCN_1,
17501750

17511751
#region BOM (Australia) Tests
17521752

1753+
// Live BOM checks remain available locally but are excluded from PR integration.
1754+
// Run them with: dotnet test -c Release --filter "TestCategory=BOMIntegration"
1755+
17531756
/// <summary>
17541757
/// Validates a full-period-of-record download for the BOM Cotter River station (discharge).
17551758
/// </summary>
1756-
[TestMethod, TestCategory("Integration")]
1759+
[TestMethod, TestCategory("Integration"), TestCategory("BOMIntegration")]
17571760
public async Task BOM_FullPor_CotterRiver_Discharge()
17581761
{
17591762
if (!await BomAvailable()) return;
@@ -1764,7 +1767,7 @@ public async Task BOM_FullPor_CotterRiver_Discharge()
17641767
/// <summary>
17651768
/// Validates a full-period-of-record download for the BOM Goodradigbee River station (discharge).
17661769
/// </summary>
1767-
[TestMethod, TestCategory("Integration")]
1770+
[TestMethod, TestCategory("Integration"), TestCategory("BOMIntegration")]
17681771
public async Task BOM_FullPor_Goodradigbee_Discharge()
17691772
{
17701773
if (!await BomAvailable()) return;
@@ -1775,7 +1778,7 @@ public async Task BOM_FullPor_Goodradigbee_Discharge()
17751778
/// <summary>
17761779
/// Validates a full-period-of-record download for the BOM Murray River station (stage).
17771780
/// </summary>
1778-
[TestMethod, TestCategory("Integration")]
1781+
[TestMethod, TestCategory("Integration"), TestCategory("BOMIntegration")]
17791782
public async Task BOM_FullPor_MurrayRiver_Stage()
17801783
{
17811784
if (!await BomAvailable()) return;
@@ -1786,7 +1789,7 @@ public async Task BOM_FullPor_MurrayRiver_Stage()
17861789
/// <summary>
17871790
/// Tests discharge unit conversions (cms ↔ cfs) for BOM data.
17881791
/// </summary>
1789-
[TestMethod, TestCategory("Integration")]
1792+
[TestMethod, TestCategory("Integration"), TestCategory("BOMIntegration")]
17901793
public async Task BOM_UnitConversion_Discharge_CmsCfs()
17911794
{
17921795
if (!await BomAvailable()) return;
@@ -1812,7 +1815,7 @@ public async Task BOM_UnitConversion_Discharge_CmsCfs()
18121815
/// <summary>
18131816
/// Tests stage unit conversions (m ↔ ft) for BOM data.
18141817
/// </summary>
1815-
[TestMethod, TestCategory("Integration")]
1818+
[TestMethod, TestCategory("Integration"), TestCategory("BOMIntegration")]
18161819
public async Task BOM_UnitConversion_Stage_MFt()
18171820
{
18181821
if (!await BomAvailable()) return;
@@ -1859,7 +1862,7 @@ await TimeSeriesDownload.FromABOM(BOM_1,
18591862
/// <summary>
18601863
/// Tests BOM with a windowed date range to verify date filtering works correctly.
18611864
/// </summary>
1862-
[TestMethod, TestCategory("Integration")]
1865+
[TestMethod, TestCategory("Integration"), TestCategory("BOMIntegration")]
18631866
public async Task BOM_WindowedDownload_Works()
18641867
{
18651868
if (!await BomAvailable()) return;
@@ -1884,7 +1887,7 @@ public async Task BOM_WindowedDownload_Works()
18841887
/// <summary>
18851888
/// Validates instantaneous discharge download from BOM.
18861889
/// </summary>
1887-
[TestMethod, TestCategory("Integration")]
1890+
[TestMethod, TestCategory("Integration"), TestCategory("BOMIntegration")]
18881891
public async Task BOM_InstantaneousDischarge_Works()
18891892
{
18901893
if (!await BomAvailable()) return;
@@ -1898,7 +1901,7 @@ public async Task BOM_InstantaneousDischarge_Works()
18981901
/// <summary>
18991902
/// Validates instantaneous stage download from BOM.
19001903
/// </summary>
1901-
[TestMethod, TestCategory("Integration")]
1904+
[TestMethod, TestCategory("Integration"), TestCategory("BOMIntegration")]
19021905
public async Task BOM_InstantaneousStage_Works()
19031906
{
19041907
if (!await BomAvailable()) return;
@@ -1912,7 +1915,7 @@ public async Task BOM_InstantaneousStage_Works()
19121915
/// <summary>
19131916
/// Validates daily precipitation download from BOM.
19141917
/// </summary>
1915-
[TestMethod, TestCategory("Integration")]
1918+
[TestMethod, TestCategory("Integration"), TestCategory("BOMIntegration")]
19161919
public async Task BOM_DailyPrecipitation_Works()
19171920
{
19181921
if (!await BomAvailable()) return;
@@ -1926,7 +1929,7 @@ public async Task BOM_DailyPrecipitation_Works()
19261929
/// <summary>
19271930
/// Tests precipitation unit conversions (mm ↔ inches) for BOM data.
19281931
/// </summary>
1929-
[TestMethod, TestCategory("Integration")]
1932+
[TestMethod, TestCategory("Integration"), TestCategory("BOMIntegration")]
19301933
public async Task BOM_UnitConversion_Precip_MmIn()
19311934
{
19321935
if (!await BomAvailable()) return;

0 commit comments

Comments
 (0)