Skip to content

test: Run integration tests against SQLite - #202

Merged
natemort merged 1 commit into
cadence-workflow:mainfrom
natemort:sqlite
Oct 29, 2026
Merged

natemort merged 1 commit into
cadence-workflow:mainfrom
natemort:sqlite

Conversation

@natemort

Copy link
Copy Markdown
Member

This speeds up integration test time significantly.

What changed?

  • Run integration tests against SQLite instead of Cassandra.

Why?

  • Cassandra takes over a minute to start up. This is over half of the time spent running integration tests currently.

How did you test it?

  • Ran integration tests

Potential risks

  • None. This is test only change.

Release notes

Documentation Changes

This speeds up integration test time significantly.
@codecov

codecov Bot commented Oct 29, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@gitar-bot

gitar-bot Bot commented Oct 29, 2026 •

Copy link
Copy Markdown
Code Review ✅ Approved

🟡 Medium risk · Switching integration tests from Cassandra to SQLite could leave Cassandra-specific persistence behavior uncovered by the test suite.

Switches integration tests from Cassandra to SQLite to cut test startup time, since Cassandra took over a minute to start. No issues found.

Verification PRs 1 inspected

Functional validation No results

Rules No rules evaluated

Broader impact PRs 1 inspected
Cross-PR 1 PR inspected 0 open impacts

PR 8119: chore(deps): bump ncruces/go-sqlite3 from v0.22.0 to v0.24.1 Inspected

For more verification details, comment gitar display:verbose.

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing compact verification results.

Comment with these commands to change the behavior for this request:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

@natemort
natemort merged commit 1f61685 into cadence-workflow:main Oct 29, 2026
8 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.

2 participants