Commit Graph

621 Commits

Author SHA1 Message Date
cff5fe0d5f Add pytest config 2019-08-02 15:58:39 -07:00
28e1a03dc4 remove empty yamux impl 2019-08-02 15:03:59 -07:00
d47cddee24 Clean up extraneous files in tests 2019-08-02 14:54:40 -07:00
224b54ad93 Add basic flake8 config 2019-08-02 14:38:03 -07:00
2dfd7794b6 add note about protoc version and add Make command to generate protobuf files 2019-08-02 12:20:48 -07:00
e55d9f2e60 restore generated protobuf files 2019-08-02 12:09:27 -07:00
88a3a3159e Add clarifying comment about InsecureTransport 2019-08-02 12:07:35 -07:00
b2367e35d8 Merge pull request #208 from ralexstokes/disable-mypy-ci
disable mypy check during CI temporarily and move `black --check` to Makefile
2019-08-02 12:01:47 -07:00
7b7c8ad30d run black over dangling files 2019-08-02 11:47:08 -07:00
da9d5cadec Disables mypy in CI
Also moves linting to Makefile to take advantage of globbing.
2019-08-02 11:46:43 -07:00
612330d318 Merge pull request #205 from mhchia/fix/remove-pylint-leftover
Remove pylint related lines and files
2019-08-02 10:50:44 -07:00
2e94fcf56c Remove pylint:disable 2019-08-02 23:26:06 +08:00
06a9511ab4 Remove .pylintrc 2019-08-02 23:20:30 +08:00
7a0fa7dd37 Merge pull request #196 from NIC619/fix_peer_id
Refactor `peer.ID` class and only takes in `bytes` type argument
2019-08-02 18:03:56 +08:00
568454534f Remove unused ID type conversions 2019-08-02 16:48:38 +08:00
ee290b2ac2 Fix missing asterisk 2019-08-02 16:48:32 +08:00
3a4b592a5a Fix missing asterisk 2019-08-02 15:06:39 +08:00
38423cc2a4 Merge pull request #200 from ChihChengLiang/add-typing-security
Add typing to security module
2019-08-02 14:57:24 +08:00
0173b5e0d9 Add ID.to_base58() and comparison against str type 2019-08-02 14:46:01 +08:00
924e965537 Change argument name of Network.listen and blakc format 2019-08-02 14:45:59 +08:00
cd684aad9e Update peer_id to type peer.ID in pubsub folder 2019-08-02 14:45:23 +08:00
9562cb2a46 Rename:
`KadPeerInfo.peer_id` to `KadPeerInfo.peer_id_bytes`
2019-08-02 14:43:01 +08:00
f00e80bc25 Fix wrong peer id type used in KadPeerInfo 2019-08-02 14:42:10 +08:00
b928bdb356 Convert from base58/pubkey/privkey to class method 2019-08-02 14:42:10 +08:00
80481252ca Refactor ID to take in type bytes only 2019-08-02 14:42:10 +08:00
10a8347c6a PR feedback 2019-08-02 14:12:59 +08:00
f6e456c96e Merge pull request #198 from mhchia/fix/clean-up-tests-with-fixture
Use factories and fixtures in pubsub tests
2019-08-01 22:08:03 +08:00
716c60ca6d Reflect PR feedback
- Remove leftover `int` in `GossipsubParams`
- Remove default fields in tests
2019-08-01 21:38:14 +08:00
e731f77f2d minor 2019-08-01 19:13:43 +08:00
a86f010c95 add typing to security 2019-08-01 19:12:11 +08:00
030abcc959 add vscode 2019-08-01 17:55:15 +08:00
33d233f5df Add missing comma 2019-08-01 13:31:02 +08:00
4eb846be7c Remove the assertions for connect
Because it fails in `test_security_multistream.py` and this check is
indeed not necessary inside `connect`.(Should probably be outside the
function.)
2019-08-01 13:30:09 +08:00
c72dfe1dd3 Use factories and fixtures in pubsub tests
Done
- Add factories using factory-boy
- Modify fixtures and tests to use factories
- Modify tests to use fixtures and factories
- Clean up
2019-08-01 13:30:08 +08:00
9181cf95f0 Merge pull request #199 from ralexstokes/add-black-to-repo
Add `black` to repo
2019-08-01 10:53:44 +08:00
51cc710dc0 remove pylint 2019-07-31 15:14:56 -07:00
0ae9840928 Run black over repo 2019-07-31 15:00:12 -07:00
a2133d8c7c Add black check to CI 2019-07-31 14:59:47 -07:00
936369aa5e Add black as a lint dependency 2019-07-31 14:53:28 -07:00
1727ba48d9 Merge pull request #190 from mhchia/feature/pubsub-test
Add `test_pubsub.py`
2019-07-31 16:40:38 +08:00
ce369d47e9 Refine exception message 2019-07-31 16:23:07 +08:00
c0e253a524 Update tests/pubsub/conftest.py
Co-Authored-By: NIC Lin <twedusuck@gmail.com>
2019-07-31 16:07:48 +08:00
3bb63612a9 Update tests/pubsub/conftest.py
Co-Authored-By: NIC Lin <twedusuck@gmail.com>
2019-07-31 16:07:04 +08:00
9683d5e8ac Add tests for Pubsub
- `test_handle_subscription`
- `test_handle_talk`
- `test_message_all_peers`
2019-07-31 14:15:11 +08:00
3a42d72cd9 Fix a minor bug for pb optional field
In `Pubsub.continuously_read_stream`, it checks whether this is a
control message enclosed in RPC message with `if rpc_incoming.control:`.
However, in pb2, the condition is always true because a default value is
returned when a field is not set. Solved it by changing it to
`if rpc_incoming.HasField("control"):`.
2019-07-31 14:15:11 +08:00
037b95252d Add tests for Pubsub
- `test_get_hello_packet`
- `test_continuously_read_stream`
- `test_publish`
- `test_push_msg`
2019-07-31 14:15:10 +08:00
550289a439 Combine test_subscription.py to test_pubsub.py
And add a bunch of tests for pubsub
2019-07-31 14:15:10 +08:00
96563c0d84 Add fixtures for pubsub and router
And a starting `test_pubsub.py`
2019-07-31 14:14:13 +08:00
21e97407ef Merge pull request #192 from NIC619/add_type_hint
Add type hints to host/, peer/, network/, pubsub/, routing/
2019-07-31 11:13:37 +08:00
5e215901c0 Apply PR feedback 2019-07-30 23:41:28 +08:00