Skip to content

add test for rollback in fixture#1521

Merged
collerek merged 1 commit intomasterfrom
add-tests-with-transaction-rollback-as-fixture
Feb 8, 2026
Merged

add test for rollback in fixture#1521
collerek merged 1 commit intomasterfrom
add-tests-with-transaction-rollback-as-fixture

Conversation

@collerek
Copy link
Copy Markdown
Collaborator

@collerek collerek commented Feb 8, 2026

No description provided.

@collerek collerek changed the title add test for rollback in transaction add test for rollback in fixture Feb 8, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (0129c1a) to head (f52a9ba).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##            master     #1521   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          211       212    +1     
  Lines        15135     15167   +32     
=========================================
+ Hits         15135     15167   +32     
Files with missing lines Coverage Δ
...ests/test_databases/test_transaction_as_fixture.py 100.00% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Feb 8, 2026

Merging this PR will degrade performance by 38.5%

❌ 2 (👁 2) regressed benchmarks
✅ 82 untouched benchmarks

Performance Changes

Mode Benchmark BASE HEAD Efficiency
👁 Simulation test_avg[250] 3.1 ms 4.5 ms -32.04%
👁 Simulation test_get_or_create_when_get[250] 2.4 ms 3.9 ms -38.5%

Comparing add-tests-with-transaction-rollback-as-fixture (f52a9ba) with master (0129c1a)

Open in CodSpeed

@collerek collerek merged commit 7e92b72 into master Feb 8, 2026
19 checks passed
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.

1 participant