Skip to content

Nip46 mock remote signer and nip46 tests updated#216

Merged
nogringo merged 5 commits intonip46-event-signerfrom
nip46-mock-remote-signer
Aug 14, 2025
Merged

Nip46 mock remote signer and nip46 tests updated#216
nogringo merged 5 commits intonip46-event-signerfrom
nip46-mock-remote-signer

Conversation

@nogringo
Copy link
Collaborator

Note

There is no test for the nostr connect

@codecov
Copy link

codecov bot commented Aug 13, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 71.50%. Comparing base (b8a28f3) to head (2748593).
⚠️ Report is 6 commits behind head on nip46-event-signer.

Additional details and impacted files
@@                  Coverage Diff                   @@
##           nip46-event-signer     #216      +/-   ##
======================================================
+ Coverage               67.83%   71.50%   +3.66%     
======================================================
  Files                     124      124              
  Lines                    4393     4393              
======================================================
+ Hits                     2980     3141     +161     
+ Misses                   1413     1252     -161     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@frnandu frnandu changed the base branch from master to nip46-event-signer August 13, 2025 16:55
@nogringo nogringo requested a review from frnandu August 13, 2025 17:18
Copy link
Collaborator

@frnandu frnandu left a comment

Choose a reason for hiding this comment

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

nice

@nogringo nogringo merged commit 83f2d35 into nip46-event-signer Aug 14, 2025
3 of 4 checks passed
@nogringo nogringo deleted the nip46-mock-remote-signer branch August 14, 2025 10:02
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