Skip to content
This repository has been archived by the owner on Apr 22, 2024. It is now read-only.

Add test suite to run e2e tests against the current and legacy authservice #32

Merged
merged 4 commits into from
Feb 21, 2024

Conversation

nacx
Copy link
Member

@nacx nacx commented Feb 21, 2024

Fixes #25

Adds a test suite that runs against a given authservice image, that can be set as an env var when running the tests. It also adds a matrix build in CI to run such a suite.

To try it out locally:

$ E2E_PRESERVE_LOGS=true E2E_SUITE_MODE=legacy make e2e/legacy
$ E2E_PRESERVE_LOGS=true E2E_SUITE_MODE=current make e2e/legacy

The legacy image used by default is ghcr.io/istio-ecosystem/authservice/authservice:0.5.3, but can be customized by setting the env var E2E_LEGACY_IMAGE.

@nacx nacx requested a review from sergicastro February 21, 2024 16:38
Copy link

codecov bot commented Feb 21, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (33cd379) 90.82% compared to head (5e21b57) 90.84%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #32      +/-   ##
==========================================
+ Coverage   90.82%   90.84%   +0.01%     
==========================================
  Files          17       17              
  Lines        1221     1223       +2     
==========================================
+ Hits         1109     1111       +2     
  Misses         78       78              
  Partials       34       34              
Files Coverage Δ
internal/config.go 97.05% <100.00%> (+0.05%) ⬆️

@nacx nacx enabled auto-merge (squash) February 21, 2024 16:59
KEYCLOAK_ADMIN: admin
KEYCLOAK_ADMIN_PASSWORD: admin
ports:
- "8080:8080"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is it necessary?

Suggested change
- "8080:8080"

@nacx nacx merged commit 3bef742 into main Feb 21, 2024
11 checks passed
@nacx nacx deleted the legacy branch February 21, 2024 16:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

E2E Test Against Existing Authservice alongside this one
2 participants