Skip to content

core: fix transaction test case#15021

Merged
BeryJu merged 4 commits into
mainfrom
core/fix-transaction-test-case
Jun 12, 2025
Merged

core: fix transaction test case#15021
BeryJu merged 4 commits into
mainfrom
core/fix-transaction-test-case

Conversation

@BeryJu
Copy link
Copy Markdown
Member

@BeryJu BeryJu commented Jun 12, 2025

Details

See #14997 (comment)


Checklist

  • Local tests pass (ak test authentik/)
  • The code has been formatted (make lint-fix)

If an API change has been made

  • The API schema has been updated (make gen-build)

If changes to the frontend have been made

  • The code has been formatted (make web)

If applicable

  • The documentation has been updated
  • The documentation has been formatted (make website)

BeryJu added 2 commits June 12, 2025 14:28
Signed-off-by: Jens Langhammer <jens@goauthentik.io>
Signed-off-by: Jens Langhammer <jens@goauthentik.io>
@BeryJu BeryJu requested a review from a team as a code owner June 12, 2025 12:30
@netlify
Copy link
Copy Markdown

netlify Bot commented Jun 12, 2025

Deploy Preview for authentik-storybook canceled.

Name Link
🔨 Latest commit 473b342
🔍 Latest deploy log https://app.netlify.com/projects/authentik-storybook/deploys/684b61d0144e9c00082f3c00

@BeryJu BeryJu changed the title Core/fix transaction test case core: fix transaction test case Jun 12, 2025
@netlify
Copy link
Copy Markdown

netlify Bot commented Jun 12, 2025

Deploy Preview for authentik-docs canceled.

Name Link
🔨 Latest commit 473b342
🔍 Latest deploy log https://app.netlify.com/projects/authentik-docs/deploys/684b61d0f3fb01000871bf3d

@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 12, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.76%. Comparing base (3b913ac) to head (473b342).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #15021   +/-   ##
=======================================
  Coverage   92.76%   92.76%           
=======================================
  Files         815      815           
  Lines       42106    42112    +6     
=======================================
+ Hits        39059    39065    +6     
  Misses       3047     3047           
Flag Coverage Δ
e2e 47.59% <100.00%> (-0.08%) ⬇️
integration 24.36% <91.66%> (+0.02%) ⬆️
unit 90.65% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@rissson
Copy link
Copy Markdown
Member

rissson commented Jun 12, 2025

Tenants use APITransactionTestCase which is also TransactionTestCase under the hood

Signed-off-by: Jens Langhammer <jens@goauthentik.io>
@BeryJu BeryJu force-pushed the core/fix-transaction-test-case branch from 0feeb06 to 92948d0 Compare June 12, 2025 21:30
Signed-off-by: Jens Langhammer <jens@goauthentik.io>
@BeryJu BeryJu force-pushed the core/fix-transaction-test-case branch from 3d0f655 to 473b342 Compare June 12, 2025 23:25
@github-actions
Copy link
Copy Markdown
Contributor

authentik PR Installation instructions

Instructions for docker-compose

Add the following block to your .env file:

AUTHENTIK_IMAGE=ghcr.io/goauthentik/dev-server
AUTHENTIK_TAG=gh-473b3428850147afbaa187e02098450e334fd737
AUTHENTIK_OUTPOSTS__CONTAINER_IMAGE_BASE=ghcr.io/goauthentik/dev-%(type)s:gh-%(build_hash)s

Afterwards, run the upgrade commands from the latest release notes.

Instructions for Kubernetes

Add the following block to your values.yml file:

authentik:
    outposts:
        container_image_base: ghcr.io/goauthentik/dev-%(type)s:gh-%(build_hash)s
global:
    image:
        repository: ghcr.io/goauthentik/dev-server
        tag: gh-473b3428850147afbaa187e02098450e334fd737

Afterwards, run the upgrade commands from the latest release notes.

