Skip to content
This repository has been archived by the owner on Jan 3, 2022. It is now read-only.

Add tests #28

Merged
merged 2 commits into from
Jan 10, 2019
Merged

Add tests #28

merged 2 commits into from
Jan 10, 2019

Conversation

petroav
Copy link
Contributor

@petroav petroav commented Jan 10, 2019

This brings test coverage to 88%.

Could someone also edit the description of the project in Github? I think this repo no longer contains Test utilities for spinning up swarm networks. In the README the description I've used is Utility methods for creating public and private keys for use in tests; feel free to use that.

Copy link
Member

@Stebalien Stebalien left a comment

Choose a reason for hiding this comment

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

Thanks! Yeah, this used to be for spinning up test swarms but we moved that to a subpackage of swarm (cleans up the dependency tree a bit).

README.md Outdated
go-libp2p-netutil
==================

[![](https://img.shields.io/badge/made%20by-Protocol%20Labs-blue.svg?style=flat-square)](http://ipn.io)
Copy link
Member

Choose a reason for hiding this comment

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

This should be https://protocol.ai.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

@petroav
Copy link
Contributor Author

petroav commented Jan 10, 2019

Hey @Stebalien -- thanks for taking a look! Just updated branch.

@Stebalien Stebalien merged commit d58056f into libp2p:master Jan 10, 2019
@Stebalien
Copy link
Member

Thanks! Better test coverage and docs are always welcome!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants