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

feat: add contract abstraction to connection router #267

Merged
merged 11 commits into from
Feb 16, 2024

Conversation

maancham
Copy link
Contributor

Description

Todos

  • Unit tests
  • Manual tests
  • Documentation
  • Connect epics/issues

Steps to Test

Expected Behaviour

Other Notes

Copy link

codecov bot commented Feb 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (0ced191) 89.53% compared to head (867d67b) 89.56%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #267      +/-   ##
==========================================
+ Coverage   89.53%   89.56%   +0.02%     
==========================================
  Files         164      165       +1     
  Lines       17007    17050      +43     
==========================================
+ Hits        15228    15271      +43     
  Misses       1779     1779              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@maancham maancham changed the title add contract abstraction to connection router feat: add contract abstraction to connection router Feb 12, 2024
integration-tests/src/contract.rs Outdated Show resolved Hide resolved
contracts/connection-router/tests/test.rs Outdated Show resolved Hide resolved
contracts/connection-router/tests/test.rs Outdated Show resolved Hide resolved
contracts/connection-router/tests/test.rs Outdated Show resolved Hide resolved
@maancham maancham marked this pull request as ready for review February 15, 2024 21:43
@maancham maancham requested a review from a team as a code owner February 15, 2024 21:43
@maancham maancham enabled auto-merge (squash) February 15, 2024 21:54
@maancham maancham merged commit e7a2301 into main Feb 16, 2024
7 checks passed
@maancham maancham deleted the AXE-3102-Add-Contract-Trait-to-all-Tests branch February 16, 2024 00:19
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.

2 participants