@BeryJu BeryJu merged commit 13bd406 into main Jun 12, 2025
101 checks passed
@BeryJu BeryJu deleted the core/fix-transaction-test-case branch June 12, 2025 23:48
@gergosimonyi
Copy link
Copy Markdown
Collaborator

/cherry-pick version-2025.6

@gcp-cherry-pick-bot
Copy link
Copy Markdown
Contributor

Cherry-pick failed with Merge error 13bd4069e47c70f482ad6d0a5b2e0f9818ae2927 into temp-cherry-pick-4c0062-version-2025.6

gergosimonyi pushed a commit that referenced this pull request Jun 16, 2025
* move patched ct to root

Signed-off-by: Jens Langhammer <jens@goauthentik.io>

* use our transaction test case as base

Signed-off-by: Jens Langhammer <jens@goauthentik.io>

* fix...?

Signed-off-by: Jens Langhammer <jens@goauthentik.io>

* well apparently that works

Signed-off-by: Jens Langhammer <jens@goauthentik.io>

---------

Signed-off-by: Jens Langhammer <jens@goauthentik.io>
gergosimonyi added a commit that referenced this pull request Jun 16, 2025
* core: fix transaction test case (#15021)

* move patched ct to root

Signed-off-by: Jens Langhammer <jens@goauthentik.io>

* use our transaction test case as base

Signed-off-by: Jens Langhammer <jens@goauthentik.io>

* fix...?

Signed-off-by: Jens Langhammer <jens@goauthentik.io>

* well apparently that works

Signed-off-by: Jens Langhammer <jens@goauthentik.io>

---------

Signed-off-by: Jens Langhammer <jens@goauthentik.io>

* fixup! core: fix transaction test case (#15021)

---------

Signed-off-by: Jens Langhammer <jens@goauthentik.io>
Co-authored-by: Jens L. <jens@goauthentik.io>
kensternberg-authentik added a commit that referenced this pull request Jun 19, 2025
* main:
  website/docs: also hide the postgres pool_options setting (#15023)
  blueprints: sort schema items (#15022)
  website: bump the build group in /website with 6 updates (#15027)
  core: bump astral-sh/uv from 0.7.12 to 0.7.13 (#15028)
  core: bump twilio from 9.6.2 to 9.6.3 (#15029)
  core: bump sentry-sdk from 2.29.1 to 2.30.0 (#15030)
  core: bump kubernetes from 32.0.1 to 33.1.0 (#15031)
  core, web: update translations (#15026)
  web: bump the sentry group across 1 directory with 2 updates (#15025)
  translate: Updates for file web/xliff/en.xlf in zh_CN (#15018)
  lifecycle/aws: bump aws-cdk from 2.1018.0 to 2.1018.1 in /lifecycle/aws (#15016)
  website: bump postcss from 8.5.4 to 8.5.5 in /website (#15013)
  website: bump @types/node from 24.0.0 to 24.0.1 in /website (#15014)
  core: fix transaction test case (#15021)
  translate: Updates for file web/xliff/en.xlf in zh-Hans (#15019)
  website/docs: correct minor version in release notes (#15012)
  root: test label handling and error reporting in PytestTestRunner (#14000)
  outposts/ldap: Handle comma-separated attributes in LDAP search requests (#15000)
  website/integrations: standardize application slug placeholder in docs (#15007)
  core: bump django from 5.1.10 to 5.1.11 (#14997)
cavefire pushed a commit to cavefire/authentik that referenced this pull request Jun 24, 2025
* move patched ct to root

Signed-off-by: Jens Langhammer <jens@goauthentik.io>

* use our transaction test case as base

Signed-off-by: Jens Langhammer <jens@goauthentik.io>

* fix...?

Signed-off-by: Jens Langhammer <jens@goauthentik.io>

* well apparently that works

Signed-off-by: Jens Langhammer <jens@goauthentik.io>

---------

Signed-off-by: Jens Langhammer <jens@goauthentik.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants