Skip to content
This repository was archived by the owner on Nov 24, 2025. It is now read-only.

Add TO Tests for ATS config endpoints#3941

Merged
ocket8888 merged 1 commit into
apache:masterfrom
rob05c:add-to-tests-for-ats-config
Sep 30, 2019
Merged

Add TO Tests for ATS config endpoints#3941
ocket8888 merged 1 commit into
apache:masterfrom
rob05c:add-to-tests-for-ats-config

Conversation

@rob05c
Copy link
Copy Markdown
Member

@rob05c rob05c commented Sep 30, 2019

Add TO Tests for ATS config endpoints

What does this PR (Pull Request) do?

Add TO Tests for ATS config endpoints. Verifies endpoints at least return a 200 success.

  • This PR is not related to any other Issue

Is tests.
No interface change, no documentation change.

Which Traffic Control components are affected by this PR?

  • TO API Tests

What is the best way to verify this PR?

Run TO API Tests.

If this is a bug fix, what versions of Traffic Control are affected?

Not a bug.

The following criteria are ALL met by this PR

  • This PR includes tests OR I have explained why tests are unnecessary
  • This PR includes documentation OR I have explained why documentation is unnecessary
  • This PR includes an update to CHANGELOG.md OR such an update is not necessary
  • This PR includes any and all required license headers
  • This PR ensures that database migration sequence is correct OR this PR does not include a database migration
  • This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

@rob05c rob05c added the tests related to tests and/or testing infrastructure label Sep 30, 2019
@rob05c rob05c force-pushed the add-to-tests-for-ats-config branch from 44a6828 to 2122931 Compare September 30, 2019 16:05
@mitchell852 mitchell852 added Traffic Ops related to Traffic Ops cache-config Cache config generation labels Sep 30, 2019
@rob05c rob05c force-pushed the add-to-tests-for-ats-config branch from 2122931 to e58ef14 Compare September 30, 2019 16:06
@asf-ci
Copy link
Copy Markdown
Contributor

asf-ci commented Sep 30, 2019

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4347/

Copy link
Copy Markdown
Contributor

@mhoppa mhoppa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good - Pulled and tested locally and everything passed good addition.

@ocket8888 ocket8888 merged commit 646c28e into apache:master Sep 30, 2019
@asf-ci
Copy link
Copy Markdown
Contributor

asf-ci commented Sep 30, 2019

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4348/

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cache-config Cache config generation tests related to tests and/or testing infrastructure Traffic Ops related to Traffic Ops

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants