Skip to content

Added transportMedia check to Platform GET endpoint integration test#377

Merged
adamkorynta merged 2 commits intoopendcs:mainfrom
zack-rma:bugfix/platform_tm
Mar 26, 2025
Merged

Added transportMedia check to Platform GET endpoint integration test#377
adamkorynta merged 2 commits intoopendcs:mainfrom
zack-rma:bugfix/platform_tm

Conversation

@zack-rma
Copy link
Collaborator

@zack-rma zack-rma commented Mar 24, 2025

Problem Description

Fixes #374.

Solution

Added transport media check to Platform GET endpoint integration test.

how you tested the change

Integration tested against OpenTSDB and CWMS database instances.

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
Copy link
Collaborator Author

Build will fail without Toolkit changes found here: opendcs/opendcs#999

@zack-rma zack-rma marked this pull request as ready for review March 24, 2025 21:30
@adamkorynta
Copy link
Contributor

@zack-rma - new version of the toolkit is available: https://central.sonatype.com/artifact/org.opendcs/opendcs/7.5.1-RC03

@zack-rma zack-rma force-pushed the bugfix/platform_tm branch from 63a23a5 to 0bb16ae Compare March 26, 2025 16:00
@zack-rma zack-rma requested a review from adamkorynta March 26, 2025 16:17
@adamkorynta adamkorynta merged commit 5ebc788 into opendcs:main Mar 26, 2025
7 checks passed
@zack-rma zack-rma deleted the bugfix/platform_tm branch March 26, 2025 23:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Platform endpoint missing transport media

2 participants