Bugfix/cda 46 rating vertical datum#1355

Open

RyanM-RMA wants to merge 16 commits intoUSACE:developfrom

RyanM-RMA:bugfix/CDA-46_rating_vertical_datum

Commits on Jan 23, 2026

Adding controller integration tests for supporting vertical datum.

RyanM-RMA
authored and
rma-bryson
committed
Adding missing controllers and vertical datum enum

RyanM-RMA
authored and
rma-bryson
committed
Refactor `RatingsControllerTestIT` by moving vertical datum tests to new `RatingsControllerTestVerticalDatumIT`. Enhance `RatingController` to handle vertical datum conversions.

RyanM-RMA
authored and
rma-bryson
committed
Improve vertical datum handling in `VerticalDatum` and update `RatingsControllerTestVerticalDatumIT` to include additional test cases for datum combinations. Enhance `RatingController` to log verti…

RyanM-RMA
authored and
rma-bryson
committed
Trim input string in `VerticalDatum` and expand test cases in `RatingsControllerTestVerticalDatumIT` to cover additional vertical datum combinations and query scenarios.

RyanM-RMA
authored and
rma-bryson
committed
Add `DeleteMeTest` for testing vertical datum rating conversions and assertions.

RyanM-RMA
authored and
rma-bryson
committed
CDA-46 - Implements vertical datum support for create, update, and retrieval. Adds integration tests.
rma-bryson
committed
CDA-46 - Removed unused vertical datum constant
rma-bryson
committed
CDA-46 - Adds setting of local datum for conversions to work similar to setting default vert datum.
rma-bryson
committed
CDA-46 - Fixed validation error on datum param if its empty
rma-bryson
committed
CDA-46 - Updated datum check to handle dashes.
rma-bryson
committed
CDA-46 - updated to check for "unknown" datum first to prevent exception
rma-bryson
committed
CDA-46 - Reverted RatingsControllerTestIT change that uses vertical datum location since we test that in its own test class.
rma-bryson
committed
CDA-46 - Adds check to ignore vertical datum conversion if no target vertical datum is passed in.
rma-bryson
committed
CDA-46 - Fixes failing test due to location not existing
rma-bryson
committed
CDA-46 - integrating cwms-rating with matching jooq-codegen version
rma-bryson
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Bugfix/cda 46 rating vertical datum#1355

Bugfix/cda 46 rating vertical datum#1355
RyanM-RMA wants to merge 16 commits intoUSACE:developfrom
RyanM-RMA:bugfix/CDA-46_rating_vertical_datum

Commits on Jan 23, 2026

Adding controller integration tests for supporting vertical datum.

Adding missing controllers and vertical datum enum

Refactor `RatingsControllerTestIT` by moving vertical datum tests to new `RatingsControllerTestVerticalDatumIT`. Enhance `RatingController` to handle vertical datum conversions.

Improve vertical datum handling in `VerticalDatum` and update `RatingsControllerTestVerticalDatumIT` to include additional test cases for datum combinations. Enhance `RatingController` to log verti…

Trim input string in `VerticalDatum` and expand test cases in `RatingsControllerTestVerticalDatumIT` to cover additional vertical datum combinations and query scenarios.

Add `DeleteMeTest` for testing vertical datum rating conversions and assertions.

CDA-46 - Implements vertical datum support for create, update, and retrieval. Adds integration tests.

CDA-46 - Removed unused vertical datum constant

CDA-46 - Adds setting of local datum for conversions to work similar to setting default vert datum.

CDA-46 - Fixed validation error on datum param if its empty

CDA-46 - Updated datum check to handle dashes.

CDA-46 - updated to check for "unknown" datum first to prevent exception

CDA-46 - Reverted RatingsControllerTestIT change that uses vertical datum location since we test that in its own test class.

CDA-46 - Adds check to ignore vertical datum conversion if no target vertical datum is passed in.

CDA-46 - Fixes failing test due to location not existing

CDA-46 - integrating cwms-rating with matching jooq-codegen version