Skip to content

Commit

Permalink
Ignore tachyon related flaky tests
Browse files Browse the repository at this point in the history
  • Loading branch information
geekingfrog committed Jun 27, 2024
1 parent 6459ac4 commit 73ffb76
Show file tree
Hide file tree
Showing 13 changed files with 26 additions and 8 deletions.
2 changes: 2 additions & 0 deletions test/teiserver/account/accolade_bot_test.exs
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,8 @@ defmodule Teiserver.Account.AccoladeBotTest do
import Teiserver.TeiserverTestLib,
only: [tachyon_auth_setup: 0, _tachyon_recv: 1, _tachyon_send: 2]

@moduletag :needs_attention

setup do
AccoladeLib.start_accolade_server()

Expand Down
2 changes: 2 additions & 0 deletions test/teiserver/battle/lobby_cache_test.exs
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,8 @@ defmodule Teiserver.Lobby.LobbyLibTest do
import Teiserver.TeiserverTestLib,
only: [tachyon_auth_setup: 0, _tachyon_send: 2, _tachyon_recv: 1]

@moduletag :needs_attention

setup do
%{socket: hsocket, user: host} = tachyon_auth_setup()
%{socket: psocket, user: player} = tachyon_auth_setup()
Expand Down
3 changes: 3 additions & 0 deletions test/teiserver/coordinator/automod_test.exs
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@ defmodule Teiserver.Coordinator.AutomodTest do
import Teiserver.TeiserverTestLib,
only: [new_user: 0, tachyon_auth_setup: 1, _tachyon_send: 2]

@moduletag :needs_attention

setup do
account = CoordinatorServer.get_coordinator_account()
Teiserver.cache_put(:application_metadata_cache, "teiserver_coordinator_userid", account.id)
Expand Down Expand Up @@ -142,6 +144,7 @@ defmodule Teiserver.Coordinator.AutomodTest do
assert result == "Banned user"
end

@tag :needs_attention
test "delayed data", %{banned_user: banned_user} do
{:ok, _ban} =
Moderation.create_ban(%{
Expand Down
2 changes: 2 additions & 0 deletions test/teiserver/coordinator/consul_chat_test.exs
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,8 @@ defmodule Teiserver.Coordinator.ConsulChatTest do
import Teiserver.TeiserverTestLib,
only: [tachyon_auth_setup: 0, _tachyon_send: 2, _tachyon_recv: 1]

@moduletag :needs_attention

setup do
Coordinator.start_coordinator()
%{socket: hsocket, user: host} = tachyon_auth_setup()
Expand Down
7 changes: 2 additions & 5 deletions test/teiserver/coordinator/coordinator_commands_test.exs
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,8 @@ defmodule Teiserver.Coordinator.CoordinatorCommandsTest do
import Teiserver.TeiserverTestLib,
only: [tachyon_auth_setup: 0, _tachyon_send: 2, _tachyon_recv: 1, new_user: 0]

@moduletag :needs_attention

setup do
Coordinator.start_coordinator()
%{socket: socket, user: user} = tachyon_auth_setup()
Expand Down Expand Up @@ -34,7 +36,6 @@ defmodule Teiserver.Coordinator.CoordinatorCommandsTest do
assert String.contains?(message, "No command of name 'creativecommandname'")
end

@tag :needs_attention
test "help", %{socket: socket, user: user} do
message_coordinator(socket, "$help")
[reply] = _tachyon_recv(socket)
Expand Down Expand Up @@ -81,7 +82,6 @@ defmodule Teiserver.Coordinator.CoordinatorCommandsTest do
refute String.contains?(message, "Displays this help text.")
end

@tag :needs_attention
test "help pull", %{socket: socket, user: user} do
# Normal pull test
message_coordinator(socket, "$help pull")
Expand Down Expand Up @@ -109,7 +109,6 @@ defmodule Teiserver.Coordinator.CoordinatorCommandsTest do
refute String.contains?(message, "Displays this help text.")
end

@tag :needs_attention
test "whoami", %{socket: socket, user: user, coordinator_userid: coordinator_userid} do
message_coordinator(socket, "$whoami")
[reply] = _tachyon_recv(socket)
Expand All @@ -132,7 +131,6 @@ defmodule Teiserver.Coordinator.CoordinatorCommandsTest do
}
end

@tag :needs_attention
test "whois", %{socket: socket, coordinator_userid: coordinator_userid} do
other_user = new_user()

Expand Down Expand Up @@ -183,7 +181,6 @@ defmodule Teiserver.Coordinator.CoordinatorCommandsTest do
}
end

@tag :needs_attention
test "mute user command", %{socket: socket, user: user, coordinator_userid: coordinator_userid} do
%{user: user2} = tachyon_auth_setup()

