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

Fix FIM test_large_changes module #4219

Merged
merged 5 commits into from
Jun 2, 2023
Merged

Fix FIM test_large_changes module #4219

merged 5 commits into from
Jun 2, 2023

Conversation

Deblintrake09
Copy link
Contributor

Related issue
#3860

Description

This PR fixes and re-enables the test_fim/test_files/test_report_changes/test_large_changes.py module.

Updated

  • Fixed test_fim/test_files/test_report_changes/test_large_changes.py

Testing performed

Tester Test path Jenkins Local OS Commit Notes
@Deblintrake09 (Developer) test_fim 🟢🟢🟢 🟢🟢🟢 Manager 9ee8732 Nothing to highlight
@Deblintrake09 (Developer) test_fim 🟢🟢🟢 Centos Agent 9ee8732 Nothing to highlight
@Deblintrake09 (Developer) test_fim 🟢🟢🟢 🟢🟢🟢 Windows Agent 9ee8732 Nothing to highlight
@Deblintrake09 (Developer) test_fim 🟢🟢🟢 Solaris & macOS 9ee8732 Nothing to highlight

@Deblintrake09 Deblintrake09 linked an issue Jun 1, 2023 that may be closed by this pull request
Fix FIM flaky test test_report_changes/test_large_changes.py #3860
Closed
@Deblintrake09 Deblintrake09 changed the base branch from master to 4.5 June 1, 2023 18:32
@damarisg damarisg self-requested a review June 1, 2023 19:06
damarisg
damarisg requested changes Jun 2, 2023
View reviewed changes
...integration/test_fim/test_files/test_report_changes/data/test_cases/cases_large_changes.yaml Outdated Show resolved Hide resolved
...integration/test_fim/test_files/test_report_changes/data/test_cases/cases_large_changes.yaml Outdated Show resolved Hide resolved
tests/integration/test_fim/test_files/test_report_changes/test_large_changes.py Show resolved Hide resolved
tests/integration/test_fim/test_files/test_report_changes/test_large_changes.py Outdated Show resolved Hide resolved
tests/integration/test_fim/test_files/test_report_changes/test_large_changes.py Outdated Show resolved Hide resolved
tests/integration/test_fim/test_files/test_report_changes/test_large_changes.py Outdated Show resolved Hide resolved
tests/integration/test_fim/test_files/test_report_changes/test_large_changes.py Outdated Show resolved Hide resolved
@damarisg damarisg merged commit 3001be5 into 4.5 Jun 2, 2023
@damarisg damarisg deleted the 3860-fix-test-large-changes branch June 2, 2023 16:01
QU3B1M pushed a commit that referenced this pull request Jun 9, 2023
@Deblintrake09 @QU3B1M
Fix FIM test_large_changes module (#4219)
46c4f86 
* docs(#3860): update changelog

* feat(#3860): add new fixtures

* refactor(#3860): update test_large_changes

* docs(#3860): remove typo

* docs(#3860): updated docu and cases description
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@damarisg damarisg damarisg approved these changes

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

Successfully merging this pull request may close these issues.

Fix FIM flaky test test_report_changes/test_large_changes.py
2 participants
@Deblintrake09 @damarisg