Skip to content

Use trybuild for testing macros#423

Merged
jayvdb merged 6 commits intoElectron100:masterfrom
jayvdb:trybuild
Nov 11, 2025
Merged

Use trybuild for testing macros#423
jayvdb merged 6 commits intoElectron100:masterfrom
jayvdb:trybuild

Conversation

@jayvdb
Copy link
Collaborator

@jayvdb jayvdb commented Nov 9, 2025

Fixes #419
Closes #365

@jayvdb jayvdb mentioned this pull request Nov 9, 2025
Closed
@jayvdb
Copy link
Collaborator Author

jayvdb commented Nov 9, 2025

Having trybuild tests in butane/tests doesnt work as-is, because this happens

     Running trybuild.rs (/Users/jayvdb/rust/butane/target/debug/deps/trybuild-a279236d5b58048f)

running 1 test
   Compiling libsqlite3-sys v0.35.0
   Compiling tokio v1.48.0
   Compiling butane_tests v0.8.1 (/Users/jayvdb/rust/butane/butane/tests)
warning: butane_tests@0.8.1: Deleting .butane directory
...

Then running unmigrate, which happens after trybuild, fails because .butane was deleted.

@jayvdb jayvdb marked this pull request as ready for review November 9, 2025 23:05
@jayvdb jayvdb requested a review from Electron100 November 9, 2025 23:05
Electron100
Electron100 reviewed Nov 11, 2025
View reviewed changes
@jayvdb jayvdb merged commit b834428 into Electron100:master Nov 11, 2025
6 of 7 checks passed
@jayvdb jayvdb deleted the trybuild branch November 11, 2025 03:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Electron100 Electron100 Electron100 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

cargo test & cargo nextest fails listing tests in butane_codegen Testing macro expansion failures

2 participants

@jayvdb @Electron100