Netskope - fix trigger configuration

[lvoloshyn-sekoia]

https://github.com/SekoiaLab/integration/issues/324

[github-actions]

Test Results

22 tests   - 74   20 ✅  - 75   1m 4s ⏱️ + 1m 2s
 1 suites ± 0    2 💤 + 1 
 1 files   ± 0    0 ❌ ± 0 

Results for commit c403d02. ± Comparison against base commit 08530d1.

This pull request removes 96 and adds 22 tests. Note that renamed tests count towards both.

tests.ic_oc_triggers.test_alerts ‑ test_alert_trigger_filter_by_rule
tests.ic_oc_triggers.test_alerts ‑ test_comment_trigger_filter_by_rule
tests.ic_oc_triggers.test_alerts ‑ test_comment_trigger_filter_notification_function
tests.ic_oc_triggers.test_alerts ‑ test_invalid_events_dont_triggers_comments_added
tests.ic_oc_triggers.test_alerts ‑ test_securityalertstrigger_handle_alert_invalid_message
tests.ic_oc_triggers.test_alerts ‑ test_securityalertstrigger_handle_alert_send_message
tests.ic_oc_triggers.test_alerts ‑ test_securityalertstrigger_handler_dispatch_alert_message
tests.ic_oc_triggers.test_alerts ‑ test_securityalertstrigger_init
tests.ic_oc_triggers.test_alerts ‑ test_securityalertstrigger_retrieve_alert_from_api
tests.ic_oc_triggers.test_alerts ‑ test_securityalertstrigger_retrieve_alert_from_api_exp_raised
…

tests.test_helpers ‑ test_get_index_name
tests.test_helpers ‑ test_get_iterator_name
tests.test_helpers ‑ test_get_tenant_hostname
tests.test_pubsub_lite ‑ test_configuration
tests.test_pubsub_lite ‑ test_process_messages[data1\ndata2\ndata3-expected_events0]
tests.test_pubsub_lite ‑ test_process_messages[data1\ndata\xd8\ndata3-None]
tests.test_pubsub_lite ‑ test_run
tests.test_pull_events_v2_connector ‑ test_create_iterators
tests.test_pull_events_v2_connector ‑ test_fetch_next_batch_integration
tests.test_pull_events_v2_connector ‑ test_long_next_batch_should_not_sleep
…

This pull request removes 1 skipped test and adds 2 skipped tests. Note that renamed tests count towards both.

tests.ic_oc_triggers.test_intelligence ‑ test_run

tests.test_pull_events_v2_connector ‑ test_fetch_next_batch_integration
tests.test_pull_events_v2_connector ‑ test_run_integration

♻️ This comment has been updated with latest results.

[squioc]

The code looks good to me.
However, can you add an entry in the CHANGELOG.md file? Thank you

[codecov]

Codecov Report

Attention: Patch coverage is 83.33333% with 1 line in your changes missing coverage. Please review.

Project coverage is 90.54%. Comparing base (08530d1) to head (b5b4ee5).

Files with missing lines	Patch %	Lines
...skope/netskope_modules/connector_pull_events_v2.py	83.33%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1230      +/-   ##
==========================================
- Coverage   91.71%   90.54%   -1.17%     
==========================================
  Files          20       29       +9     
  Lines         953     1280     +327     
==========================================
+ Hits          874     1159     +285     
- Misses         79      121      +42     

Flag	Coverage Δ
Netskope	87.15% <83.33%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[squioc]

LGTM