starred/openobserve
1
0
Fork 0
mirror of https://github.com/openobserve/openobserve.git synced 2026-11-06 00:09:22 +00:00
Code Issues Packages Projects Releases Wiki Activity

test: fixed logs breaking testcases (#2538)

Browse Source 
- Fixed 3 testcases under logs that were breaking due to the element
getting hidden from div
This commit is contained in:
Neha P
2024-01-24 11:26:47 +05:30
committed by GitHub
parent 99ec2221d8
commit e752baf02e
4 changed files with 28 additions and 24 deletions
Download Patch File Download Diff File Expand all files Collapse all files
.github/workflows/ui-testing.yml
+2 -1
View File
@@ -19,6 +19,7 @@ env:
ZO_BASE_URL: http://localhost:5080
# these are cypress related env vars
CYPRESS_ORGNAME: default
CYPRESS_CLOUD_TOKEN: ${{ secrets.CYPRESS_CLOUD_TOKEN }}
jobs:
ui_integration_tests:
@@ -77,4 +78,4 @@ jobs:
# browser: ${{ matrix.browser }}
# spec: cypress/e2e/tests
- name: Cypress run
run: cd tests/ui-testing && npm install && npx cypress run --env EMAIL=${ZO_ROOT_USER_EMAIL},PASSWORD=${ZO_ROOT_USER_PASSWORD},BASEURL=${ZO_BASE_URL},ORGNAME=${CYPRESS_ORGNAME} --headless --browser ${{ matrix.browser }} --spec ./cypress/e2e/tests/logs_tests.cy.js
run: cd tests/ui-testing && npm install && npx cypress run --record --key ${CYPRESS_CLOUD_TOKEN} --env EMAIL=${ZO_ROOT_USER_EMAIL},PASSWORD=${ZO_ROOT_USER_PASSWORD},BASEURL=${ZO_BASE_URL},ORGNAME=${CYPRESS_ORGNAME} --headless --browser ${{ matrix.browser }} --spec ./cypress/e2e/tests/logs_tests.cy.js
tests/ui-testing/cypress.config.js
+2 -2
View File
@@ -6,7 +6,7 @@ try {
} catch (err) {}
module.exports = defineConfig({
projectId: 'fmd3am',
projectId: '64nwum',
//defaultCommandTimeout:4000,
e2e: {
setupNodeEvents(on, config) {
@@ -19,7 +19,7 @@ module.exports = defineConfig({
"experimentalOriginDependencies":true,
"testIsolation": true,
"experimentalRunAllSpecs": true,
"projectId": 'fmd3am',
"projectId": '64nwum',
"viewportHeight": 1024,
"viewportWidth": 1280
},
tests/ui-testing/cypress/e2e/allfunctions/logs.js
+1 -1
View File
@@ -41,7 +41,7 @@ export function clickHistogramToggle() {
/** Verify if the chart is hidden */
export function confirmLogsSearchHidden() {
cy.get(logsSearchBarChart, { timeout: 2000 }).should("be.hidden");
cy.get(logsSearchBarChart, { timeout: 2000 }).should("not.exist",{ timeout: 10000, interval: 1000 });
}
/** Verify if vrl function editor is visible */
tests/ui-testing/cypress/e2e/tests/logs_tests.cy.js
+23 -20
View File
@@ -71,10 +71,11 @@ describe("Logs testcases", () => {
cy.intercept("GET", "**/api/default/organizations**").as("allorgs");
cy.intercept("GET", "**/api/default/functions**").as("functions");
cy.visit(`${logData.logsUrl}?org_identifier=${Cypress.env("ORGNAME")}`);
cy.intercept('POST', '**/api/default/_search**').as('allsearch')
cy.wait("@allsearch")
cy.selectStreamAndStreamTypeForLogs(logData.Stream);
cy.intercept("GET", "**/api/default/streams**").as("streams");
cy.intercept('GET', '**/api/default/_search**').as('allsearch')
});
// This is a test case to navigate to the logs page
@@ -90,21 +91,23 @@ describe("Logs testcases", () => {
});
// This test checks if the histogram toggle button works correctly by clicking it and verifying that the chart is hidden.
it("Should toggle chart when clicking on the histogram toggled", () => {
logstests.clickHistogramToggle();
logstests.confirmLogsSearchHidden();
});
// This test checks that clicking on the histogram toggle button in SQL mode does not toggle the chart
it("Should not toggle chart when clicking on the histogram toggle in the sql mode", () => {
it("should not toggle chart when clicking on the histogram toggle in the sql mode", () => {
logstests.clickHistogramToggle();
logstests.clickSearchBarSqlMode();
logstests.confirmLogsSearchHidden();
logstests.histogramToggleDisabled();
});
it.skip("should toggle chart when clicking on the histogram toggle", () => {
cy.wait(3000)
logstests.clickHistogramToggle();
cy.wait(3000)
logstests.confirmLogsSearchHidden();
});
// This test case checks if the function editor is toggled on/off when the 'functions toggle' button is clicked
it("Should toggle the function editor on functions toggle click", () => {
it("should toggle the function editor on functions toggle click", () => {
logstests.displayVrlFunctionEditor();
logstests.clickLogsSearchQueryToggle();
logstests.vrlFunctionEditorHidden();
@@ -112,7 +115,7 @@ describe("Logs testcases", () => {
logstests.displayVrlFunctionEditor();
});
it("Should clear the value of a function on functions toggle click", () => {
it("should clear the value of a function on functions toggle click", () => {
// Wait for 5 seconds
cy.wait(5000);
logstests.enterVrlFunctionvalue();
@@ -121,7 +124,7 @@ describe("Logs testcases", () => {
logstests.attributeToNotHaveVrlFunctionvalue();
});
it("Should load the values of a field when the field is expanded", () => {
it("should load the values of a field when the field is expanded", () => {
cy.get('[data-cy="date-time-button"]').click({ force: true });
cy.get('[data-test="date-time-relative-6-w-btn"] > .q-btn__content').click({
force: true,
@@ -134,7 +137,7 @@ describe("Logs testcases", () => {
);
});
it("Should add the field to the editor when + is clicked", () => {
it("should add the field to the editor when + is clicked", () => {
// Wait for 5 seconds
cy.wait(4000);
// get the value of the field name
@@ -151,7 +154,7 @@ describe("Logs testcases", () => {
applyQueryButton();
});
it("Should add the value to the editor when the = is clicked next to the field value", () => {
it("should add the value to the editor when the = is clicked next to the field value", () => {
// Wait for 2 seconds
cy.wait(2000);
// Type the value of a variable into an input field
@@ -184,7 +187,7 @@ describe("Logs testcases", () => {
applyQueryButton();
});
it("Should add the value to the editor when the != is clicked next to the field value", () => {
it("should add the value to the editor when the != is clicked next to the field value", () => {
// Wait for 2 seconds
cy.wait(5000);
// Type the value of a variable into an input field
@@ -251,7 +254,7 @@ describe("Logs testcases", () => {
applyQueryButton();
});
it("Should add invalid query and display error", () => {
it("should add invalid query and display error", () => {
// Type the value of a variable into an input field
cy.intercept("GET", logData.ValueQuery).as("value");
@@ -270,7 +273,7 @@ describe("Logs testcases", () => {
cy.get('[data-test="logs-search-error-message"]').should("be.visible");
});
it("Should click run query after SQL toggle on but without any query", () => {
it("should click run query after SQL toggle on but without any query", () => {
// Wait for 2 seconds
cy.wait(3000);
// Type the value of a variable into an input field
@@ -285,7 +288,7 @@ describe("Logs testcases", () => {
cy.contains("Invalid SQL Syntax").should("be.visible");
});
it("Should enter a valid SQL query", () => {
it("should enter a valid SQL query", () => {
// Wait for 2 seconds
cy.wait(2000);
// Type the value of a variable into an input field
@@ -328,7 +331,7 @@ describe("Logs testcases", () => {
applyQueryButton();
});
it("Should contain options to include, exclude and add field to table under Json", () => {
it("should contain options to include, exclude and add field to table under Json", () => {
// Wait for 2 seconds
cy.wait(2000);
// Type the value of a variable into an input field
@@ -419,7 +422,7 @@ describe("Logs testcases", () => {
});
});
it("Should contain options to include, exclude and add field to table under TABLE", () => {
it("should contain options to include, exclude and add field to table under TABLE", () => {
// Wait for 2 seconds
cy.wait(2000);
// Type the value of a variable into an input field
@@ -998,7 +1001,7 @@ describe("Logs testcases", () => {
cy.get('[aria-label="SQL Mode"] > .q-toggle__label').click({ force: true });
applyQueryButton();
cy.get('[aria-label="SQL Mode"] > .q-toggle__label').click({ force: true });
cy.get('.q-spinner').should('not.be.visible')
cy.get('.q-spinner').should('not.exist')
});
// it.only("should enter function, edit and then delete the function", () => {