Skip to content

Conversation

@patwey
Copy link
Collaborator

@patwey patwey commented Oct 15, 2025

RFC Pull Request

RFC Number: 0008
RFC Title: Program Testing Strategy
Status: Discussion

Summary

Checklist

  • RFC follows the template structure
  • RFC number is the next available number
  • Title is clear and descriptive
  • All required sections are filled out
  • Related issues/PRs are linked
  • Relevant stakeholders are tagged

By submitting this RFC, I agree to engage constructively with feedback and work toward consensus.

@patwey patwey requested review from catonph, jmejia and kbrey October 15, 2025 23:34
@patwey patwey self-assigned this Oct 15, 2025
@patwey patwey marked this pull request as draft October 16, 2025 21:42
@patwey patwey removed request for catonph, jmejia and kbrey October 16, 2025 21:42
@patwey
Copy link
Collaborator Author

patwey commented Oct 16, 2025

After our conversation today, this likely needs to be adjusted and scoped to something actionable and inline with the program architecture refactor

@patwey patwey marked this pull request as ready for review October 16, 2025 22:32
@patwey patwey requested review from catonph, jmejia and kbrey October 16, 2025 22:32
Comment on lines +62 to +63
- Add coverage tooling
* Add `coverage.py` for BE coverage reporting
* Add GH workflow to upload BE coverage report to CodeCov
* Add GH workflow to upload FE coverage report to CodeCov
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I could see an argument for moving this to Phase 2 if we wanted to get to the new architecture and TX implementation ASAP.

I have it in Phase 1 because we want our TX implementation to be exemplary and I think it'll help us ensure our tests are comprehensive.

Copy link
Collaborator

@kbrey kbrey left a comment

Choose a reason for hiding this comment

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

Looks great, excited for this! Left a few thoughts.

@patwey patwey force-pushed the pat/rfc-program-testing-strategy branch from 73f985b to 4d2062f Compare October 22, 2025 17:26
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.

3 participants