testPolygonPolygon and testPolygonCircle APIs differ on definition of collision

[brownstein]

Found that testPolygonPolygon returns true when one polygon is inside another, but testPolygonCircle returns false when a circle is inside a containing polygon. Looks like this is caused by line #840 of SAT.js, which returns false regardless of containment for circle tests, and line #983, which returns true when two polygons do not have a separating axis.

Requesting that the next release unify the definition of collisions to either include or not-include containment cases. Issue is mitigated by inspecting Response payloads, but this is still a confusing API behavior.

[brownstein]

@jriecken Looks like I'm in error - had an inverted polygon in the mix which was causing the observed behavior. Closing the issue.

[jriecken]

Thanks for double-checking!