[core] Split out runtime_env unit tests (ray-project#53477)

- Move unit tests to `unit/` directory.
- Add enforcement in the `unit/` directory that Ray is never
initialized.
- Combine the two `test_runtime_env_validation.py` files.
- Split out `test_runtime_env_get_wheel_names.py`.

---------

Signed-off-by: Edward Oakes <[email protected]>

Author: edoakes

.pre-commit-config.yaml

@@ -29,7 +29,7 @@ repos:
         exclude: |
           (?x)^(
             # Intentionally bad json schema
-            python/ray/tests/test_runtime_env_validation_bad_2_schema.json|
+            python/ray/tests/unit/test_runtime_env_validation_bad_schema.json|
             # json5 comments prevent parsing
             python/asv.conf.json|
             rllib/asv.conf.json

python/ray/tests/BUILD

@@ -477,6 +477,7 @@ py_test_module_list(
         "test_raylet_output.py",
         "test_reconstruction_stress.py",
         "test_reconstruction_stress_spill.py",
+        "test_runtime_env_get_wheel_names.py",
         "test_state_api_summary.py",
         "test_streaming_generator_regression.py",
         "test_system_metrics.py",
@@ -1175,25 +1176,6 @@ py_test(
     ],
 )
 
-py_test(
-    name = "test_runtime_env_validation",
-    size = "small",
-    srcs = ["test_runtime_env_validation.py"],
-    data = [
-        "test_runtime_env_validation_1_schema.json",
-        "test_runtime_env_validation_2_schema.json",
-    ],
-    tags = [
-        "exclusive",
-        "small_size_python_tests",
-        "team:core",
-    ],
-    deps = [
-        ":conftest",
-        "//:ray_lib",
-    ],
-)
-
 py_test(
     name = "kuberay/test_autoscaling_e2e",
     size = "large",

python/ray/tests/conftest.py

@@ -24,7 +24,6 @@
 import ray._private.ray_constants as ray_constants
 from ray._private.conftest_utils import set_override_dashboard_url  # noqa: F401
 from ray._private.runtime_env import virtualenv_utils
-from ray._private.runtime_env.plugin_schema_manager import RuntimeEnvPluginSchemaManager
 
 from ray._private.test_utils import (
     get_and_run_resource_killer,
@@ -1457,18 +1456,6 @@ def set_runtime_env_plugins(request):
         del os.environ["RAY_RUNTIME_ENV_PLUGINS"]
 
 
-@pytest.fixture
-def set_runtime_env_plugin_schemas(request):
-    runtime_env_plugin_schemas = getattr(request, "param", "0")
-    try:
-        os.environ["RAY_RUNTIME_ENV_PLUGIN_SCHEMAS"] = runtime_env_plugin_schemas
-        # Clear and reload schemas.
-        RuntimeEnvPluginSchemaManager.clear()
-        yield runtime_env_plugin_schemas
-    finally:
-        del os.environ["RAY_RUNTIME_ENV_PLUGIN_SCHEMAS"]
-
-
 @pytest.fixture(scope="function")
 def temp_file(request):
     with tempfile.NamedTemporaryFile("r+b") as fp: