refactor: clean up testing files

[jsjoeio]

I accidentally merged a PR that didn't have a file formatted correctly and it made CI on main fail. This fixes that but also clean ups some testing files to be better organized.

[jawnsy]

Ah, just curious, shouldn't the same formatting checks run on PRs and main? How'd it get merged in the first place?

[jsjoeio]

I hit enable auto-merge which says "Automatically merge when all requirements are met." - let me checking the repo settings

[jsjoeio]

Weird, pre-build is not required, but should be? Enabling now: cc @oxy and @code-asher

[jsjoeio]

actually, i bet i broke it when i changed it to "Pre-Build" instead of "Pre-build"

[jsjoeio]

argh, auto update imports didn't work lol fixing

[jawnsy]

Haha, I dislike auto merge because of these sorts of problems, but I see how it can be helpful 🤷‍♂️

[jsjoeio]

I feel ya, luckily this has only happened once or twice

[jsjoeio]

Moment of truth

[codecov]

Codecov Report

Merging #3769 (0f03802) into main (002fa08) will increase coverage by 1.33%.
The diff coverage is 100.00%.

❗ Current head 0f03802 differs from pull request most recent head fc16f7e. Consider uploading reports for the commit fc16f7e to get more accurate results

@@            Coverage Diff             @@
##             main    #3769      +/-   ##
==========================================
+ Coverage   61.55%   62.89%   +1.33%     
==========================================
  Files          35       35              
  Lines        1813     1838      +25     
  Branches      365      371       +6     
==========================================
+ Hits         1116     1156      +40     
+ Misses        588      577      -11     
+ Partials      109      105       -4     

Impacted Files	Coverage Δ
src/browser/pages/vscode.ts	76.36% <100.00%> (+15.75%)	⬆️
src/node/util.ts	80.20% <100.00%> (+8.03%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a5cf018...fc16f7e. Read the comment docs.

[code-asher]

Hmm but we require both build and end-to-end tests which require pre-build? Is it sneaking in the auto-merge in the gaps before the next check starts??

[jsjoeio]

I guess it might be? Maybe it's a bug? I went into the settings and added "Pre-Build" as a required check. Maybe it's sneaking in because the previous run had both of those passing? I'm not too sure