update based on PR feedback

Signed-off-by: greerdv <greerdv@amazon.com>

Code/Framework/AzFramework/AzFramework/Physics/Collision/CollisionGroups.cpp

@@ -36,8 +36,8 @@ namespace AzPhysics
 
     void CollisionGroupScriptConstructor(CollisionGroup* thisPtr, AZ::ScriptDataContext& scriptDataContext)
     {
-        int numArgs = scriptDataContext.GetNumArguments();
-        if (numArgs != 1)
+        if (int numArgs = scriptDataContext.GetNumArguments();
+            numArgs != 1)
         {
             scriptDataContext.GetScriptContext()->Error(AZ::ScriptContext::ErrorType::Error, true,
                 "CollisionGroup() accepts only 1 argument, not %d", numArgs);

Gems/PhysX/Code/Tests/PhysXScriptTest.cpp

@@ -34,6 +34,8 @@ namespace PhysX
         return AZ::EntityId();
     }
 
+    // this allows EXPECT_TRUE to be exposed to the behavior context and used inside blocks of lua code which
+    // are executed in tests
     void ExpectTrue(bool check)
     {
         EXPECT_TRUE(check);