Skip to content

Commit

Permalink
test latest clickhouse by default, but include Plausible-specific ver…
Browse files Browse the repository at this point in the history
…sion in the matrix
  • Loading branch information
ruslandoga committed Jan 12, 2024
1 parent 1212a1a commit 4611808
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 5 deletions.
13 changes: 10 additions & 3 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,17 +16,24 @@ jobs:
matrix:
elixir: ["1.14", "1.15", "1.16"]
otp: ["25", "26"]
clickhouse: ["23.3.7.5", "latest"]
clickhouse: ["latest"]
timezone: ["UTC"]
include:
- elixir: "1.16"
otp: "26"
clickhouse: "23.3.7.5"
clickhouse: "latest"
timezone: "Europe/Berlin"
# Plausible
# - https://github.com/plausible/analytics/blob/master/.tool-versions
# - https://github.com/plausible/analytics/blob/master/.github/workflows/elixir.yml
- elixir: "1.16.0"
otp: "26.2.1"
clickhouse: "23.3.7.5"
timezone: "UTC"

services:
clickhouse:
image: clickhouse/clickhouse-server:${{ matrix.clickhouse }}-alpine
image: clickhouse/clickhouse-server:${{ matrix.clickhouse }}
ports:
- 8123:8123
env:
Expand Down
5 changes: 3 additions & 2 deletions test/ch/connection_test.exs
Original file line number Diff line number Diff line change
Expand Up @@ -1282,8 +1282,9 @@ defmodule Ch.ConnectionTest do
assert {:error, %Ch.Error{code: 81} = error} =
Ch.query(conn, "select 1 + 1", _params = [], database: "no-db")

assert Exception.message(error) =~
"Code: 81. DB::Exception: Database `no-db` doesn't exist. (UNKNOWN_DATABASE)"
assert Exception.message(error) =~ "Code: 81."
assert Exception.message(error) =~ "`no-db`"
assert Exception.message(error) =~ "(UNKNOWN_DATABASE)"
end

test "can provide custom database", %{conn: conn} do
Expand Down

0 comments on commit 4611808

Please sign in to comment.