mirror of
https://github.com/zulip/zulip.git
synced 2025-11-13 02:17:19 +00:00
puppeteer: Remove failing linkifier tests.
I'm not sure that settings UI is particularly worth having puppeteer tests for, so it's possible we shouldn't bring these back at all. But in any case, it's worth disabling them as they've been failing for some time.
This commit is contained in:
@@ -1,140 +0,0 @@
|
||||
import {strict as assert} from "assert";
|
||||
|
||||
import type {Page} from "puppeteer";
|
||||
|
||||
import common from "../puppeteer_lib/common";
|
||||
|
||||
async function test_add_linkifier(page: Page): Promise<void> {
|
||||
await page.waitForSelector(".admin-linkifier-form", {visible: true});
|
||||
await common.fill_form(page, "form.admin-linkifier-form", {
|
||||
pattern: "#(?P<id>[0-9]+)",
|
||||
url_format_string: "https://trac.example.com/ticket/%(id)s",
|
||||
});
|
||||
await page.click("form.admin-linkifier-form button.button");
|
||||
|
||||
const admin_linkifier_status_selector = "div#admin-linkifier-status";
|
||||
await page.waitForSelector(admin_linkifier_status_selector, {visible: true});
|
||||
const admin_linkifier_status = await common.get_text_from_selector(
|
||||
page,
|
||||
admin_linkifier_status_selector,
|
||||
);
|
||||
assert.strictEqual(admin_linkifier_status, "Custom linkifier added!");
|
||||
|
||||
await page.waitForSelector(".linkifier_row", {visible: true});
|
||||
assert.strictEqual(
|
||||
await common.get_text_from_selector(page, ".linkifier_row span.linkifier_pattern"),
|
||||
"#(?P<id>[0-9]+)",
|
||||
);
|
||||
assert.strictEqual(
|
||||
await common.get_text_from_selector(
|
||||
page,
|
||||
".linkifier_row span.linkifier_url_format_string",
|
||||
),
|
||||
"https://trac.example.com/ticket/%(id)s",
|
||||
);
|
||||
}
|
||||
|
||||
async function test_delete_linkifier(page: Page): Promise<void> {
|
||||
await page.click(".linkifier_row .delete");
|
||||
await page.waitForSelector(".linkifier_row", {hidden: true});
|
||||
}
|
||||
|
||||
async function test_add_invalid_linkifier_pattern(page: Page): Promise<void> {
|
||||
await page.waitForSelector(".admin-linkifier-form", {visible: true});
|
||||
await common.fill_form(page, "form.admin-linkifier-form", {
|
||||
pattern: "a$",
|
||||
url_format_string: "https://trac.example.com/ticket/%(id)s",
|
||||
});
|
||||
await page.click("form.admin-linkifier-form button.button");
|
||||
|
||||
await page.waitForSelector("div#admin-linkifier-pattern-status", {visible: true});
|
||||
assert.strictEqual(
|
||||
await common.get_text_from_selector(page, "div#admin-linkifier-pattern-status"),
|
||||
"Failed: Invalid linkifier pattern. Valid characters are [ a-zA-Z_#=/:+!-].",
|
||||
);
|
||||
}
|
||||
|
||||
async function test_edit_linkifier(page: Page): Promise<void> {
|
||||
await page.click(".linkifier_row .edit");
|
||||
await page.waitForFunction(() => document.activeElement?.id === "linkifier-edit-form-modal");
|
||||
await common.fill_form(page, "form.linkifier-edit-form", {
|
||||
pattern: "(?P<num>[0-9a-f]{40})",
|
||||
url_format_string: "https://trac.example.com/commit/%(num)s",
|
||||
});
|
||||
await page.click(".submit-linkifier-info-change");
|
||||
|
||||
await page.waitForSelector("#linkifier-edit-form-modal", {hidden: true});
|
||||
await page.waitForSelector(".linkifier_row", {visible: true});
|
||||
await page.waitForFunction(
|
||||
() => document.querySelector(".linkifier_pattern")?.textContent === "(?P<num>[0-9a-f]{40})",
|
||||
);
|
||||
assert.strictEqual(
|
||||
await common.get_text_from_selector(
|
||||
page,
|
||||
".linkifier_row span.linkifier_url_format_string",
|
||||
),
|
||||
"https://trac.example.com/commit/%(num)s",
|
||||
);
|
||||
}
|
||||
|
||||
async function test_edit_invalid_linkifier(page: Page): Promise<void> {
|
||||
await page.click(".linkifier_row .edit");
|
||||
await page.waitForFunction(() => document.activeElement?.id === "linkifier-edit-form-modal");
|
||||
await common.fill_form(page, "form.linkifier-edit-form", {
|
||||
pattern: "####",
|
||||
url_format_string: "####",
|
||||
});
|
||||
await page.click(".submit-linkifier-info-change");
|
||||
|
||||
const edit_linkifier_pattern_status_selector = "div#edit-linkifier-pattern-status";
|
||||
await page.waitForSelector(edit_linkifier_pattern_status_selector, {visible: true});
|
||||
const edit_linkifier_pattern_status = await common.get_text_from_selector(
|
||||
page,
|
||||
edit_linkifier_pattern_status_selector,
|
||||
);
|
||||
assert.strictEqual(
|
||||
edit_linkifier_pattern_status,
|
||||
"Failed: Invalid linkifier pattern. Valid characters are [ a-zA-Z_#=/:+!-].",
|
||||
);
|
||||
|
||||
const edit_linkifier_format_status_selector = "div#edit-linkifier-format-status";
|
||||
await page.waitForSelector(edit_linkifier_format_status_selector, {visible: true});
|
||||
const edit_linkifier_format_status = await common.get_text_from_selector(
|
||||
page,
|
||||
edit_linkifier_format_status_selector,
|
||||
);
|
||||
assert.strictEqual(
|
||||
edit_linkifier_format_status,
|
||||
"Failed: Enter a valid URL.,Invalid URL format string.",
|
||||
);
|
||||
|
||||
await page.click(".cancel-linkifier-info-change");
|
||||
await page.waitForSelector("#linkifier-edit-form-modal", {hidden: true});
|
||||
|
||||
await page.waitForSelector(".linkifier_row", {visible: true});
|
||||
assert.strictEqual(
|
||||
await common.get_text_from_selector(page, ".linkifier_row span.linkifier_pattern"),
|
||||
"(?P<num>[0-9a-f]{40})",
|
||||
);
|
||||
assert.strictEqual(
|
||||
await common.get_text_from_selector(
|
||||
page,
|
||||
".linkifier_row span.linkifier_url_format_string",
|
||||
),
|
||||
"https://trac.example.com/commit/%(num)s",
|
||||
);
|
||||
}
|
||||
|
||||
async function linkifier_test(page: Page): Promise<void> {
|
||||
await common.log_in(page);
|
||||
await common.manage_organization(page);
|
||||
await page.click("li[data-section='linkifier-settings']");
|
||||
|
||||
await test_add_linkifier(page);
|
||||
await test_edit_linkifier(page);
|
||||
await test_edit_invalid_linkifier(page);
|
||||
await test_add_invalid_linkifier_pattern(page);
|
||||
await test_delete_linkifier(page);
|
||||
}
|
||||
|
||||
common.run_test(linkifier_test);
|
||||
Reference in New Issue
Block a user