Testing isinstance

[sanders41]

There are several tests in the test suite that test if a value is an instance of object. In Python everything behind the scenes in an object so these tests are always going to be True. For example isinstance('test', object), isinstance(1, object), and isinstance({"a": 1}, object will all return True.

I am assuming the thinking for these tests was object in a javascript context since many are testing what should be something like {"updateId": 1}. In this case the assert would instead be assert isinstance(test_value, dict) instead of assert isinstance(test_value, object)

[bidoubiwa]

Hey @sanders41, you are right. I think we might also have done it in a quick way to ensure it does not throw an error. But this should definitively be improved.

[eskombro]

Absolutely 👌🏽