TODO: vitest conventions

[brandonjpierce]

A common pattern we have is doing something like:

const pairs = [
  ['test title', 123],
  ['test title', 123],
  ['test title', 123],
  ['test title', 123],
]

it('the test', () => {
  for (const pair of pairs) {
    expect(pair[1]).toEqual(123);
  } 
});

However, vitest does have it.each which works on pairs like datasets. Should we switch to that method instead?

[brandonjpierce]

Perhaps move this to a discussion instead

[belsrc]

It also has ... which is what I prefer to use.

describe('description...', () => {
  for (const [a, b] of TESTS) {
    it.concurrent( ... )
  }
}

https://vitest.dev/api/#test-concurrent

Additionally, can mark the entire inner describe as well. https://vitest.dev/api/#describe-concurrent