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

E2E tests: Unify Virustotal tests #3174

Merged
merged 1 commit into from
Aug 12, 2022
Merged

E2E tests: Unify Virustotal tests #3174

merged 1 commit into from
Aug 12, 2022

Conversation

juliamagan
Copy link
Member

@juliamagan juliamagan commented Aug 12, 2022 •
edited
Loading

Related issue
#3173

Description

While documenting the Virustotal test in #2983, we have seen that the test is divided into two modules, however, this should be a single test with two test cases, since the configuration is the same, but the event generation is different.

Updated

  • Unify test_virustotal_active_response and test_virustotal_fim into test_virustotal_integration

Testing performed

Tester Test path Jenkins Local OS Commit Notes
@juliamagan (Developer) test_virustotal_integration/ 🚫 🚫 🚫 🟢 🟢 🟢 CentOS 1cca919 Nothing to highlight
@user (Reviewer) 🚫 🚫 🚫 ⚫⚫⚫ Nothing to highlight

@juliamagan juliamagan self-assigned this Aug 12, 2022
@jmv74211 jmv74211 merged commit 6a4a0f3 into 2872-tests-e2e Aug 12, 2022
@jmv74211 jmv74211 deleted the 3173-refactor-virustotal-e2e branch August 12, 2022 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jmv74211 jmv74211 jmv74211 approved these changes

Assignees

@juliamagan juliamagan

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@juliamagan @jmv74211