Wrong comparison of Option with vitest's expect(...).not.toStrictEqual(...) about effect HOT 9 OPEN

vitest-dev/vitest#5620

from effect.

Started playing with it in the effect codebase and seems like the expect(<option>).toStrictEqual(<another option>) always succeeds independently of what are the option values, for example this succeeds in the test/Option.test.ts

    expect(_.some(1)).toStrictEqual(_.some(2))
    expect(_.none()).toStrictEqual(_.some(2))

from effect.

Okay, seems to be the Symbol.iterator that makes the difference - which makes sense, that is the thing introduced in 3.0.4 to enable to yield without an adapter, right?

  [Symbol.iterator]() {
    return new SingleShotGen.SingleShotGen(this) as any
  },

When I remove it from the Effect prototype the comparison starts working. cc @mikearnaldi

from effect.

Yup that comes from 3.0.4, I don't see how this could ever be considered a bug of effect, it's behaviour of expect...

from effect.

The Effect way of comparing options is Equals.equals(a, b)

from effect.

In this codebase you rely on the expect().toStrictEqual() / Util.deepStrictEqual etc checks in the tests as well. Therefore, from 3.0.4, tests in here doing checks on options actually test nothing because they all always succeed. And this will apply for all the consumers of effect lib as well.

from effect.

Yeah but that's a bug in expect not in Effect... like I am trying to compare two objects with a[Symbol.iterator] = [1, 2, 3, Math.random()][Symbol.iterator] and it says that they are equal, why on earth???

from effect.

seems like a vitest issue, jest had a similar jestjs/jest#8280

from effect.

The following works:

import { jestExpect as expect } from "@jest/expect"
import { describe, it } from "vitest"

describe("Vitest Issue", () => {
  it("compares", () => {
    const a = {
      value: 0,
      [Symbol.iterator]: [0, 1, 2, Math.random()][Symbol.iterator]
    }
    const b = {
      value: 1,
      [Symbol.iterator]: [0, 1, 2, Math.random()][Symbol.iterator]
    }
    expect(a).toStrictEqual(b)
  })
})

from effect.