Skip to content

Issue 6753 - Remove ticket 48497 test#7482

Open
mirielka wants to merge 1 commit into389ds:mainfrom
mirielka:48497
Open

Issue 6753 - Remove ticket 48497 test#7482
mirielka wants to merge 1 commit into389ds:mainfrom
mirielka:48497

Conversation

@mirielka
Copy link
Copy Markdown
Contributor

@mirielka mirielka commented May 6, 2026

Description:
Ticket 48497 tests are already covered in dirsrvtests/suites/indexes/regression_test.py, namely in test_reindex_homedirectory_matching_rules and test_reindex_homedirectory_mixed_value tests), therefore the ticket 48497 can be removed.

Relates: #6753
Author: Lenka Doudova
Reviewer: ???

Summary by Sourcery

Tests:

  • Delete the dedicated ticket48497_test regression test file that is now redundant.

Description:
Ticket 48497 tests are already covered in dirsrvtests/suites/indexes/regression_test.py,
namely in test_reindex_homedirectory_matching_rules and test_reindex_homedirectory_mixed_value tests),
therefore the ticket 48497 can be removed.

Relates: 389ds#6753
Author: Lenka Doudova
Reviewer: ???
Copy link
Copy Markdown
Contributor

@sourcery-ai sourcery-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@packit-as-a-service
Copy link
Copy Markdown

Congratulations! One of the builds has completed. 🍾

You can install the built RPMs by following these steps:

  • sudo dnf install -y 'dnf*-command(copr)'
  • dnf copr enable packit/389ds-389-ds-base-7482
  • And now you can install the packages.

Please note that the RPMs should be used only in a testing environment.

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.

2 participants