Skip to content

test: Add tests for schema and check helper methods - #289

Merged
spencewenski merged 1 commit into
mainfrom
schema-tests
Jul 24, 2024
Merged

spencewenski merged 1 commit into
mainfrom
schema-tests

Conversation

@spencewenski

Copy link
Copy Markdown
Member

Also fix a minor issue with how TestCase parses thread names to create a description

Also fix a minor issue with how `TestCase` parses thread names to create
a description
@codecov

codecov Bot commented Jul 24, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 47.35%. Comparing base (d1cc47a) to head (a4db2ec).

Files Coverage Δ
src/migration/check.rs 100.00% <100.00%> (+33.33%) ⬆️
src/migration/schema.rs 100.00% <100.00%> (+46.34%) ⬆️
src/testing/snapshot.rs 100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d1cc47a...a4db2ec. Read the comment docs.

@spencewenski
spencewenski merged commit 93c5bb6 into main Jul 24, 2024
@spencewenski
spencewenski deleted the schema-tests branch July 24, 2024 21:04
@github-actions github-actions Bot mentioned this pull request Jul 25, 2024
spencewenski pushed a commit that referenced this pull request Jul 25, 2024
## 🤖 New release
* `roadster`: 0.5.9 -> 0.5.10

<details><summary><i><b>Changelog</b></i></summary><p>

<blockquote>

##
[0.5.10](roadster-v0.5.9...roadster-v0.5.10)
- 2024-07-25

### Added
- Use IDENTITY column for int primary keys instead of BIGSERIAL
([#293](#293))

### Fixed
- Add "if exists" to user's drop_table migration statement
([#292](#292))

### Other
- Add tests for schema and check helper methods
([#289](#289))
</blockquote>


</p></details>

---
This PR was generated with
[release-plz](https://github.com/MarcoIeni/release-plz/).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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