Skip to content

Use POST for integration add-multiple commands#225

Open
Rohithmatham12 wants to merge 1 commit into
cortexapps:mainfrom
Rohithmatham12:fix-integration-add-multiple-post
Open

Use POST for integration add-multiple commands#225
Rohithmatham12 wants to merge 1 commit into
cortexapps:mainfrom
Rohithmatham12:fix-integration-add-multiple-post

Conversation

@Rohithmatham12

Copy link
Copy Markdown

Summary

  • switch generated add-multiple integration commands from PUT to POST for configuration creation endpoints
  • keep AWS unchanged because its command already has separate add/update semantics
  • align add-multiple valid-input tests with the POST behavior already expected by the basic add-multiple tests

Fixes #140.

Testing

  • CORTEX_BASE_URL=http://localhost CORTEX_API_KEY=test POETRY_CACHE_DIR=/private/tmp/cortexapps-cli-poetry-cache poetry run pytest tests/test_integrations_*.py -k add_multiple -q

Signed-off-by: Rohithmatham12 <rohithmatham@gmail.com>
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.

Wrong endpoint called for add multiple configurations for several integrations

1 participant