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

[6.14.z] notifications: new test for notification_recipients endpoint #14694

Conversation

pnovotny
Copy link
Contributor

@pnovotny pnovotny commented Apr 9, 2024

Cherrypick of PR: #14592

Fixes cherry-pick issue: #14662

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.

JacobCallahan and others added 30 commits November 23, 2023 00:22
This change adds a new fixture_cli.py script under the scripts
directory.

The basic usage of this script is to take in a space-separated list
of global fixtures and run them together in a single temporary test.

e.g. python scripts/fixture_cli.py module_ak_with_synced_repo module_lce

Additionally, I had to make some minor adjustments to a couple of
plugins since this temporary test doesn't follow the same rules as the
rest of our framework.

(cherry picked from commit 150e7a2)
* Add test to check upgrade warning when KA enabled

* Address PR comments
…QE#13182)

recording ui-session-id for report portal logging (SatelliteQE#13055)

* recording ui-session-id for report portal logging

* moving the fixture to common location and adding fspath check

(cherry picked from commit 3e4abae)

Co-authored-by: Omkar Khatavkar <[email protected]>
…#13216)

added the missing direcotry path for ui_session (SatelliteQE#13212)

(cherry picked from commit 2fa446e)

Co-authored-by: Omkar Khatavkar <[email protected]>
…E#12946)

* test for syncing multiple large repos

* Adressing comments

* addressing comments

* precommit fix
Pytest FixtureCollection external plugin (SatelliteQE#13176)

(cherry picked from commit a9ffccd)

Co-authored-by: Jitendra Yejare <[email protected]>
…ype virtwho_config delete_host& manifest content move (SatelliteQE#13228)

fixture support for virt-who config cli : data_form deploy_type virtwho_config delete_host& manifest content move (SatelliteQE#12619)

* fixture support for virt-who config cli : data_form deploy_type virtwho_config & manifest content move

* add fixture for virt-who config

* Update virt-who config ui fixture data_form_ui & virtwho_config_ui deploy_type_ui

* remove used import

* Update virtwho_config.py

update the format for import

* Update taxonomy.py

Revert the changes

* Update test_libvirt_sca.py

* Update test_libvirt.py

* Update test_nutanix.py

* Update test_nutanix_sca.py

* Update test_esx_sca.py

* fix pre-commit issue

* use pytest.mark.usefixtures for delete_host

* put delete_host to virtwho_config fixture

(cherry picked from commit d4079a6)

Co-authored-by: yanpliu <[email protected]>
…1544) (SatelliteQE#13224)

* Remove cli.factory dependencies from the repo (SatelliteQE#11544)

Remove cli_factory

* Add default content type as yum for make_repository helper (SatelliteQE#13196)

Signed-off-by: Gaurav Talreja <[email protected]>

---------

Signed-off-by: Gaurav Talreja <[email protected]>
Co-authored-by: Shweta Singh <[email protected]>
Co-authored-by: Gaurav Talreja <[email protected]>
…atelliteQE#13258)

adding customerscenario tag to libvirt e2e provision test (SatelliteQE#13252)

(cherry picked from commit 5703ea7)

Co-authored-by: Peter Ondrejka <[email protected]>
Bump navmazing from 1.1.6 to 1.2.2 (SatelliteQE#13259)

(cherry picked from commit 7b90a09)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…liteQE#13245)

* Add coverage for BZ#2092039 and update few docstrings

* Use better timeout format

* Clear import path before import
)

Bump pytest-reportportal from 5.3.0 to 5.3.1 (SatelliteQE#13297)

(cherry picked from commit 11d1366)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Test Coverage for FDI version verification (SatelliteQE#13209)

* Test Coverage for FDI version verification

* Removed unwanted code

* Using foreman-maintain for installation

* Using Version

(cherry picked from commit 18acd55)

Co-authored-by: Adarsh dubey <[email protected]>
Add closed loop for BZ#1841048 (SatelliteQE#13282)

Signed-off-by: Shubham Ganar <[email protected]>
(cherry picked from commit a8ab807)

Co-authored-by: Shubham Ganar <[email protected]>
…uct WFs (SatelliteQE#13275)

Create a distinction between the product-ready OS and the product WFs

(cherry picked from commit ec88ed9)

Co-authored-by: Ondřej Gajdušek <[email protected]>
…atelliteQE#13313)

Add default provider/url for make_compute_resource helper (SatelliteQE#13283)

Signed-off-by: Gaurav Talreja <[email protected]>
(cherry picked from commit 8015e0a)

Co-authored-by: Gaurav Talreja <[email protected]>
The changes included in this commit are:
- Run fixtures in separate tests so failing fixtures don't block others
- Add the ability to run fixtures that are indirectly parametrized
- Add an option to switch to verbose mode, showing test execution and
  results. This should help with debugging and generally seeing what'
  happening
- Add an option to run the fixtures in multiple xdist workers

(cherry picked from commit e86c436)
Satellite-QE and others added 19 commits April 2, 2024 11:55
…eQE#14601)

zStream Checks for Python 3.12 (SatelliteQE#14558)

Add Python 3.12 to supported versions (SatelliteQE#12793)

* Add Python 3.12 to supported versions

Python 3.12 is out and should be supported by Robottelo. Also switched
the weekly workflow to use Python 3.12 instead of 3.11.

* update unittest deprecations

* Adjust unit tests for python 3.12

This commit introduces a few changes relating to unittest changes and
differences post-removal of old behavior.

Co-authored-by: Jake Callahan <[email protected]>
…#14611)

Move TestAnsibleREX CLI tests to Ansible module (SatelliteQE#14577)

Signed-off-by: Gaurav Talreja <[email protected]>
(cherry picked from commit 9687508)
…ate (SatelliteQE#14625)

Repair setup for package w/ swidtag, install and modular update
Bump testimony from 2.3.0 to 2.4.0 (SatelliteQE#14648)

(cherry picked from commit dc21432)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…E#14665)

Make oscap profile configurable in settings file (SatelliteQE#14535)

to enable tests for OS other than rhel

(cherry picked from commit ca8ab78)

Co-authored-by: dosas <[email protected]>
Add filter for ldap related tests (SatelliteQE#14540)

(cherry picked from commit 21580fc)

Co-authored-by: dosas <[email protected]>
)

(cherry picked from commit 8c2a4c6)

Signed-off-by: Gaurav Talreja <[email protected]>
* Get rid of deprecated DRPM repo tests

* Fix SRPM repo tests
…teQE#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 pnovotny added the No-CherryPick PR doesnt need CherryPick to previous branches label Apr 9, 2024
@pnovotny pnovotny self-assigned this Apr 9, 2024
@pnovotny pnovotny requested review from pondrejk, lhellebr and a team April 9, 2024 13:08
@pnovotny pnovotny closed this Apr 9, 2024
@pnovotny
Copy link
Contributor Author

pnovotny commented Apr 9, 2024

Wrong target branch, this is intended for 6.14.z.

@pnovotny pnovotny removed request for pondrejk, lhellebr and a team April 9, 2024 13:12
@pnovotny pnovotny deleted the cherry-pick-6.14.z-notification_recipients branch April 30, 2024 10:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
No-CherryPick PR doesnt need CherryPick to previous branches
Projects
None yet
Development

Successfully merging this pull request may close these issues.