[Core] Remove core worker test

.buildkite/pipeline.macos.yml

@@ -66,7 +66,7 @@ steps:
     - *prelude_commands
     - TORCH_VERSION=1.6 ./ci/travis/install-dependencies.sh
     - bazel test --config=ci --test_env=CI $(./scripts/bazel_export_options) --build_tests_only --test_tag_filters=-post_wheel_build --
-      //:all python/ray/serve/... python/ray/dashboard/... -rllib/... -core_worker_test
+      //:all python/ray/serve/... python/ray/dashboard/... -rllib/...
     # clang-format is needed by java/test.sh
     - pip install clang-format==12.0.1
     - ./java/test.sh

.buildkite/pipeline.yml

@@ -171,7 +171,7 @@
     - cleanup() { if [ "${BUILDKITE_PULL_REQUEST}" = "false" ]; then ./ci/travis/upload_build_info.sh; fi }; trap cleanup EXIT
     - bazel test --config=ci --config=llvm $(./scripts/bazel_export_options)
       --build_tests_only
-      -- //:all -rllib/... -core_worker_test
+      -- //:all -rllib/...
 
 - label: ":cpp: Tests (ASAN)"
   conditions: [ "RAY_CI_CORE_CPP_AFFECTED" ]
@@ -180,7 +180,7 @@
     - bazel test --config=ci --config=asan-clang $(./scripts/bazel_export_options)
       --build_tests_only
       --jobs=2
-      -- //:all -//:core_worker_test
+      -- //:all
 
 - label: ":cpp: Tests (UBSAN)"
   conditions: [ "RAY_CI_CORE_CPP_AFFECTED" ]
@@ -189,7 +189,7 @@
     - bazel test --config=ci --config=ubsan $(./scripts/bazel_export_options)
       --build_tests_only
       --jobs=2
-      -- //:all -//:core_worker_test -//:logging_test
+      -- //:all -//:logging_test
 
 - label: ":cpp: Tests (TSAN)"
   conditions: [ "RAY_CI_CORE_CPP_AFFECTED" ]
@@ -198,7 +198,7 @@
     - bazel test --config=ci --config=tsan-clang $(./scripts/bazel_export_options)
       --build_tests_only
       --jobs=2
-      -- //:all -//:core_worker_test -//:event_test -//:gcs_actor_manager_test
+      -- //:all -//:event_test -//:gcs_actor_manager_test
       -//:gcs_placement_group_manager_test -//:gcs_placement_group_scheduler_test
       -//:gcs_server_rpc_test -//:gcs_server_test
       -//:metric_exporter_client_test -//:stats_test -//:worker_pool_test

BUILD.bazel

@@ -744,7 +744,6 @@ cc_library(
         ],
         exclude = [
             "src/ray/core_worker/**/*_test.cc",
-            "src/ray/core_worker/mock_worker.cc",
         ],
     ),
     hdrs = glob([
@@ -773,56 +772,6 @@ cc_library(
     ],
 )
 
-cc_library(
-    name = "mock_worker_lib",
-    srcs = ["src/ray/core_worker/test/mock_worker.cc"],
-    hdrs = glob([
-        "src/ray/core_worker/test/*.h",
-    ]),
-    copts = COPTS,
-    strip_include_prefix = "src",
-    deps = [
-        ":core_worker_lib",
-    ],
-)
-
-cc_binary(
-    name = "mock_worker",
-    copts = COPTS,
-    deps = [
-        ":mock_worker_lib",
-    ],
-)
-
-cc_test(
-    name = "core_worker_test",
-    size = "small",
-    srcs = ["src/ray/core_worker/test/core_worker_test.cc"],
-    args = [
-        "$(location raylet)",
-        "$(location mock_worker)",
-        "$(location gcs_server)",
-        "$(location redis-cli)",
-        "$(location redis-server)",
-    ],
-    copts = COPTS,
-    data = [
-        "//:gcs_server",
-        "//:mock_worker",
-        "//:raylet",
-        "//:redis-cli",
-        "//:redis-server",
-    ],
-    tags = ["team:core"],
-    deps = [
-        ":core_worker_lib",
-        ":gcs",
-        "@com_google_absl//absl/container:flat_hash_map",
-        "@com_google_absl//absl/container:flat_hash_set",
-        "@com_google_googletest//:gtest_main",
-    ],
-)
-
 cc_test(
     name = "memory_store_test",
     size = "small",

ci/travis/ci.sh

@@ -120,8 +120,6 @@ test_core() {
   case "${OSTYPE}" in
     msys)
       args+=(
-        -//:core_worker_test
-        -//:event_test
         -//:gcs_pub_sub_test
         -//:gcs_server_test
         -//:gcs_server_rpc_test

doc/source/getting-involved.rst

@@ -222,9 +222,6 @@ The `CI`_ test folder contains all integration test scripts and they
 invoke other test scripts via ``pytest``, ``bazel``-based test or other bash
 scripts. Some of the examples include:
 
-* Raylet integration tests commands:
-    * ``bazel test //:core_worker_test``
-
 * Bazel test command:
     * ``bazel test --build_tests_only //:all``
 
@@ -264,7 +261,7 @@ In Python APIs, consider forcing the use of kwargs instead of positional argumen
         pass
 
 For callback APIs, consider adding a ``**kwargs`` placeholder as a "forward compatibility placeholder" in case more args need to be passed to the callback in the future, e.g.:
- 
+
 .. code-block:: python
 
     def tune_user_callback(model, score, **future_kwargs):

src/ray/common/test_util.h

@@ -40,9 +40,6 @@ static inline std::vector<rpc::ObjectReference> ObjectIdsToRefs(
 class Buffer;
 class RayObject;
 
-// Magic argument to signal to mock_worker we should check message order.
-static const int64_t SHOULD_CHECK_MESSAGE_ORDER = 123450000;
-
 /// Wait until the future is ready, or timeout is reached.
 ///
 /// \param[in] future The future to wait for.