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

Issue/6007 e2e removal #6069

Closed
wants to merge 8 commits into from
Closed

Issue/6007 e2e removal #6069

wants to merge 8 commits into from

Conversation

matborowczyk
Copy link
Collaborator

Description

Added e2e for inter-service relations, fixed issue with removing inter-service relations for instance that could have an array of those, added constraint to deploy button, fixed issue with relation removal didn't trigger the update of the cell

closes #6007

Self Check:

Strike through any lines that are not applicable (~~line~~) then check the box

  • Attached issue to pull request
  • Changelog entry
  • Code is clear and sufficiently documented
  • Sufficient test cases (reproduces the bug/tests the requested feature)
  • Correct, in line with design
  • End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )

cy.get('[aria-label="Environment card"]')
.contains("lsm-frontend")
.click();
cy.get(".pf-v5-c-nav__item").contains("Service Catalog").click();
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If we can avoid using the class, that would be great :)

@matborowczyk matborowczyk added the merge-tool-ready This ticket is ready to be merged in label Nov 26, 2024
@inmantaci
Copy link
Contributor

Processing this pull request

@inmantaci
Copy link
Contributor

Merged into branches master in 22d0e0a

inmantaci pushed a commit that referenced this pull request Nov 26, 2024
… PR #6069)

# Description

Added e2e for inter-service relations, fixed issue with removing inter-service relations for instance that could have an array of those, added constraint to deploy button, fixed issue with relation removal didn't trigger the update of the cell

closes #6007

# Self Check:

Strike through any lines that are not applicable (`~~line~~`) then check the box

- [ ] Attached issue to pull request
- [ ] Changelog entry
- [ ] Code is clear and sufficiently documented
- [ ] Sufficient test cases (reproduces the bug/tests the requested feature)
- [ ] Correct, in line with design
- [ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )
@inmantaci inmantaci closed this Nov 26, 2024
@inmantaci inmantaci deleted the issue/6007-e2e-removal branch November 26, 2024 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge-tool-ready This ticket is ready to be merged in
Projects
None yet
Development

Successfully merging this pull request may close these issues.

add e2e case for removal of inter-service relation
3 participants