Skip to content

Commit

Permalink
Merge pull request getsentry#497 from getsentry/1.13-ci
Browse files Browse the repository at this point in the history
add 1.13.x to CI
  • Loading branch information
mitchellhenke authored Feb 13, 2022
2 parents 1c1d311 + b25a0d9 commit 3619d31
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 11 deletions.
13 changes: 8 additions & 5 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ jobs:
continue-on-error: ${{ matrix.development }}
strategy:
matrix:
elixir: ['1.10.x', '1.11.x', '1.12.x']
elixir: ['1.10.x', '1.11.x', '1.12.x', '1.13.x']
otp: ['21.3.x', '22.3.x', '23.3.x', '24.1.x']
development: [false]
exclude:
Expand All @@ -26,6 +26,9 @@ jobs:
- elixir: '1.12.x'
otp: '21.3.x'
development: false
- elixir: '1.13.x'
otp: '21.3.x'
development: false
include:
- elixir: 'master'
otp: '24.1.x'
Expand Down Expand Up @@ -55,7 +58,7 @@ jobs:
mix deps.get
- name: Check Formatting
run: mix format --check-formatted
if: matrix.elixir == '1.11.x'
if: matrix.elixir == '1.13.x'

- name: Check Compile Warnings
run: mix compile --warnings-as-errors
Expand All @@ -66,16 +69,16 @@ jobs:
- name: Retrieve PLT Cache
uses: actions/cache@v1
id: plt-cache
if: matrix.elixir == '1.11.x'
if: matrix.elixir == '1.13.x'
with:
path: priv/plts
key: ${{ runner.os }}-${{ matrix.otp }}-${{ matrix.elixir }}-plts-${{ hashFiles(format('{0}{1}', github.workspace, '/mix.lock')) }}

- name: Create PLTs
if: steps.plt-cache.outputs.cache-hit != 'true' && matrix.elixir == '1.11.x'
if: steps.plt-cache.outputs.cache-hit != 'true' && matrix.elixir == '1.13.x'
run: |
mkdir -p priv/plts
mix dialyzer --plt
- name: Run dialyzer
run: mix dialyzer --no-check --halt-exit-status
if: matrix.elixir == '1.11.x'
if: matrix.elixir == '1.13.x'
8 changes: 2 additions & 6 deletions test/client_test.exs
Original file line number Diff line number Diff line change
Expand Up @@ -11,19 +11,15 @@ defmodule Sentry.ClientTest do
{_endpoint, public_key, private_key} = Client.get_dsn()

assert Client.authorization_header(public_key, private_key) =~
~r/^Sentry sentry_version=5, sentry_client=sentry-elixir\/#{
Application.spec(:sentry, :vsn)
}, sentry_timestamp=\d{10}, sentry_key=public, sentry_secret=secret$/
~r/^Sentry sentry_version=5, sentry_client=sentry-elixir\/#{Application.spec(:sentry, :vsn)}, sentry_timestamp=\d{10}, sentry_key=public, sentry_secret=secret$/
end

test "authorization without secret" do
modify_env(:sentry, dsn: "https://public@app.getsentry.com/1")
{_endpoint, public_key, private_key} = Client.get_dsn()

assert Client.authorization_header(public_key, private_key) =~
~r/^Sentry sentry_version=5, sentry_client=sentry-elixir\/#{
Application.spec(:sentry, :vsn)
}, sentry_timestamp=\d{10}, sentry_key=public$/
~r/^Sentry sentry_version=5, sentry_client=sentry-elixir\/#{Application.spec(:sentry, :vsn)}, sentry_timestamp=\d{10}, sentry_key=public$/
end

test "get dsn with default config" do
Expand Down

0 comments on commit 3619d31

Please sign in to comment.