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.

Sign up for GitHub

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

Jump to bottom

test(recordings): Add more target recording handler 4xx itests #598

Merged
merged 3 commits into from
Jul 23, 2021
Merged

test(recordings): Add more target recording handler 4xx itests #598

merged 3 commits into from
Jul 23, 2021

Conversation

jan-law
Copy link
Contributor

@jan-law jan-law commented Jul 23, 2021

Related #474

I decided to rename TargetRecordingsPostIT to TargetRecordingsClientErrorIT and put all the 4xx tests related to the target recording handlers into one file.

These tests should cover all the 4xx scenarios caused by the TargetRecordings GET, DELETE, PATCH, PATCH options, and upload POST handlers.

@jan-law jan-law added the test label Jul 23, 2021
@jan-law jan-law requested a review from andrewazores July 23, 2021 19:24
@andrewazores andrewazores merged commit 56905e4 into cryostatio:main Jul 23, 2021
@jan-law jan-law deleted the itest-target-recordings branch July 29, 2021 14:07
tthvo pushed a commit to tthvo/cryostat-legacy that referenced this pull request Dec 9, 2024
@andrewazores
fix(matchexpr): expose 'isAgent' field for evaluation (cryostatio#598)
f319134
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andrewazores andrewazores andrewazores approved these changes

Assignees
No one assigned
Labels
test
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jan-law @andrewazores