Skip to content

Commit

Permalink
Merge pull request #883 from dolthub/liuliu/skip-corona-virus-db-for-now
Browse files Browse the repository at this point in the history
liuliu/skip-corona-virus-db-for-now
  • Loading branch information
liuliu-dev authored Oct 10, 2024
2 parents 78a8e16 + 8af8526 commit 4f135de
Show file tree
Hide file tree
Showing 27 changed files with 41 additions and 41 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ describe(`${pageName} renders expected components on different devices`, () => {
),
];

const skip = false;
const skip = true;
const devices = [macbook15ForAppLayout(pageName, tests, false, loggedIn)];
runTestsForDevices({ currentPage, devices, skip, loggedIn });
});
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,6 @@ describe(`${pageName} expected components on different devices`, () => {
];

const devices = [macbook15ForAppLayout(pageName, tests, false, loggedIn)];
const skip = false;
const skip = true;
runTestsForDevices({ currentPage, devices, skip, loggedIn });
});
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,6 @@ describe(`${pageName} renders expected components on different devices`, () => {

const devices = [macbook15ForAppLayout(pageName, tests, false, loggedIn)];

const skip = false;
const skip = true;
runTestsForDevices({ currentPage, devices, skip, loggedIn });
});
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,6 @@ describe(`${pageName} renders expected components on different devices`, () => {
];
const loggedIn = true;
const devices = [macbook15ForAppLayout(pageName, tests, false, loggedIn)];
const skip = false;
const skip = true;
runTestsForDevices({ currentPage, devices, skip, loggedIn });
});
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,6 @@ describe(`${pageName} renders expected components on different devices`, () => {
];

const devices = [macbook15ForAppLayout(pageName, tests, false, loggedIn)];
const skip = false;
const skip = true;
runTestsForDevices({ currentPage, devices, skip, loggedIn });
});
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,6 @@ describe(`${pageName} renders expected components on different devices`, () => {
];

const devices = [macbook15ForAppLayout(pageName, tests, false, loggedIn)];
const skip = false;
const skip = true;
runTestsForDevices({ currentPage, devices, skip, loggedIn });
});
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,6 @@ describe(`${pageName} renders expected components on different devices`, () => {
];

const devices = [macbook15ForAppLayout(pageName, tests, false, loggedIn)];
const skip = false;
const skip = true;
runTestsForDevices({ currentPage, devices, skip, loggedIn });
});
30 changes: 15 additions & 15 deletions cypress/e2e/dolthub/publicPaths/api/index.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ const selectQuery = "SELECT * FROM `places` ORDER BY place_id ASC LIMIT 5;";
const badQuery = "heatdome";