Expand Down
2 changes: 2 additions & 0 deletions test/teiserver/coordinator/host_update_test.exs
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,8 @@ defmodule Teiserver.Coordinator.HostUpdateTest do
import Teiserver.TeiserverTestLib,
only: [tachyon_auth_setup: 0, _tachyon_send: 2, _tachyon_recv: 1]

@moduletag :needs_attention

setup do
Coordinator.start_coordinator()
%{socket: hsocket, user: host} = tachyon_auth_setup()
Expand Down
3 changes: 2 additions & 1 deletion test/teiserver/coordinator/joining_test.exs
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,8 @@ defmodule Teiserver.Coordinator.JoiningTest do
import Teiserver.TeiserverTestLib,
only: [tachyon_auth_setup: 0, _tachyon_send: 2, _tachyon_recv: 1]

@tag :needs_attention
@moduletag :needs_attention

setup do
Coordinator.start_coordinator()
%{socket: socket, user: user} = tachyon_auth_setup()
Expand Down
2 changes: 2 additions & 0 deletions test/teiserver/coordinator/lock_test.exs
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,8 @@ defmodule Teiserver.Coordinator.LockTest do
import Teiserver.TeiserverTestLib,
only: [tachyon_auth_setup: 0, _tachyon_send: 2, _tachyon_recv: 1]

@moduletag :needs_attention

setup do
Coordinator.start_coordinator()
%{socket: hsocket, user: host} = tachyon_auth_setup()
Expand Down
2 changes: 2 additions & 0 deletions test/teiserver/coordinator/match_monitor_server_test.exs
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,8 @@ defmodule Teiserver.Coordinator.MatchMonitorServerTest do
import Teiserver.TeiserverTestLib,
only: [tachyon_auth_setup: 0, _tachyon_send: 2, _tachyon_recv: 1]

@moduletag :needs_attention

setup do
account = CoordinatorServer.get_coordinator_account()
Teiserver.cache_put(:application_metadata_cache, "teiserver_coordinator_userid", account.id)
Expand Down
2 changes: 2 additions & 0 deletions test/teiserver/coordinator/moderation_test.exs
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,8 @@ defmodule Teiserver.Coordinator.ModerationTest do
import Teiserver.TeiserverTestLib,
only: [new_user: 0, tachyon_auth_setup: 1, _tachyon_send: 2, _tachyon_recv: 1]

@moduletag :needs_attention

setup do
Coordinator.start_coordinator()
user = new_user()
Expand Down
1 change: 1 addition & 0 deletions test/teiserver/coordinator/setup_test.exs
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ defmodule Teiserver.Protocols.Coordinator.SetupTest do
only: [tachyon_auth_setup: 0, _tachyon_recv: 1]

@sleep 50
@moduletag :needs_attention

setup do
%{socket: socket, user: user, pid: pid} = tachyon_auth_setup()
Expand Down
4 changes: 2 additions & 2 deletions test/teiserver/coordinator/split_test.exs
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@ defmodule Teiserver.Coordinator.SplitTest do
import Teiserver.TeiserverTestLib,
only: [tachyon_auth_setup: 0, _tachyon_send: 2, _tachyon_recv: 1]

@moduletag :needs_attention

setup do
Coordinator.start_coordinator()
%{socket: hsocket, user: host} = tachyon_auth_setup()
Expand Down Expand Up @@ -89,7 +91,6 @@ defmodule Teiserver.Coordinator.SplitTest do
empty_lobby_id: empty_lobby_id}
end

@tag :needs_attention
test "basic split test", %{
host: _host,
player: player1,
Expand Down Expand Up @@ -499,7 +500,6 @@ defmodule Teiserver.Coordinator.SplitTest do
}
end

@tag :needs_attention
test "test minimum player split", %{
host: _host,
player: player1,
Expand Down
2 changes: 2 additions & 0 deletions test/teiserver/protocols/spring/spring_token_test.exs
Original file line number Diff line number Diff line change
Expand Up @@ -106,6 +106,7 @@ defmodule Teiserver.SpringTokenTest do
assert reply == "DENIED token_login_failed\n"
end

@tag :needs_attention
test "c.user.get_token_by_email - incorrect" do
%{socket: socket} = spring_tls_setup()
_welcome = _recv_raw(socket)
Expand All @@ -119,6 +120,7 @@ defmodule Teiserver.SpringTokenTest do
assert reply == "NO cmd=c.user.get_token_by_email\tbad format\n"
end

@tag :needs_attention
test "c.user.get_token_by_name - incorrect" do
%{socket: socket} = spring_tls_setup()
_welcome = _recv_raw(socket)
Expand Down

0 comments on commit 73ffb76

Please sign in to comment.