Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

replace blackbox testing macros with build patterns #1927

Open
ethanoroshiba opened this issue Jan 24, 2025 · 0 comments
Open

replace blackbox testing macros with build patterns #1927

ethanoroshiba opened this issue Jan 24, 2025 · 0 comments
Labels
conductor pertaining to the astria-conductor crate tech-debt testing

Comments

@ethanoroshiba
Copy link
Contributor

ethanoroshiba commented Jan 24, 2025

The blackbox testing macros for Conductor have become unwieldy, especially as we've added more optional fields and tried to scale them. Additionally, they're difficult to learn and to work with when creating new blackbox tests or changing existing ones. We should aim to replace these with some sort of build pattern instead.

┆Issue Number: ENG-1078

@ethanoroshiba ethanoroshiba added conductor pertaining to the astria-conductor crate tech-debt testing labels Jan 24, 2025
@SuperFluffy SuperFluffy changed the title chore(conductor): replace blackbox testing macros with build patterns replace blackbox testing macros with build patterns Jan 30, 2025
@SuperFluffy SuperFluffy added the ignore-stale Override for issues or PRs which should not be removed if stale. label Jan 30, 2025
@syedali012 syedali012 removed the ignore-stale Override for issues or PRs which should not be removed if stale. label Jan 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
conductor pertaining to the astria-conductor crate tech-debt testing
Projects
None yet
Development

No branches or pull requests

3 participants