Skip to content

Routing Spec Property Mapping Fix#408

Merged
adamkorynta merged 5 commits intoopendcs:mainfrom
zack-rma:bugfix/routing_spec_props
Apr 7, 2025
Merged

Routing Spec Property Mapping Fix#408
adamkorynta merged 5 commits intoopendcs:mainfrom
zack-rma:bugfix/routing_spec_props

Conversation

@zack-rma
Copy link
Collaborator

@zack-rma zack-rma commented Apr 4, 2025

Problem Description

Fixes missing/incorrect mapping of RoutingSpec properties.

Solution

Updated both unit and integration tests with proper routing property mapping.

how you tested the change

Unit tested and integration tested against OpenTSDB.

Where the following done:

  • Tests. Check all that apply:
    • Unit tests created or modified that run during ant test.
    • Integration tests created or modified that run during integration testing
      (Formerly called regression tests.)
    • Test procedure descriptions for manual testing
  • Was relevant documentation updated?
  • Were relevant config element (e.g. XML data) updated as appropriate

If you aren't sure leave unchecked and we will help guide you to want needs changing where.

@zack-rma zack-rma marked this pull request as ready for review April 4, 2025 19:09
@zack-rma zack-rma requested a review from adamkorynta April 4, 2025 19:09
@zack-rma zack-rma marked this pull request as draft April 4, 2025 20:16
@zack-rma zack-rma marked this pull request as ready for review April 4, 2025 21:53
@adamkorynta adamkorynta linked an issue Apr 7, 2025 that may be closed by this pull request
@adamkorynta adamkorynta merged commit 346ec9b into opendcs:main Apr 7, 2025
8 checks passed
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.

Saving a routing spec is not saving the Platform/Message Types

2 participants