Skip to content

Snapshot tests fix#692

Merged
Smartdevs17 merged 2 commits into
Smartdevs17:mainfrom
eischideraa-unn:snapshot-tests-fix
Jun 28, 2026
Merged

Snapshot tests fix#692
Smartdevs17 merged 2 commits into
Smartdevs17:mainfrom
eischideraa-unn:snapshot-tests-fix

Conversation

@eischideraa-unn

Copy link
Copy Markdown
Contributor

Summary

Added snapshot test coverage for core UI components to help detect unintended visual or structural changes during future development.

Closes #52

Changes Made

  • Added snapshot tests for:

    • Button
    • Card
    • SubscriptionCard
  • Generated and committed corresponding snapshots.

  • Improved regression protection for commonly used UI components.

Branch & Commit

  • Branch: snapshot-tests-fix
  • Commit: Add snapshot tests for Button/Card/SubscriptionCard

PR Link

https://github.com/eischideraa-unn/SubTrackr/pull/new/snapshot-tests-fix


Pull Request Checklist

Quality Gates (All must pass before merge)

  • Lint: Code passes ESLint and Prettier checks
  • Type Check: TypeScript compilation succeeds
  • Tests: All tests pass
  • Build: Project builds successfully
  • Rust Format: Smart contract formatting is correct
  • Rust Clippy: Smart contract linting passes
  • Rust Tests: All smart contract tests pass
  • Rust Build: Smart contracts compile successfully

Additional Requirements

  • New code has appropriate TypeScript types
  • No hardcoded secrets or credentials
  • New features have corresponding tests
  • Documentation updated if needed

Reviewers

  • At least 1 approval required for merge
  • All CI checks must be green

This PR will not be mergeable until all quality gates pass.

@drips-wave

drips-wave Bot commented Jun 26, 2026

Copy link
Copy Markdown

@eischideraa-unn Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Smartdevs17 Smartdevs17 merged commit 37e8860 into Smartdevs17:main Jun 28, 2026
8 of 31 checks passed
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.

🧪 Add React Native component snapshot tests

2 participants