Can't this function be defined outside the addMethod anon function?

Still looks like this function could be defined outside of Assertion.addMethod('property', function (name, val, msg) { (i.e. above line 765).

I'm not against moving this outside, but it has no general applicability outside of this code. My own personal style is to put functions that are akin to variables inside of the scope where they are used, if it is more in-line with the project's style to have this outside then I'm cool with that.

What you have done here is quite clever, however it is not very maintainable as its logic is quite dense. By moving it out of .addProperty you can provide first class documentation for its workings. Perhaps it might be relevant to add it as new utility and expose it for other plugin developers (just in case it has further use)?

Furthermore, any reason you are using typeof instead of our type-detect utility? I would also recommend just doing the type detection once to reduce overhead.

Thanks for the feedback guys. I honestly wasn't trying to be clever, just adding functionality as I added more and more tests.

Ok, here's my plan:

• A new util function getPathInfo that returns the property info struct including whether the property actually exists or not.
• Keep getPathValue for back compat, but it can easily be implemented by calling getPropertyInfo to share the bulk of the code.

This will allow me to properly document the general functionality and make it available to others.

I'll take a look at the type utility you mentioned, I'm still getting introduced to the codebase.

This is still getPathValue...