refactor(nodebuilder/p2p): Add errors for all methods (#1539)

Based on #1429 Adds an error as a return value for all methods as it is needed for elevated permission methods.
celestiaorg · Dec 23, 2022 · 0ab5df6 · 0ab5df6
1 parent bcaa431
commit 0ab5df6
Show file tree

Hide file tree

Showing 4 changed files with 147 additions and 102 deletions.
diff --git a/docs/adr/adr-009-public-api.md b/docs/adr/adr-009-public-api.md
@@ -143,10 +143,10 @@ SyncHead(ctx context.Context) (*header.ExtendedHeader, error)
  // Info returns address information about the host.
  Info(context.Context) (peer.AddrInfo, error)
  // Peers returns all peer IDs used across all inner stores.
- Peers(context.Context) []peer.ID
+ Peers(context.Context) ([]peer.ID, error)
  // PeerInfo returns a small slice of information Peerstore has on the
  // given peer.
- PeerInfo(context.Context, peer.ID) peer.AddrInfo
+ PeerInfo(context.Context, peer.ID) (peer.AddrInfo, error)
 
  // Connect ensures there is a connection between this host and the peer with
  // given peer.

diff --git a/nodebuilder/p2p/mocks/api.go b/nodebuilder/p2p/mocks/api.go