Skip to content

Commit

Permalink
fix (#226)
Browse files Browse the repository at this point in the history
  • Loading branch information
sdotson committed Jan 27, 2022
1 parent f1a1f30 commit 6955f94
Show file tree
Hide file tree
Showing 2 changed files with 30 additions and 4 deletions.
4 changes: 0 additions & 4 deletions lib/deep-equal.js
Expand Up @@ -200,10 +200,6 @@ function deepEqualCyclic(actual, expectation, match) {
!isArrayType(expectation) &&
!isArguments(expectation);
if (isActualNonArrayIterable || isExpectationNonArrayIterable) {
if (!isActualNonArrayIterable || !isExpectationNonArrayIterable) {
return false;
}

var actualArray = Array.from(actualObj);
var expectationArray = Array.from(expectation);
if (actualArray.length !== expectationArray.length) {
Expand Down
30 changes: 30 additions & 0 deletions lib/is-iterable.test.js
@@ -0,0 +1,30 @@
"use strict";

var assert = require("@sinonjs/referee").assert;
var isIterable = require("./is-iterable");

describe("isIterable", function () {
it("returns false if not object", function () {
assert.isFalse(isIterable(undefined));
});

it("returns false if object does not have iterator function", function () {
assert.isFalse(isIterable({}));
});

it("returns true if Set", function () {
assert.isTrue(isIterable(new Set()));
});

it("returns true if Set iterator", function () {
assert.isTrue(isIterable(new Set().values()));
});

it("returns true if Map", function () {
assert.isTrue(isIterable(new Map()));
});

it("returns true if Map iterator", function () {
assert.isTrue(isIterable(new Map().values()));
});
});

0 comments on commit 6955f94

Please sign in to comment.