Are we confident all tests will follow this pattern? What if there are subsequent given after a when?

I'm hesitant to say we should have these comments in all tests, because I'm not sure how else you'd write the logic anyways.

This is a structure we've been using on the front-end (I believe @vapurrmaid introduced it) - in fact one of my first PRs had me add this to tests: https://github.com/coder/m/pull/9720/files#r697678791

Some tests have an extra given / when / then - @vapurrmaid would know some examples of that. Might be nice to have this documented in a front-end code standards page on Notion (our current standards are pretty empty)

https://en.wikipedia.org/wiki/Given-When-Then#:~:text=Given%2DWhen%2DThen%20(GWT,to%20write%20down%20test%20cases.&text=When%20describes%20actions%20taken%20by,part%20of%20behavior%2Ddriven%20development.

^ This is a fairly standard pattern.

You're welcome to follow up whens and thens

ie:

// Given - all your givens

// When - thing

// Then - outcome

// When - other thing

// Then - other outcome

I probably should have reviewed the other PR more thoroughly, but I'm hesitant to have a Page component, because if it were truly to be used on all pages, it would be idiomatic to put it in _app.tsx instead.

Ah good point, that <Page /> was leftover before I moved to next. I'll port this over to our _app part of this change.

Removed the <Page /> component here: 7b50658

I still left the Page folder to contain some helper stuff for the page (like the <Footer />) - but we can revisit that folder as we start building out the UI.