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

[6.15.z] notifications: new test for notification_recipients endpoint #14663

Merged
merged 1 commit into from
Apr 9, 2024
Merged

[6.15.z] notifications: new test for notification_recipients endpoint #14663

merged 1 commit into from
Apr 9, 2024

Conversation

Satellite-QE
Copy link
Collaborator

Cherrypick of PR: #14592

This new test verifies that endpoint /notification_recipients works and returns correct data structure.

It covers bug https://bugzilla.redhat.com/2249970 where this broken endpoint caused some web UI pages to fail to load.

@pnovotny @web-flow
notifications: new test for notification_recipients endpoint (#14592)
1334ae4 
This new test verifies that endpoint `/notification_recipients` works
and returns correct data structure.

It covers bug https://bugzilla.redhat.com/2249970
where this broken endpoint caused some web UI pages to fail to load.

(cherry picked from commit 349c75a)
@Satellite-QE Satellite-QE requested a review from a team as a code owner April 8, 2024 07:07
@Satellite-QE Satellite-QE added 6.15.z Introduced in or relating directly to Satellite 6.15 Auto_Cherry_Picked Automatically cherrypicked PR using GHA No-CherryPick PR doesnt need CherryPick to previous branches labels Apr 8, 2024
@Gauravtalreja1
Copy link
Collaborator

trigger: test-robottelo
pytest: tests/foreman/api/test_notifications.py -k test_positive_notification_recipients
nailgun: 1125

@Gauravtalreja1 Gauravtalreja1 mentioned this pull request Apr 9, 2024
Merged
@Satellite-QE
Copy link
Collaborator Author

PRT Result

Build Number: 6384
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_notifications.py -k test_positive_notification_recipients --external-logging
Test Result : =========== 1 passed, 2 deselected, 4 warnings in 787.50s (0:13:07) ============

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Apr 9, 2024
@Gauravtalreja1 Gauravtalreja1 merged commit dcabb2d into 6.15.z Apr 9, 2024
15 checks passed
@Gauravtalreja1 Gauravtalreja1 deleted the cherry-pick-6.15.z-349c75a2e8b091bac289994b9dc9ed0db414dcff branch April 9, 2024 07:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Gauravtalreja1 Gauravtalreja1 Gauravtalreja1 approved these changes

Assignees

@pnovotny pnovotny

Labels
6.15.z Introduced in or relating directly to Satellite 6.15 Auto_Cherry_Picked Automatically cherrypicked PR using GHA No-CherryPick PR doesnt need CherryPick to previous branches PRT-Passed Indicates that latest PRT run is passed for the PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Satellite-QE @Gauravtalreja1 @pnovotny