Format environment test updates

Author: StafaH

source/isaaclab_tasks/test/contrib/test_cartpole_showcase_environments.py

@@ -25,9 +25,7 @@
 @pytest.mark.parametrize("num_envs, device", [(2, "cuda"), (1, "cuda")])
 @pytest.mark.parametrize(
     "task_name",
-    setup_environment(
-        factory_envs=False, multi_agent=False, teleop_envs=False, cartpole_showcase_envs=True
-    ),
+    setup_environment(factory_envs=False, multi_agent=False, teleop_envs=False, cartpole_showcase_envs=True),
 )
 @pytest.mark.isaacsim_ci
 def test_cartpole_showcase_environments(task_name, num_envs, device):

source/isaaclab_tasks/test/contrib/test_cartpole_showcase_environments_with_stage_in_memory.py

@@ -26,9 +26,7 @@
 @pytest.mark.parametrize("num_envs, device", [(2, "cuda")])
 @pytest.mark.parametrize(
     "task_name",
-    setup_environment(
-        factory_envs=False, multi_agent=False, teleop_envs=False, cartpole_showcase_envs=True
-    ),
+    setup_environment(factory_envs=False, multi_agent=False, teleop_envs=False, cartpole_showcase_envs=True),
 )
 def test_cartpole_showcase_environments_with_stage_in_memory_and_clone_in_fabric_disabled(task_name, num_envs, device):
     # skip test if stage in memory is not supported

source/isaaclab_tasks/test/contrib/test_teleop_environments.py

@@ -36,9 +36,7 @@
 
 
 @pytest.mark.parametrize("num_envs, device", [(2, "cuda"), (1, "cuda")])
-@pytest.mark.parametrize(
-    "task_name", setup_environment(factory_envs=False, multi_agent=False, teleop_envs=True)
-)
+@pytest.mark.parametrize("task_name", setup_environment(factory_envs=False, multi_agent=False, teleop_envs=True))
 @pytest.mark.isaacsim_ci
 def test_teleop_environments(task_name, num_envs, device):
     # run teleop environments without stage in memory

source/isaaclab_tasks/test/contrib/test_teleop_environments_with_stage_in_memory.py

@@ -37,9 +37,7 @@
 
 
 @pytest.mark.parametrize("num_envs, device", [(2, "cuda")])
-@pytest.mark.parametrize(
-    "task_name", setup_environment(factory_envs=False, multi_agent=False, teleop_envs=True)
-)
+@pytest.mark.parametrize("task_name", setup_environment(factory_envs=False, multi_agent=False, teleop_envs=True))
 def test_teleop_environments_with_stage_in_memory_and_clone_in_fabric_disabled(task_name, num_envs, device):
     # skip test if stage in memory is not supported
     if get_isaac_sim_version().major < 5:

source/isaaclab_tasks/test/core/test_environments.py

@@ -21,8 +21,8 @@
 # Local imports should be imported last
 from env_test_utils import _run_environments, setup_environment  # isort: skip
 
-@pytest.mark.parametrize("physics_preset_name", ["newton_mjwarp", "physx", "isaacsim_physx"])
 
+@pytest.mark.parametrize("physics_preset_name", ["newton_mjwarp", "physx", "isaacsim_physx"])
 @pytest.mark.parametrize("num_envs, device", [(2, "cuda"), (1, "cuda")])
 @pytest.mark.parametrize(
     "task_name",

source/isaaclab_tasks/test/core/test_preset_kit_decision.py

@@ -88,10 +88,11 @@ def test_isaacsim_physx_is_physics_selector():
 
 
 def test_registered_task_physx_presets_keep_auto_selection_explicit():
-    """PhysX defaults are concrete while ``physx`` remains the automatic selector."""
+    """Core tasks retain explicit PhysX variants alongside automatic ``physx``."""
 
     for task_id, task_spec in gym.registry.items():
-        if not task_id.startswith(("Isaac-", "IsaacContrib-")) or "env_cfg_entry_point" not in task_spec.kwargs:
+        entry_point = task_spec.kwargs.get("env_cfg_entry_point", "")
+        if not task_id.startswith("Isaac-") or "isaaclab_tasks.core" not in str(entry_point):
             continue
         env_cfg = load_cfg_from_registry(task_id, "env_cfg_entry_point")
         presets = collect_presets(env_cfg)

source/isaaclab_tasks/test/env_test_utils.py

@@ -173,7 +173,6 @@ def setup_environment(
                 continue
         # if None: no filter
 
-
         registered_tasks.append(task_spec.id)
 
     # sort environments alphabetically