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
Add Central Connectivity Validator fast integration tests #11676
Add Central Connectivity Validator fast integration tests #11676
Conversation
✔️ 🥰 Documentation preview ready! 🥰 🔨 Explore the source changes: af93699 🔍 Inspect the deploy log: https://app.netlify.com/sites/kyma-project-docs-preview/deploys/61027c745a1c6e0007e3df23 😎 Browse the preview: https://deploy-preview-11676--kyma-project-docs-preview.netlify.app |
c2096a6
to
6203ac2
Compare
/test pre-main-kyma-integration-k3s-compass-dev |
1 similar comment
/test pre-main-kyma-integration-k3s-compass-dev |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Only one minor thing
@@ -44,15 +42,15 @@ describe("Kyma with Compass test", async function() { | |||
return; | |||
} | |||
|
|||
await installer.installKyma({withCompass: true, skipComponents, withCentralApplicationGateway}); | |||
await installer.installKyma({skipComponents: componentsToSkip, withCompass: true, withCentralAppConnectivity: withCentralAppConnectivity}); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can leave shorthand property names notation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Now it's like this:
await installer.installKyma({withCompass: true, withCentralAppConnectivity: withCentralAppConnectivity, componentsToSkip});
@@ -42,7 +42,7 @@ describe("Kyma with Compass test", async function() { | |||
return; | |||
} | |||
|
|||
await installer.installKyma({skipComponents: componentsToSkip, withCompass: true, withCentralAppConnectivity: withCentralAppConnectivity}); | |||
await installer.installKyma({withCompass: true, withCentralAppConnectivity: withCentralAppConnectivity, componentsToSkip}); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The shorthand for object initialization works when key and value match (you can skip value). In case of skipComponents
it is not a case. Rename componentsToSkip
variable or keep the long syntax.
Description
Changes proposed in this pull request:
Related issue(s)
#11654