Skip to content

Conditionally skip with hasFeatureFlag in vulnmanagement integration tests#3308

Merged
pedrottimark merged 1 commit intomasterfrom
cypress-vulnmanagement-skip-hasFeatureFlag
Oct 5, 2022
Merged

Conditionally skip with hasFeatureFlag in vulnmanagement integration tests#3308
pedrottimark merged 1 commit intomasterfrom
cypress-vulnmanagement-skip-hasFeatureFlag

Conversation

@pedrottimark
Copy link
Contributor

@pedrottimark pedrottimark commented Oct 4, 2022

Description

Follow up after #3131

Run tests when 'ROX_POSTGRES_DATASTORE' feature flag is enabled.

  1. Edit cypress/integration/vulnmanagement/clusterCvesListPages.test.js

    • Delete skip because problem has been fixed.
  2. Edit cypress/integration/vulnmanagement/dashboard.test.js

    • Delete 2 comments and replace unconditional skip with conditional skip in 5 tests.

Residue

Investigate whether not yet supported or incorrect field in payload.

  1. Severity

    • cypress/integration/vulnmanagement/imageCvesListPages.test.js
    • cypress/integration/vulnmanagement/nodeCvesListPages.test.js
  2. Top CVSS

    • cypress/integration/vulnmanagement/imageComponentsListPages.test.js
    • cypress/integration/vulnmanagement/imagesListPages.test.js
    • cypress/integration/vulnmanagement/nodeComponentsListPages.test.js
    • cypress/integration/vulnmanagement/nodesListPages.test.js

Checklist

  • Investigated and inspected CI test results
  • Edited integration tests

Testing Performed

@ghost
Copy link

ghost commented Oct 4, 2022

Images are ready for the commit at dbac503.

To use with deploy scripts, first export MAIN_IMAGE_TAG=3.72.x-216-gdbac5035ca.

@pedrottimark pedrottimark requested a review from vjwilson October 4, 2022 21:57
@pedrottimark pedrottimark merged commit f31e85d into master Oct 5, 2022
@pedrottimark pedrottimark deleted the cypress-vulnmanagement-skip-hasFeatureFlag branch October 5, 2022 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants