Skip to content

Commit

Permalink
acceptance: disable Elixir tests
Browse files Browse the repository at this point in the history
Elixir requires network to install dependencies. This can be changed
(painfully), but an upgrade on Elixir is causing even more problems. I'm
in favour of removing the acceptance (which I wrote) to fix this for now
as I have plentiful unit tests covering the same thing. Maybe someone
can come back to this if it's a large issue...

Release note: None
  • Loading branch information
otan committed Aug 4, 2020
1 parent 630af18 commit b1947cd
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions pkg/acceptance/adapter_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,8 @@ func TestDockerJava(t *testing.T) {
}

func TestDockerElixir(t *testing.T) {
t.Skip("Elixir requires network to run, which can flake. When attempting to update this (#52341), the new Elixir version does not work with CRDB/TLS.")

s := log.Scope(t)
defer s.Close(t)

Expand Down

0 comments on commit b1947cd

Please sign in to comment.