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
Basic pipeline test #3813
Basic pipeline test #3813
Conversation
a1ded31
to
488ad68
Compare
/assign @christianvogt |
expect(createPipelineYamlError.isPresent()).toBe(false); | ||
await browser.wait(until.visibilityOf(pipelineOverviewName)); | ||
expect(pipelineOverviewName.getText()).toContain('new-pipeline'); | ||
await execSync(`oc create -f ./packages/dev-console/integration-tests/views/simple-pipeline-demo.yaml`); |
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.
This file is missing from the pull request.
simple-pipeline-demo.yaml
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.
Added now thanks
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.
OK - now the test runs cleanly for me - thx!
9e398a1
to
53f1a2c
Compare
lgtm (as soon as all the tests are passing ;-) |
So, I don't know a ton about running the protractor tests... I thought I needed to make my Chromedriver be the version of my Chrome installed (mac setup; so I can't install a separate instance). But I can't seem to get webdriver-manager to update to Chrome 79. Any assistance would be appreciated. I have some questions about the script but I don't have anyway to test my theories :) |
Get the version from Chrome => About menu |
/test analyze |
/test frontend |
@sanketpathak You'll want to run |
I can't seem to run these tests... If I try to run all of the tests, it freezes and then fails on pod delete.
If I try to run just devconsole, it fails on git import flow
If I redirect all tests to just Pipeline by changing the package.json to point at specifically the Pipeline scenario:
I get this error:
Clearly something is up with my environment. I'll have to debug this with someone from QE late Dec / next year. I'm off next week, but I can't validate this PR so I am not comfortable approving it. |
@andrewballantyne I think you are testing in a cluster where pipeline operator is already installed, so you are getting this error.
We can debug this if you still get any error while running the tests once you are back |
53f1a2c
to
1a516e1
Compare
/test e2e-gcp-console |
3 similar comments
/test e2e-gcp-console |
/test e2e-gcp-console |
/test e2e-gcp-console |
I tried running this again... no luck. It seemed to get further down the line but failed an assertion. @sanketpathak Perhaps we just need to go over this on bluejeans where you show me the tests pass. I hope it's just me, but we definitely don't need a test that fails sneaking it's way into the e2e tests for all PRs. |
/test e2e-gcp-console |
1a516e1
to
6aefdc0
Compare
/test analyze |
@sanketpathak Per our testing of your Pipeline, these are changes needed:
|
6aefdc0
to
12eb433
Compare
/test analyze |
12eb433
to
df26495
Compare
/test e2e-gcp-console |
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.
Works pretty good. My only concern is the still the roll of the dice to getting the operator installed. It's highly dependant on other operators / number of namespaces.
/lgtm I cannot give an approval due to the |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: andrewballantyne, christianvogt, sanketpathak The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/test analyze |
/retest Please review the full test history for this PR and help us cut down flakes. |
4 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
Pipeline automation : https://issues.redhat.com/browse/ODC-1506