fix!: update separators

[jly36963]

Should Close #35

[gustavoguichard]

good one!

[jly36963]

I wish for a DRYer way to write this test suite -- they're all the exact same.

There isn't a way to do the following, is there?

const text = 'abc'

const resultExpectedMap = new Map([
  [subject.toUpperCase(text), 'ABC'],
  // ...
]) 

for (const [result, expected] of resultExpectedMap.entries() {
  expect(result).toEqual(expected)
  type test = Expect<Equal<typeof result, typeof expected>>
}

[jly36963]

The closest I could get is this

Try 1

describe('with various separators', () => {
  const text = 'abc'

  const resultExpectedMap = new Map([
    [subject.toUpperCase(text), 'ABC' as const],
    [subject.toLowerCase(text), 'abc' as const],
    // ...
  ])

  for (const [result, expected] of resultExpectedMap.entries()) {
    expect(result).toEqual(expected)
    type test = Expect<Equal<typeof result, typeof expected>>
  }
})

The problem with the above code is that I could switch ABC and abc on the expected side of the map and the type union would still be equal

Try 2

describe('with various separators', () => {
  const text = 'abc'
  const a = {
    [subject.toUpperCase(text)]: 'ABC' as const,
    [subject.toLowerCase(text)]: 'abc' as const,
  } as const

  for (const [result, expected] of Object.entries(a)) {
    expect(result).toEqual(expected)
    type test = Expect<Equal<typeof result, typeof expected>>
  }
})

The problem with this one is that the type assertion fails (string !== 'abc' | 'ABC')

[gustavoguichard]

It could be easier... I personally am not a fan of too much abstractions in tests.. I tend to just repeat LOCs to avoid potential footguns.
But not against the trials

[gustavoguichard]

Lemme know when it is ready for review