Skip to content

Add Test for CONUS Standard AnA and Refactor Test Configuration#148

Open
mxkpp wants to merge 19 commits into
developmentfrom
maxkipp-add-tests
Open

Add Test for CONUS Standard AnA and Refactor Test Configuration#148
mxkpp wants to merge 19 commits into
developmentfrom
maxkipp-add-tests

Conversation

@mxkpp
Copy link
Copy Markdown

@mxkpp mxkpp commented Apr 30, 2026

This refactors the test configurations to support extensibility, and extends the tests by adding one for CONUS Standard Analysis & Assimilation (AnA).

Additions

  • AnA test: ./tests/ana/test_ana.py
  • Run from Dev Container via:
    • ( cd src/ngen-forcing && pytest tests/ana )
    • ( cd src/ngen-forcing && mpirun -n 2 pytest tests/ana )

Removals

Changes

  • Refactored test configuration structure

Testing

  1. Ran existing tests. InputForcing -n 2 is not currently passing but InputForcings -n 1 is (unrelated to this PR).
  2. Created new test data (expected results) for the new AnA case.

Screenshots

Notes

Todos

Checklist

  • PR has an informative and human-readable title
  • Changes are limited to a single goal (no scope creep)
  • Code can be automatically merged (no conflicts)
  • Code follows project standards (link if applicable)
  • Passes all existing automated tests
  • Any change in functionality is tested
  • New functions are documented (with a description, list of inputs, and expected output)
  • Placeholder code is flagged / future todos are captured in comments
  • Visually tested in supported browsers and devices (see checklist below 👇)
  • Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)
  • Reviewers requested with the Reviewers tool ➡️

Testing checklist

Target Environment support

  • Linux

@mxkpp mxkpp changed the title Maxkipp add tests Add Test for CONUS Standard AnA and Refactor Test Configuration Apr 30, 2026
@mxkpp mxkpp mentioned this pull request May 2, 2026
12 tasks
@mxkpp mxkpp changed the base branch from maxkipp-pre-dev to development May 7, 2026 14:26
@mxkpp mxkpp changed the base branch from development to maxkipp-pre-dev May 7, 2026 16:55
@mxkpp mxkpp changed the base branch from maxkipp-pre-dev to development May 7, 2026 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants