Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Declarative usage ofin:A,B does not validate B #73

Open
christoph-kluge opened this issue Feb 4, 2025 · 0 comments
Open

Declarative usage ofin:A,B does not validate B #73

christoph-kluge opened this issue Feb 4, 2025 · 0 comments

Comments

@christoph-kluge
Copy link

christoph-kluge commented Feb 4, 2025

@ozziest it seems that #70 and #71 didn't solved the issue described in #67.

I checked quickly my test-case in #67 and I realised that the test-case didn't represented the bug correctly. The declarative way only validates the first option correctly but does not include all other options. Here's the updated test, which validates that the bug still exists.

  it("should be able to work with spread-string options", async () => {
-    expect(isIn("A", ...["A,B,B"])).toBe(true);
-    expect(isIn("A", ...["A","B","C"])).toBe(true);
+    expect(isIn("B", ...["A,B,B"])).toBe(true);
+    expect(isIn("B", ...["A","B","C"])).toBe(true);
  });
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant