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

PrismaCloudCompute Test Playbook fix #37495

Merged
merged 2 commits into from
Dec 2, 2024

Conversation

YaelShamai
Copy link
Contributor

@YaelShamai YaelShamai commented Dec 1, 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

relates: link to the issue

Description

The command 'prisma-cloud-compute-container-list' is capable of returning more than one container. However, the subsequent command in the test playbook, 'prisma-cloud-compute-container-host-list', is designed to retrieve only one container ID.
To ensure consistency and proper testing, the test playbook has been modified accordingly, now allowing only one container ID to be sent. We have thoroughly reviewed our changes, as well as the PrismaCloudCompute side, and confirmed that no unintended modifications were made.
While it is rare for multiple containers to be returned, it's important to handle this scenario appropriately.

Must have

  • Tests
  • Documentation

@YaelShamai YaelShamai merged commit 886d821 into master Dec 2, 2024
29 of 33 checks passed
@YaelShamai YaelShamai deleted the PrismaCloudCompute-testplaybook-fix branch December 2, 2024 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants