changelog.md

@@ -18,30 +18,37 @@ Change log
 
 ### Upcoming
 
-- *Nothing so far*
+- *Noothing yet*
+
+### 4.1.6
+_2022-05-19_
+
+- `.deepEquals()` now ignores non-enumerable keys. This makes `seamless-immutable` objects comparable and fixes [#24](https://github.com/MithrilJS/ospec/issues/24).
 
 ### 4.1.5
+_2022-05-19_
 
-#### Bug fixes
+#### Bug fix
 
 - Properly interpolate values when using assertions to tag templages
 
   ```JS
   o(x).equals(y)`Description that interpolates ${context}`
   ```
 
-  Will work as expected. This fixes #43
+  Will work as expected. This fixes [#43](https://github.com/MithrilJS/ospec/issues/43)
 
 ### 4.1.4
+_2022-05-19_
 
-#### Bug fixes
+#### Bug fix
 
 - Work around a Rollup limitation, fixes [#25](https://github.com/MithrilJS/ospec/issues/25) Thanks to [Ivan Kupalov](https://github.com/charlag) for the report and preliminary fix.
 - Properly handle objects with a null prototype in `.deepEquals` assertions. Fixes [#41](https://github.com/MithrilJS/ospec/issues/41)
 
 ### 4.1.3
 
-#### Bug fixes
+#### Bug fix
 _2022-05-18_
 - Fix post-install crash introduced in v4.1.2
 

ospec.js

@@ -526,7 +526,11 @@ else window.o = m()
 			return true
 		}
 	}
-
+	function getEnumerableProps(x) {
+		var desc = Object.getOwnPropertyDescriptors(x)
+		return Object.keys(desc).filter(function(k){return desc[k].enumerable})
+	}
+
 	function deepEqual(a, b) {
 		if (a === b) return true
 		if (a === null ^ b === null || a === undefined ^ b === undefined) return false // eslint-disable-line no-bitwise
@@ -542,7 +546,7 @@ else window.o = m()
 				return true
 			}
 			if (a.length === b.length && (Array.isArray(a) && Array.isArray(b) || aIsArgs && bIsArgs)) {
-				var aKeys = Object.getOwnPropertyNames(a), bKeys = Object.getOwnPropertyNames(b)
+				var aKeys = getEnumerableProps(a), bKeys = getEnumerableProps(b)
 				if (aKeys.length !== bKeys.length) return false
 				for (var i = 0; i < aKeys.length; i++) {
 					if (!hasOwn.call(b, aKeys[i]) || !deepEqual(a[aKeys[i]], b[aKeys[i]])) return false

package.json

@@ -1,6 +1,6 @@
 {
   "name": "ospec",
-  "version": "4.1.5",
+  "version": "4.1.6",
   "description": "Noiseless testing framework",
   "main": "ospec.js",
   "unpkg": "ospec.js",
@@ -34,6 +34,6 @@
     "cmd-shim": "4.0.2",
     "compose-regexp": "^0.6.22",
     "eslint": "^6.8.0",
-    "ospec-stable": "npm:ospec@4.1.4"
+    "ospec-stable": "npm:ospec@4.1.5"
   }
 }

tests/test-api.js

@@ -60,7 +60,7 @@ o.spec("no output", function() {
 	if (spec.toString().indexOf("console") !== -1) {
 		throw new Error("Avoid referencing the console in the 'no output' test suite. For logging, use the `LOG(...)` helper.")
 	} else {
-		o.spec("", spec)
+		o.spec("-", spec)
 	}
 	function spec(){
 		o("API", function() {
@@ -480,6 +480,19 @@ o.spec("no output", function() {
 						oo({__proto__: null}).notDeepEquals({__proto__: null, a: 1})
 						oo({__proto__: null, a: 1}).notDeepEquals({__proto__: null})
 
+						// #29
+						var im1 = [1, 2]
+						Object.defineProperty(im1, "x", {value: 5, enumrable: false})
+						var im2 = [1, 2]
+						Object.defineProperty(im2, "x", {value: 4, enumrable: false})
+						oo(im1).deepEquals(im2)
+
+						var im3 = {y: 4, z: 5}
+						Object.defineProperty(im3, "x", {value: 5, enumrable: false})
+						var im4 = {y: 4, z: 5}
+						Object.defineProperty(im4, "x", {value: 4, enumrable: false})
+						oo(im3).deepEquals(im4)
+
 
 						oo(function(){throw new Error()}).throws(Error)
 						oo(function(){"ayy".foo()}).throws(TypeError)
@@ -1338,11 +1351,22 @@ o.spec("no output", function() {
 					oo("description", function() {
 						oo(1).equals(2)("howdy")
 					})
+					try {
+						eval("const a = `${5}`")
+						eval("oo('taggedTemplate', ()=>{oo(1).equals(2)`h${'o'}w${'d'}y`})")
+					} catch(e) {
+						oo("taggedTemaplate", function() {
+							oo(1).equals(2)(['h', 'w', 'y'], 'o', 'd')
+
+						})	
+					}
 					oo.run(function(results) {
 						try {
-							o(results.length).equals(2)
+							o(results.length).equals(3)
 							o(results[1].message).equals("howdy\n\n"+results[0].message)
 							o(results[1].pass).equals(false)
+							o(results[2].message).equals("howdy\n\n"+results[0].message)
+							o(results[2].pass).equals(false)
 
 							done()
 						} catch (e) {