Skip to content
Snippets Groups Projects
Unverified Commit aa77fcad authored by Nemanja Glumac's avatar Nemanja Glumac Committed by GitHub
Browse files

Fix check for disabled button in `add.cy.spec.js` (#14833)

parent 17e3223e
No related branches found
No related tags found
No related merge requests found
......@@ -43,12 +43,11 @@ describe("scenarios > admin > databases > add", () => {
typeField("Database name", "test_postgres_db");
typeField("Username", "uberadmin");
cy.findByText("Save")
cy.findByRole("button", { name: "Save" })
.should("not.be.disabled")
.click();
cy.wait("@createDatabase");
cy.url().should("match", /\/admin\/databases\?created=42$/);
});
......@@ -59,11 +58,13 @@ describe("scenarios > admin > databases > add", () => {
typeField("Database name", "test_postgres_db");
typeField("Username", "uberadmin");
cy.findByText("Save").should("not.be.disabled");
cy.findByRole("button", { name: "Save" })
.should("not.be.disabled")
.click();
toggleFieldWithDisplayName("let me choose when Metabase syncs and scans");
cy.findByText("Next")
cy.findByRole("button", { name: "Next" })
.should("not.be.disabled")
.click();
......@@ -82,17 +83,17 @@ describe("scenarios > admin > databases > add", () => {
typeField("Database name", "test_postgres_db");
typeField("Username", "uberadmin");
cy.findByText("Save").should("not.be.disabled");
cy.findByRole("button", { name: "Save" }).should("not.be.disabled");
toggleFieldWithDisplayName("let me choose when Metabase syncs and scans");
cy.findByText("Next")
cy.findByRole("button", { name: "Next" })
.should("not.be.disabled")
.click();
cy.findByText("Never, I'll do this manually if I need to").click();
cy.findByText("Save").click();
cy.findByRole("button", { name: "Save" }).click();
cy.wait("@createDatabase").then(({ request }) => {
expect(request.body.engine).to.equal("postgres");
......@@ -101,8 +102,7 @@ describe("scenarios > admin > databases > add", () => {
});
cy.url().should("match", /\/admin\/databases\?created=42$/);
cy.findByText("Your database has been added!").should("exist");
cy.findByText("Your database has been added!");
});
it("should show error correctly on server error", () => {
......@@ -120,10 +120,9 @@ describe("scenarios > admin > databases > add", () => {
typeField("Database name", "test_postgres_db");
typeField("Username", "uberadmin");
cy.findByText("Save").click();
cy.findByRole("button", { name: "Save" }).click();
cy.wait("@createDatabase");
cy.findByText("DATABASE CONNECTION ERROR").should("exist");
});
......@@ -167,7 +166,7 @@ describe("scenarios > admin > databases > add", () => {
}).as("createDatabase");
// submit form and check that the file's body is included
cy.contains("Save").click();
cy.findByRole("button", { name: "Save" }).click();
cy.wait("@createDatabase").should(xhr => {
expect(xhr.request.body.details["service-account-json"]).to.equal(
'{"foo": 123}',
......@@ -197,7 +196,6 @@ describe("scenarios > admin > databases > add", () => {
cy.visit("/admin/databases/123");
cy.contains("Connect to a Service Account instead");
cy.contains("generate a Client ID and Client Secret for your project");
});
});
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment