Skip to content

Move TestEnv and tests to tests/#207

Merged
luisschwab merged 1 commit into
bitcoindevkit:masterfrom
luisschwab:chore/move-tests
May 4, 2026
Merged

Move TestEnv and tests to tests/#207
luisschwab merged 1 commit into
bitcoindevkit:masterfrom
luisschwab:chore/move-tests

Conversation

@luisschwab
Copy link
Copy Markdown
Member

@luisschwab luisschwab commented May 3, 2026

Closes #199

Moves the TestEnv and tests from src/lib.rs to tests/.

Changelog

## Changed
- Move `TestEnv` to `tests/testenv/mod.rs`
- Move tests to `tests/*`

@luisschwab luisschwab self-assigned this May 3, 2026
@luisschwab luisschwab requested a review from oleonardolima as a code owner May 3, 2026 18:22
@luisschwab
Copy link
Copy Markdown
Member Author

luisschwab commented May 3, 2026

ping @oleonardolima

oleonardolima
oleonardolima reviewed May 4, 2026
View reviewed changes
Copy link
Copy Markdown
Collaborator

@oleonardolima oleonardolima left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cACK 88f1bbd

Comment thread tests/testenv/mod.rs Outdated
oleonardolima
oleonardolima approved these changes May 4, 2026
View reviewed changes
Copy link
Copy Markdown
Collaborator

@oleonardolima oleonardolima left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK b8a353d

@oleonardolima
Copy link
Copy Markdown
Collaborator

oleonardolima commented May 4, 2026

@luisschwab :shipit:

@luisschwab luisschwab merged commit b0ab0a4 into bitcoindevkit:master May 4, 2026
8 checks passed
@luisschwab luisschwab deleted the chore/move-tests branch May 4, 2026 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@oleonardolima oleonardolima oleonardolima approved these changes

Assignees

@luisschwab luisschwab

Labels

chore

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Move tests and TestEnv to tests/

2 participants

@luisschwab @oleonardolima