Stepping back: the intent appears to be to compare regardless of order, so it's more robust and cleaner to restructure the test to not have branches for possible orderings.

Stepping back: the intent appears to be to compare regardless of order, so it's more robust and cleaner to restructure the test to not have branches for possible orderings.

I looked back at when this test was added, and read through the codebase better (based on your most recent feedback on my PR). Really, the goal of the test is to make sure that:

    def __repr__(self):
        return "ClientOptions: " + repr(self.__dict__)

Serializes the class with a ClientOptions prefix, followed by a string representation of the keys. It's overkill IMO to assert that every key is set, let's just switch to a simple regex:

#497

Fixed in #449, #492, thanks for your help with review.