DOH assertEquals function deal with undefined in object in a incorrect way.

[Programus]

When I compare two objects with properties which value is undefined, the doh.assertEquals gives incorrect results. Here are some examples:

doh.assertEqual({a:undefined, b:'b'},{a:undefined, b:'b'}); // AssertFailure

doh.assertEqual({a:null, b:'b'},{a:undefined, b:'b'}); // true

doh.assertEqual({a:undefined, b:'b'},{a:null, b:'b'}); // AssertFailure

doh.assertEqual({a:undefined, b:'b'},{b:'b'}); // true

doh.assertEqual({a:null, b:'b'},{a:null, b:'b'}); // true <- this makes sense

doh.assertEqual({a:undefined},{a:undefined}); // AssertFailure

doh.assertEqual({a:null},{a:undefined}); // true

doh.assertEqual({a:undefined},{a:null}); // AssertFailure

doh.assertEqual({a:undefined},{}); // true

doh.assertEqual({a:null},{a:null}); // true <- this makes sense

[Programus]

One supplement about the version. In the package.json file under doh folder, there is a version information said "1.7.0dev", but the doh and utils are extracted from the 1.7.3 SDK source zip file.

[haysmark]

The original code for comparing objects was just:

doh._objPropEq = function(expected, actual){
	for(var x in expected){
		if(!doh.assertEqual(expected[x], actual[x])){
			return false;
		}
	}
	return true;
}

This was a sound but not complete comparison. In trunk on line 990 in runner.js doh._objPropEq wolfram tried to make the comparison complete:

for(x in actual){ // Lets check "actual" here, expected is checked below.
	if(expected[x] === undefined){
		return false;
	}
}

I suspect this code is what causes things to break. I wonder if a better test would be to enumerate the keys in a for-in loop and verify the keys are the same.

[haysmark]

I'm surprised you are using undefined instead of null in your code.

[bill]

Sounds like just need to say !(x in expected) rather than expected[x] === undefined. I'll check that in, in the interests of closing this ticket.

[bill]

In [30843]:

fix logic for comparing objects in DOH, fixes #15839 !strict