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 awx collection export tests #14728

Conversation

chrismeyersfsu
Copy link
Member

SUMMARY
  • Basic export tests
  • Added test that highlights a problem with running Schedule exports as non-root user. We rely on the POST key in the OPTIONS response to determine the fields to export for a resource. The POST key is not present if a user does NOT have create privileges.
  • Fixed up forwarding all headers from the API server back to the test code. This was causing a problem in awxkit code that checks for allowed HTTP Verbs in the headers.
ISSUE TYPE
  • Bug, Docs Fix or other nominal change
COMPONENT NAME
  • Collection
AWX VERSION
awx: 23.5.1.dev14+g8ca0d69296.d20231215
ADDITIONAL INFORMATION

I added these tests to be able to validate this PR #14626

@github-actions github-actions bot added the component:awx_collection issues related to the collection for controlling AWX label Dec 15, 2023
@shanemcd shanemcd changed the title add awx collection export tests Add awx collection export tests Dec 15, 2023
@chrismeyersfsu chrismeyersfsu force-pushed the improvement-awx_collection_export_tests branch 13 times, most recently from 4eb95a3 to 0536ade Compare December 18, 2023 15:30
* Basic export tests
* Added test that highlights a problem with running Schedule exports as
  non-root user. We rely on the POST key in the OPTIONS response to
  determine the fields to export for a resource. The POST key is not
  present if a user does NOT have create privileges.
* Fixed up forwarding all headers from the API server back to the test
  code. This was causing a problem in awxkit code that checks for
  allowed HTTP Verbs in the headers.
* Move awxkit import code into a pytest fixture to better control when
  the import happens
* Ensure /awx_devel/awxkit is added to sys path before awxkit import
  runs
@chrismeyersfsu chrismeyersfsu merged commit 2a6cf03 into ansible:devel Dec 18, 2023
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:awx_collection issues related to the collection for controlling AWX
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants