civisanalytics / civis-python

Civis API Python Client
BSD 3-Clause "New" or "Revised" License
34 stars 26 forks source link

TST update test api specs #281

Closed elsander closed 5 years ago

elsander commented 5 years ago

This PR updates the API specs in the tests, including some feature-flagged endpoints.

elsander commented 5 years ago

Closing this PR in favor of opening a new one with a spec built from the robot user.

elsander commented 5 years ago

After offline conversation with @stephen-hoover and @keithing, I'm reopening this with api specs which include feature flags.

stephen-hoover commented 5 years ago

Just checking -- are the specs edited at all, aside from removing channels from civis/tests/civis_api_spec.json?

elsander commented 5 years ago

I removed all of the admin endpoints from both, otherwise no.

elsander commented 5 years ago

[deleted a comment that I had left in a stray admin endpoint-- I was looking at the wrong local branch. The merged branch doesn't have any admin endpoints]