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

Qualysv2: added start date to the test-module #34026

Merged
merged 6 commits into from Apr 22, 2024
Merged

Conversation

moishce
Copy link
Contributor

@moishce moishce commented Apr 17, 2024

Contributing to Cortex XSOAR Content

Make sure to register your contribution by filling the contribution registration form

The Pull Request will be reviewed only after the contribution registration form is filled.

Status

  • In Progress
  • Ready
  • In Hold - (Reason for hold)

Related Issues

fixes: link to the issue

Description

added start date to the test-module to prevent timeout.

Must have

  • Tests
  • Documentation

@@ -3048,7 +3048,7 @@ def test_module(client: Client, params: dict[str, Any], first_fetch_time: str) -
Returns:
str: 'ok' if test passed, anything else will raise an exception and will fail the test.
"""
build_args_dict(None, COMMANDS_ARGS_DATA["test-module"], False)
build_args_dict({'launched_after_datetime': 'one day'}, COMMANDS_ARGS_DATA["test-module"], False)
Copy link
Contributor

Choose a reason for hiding this comment

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

why not use the first_fetch?

@moishce moishce merged commit d4a52cc into master Apr 22, 2024
19 of 20 checks passed
@moishce moishce deleted the qualys-test-timeout branch April 22, 2024 09:12
barryyosi-panw pushed a commit that referenced this pull request Apr 24, 2024
* added start date to the test-module

* added rn

* improvements

* update docker
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants