Skip to content

Commit

Permalink
Remove "ccm" tests
Browse files Browse the repository at this point in the history
They don't work.
  • Loading branch information
whatyouhide committed Oct 16, 2023
1 parent cd58f64 commit 032dc00
Show file tree
Hide file tree
Showing 4 changed files with 3 additions and 104 deletions.
18 changes: 2 additions & 16 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -122,20 +122,6 @@ jobs:
docker-compose up --detach --build
./test/docker/health-check-services.sh
- name: Install Python (for ccm)
uses: actions/setup-python@v4
with:
python-version: "3.9.16"

- name: Install ccm
run: pip3 install ccm

- name: Install OpenJDK
uses: actions/setup-java@v2
with:
distribution: "zulu"
java-version: "11"

- name: Install OTP and Elixir
uses: erlef/setup-beam@v1
with:
Expand All @@ -161,11 +147,11 @@ jobs:
# TODO: eventually figure out why we can't run encryption tests on CI.
- name: Run tests for Cassandra and Scylla
if: ${{ !matrix.test_only_cassandra }}
run: mix test.all --trace --exclude encryption --exclude ccm --exclude toxiproxy
run: mix test.all --trace --exclude encryption --exclude toxiproxy

- name: Run tests for Cassandra only
if: ${{ matrix.test_only_cassandra }}
run: mix test --trace --exclude encryption --exclude ccm --exclude toxiproxy
run: mix test --trace --exclude encryption --exclude toxiproxy

- name: Dump Docker logs on failure
uses: jwalton/gh-docker-logs@v1
Expand Down
2 changes: 1 addition & 1 deletion mix.exs
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ defmodule Xandra.Mixfile do
System.put_env("CASSANDRA_PORT", "9062")
System.put_env("CASSANDRA_WITH_AUTH_PORT", "9063")
end,
"test --exclude cassandra_specific --exclude encryption --exclude ccm --include scylla_specific"
"test --exclude cassandra_specific --exclude encryption --include scylla_specific"
],
"test.all": fn args ->
Mix.Task.run(:test, args)
Expand Down
13 changes: 0 additions & 13 deletions test/test_helper.exs
Original file line number Diff line number Diff line change
Expand Up @@ -13,19 +13,6 @@ excluded =
nil -> [:skip_for_native_protocol_v4, :skip_for_native_protocol_v3]
end

excluded =
if System.find_executable("ccm") do
excluded
else
message = """
ccm was not found in your PATH. Xandra requires it in order to run cluster tests, so \
we're skipping ccm-based tests for now. Make sure you can run "ccm" in your shell.\
"""

IO.puts(IO.ANSI.format([:yellow, "WARNING: ", :reset, message, ?\n]))
excluded ++ [:ccm]
end

Mox.defmock(LBPMock, for: Xandra.Cluster.LoadBalancingPolicy)

ExUnit.start(exclude: excluded, assert_receive_timeout: 1_000)
74 changes: 0 additions & 74 deletions test/xandra/cluster/ccm_test.exs

This file was deleted.

0 comments on commit 032dc00

Please sign in to comment.