describe("API returns 404 for invalid endpoints", () => {
it("gets 404 responses from non-existent endpoints", () => {
xit("gets 404 responses from non-existent endpoints", () => {
cy.request({
url: `/api/`,
failOnStatusCode: false,
Expand All @@ -32,10 +32,10 @@ describe("API returns 404 for invalid endpoints", () => {

describe(`API returns query results for '${defaultQuery}' from ${defaultBranch} without branch or query specified`, () => {
const earl = `/api/${apiVersion}/${repoOwner}/${repoName}`;
it("gets a success response from the API", () => {
xit("gets a success response from the API", () => {
cy.request({ url: earl }).its("status").should("equal", 200);
});
it("contains the correct query metadata in the response body", () => {
xit("contains the correct query metadata in the response body", () => {
cy.request({ url: earl })
.its("body.sql_query")
.should("equal", `${defaultQuery}`);
Expand All @@ -55,7 +55,7 @@ describe(`API returns query results for '${defaultQuery}' from ${defaultBranch}
.its("body.query_execution_message")
.should("equal", "");
});
it("contains the correct query result schema in the response body", () => {
xit("contains the correct query result schema in the response body", () => {
cy.request({ url: earl })
.its("body.schema")
.should("deep.equal", [
Expand All @@ -65,7 +65,7 @@ describe(`API returns query results for '${defaultQuery}' from ${defaultBranch}
},
]);
});
it("contains the correct query result rows in the response body", () => {
xit("contains the correct query result rows in the response body", () => {
cy.request({ url: earl })
.its("body.rows")
.should("deep.equal", [
Expand Down Expand Up @@ -101,10 +101,10 @@ describe(`API returns query results for '${defaultQuery}' from ${defaultBranch}

describe(`API returns query results for '${defaultQuery}' from branch ${otherBranch} without query specified`, () => {
const earl = `/api/${apiVersion}/${repoOwner}/${repoName}/${otherBranch}`;
it("gets a success response from the API", () => {
xit("gets a success response from the API", () => {
cy.request({ url: earl }).its("status").should("equal", 200);
});
it("contains the correct query metadata in the response body", () => {
xit("contains the correct query metadata in the response body", () => {
cy.request({ url: earl })
.its("body.sql_query")
.should("equal", `${defaultQuery}`);
Expand All @@ -124,7 +124,7 @@ describe(`API returns query results for '${defaultQuery}' from branch ${otherBra
.its("body.query_execution_message")
.should("equal", "");
});
it("contains the correct query result schema in the response body", () => {
xit("contains the correct query result schema in the response body", () => {
cy.request({ url: earl })
.its("body.schema")
.should("deep.equal", [
Expand All @@ -134,7 +134,7 @@ describe(`API returns query results for '${defaultQuery}' from branch ${otherBra
},
]);
});
it("contains the correct query result rows in the response body", () => {
xit("contains the correct query result rows in the response body", () => {
cy.request({ url: earl })
.its("body.rows")
.should("deep.equal", [
Expand Down Expand Up @@ -163,10 +163,10 @@ describe(`API returns query results for '${selectQuery}' from ${defaultBranch} w
const earl = `/api/${apiVersion}/${repoOwner}/${repoName}?q=${encodeURI(
selectQuery,
)}`;
it("gets a success response from the API", () => {
xit("gets a success response from the API", () => {
cy.request({ url: earl }).its("status").should("equal", 200);
});
it("contains the correct query metadata in the response body", () => {
xit("contains the correct query metadata in the response body", () => {
cy.request({ url: earl })
.its("body.sql_query")
.should("equal", `${selectQuery}`);
Expand All @@ -186,7 +186,7 @@ describe(`API returns query results for '${selectQuery}' from ${defaultBranch} w
.its("body.query_execution_message")
.should("equal", "");
});
it("contains the correct query result schema in the response body", () => {
xit("contains the correct query result schema in the response body", () => {
cy.request({ url: earl })
.its("body.schema")
.should("deep.equal", [
Expand All @@ -212,7 +212,7 @@ describe(`API returns query results for '${selectQuery}' from ${defaultBranch} w
},
]);
});
it("contains the correct query result rows in the response body", () => {
xit("contains the correct query result rows in the response body", () => {
cy.request({ url: earl })
.its("body.rows")
.should("deep.equal", [
Expand Down Expand Up @@ -259,10 +259,10 @@ describe(`API returns query error for invalid query '${badQuery}'`, () => {
const earl = `/api/${apiVersion}/${repoOwner}/${repoName}?q=${encodeURI(
badQuery,
)}`;
it("gets a success response from the API", () => {
xit("gets a success response from the API", () => {
cy.request({ url: earl }).its("status").should("equal", 200);
});
it("contains the correct query metadata in the response body", () => {
xit("contains the correct query metadata in the response body", () => {
cy.request({ url: earl })
.its("body.sql_query")
.should("equal", `${badQuery}`);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ describe(`${pageName} renders expected components on different devices`, () => {
const devices = [
macbook15ForAppLayout(pageName, changeBranch(changeBranchParams)),
];
const skip = false;
const skip = true;
runTestsForDevices({
currentPage: `${currentPage}${currentBranch}`,
devices,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -97,6 +97,6 @@ describe(`${pageName} renders expected components on different devices`, () => {
iPad2ForAppLayout(pageName, tests),
iPhoneXForAppLayout(pageName, mobileTests),
];
const skip = false;
const skip = true;
runTestsForDevices({ currentPage, devices, skip });
});
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,6 @@ describe(`${pageName} renders expected components on different devices`, () => {
};

const devices = [macbook15ForAppLayout(pageName, changeBranch(testParams))];
const skip = false;
const skip = true;
runTestsForDevices({ currentPage, devices, skip });
});
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,6 @@ describe(`${pageName} renders expected components on different devices`, () => {
];

const devices = [macbook15ForAppLayout(pageName, tests())];
const skip = false;
const skip = true;
runTestsForDevices({ currentPage, devices, skip });
});
Original file line number Diff line number Diff line change
Expand Up @@ -115,6 +115,6 @@ describe(`${pageName} renders expected components on different devices`, () => {
iPad2ForAppLayout(pageName, tests),
iPhoneXForAppLayout(pageName, mobileTests),
];
const skip = false;
const skip = true;
runTestsForDevices({ currentPage, devices, skip });
});
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,6 @@ describe(`${pageName} expected components on different devices`, () => {
];

const devices = [macbook15ForAppLayout(pageName, tests)];
const skip = false;
const skip = true;
runTestsForDevices({ currentPage, devices, skip });
});
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,6 @@ describe(`${pageName} renders expected component on different devices`, () => {
];

const devices = [macbook15ForAppLayout(pageName, tests)];
const skip = false;
const skip = true;
runTestsForDevices({ currentPage, devices, skip });
});
Original file line number Diff line number Diff line change
Expand Up @@ -60,6 +60,6 @@ describe(`${pageName} renders expected component on different devices`, () => {
];

const devices = [macbook15ForAppLayout(pageName, tests)];
const skip = false;
const skip = true;
runTestsForDevices({ currentPage, devices, skip });
});
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,6 @@ describe(`${pageName} renders expected components on different devices`, () => {
];

const devices = [macbook15ForAppLayout(pageName, tests)];
const skip = false;
const skip = true;
runTestsForDevices({ currentPage, devices, skip });
});
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,6 @@ describe(`${pageName} renders expected components on different devices`, () => {
];

const devices = [macbook15ForAppLayout(pageName, tests)];
const skip = false;
const skip = true;
runTestsForDevices({ currentPage, devices, skip });
});
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,7 @@ describe(`${pageName} renders expected components on different devices`, () => {
),
),
];
const skip = false;
const skip = true;
runTestsForDevices({
currentPage: `${currentPage}${currentBranch}`,
devices,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ const currentPage = `repositories/${currentOwner}/${currentRepo}/pulls/${current
const destinationBranch = "archived";

// Need to investigate why the tests for this page are flaky
const skip = false;
const skip = true;

describe(`${pageName} renders expected components on different devices`, () => {
const beVisible = newShouldArgs("be.visible");
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ const currentRepo = "corona-virus";
const currentPullId = "1";
const currentPage = `repositories/${currentOwner}/${currentRepo}/pulls/${currentPullId}/compare`;

const skip = false;
const skip = true;

describe(`${pageName} renders expected components on different devices`, () => {
const beVisible = newShouldArgs("be.visible");
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ const currentPage = `repositories/${currentOwner}/${currentRepo}/pulls/${current

describe(`${pageName} renders expected components on different devices`, () => {
const beVisible = newShouldArgs("be.visible");
const skipAll = false;
const skipAll = true;

const tests = [
newExpectation(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,6 @@ describe(`${pageName} renders expected components on different devices`, () => {
];

const devices = [macbook15ForAppLayout(pageName, tests)];
const skip = false;
const skip = true;
runTestsForDevices({ currentPage, devices, skip });
});
Original file line number Diff line number Diff line change
Expand Up @@ -102,6 +102,6 @@ describe(`${pageName} renders expected components on different devices`, () => {
];

const devices = [macbook15ForAppLayout(pageName, tests)];
const skip = false;
const skip = true;
runTestsForDevices({ currentPage, devices, skip });
});
2 changes: 1 addition & 1 deletion cypress/e2e/dolthub/publicPaths/render/pulls/index.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -96,6 +96,6 @@ describe(`${pageName} renders expected components on different devices`, () => {
),
),
];
const skip = false;
const skip = true;
runTestsForDevices({ currentPage, devices, skip });
});
Original file line number Diff line number Diff line change
Expand Up @@ -80,6 +80,6 @@ describe(`${pageName} renders expected components on different devices`, () => {
];

const devices = [macbook15ForAppLayout(pageName, tests)];
const skip = false;
const skip = true;
runTestsForDevices({ currentPage, devices, skip });
});
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,6 @@ describe(`${pageName} renders expected components on different devices`, () => {
),
),
];
const skip = false;
const skip = true;
runTestsForDevices({ currentPage, devices, skip });
});

0 comments on commit 4f135de

Please sign in to comment.