We might want to instead grab the link from that button, modify it with the query parameter, then click it.

I thought about that, but I'm not crazy about the feeling of reaching into the app, making changes from the outside, and then using this newly tweaked behavior as the one that we test. 😅 Feels a bit too much like we're not testing the button anymore.

Also it doesn't even look like that's something that Playwright supports, but I might not be seeing something. I guess you can use evaluateHandle but that just further signals to me that this isn't a thing you're intended to do.

I tried it anyway just to see, and something is getting lost in translation.

    // This assertion passes, everything seems fine.
    expect(await page.getByTestId("terminal").getAttribute("href")).toBe(
      `/@admin/${workspaceName}.dev/terminal${commandQuery}`,
    );
    // This one does not. Somehow, the change isn't _actually_ taking effect, even though it seems to be reflected.
    await expect(terminal).toHaveURL(
      `/@admin/${workspaceName}.dev/terminal${commandQuery}`,
    );

...so I'm just gonna go with the way it is. 😅