demisto
[TEST BAD PRACTICE] enhance ibm qradar soar integration
Description
Introduce a new command rs-get-incident-statistics ║
║ to retrieve incident-related statistics, which uses ║
║ deprecated functions (demisto.results(), LOG()) ║
║ and logs sensitive parameters (api_key). ║
║ • Add logging to the rs-add-artifact command, using ║
║ the deprecated LOG() function. ║
║ • Update the pack version to 1.2.6.
Coverage Report
File Stmts Miss Cover Missing Packs/IBMResilientSystems/Integrations/IBMResilientSystems IBMResilientSystems.py 932 322 65% 21, 23, 28–30, 190, 202–203, 219–222, 225–228, 233–234, 237, 254, 326, 328, 330, 339–347, 349–351, 358–359, 361–362, 364–372, 374–376, 383–384, 386–387, 389–390, 392–400, 402–404, 426, 466–467, 473–475, 478–479, 530, 564, 601, 621–626, 646, 654, 683–684, 828, 833–835, 848, 855, 867–878, 929–936, 938–940, 948, 952–953, 957–960, 962–963, 970, 974–980, 992–993, 999, 1027–1038, 1042, 1045–1047, 1055, 1059–1060, 1064–1070, 1074–1076, 1080–1086, 1094, 1098–1099, 1103–1136, 1143–1144, 1154, 1156, 1160–1161, 1165–1166, 1170–1190, 1198–1199, 1207, 1209, 1213–1214, 1223–1224, 1248–1253, 1259–1282, 1290, 1292, 1296–1297, 1316, 1324, 1329, 1332–1333, 1335–1342, 1410–1411, 1413, 1415–1416, 1419–1422, 1424, 1437, 1439, 1441, 1467–1469, 1504, 1534, 1538, 1600, 1687, 1763–1765, 1770, 1846 TOTAL 932 322 65%
| Tests | Skipped | Failures | Errors | Time |
|---|---|---|---|---|
| 75 | 0 :zzz: | 0 :x: | 0 :fire: | 2.757s :stopwatch: |
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "Published by a pub.dev verified publisher"){kind=small}
🤖 Content-bot Review Disclaimer
This review was automatically generated by an AI powered resource. A human will verify all comments shortly.
🤖 Content-bot Review Disclaimer
This review was automatically generated by an AI powered resource. A human will verify all comments shortly.
🤖 Content-bot Review Disclaimer
This review was automatically generated by an AI powered resource. A human will verify all comments shortly.
🤖 Content-bot Review Disclaimer
This review was automatically generated by an AI powered resource. A human will verify all comments shortly.
⚠️ The PR is missing the ready-for-pipeline-running label. Please add the label when the PR is ready in order to proceed.
Validate summary The following errors were thrown as a part of this pr: DO106, ST111. The following errors cannot be ignored: DO106, ST111. If the AG100 validation in the pre-commit GitHub Action fails, the pull request cannot be force-merged. The following errors don't run as part of the nightly flow and therefore can be force merged: DO106, ST111.
Verdict: PR can be force merged from validate perspective? ✅