Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add CWMS integration testing coverage to the AppResources controller #354

Draft
wants to merge 17 commits into
base: feature/task_cwms_support
Choose a base branch
from

Conversation

zack-rma
Copy link
Collaborator

Problem Description

Fixes #293.

Based on changes found here: #311. Will require rebase when those changes are merged.

Requires Toolkit changes found here: opendcs/opendcs#930.

Solution

Moved test data to DEFAULT directory, updated object mapping, removed start, stop and events endpoints and associated tests.

how you tested the change

Integration tested against CWMS and OpenTSDB 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 failures should be resolved by: opendcs/opendcs#930

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.

1 participant