Turtle
98b3dcae7b
Test that server validates signature properly
2019-11-28 09:04:43 -05:00
Turtle
7713a3cd47
Server validates appointment signature upon reception
2019-11-26 23:21:07 -05:00
Turtle
79d986677d
Client signs appointment before sending it to server
2019-11-26 23:21:00 -05:00
Sergi Delgado Segura
d2b7216a96
Addresses requested changes
2019-11-22 14:48:38 +00:00
Sergi Delgado Segura
2183c57f53
Updates handle_reorgs and deletes check_tx_in_chain
...
Updates handle_reorg to work with the current version of the Responder (the old code was outdated and broken).
`check_tx_in_chain` was only used by `handle_reorgs`, and made not much sense at this point. The method need to check if the transaction is in mempool or blockchain, so it seems to make more sense bulding on top of `get_transaction`.
2019-11-20 15:41:50 +00:00
Sergi Delgado Segura
3dad5b7c71
Adds tests for check_tx_in_chain
2019-11-20 13:01:56 +00:00
Sergi Delgado Segura
05961f1632
Moves tx_in_chain to Carrier
...
Also integrates it properly so it uses self.get_transaction() instead of bitcoin_cli straightaway. Error messages have also been merged / modified
2019-11-20 13:01:27 +00:00
Sergi Delgado Segura
f91413ebd8
Properly handles mempool checks
...
Transactions were included in the mempool as rawtxs but checked as txids, so effectively every time we checked if a transaction was in mempool it returned false
2019-11-20 12:59:29 +00:00
Sergi Delgado Segura
287dfeeee3
Adds missing test on BlockProessor
2019-11-18 16:39:50 +00:00
Sergi Delgado Segura
764e513ab9
Replaces asserts for Exceptions in simulator
2019-11-18 16:39:23 +00:00
Sergi Delgado Segura
a1c0eeadbc
Add missing EncryptedBlob tests
2019-11-18 15:25:15 +00:00
Sergi Delgado Segura
6ff580ed8f
Updates test_api according to 1b78060 and add missing cases
2019-11-14 17:38:26 +00:00
Sergi Delgado Segura
3a1bf0cc8a
Updates API to use DB on get methods
2019-11-14 17:38:26 +00:00
Sergi Delgado Segura
f33d2d61ac
Moves start api to test_api since it was the only place where it's used
2019-11-14 17:38:19 +00:00
Sergi Delgado Segura
1d1efc9ccf
Adds some missing Responder unit tests
...
Excluding the basic reorgs hangle, we need a better bitcoind mock or a different testing approach to reach a higher code coverage. This is as far as it can get for the Responder at the moment
2019-11-14 15:05:45 +00:00
Sergi Delgado Segura
ca3f6ee1b1
Fixes memory leak bug regarding test db
2019-11-14 15:04:55 +00:00
Sergi Delgado Segura
f65e2af675
Deletes missed print
2019-11-14 15:03:03 +00:00
Sergi Delgado Segura
c54d10f92c
Fixes Responder
...
There were a couple of wrong things going on in the `Responder`:
- `check_confirmations` included in 8a10979db0 was not doing a good job keeping track of the missed confirmations, probably due to a overlooked refactor.
- There was an edge case when adding txs to `unconfirmed_txs` on `create_job` that may lead to a transaction being confirmed and unconfirmed at the same time from the `Responder's` pow. It could be triggered by adding two jobs with the same justice_txid.
2019-11-11 17:50:28 +00:00
Sergi Delgado Segura
f81cb04b74
Fixes get_distance_to_tip
2019-11-11 14:03:18 +00:00
Sergi Delgado Segura
fb7dfd4df8
Add missing Cleaner unit tests
2019-11-11 13:28:24 +00:00
Sergi Delgado Segura
3c6f13ef00
Rename delete_completed_appointments and rearange arguments
2019-11-11 13:27:57 +00:00
Sergi Delgado Segura
a1a1206e0c
Adds new Watcher unit tests
2019-11-08 18:25:14 +00:00
Sergi Delgado Segura
61a36048de
Removes redundant tests
2019-11-08 18:24:33 +00:00
Sergi Delgado Segura
4233c9d5cc
Refactor get_matches and filter_valid_matches to not be static
2019-11-08 18:21:42 +00:00
Sergi Delgado Segura
e65b02c473
Adds Cryptographer unit tests
2019-11-08 18:20:25 +00:00
Sergi Delgado Segura
da61ea9b8e
Makes sure that blobs to be decrypred are even-length
2019-11-08 18:20:05 +00:00
Sergi Delgado Segura
8a10979db0
Refactors Responder and BlockProcessor
...
Brings check_confirmations back to the Responder
2019-11-08 16:25:40 +00:00
Sergi Delgado Segura
20faa04c4c
Refactord Watcher and BlockProcessor. Closes #36 .
...
Brings back check_potential_matches and check_matches to the Watcher.
The methods have now been renamed to check_matches (old check_potential_matches) and filter_valid_matches (old check_matches) to provide a better description of their purpose.
decode_raw_transaction has been added to BlockProcessor
2019-11-08 16:25:35 +00:00
Sergi Delgado Segura
c08013abd0
Creates Cryptographer. Separates decryptiomn from encrypted_blob
2019-11-08 16:24:31 +00:00
Sergi Delgado Segura
f4f77fb39a
Improves test appointment and add missing tests
2019-11-07 19:16:39 +00:00
Sergi Delgado Segura
c384bc78c2
Changes code to use bitcoin_cli from pisa/tools
2019-11-07 19:16:18 +00:00
Sergi Delgado Segura
eb0417858e
Adds missing test for EncryptedBlob
2019-11-07 19:16:18 +00:00
Sergi Delgado Segura
8b2f0efa2f
Moves appointment deletion + db update to the cleaner
2019-11-07 19:16:05 +00:00
Sergi Delgado Segura
82f9de1717
Adds missing Watcher test
2019-11-07 16:56:31 +00:00
Sergi Delgado Segura
2b1640ea86
Simplifies Inspector using Appointment.from_json() constructor
2019-11-07 16:56:30 +00:00
Sergi Delgado Segura
8f7505c17a
Improves generate_dummy_appointment
2019-11-07 16:56:15 +00:00
Sergi Delgado Segura
1eb4423e58
Updates gitignore
2019-11-07 15:56:49 +00:00
Sergi Delgado Segura
20f296c9d4
Reformats code to match code guidelines
2019-11-07 15:47:35 +00:00
Sergi Delgado Segura
b16775a4aa
Updates style guidelines
2019-11-07 15:08:06 +00:00
Sergi Delgado Segura
8174ec61af
Generalizes get_dummy_appointment (there were multiple instances of it in different tests)
...
Also refactors some unused imports and modifies the watcher tests to use .serialize instead of to_json
2019-11-07 15:08:06 +00:00
Sergi Delgado Segura
e6e577c99e
Updates Watcher to use serialize instead of to_json
2019-11-07 15:08:05 +00:00
Sergi Delgado Segura
30159399f0
Adds serialize method
...
After adding the triggered flag we can not just use to_json to serialize since the two data blobs won't match. This also sets ground towards having a generalized serialization function.
2019-11-07 15:08:05 +00:00
Sergi Delgado Segura
99bda73eff
Applies fixes after PR review
2019-11-07 15:07:58 +00:00
Salvatore Ingala
00a27b68e6
Various fixes from PR review
2019-11-07 15:04:21 +00:00
Salvatore Ingala
c786e3d225
Refactored add_appointment and get_appointment to avoid block nesting
2019-11-07 15:04:21 +00:00
Salvatore Ingala
24a451c108
Avoiding long strings by using random dummy txids; code formatting
2019-11-07 15:04:21 +00:00
Salvatore Ingala
21db172727
Refactored pisa_cli to improve testability; added test with wrong signature returned
2019-11-07 15:04:21 +00:00
Salvatore Ingala
956d7091eb
Initial tests for pisa-cli
2019-11-07 15:04:21 +00:00
Sergi Delgado Segura
76059d569e
Adds db section to sample_conf
2019-11-06 17:34:18 +00:00
Sergi Delgado Segura
31afb1373a
Integrates the locator map dumping to the Watcher and Cleaner
2019-11-06 17:08:42 +00:00