mhchia
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
mhchia
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
Kevin Mai-Husan Chia
9181cf95f0
Merge pull request #199 from ralexstokes/add-black-to-repo
...
Add `black` to repo
2019-08-01 10:53:44 +08:00
Alex Stokes
51cc710dc0
remove pylint
2019-07-31 15:14:56 -07:00
Alex Stokes
0ae9840928
Run black
over repo
2019-07-31 15:00:12 -07:00
Alex Stokes
a2133d8c7c
Add black
check to CI
2019-07-31 14:59:47 -07:00
Alex Stokes
936369aa5e
Add black
as a lint dependency
2019-07-31 14:53:28 -07:00
Kevin Mai-Husan Chia
1727ba48d9
Merge pull request #190 from mhchia/feature/pubsub-test
...
Add `test_pubsub.py`
2019-07-31 16:40:38 +08:00
mhchia
ce369d47e9
Refine exception message
2019-07-31 16:23:07 +08:00
Kevin Mai-Husan Chia
c0e253a524
Update tests/pubsub/conftest.py
...
Co-Authored-By: NIC Lin <twedusuck@gmail.com>
2019-07-31 16:07:48 +08:00
Kevin Mai-Husan Chia
3bb63612a9
Update tests/pubsub/conftest.py
...
Co-Authored-By: NIC Lin <twedusuck@gmail.com>
2019-07-31 16:07:04 +08:00
mhchia
9683d5e8ac
Add tests for Pubsub
...
- `test_handle_subscription`
- `test_handle_talk`
- `test_message_all_peers`
2019-07-31 14:15:11 +08:00
mhchia
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
mhchia
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
mhchia
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
mhchia
96563c0d84
Add fixtures for pubsub and router
...
And a starting `test_pubsub.py`
2019-07-31 14:14:13 +08:00
NIC Lin
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
NIC619
5e215901c0
Apply PR feedback
2019-07-30 23:41:28 +08:00
NIC619
76de01a17d
Add duplicate-code
and cyclic-import
to pylintrc
2019-07-30 18:01:01 +08:00
NIC619
437b7665c4
Fix:
...
type object not subscriptable
2019-07-30 18:00:30 +08:00
NIC619
60d6703964
Temporary disable pylint on tests folder
2019-07-30 17:41:46 +08:00
NIC619
c4105688d1
Fix after rebase
2019-07-30 17:31:08 +08:00
NIC619
e53727d301
Apply PR feedback: fix type hints
2019-07-30 16:28:25 +08:00
NIC619
2d4e23cfe2
Fix cyclic import and lint
2019-07-30 16:28:05 +08:00
NIC619
a4a0d79f6d
Improve import layout
2019-07-30 16:27:29 +08:00
NIC619
e7ac09cb94
Fix:
...
Add Gossipsub attribute `peers_protocol` and do cleanup when peer removed
2019-07-30 16:26:21 +08:00
NIC619
d716e90e17
Fix on type hints
2019-07-30 16:25:33 +08:00
NIC619
edd164c878
Add type hints to network folder
2019-07-30 16:25:17 +08:00
NIC619
e1592997a8
Add type hints to routing folder
2019-07-30 16:24:52 +08:00
NIC619
f2de986c74
Add type hints to peer folder
2019-07-30 16:24:51 +08:00
NIC619
b695b0e1ec
Add type hint to host folder
2019-07-30 16:24:34 +08:00
NIC619
b2f496d081
Fix type hints except pb msg in pubsub folder
2019-07-30 16:24:34 +08:00
NIC619
a0aa105867
Add type hint to pubsub notifee/interface
2019-07-30 16:23:15 +08:00
NIC619
3549f2ff8b
Add type hints to mcache.py
2019-07-30 16:20:48 +08:00
NIC619
63014eeaae
Add type hints to floodsub.py
2019-07-30 16:20:47 +08:00
NIC619
b920955db6
Add type hints to gossipsub.py
2019-07-30 15:39:50 +08:00
NIC619
8eb6a230ff
Fix and add type hints to pubsub.py
2019-07-30 15:32:58 +08:00
Kevin Mai-Husan Chia
f0046fa3e0
Merge pull request #187 from mhchia/feature/pubsub-publish
...
Add `Pubsub.publish`
2019-07-29 12:29:34 +08:00
mhchia
f02d38c0ee
Reflect PR feedback
...
* Rename `src` to `msg_forwarder` in pubsub/floodsub/gossipsub
* Rename Variables
* Sort imports
* Clean up
2019-07-29 12:09:35 +08:00
mhchia
74d831d4e2
Reflect PR feedback
2019-07-28 18:06:38 +08:00
Kevin Mai-Husan Chia
70c5c84f32
Update libp2p/pubsub/floodsub.py
...
Co-Authored-By: NIC Lin <twedusuck@gmail.com>
2019-07-28 16:09:01 +08:00
Kevin Mai-Husan Chia
a1e20caebe
Update libp2p/pubsub/floodsub.py
...
Co-Authored-By: NIC Lin <twedusuck@gmail.com>
2019-07-28 16:07:11 +08:00
Kevin Mai-Husan Chia
ffb3920468
Update libp2p/pubsub/floodsub.py
...
Co-Authored-By: NIC Lin <twedusuck@gmail.com>
2019-07-28 16:06:03 +08:00
Kevin Mai-Husan Chia
c252c62009
Update libp2p/pubsub/pubsub.py
...
Co-Authored-By: NIC Lin <twedusuck@gmail.com>
2019-07-28 16:05:29 +08:00
mhchia
766d8ba1e1
A little bit clean up
2019-07-27 12:06:36 +08:00
mhchia
c028aef2de
Fix all tests
...
- Dedup `perform_test_from_obj` and the test cases used in both
`test_floodsub` and `test_gossipsub_backward_compatibility.py`.
Therefore, they are put in the standalone file
`tests/pubsub/floodsub_integration_test_settings.py`. The functions
and testcases are imported from there then.
- IMO still need a refactor on the tests. There are still some duplicate
code.
2019-07-27 11:49:03 +08:00
mhchia
65aedcb25a
Fix several tests
2019-07-26 18:35:25 +08:00
mhchia
035d08b8bd
Fix test_floodsub.py
2019-07-25 23:11:27 +08:00
mhchia
dadcf8138e
Fix the tests according to pubsub.Publish
...
And refactored a bit.
2019-07-25 16:58:00 +08:00
mhchia
cae4f34034
Refactor floodsub.publish
...
Passed the first test of floodsub
2019-07-25 14:08:16 +08:00