opensource/galaxy
3
0
Fork 0
mirror of https://github.com/galaxyproject/galaxy.git synced 2026-09-24 16:30:27 +08:00
Code Issues Packages Projects Releases Wiki Activity

Update expected tooltip text for switch action

Browse Source 
Update SwitchToHistoryLink tests to reflect the new tooltip
label "Switch to history and view dataset" when switching
histories with filters applied, ensuring assertions match
current UI behavior and intent.
This commit is contained in:
Alireza Heidari
2026-01-28 23:48:30 +01:00
parent 386b950b22
commit 936113eb76
1 changed files with 9 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
client/src/components/History/SwitchToHistoryLink.test.ts
+9 -5
View File
@@ -127,7 +127,11 @@ function mountSwitchToHistoryLinkForHistory(history: HistorySummaryExtended, has
* @param setsFilters Whether filters are applied to the current history on click
*/
async function expectActionForHistory(
tooltip: "Switch to this history" | "This is your current history" | "View in new tab" | "Show in history",
tooltip:
| "Switch to this history"
| "This is your current history"
| "View in new tab"
| "Switch to history and view dataset",
history: HistorySummaryExtended,
opensInNewTab = false,
hasFilters = false,
@@ -192,7 +196,7 @@ describe("SwitchToHistoryLink", () => {
await expectActionForHistory("Switch to this history", history, false, false, true, false);
// Since history was not current, we switch to it AND apply filters
await expectActionForHistory("Show in history", history, false, true, true, true);
await expectActionForHistory("Switch to history and view dataset", history, false, true, true, true);
});
it("only applies filters when the history is the Current history", async () => {
@@ -209,7 +213,7 @@ describe("SwitchToHistoryLink", () => {
await expectActionForHistory("This is your current history", history);
// Since history is already current, we only apply filters
await expectActionForHistory("Show in history", history, false, true, false, true);
await expectActionForHistory("Switch to history and view dataset", history, false, true, false, true);
});
it("opens purged history in new tab or applies filters", async () => {
@@ -226,7 +230,7 @@ describe("SwitchToHistoryLink", () => {
await expectActionForHistory("View in new tab", history, true);
// We switch to the purged history and apply filters
await expectActionForHistory("Show in history", history, false, true, true, true);
await expectActionForHistory("Switch to history and view dataset", history, false, true, true, true);
});
it("opens archived history in new tab or applies filters", async () => {
@@ -243,7 +247,7 @@ describe("SwitchToHistoryLink", () => {
await expectActionForHistory("View in new tab", history, true);
// We switch to the archived history and apply filters
await expectActionForHistory("Show in history", history, false, true, true, true);
await expectActionForHistory("Switch to history and view dataset", history, false, true, true, true);
});
it("only opens an accessible unowned history in new tab", async () => {