Skip to content
This repository has been archived by the owner on Dec 13, 2021. It is now read-only.

Commit

Permalink
switched to contains to minimize failures on admin_setup (metabase#13035
Browse files Browse the repository at this point in the history
)
  • Loading branch information
Opalevanescence authored Jul 31, 2020
1 parent 55575cf commit 24775a3
Showing 1 changed file with 4 additions and 5 deletions.
9 changes: 4 additions & 5 deletions frontend/test/metabase-smoketest/admin_setup.cy.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -890,8 +890,7 @@ describe("smoketest > admin_setup", () => {
cy.findByLabelText("Name")
.clear()
.wait(1)
.type("sub-collection question");
cy.findByText("sub-collection question");
.type(" sub-collection question ");
cy.findByText("Robert Tableton's Personal Collection").click();

cy.findByText("My personal collection");
Expand All @@ -902,12 +901,12 @@ describe("smoketest > admin_setup", () => {
.click();
cy.findByText("Not now").click();

cy.findByText("test sub-collection").click();
cy.contains("test sub-collection").click();

cy.findByText("Sorry, you don’t have permission to see that.").should(
"not.exist",
);
cy.findByText("sub-collection question");
cy.contains("sub-collection question");

// Check access as no collection user

Expand All @@ -929,7 +928,7 @@ describe("smoketest > admin_setup", () => {
signIn("nocollection");

cy.visit("/question/4");
cy.findByText("sub-collection question").should("not.exist");
cy.contains("sub-collection question").should("not.exist");
cy.findByText("Sorry, you don’t have permission to see that.");
});

Expand Down

0 comments on commit 24775a3

Please sign in to comment.