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

JBEAP-24957 Removing Reset test for Configuration => Subsystem => Distributable Web => Configuration #35

Merged
merged 1 commit into from
Jun 13, 2023

Conversation

mchoma
Copy link
Contributor

@mchoma mchoma commented Jun 9, 2023

@mchoma
Copy link
Contributor Author

mchoma commented Jun 9, 2023

@spriadka please review. I am not able to assign for some reason.

it("Reset", () => {
cy.navigateTo(managementEndpoint, "distributable-web");
cy.get("#dw-configuration-item").click();
cy.resetForm(configurationFormId, managementEndpoint, ["subsystem", "distributable-web"]);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Alternatively this line can be written to expect current warning, but don't know how to write that.

Copy link
Contributor

Choose a reason for hiding this comment

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

Let's remove the test for now. We'll update this method in scope of #44.

@OndrejKotek
Copy link
Contributor

@mchoma Can you update the description of the PR and commit to be more descriptive please? Something like "JBEAP-24957 Removing Reset test for Configuration => Subsystem => Distributable Web => Configuration" could work.

@mchoma mchoma changed the title Turns out Reset is not expected to work in this context. JBEAP-24957 Removing Reset test for Configuration => Subsystem => Distributable Web => Configuration Jun 13, 2023
@mchoma
Copy link
Contributor Author

mchoma commented Jun 13, 2023

Can you update the description of the PR and commit

Done.

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.

None yet

2 participants