Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat/zked #1148

Merged
merged 59 commits into from
Jun 27, 2019
Merged

Feat/zked #1148

merged 59 commits into from
Jun 27, 2019

Conversation

mikiquantum
Copy link
Contributor

@mikiquantum mikiquantum commented Jun 26, 2019

Updated to latest develop + Milestone 1 (https://centrifuge.hackmd.io/smRGRtyeQnSBYuu_fdswbQ?both)

mikiquantum and others added 30 commits May 6, 2019 17:42
* add core api handler

* add types test

* review changes

* add middleware tests

* working api

* add test world test

* update tests

* fix testworld

* remove middleware

* add accounts to header rule
* add update endpoint

* add testworld case

* fix testworld
* added protobuf for transfer from

* renamed unpaidInvoice to service in nft package

* added transferFromJob

* added transferFrom service method

* added TokenTransfer handler method

* added mock for transferFrom

* added integration test for nftTransfer

* added fail transactionFrom integration test

* added handler unit tests

* handler unit test

* fmt

* fmt

* clean up

* increased gas for transferFrom

* swagger

* better gaslimit for transferFrom

* added protobuf for ownerOf

* added handler for ownerOf

* fixed handler ownerOf

* added handler test for owner of

* better ownerOf response

* added testworld test for nftTransfer

* added testworld test owner of

* fmt

* change to new nft rest api paths

* fixed spacing

* fmt
* add incorrect node version, incorrect body tests

* remove unnecessary test function in incorrect_proto_test

* use updated peer interface to getPeerID

* removed oskar

* renamed messenger_testworld file

* add linting

* Revert "add linting"
fix linting issue
This reverts commit 8d85b3d.

* linting fix

* remove local account check

* lint fix

* lint fix

* add message over max size test

* lint fix

* add specific assertions
* add update endpoint

* add testworld case

* fix testworld

* migrate jobs to rest

* migrate jobs to rest

* fix unit tests

* add get and get version

* update swagger

* update testworld tests
* add update endpoint

* add testworld case

* fix testworld

* migrate jobs to rest

* migrate jobs to rest

* fix unit tests

* add get and get version

* update swagger

* update testworld tests

* add proofs endpoints

* add proofs endpoints

* update testworld tests

* better test coverage
mikiquantum and others added 22 commits June 11, 2019 10:39
* range for webhook http status code job log
* added test
* entity data

* add model methods

* add entity to the core apis

* add converters tests

* add test world cases

* fix testworld
* entity data

* add model methods

* add entity to the core apis

* add converters tests

* add test world cases

* fix testworld

* migrate min nft

* migrate transfer nft

* migrate owner nft

* fix unit tests

* fix test world

* fix nft tests

* fix unit and tes world tests
* add Byzantine test

* add client signature verification

* adjust unit test according changed function signature

* adjust unit test

* adjust unit test

* goimports

* fix signing root calculating

* adjust server_testworld test

* Adjust TestHost_RevokedSigningKey

* add another two signature test

* adjust TestHost_RevokedSigningKey

* no content, force a CI build, it is ok on my site

* add print information for debugging

* change according to comments

* adjust format

* increase timeout limit to 30 min
* version accounts

* funding service apis

* invoices and nft

* pos

* Byzantine fault behaviour tests 1 centrifuge#876 (centrifuge#1077)

* add Byzantine test

* add client signature verification

* adjust unit test according changed function signature

* adjust unit test

* adjust unit test

* goimports

* fix signing root calculating

* adjust server_testworld test

* Adjust TestHost_RevokedSigningKey

* add another two signature test

* adjust TestHost_RevokedSigningKey

* no content, force a CI build, it is ok on my site

* add print information for debugging

* change according to comments

* adjust format

* increase timeout limit to 30 min

* fix testworlds

* move collaborators up

* migrate attributes

* fix tests

* api todos

* rename nfts

* fix tests

* update swagger routes

* add debug

* fix tests


Co-authored-by: null <zy_frank@hotmail.com>
* add generic doc service

* add testworkd tests

* add update to service

* update deps

* change deps
@mikiquantum mikiquantum changed the base branch from develop to feat/zked June 26, 2019 15:14
@codecov
Copy link

codecov bot commented Jun 26, 2019

Codecov Report

Merging #1148 into feat/zked will increase coverage by 1.73%.
The diff coverage is 75.67%.

Impacted file tree graph

@@              Coverage Diff              @@
##           feat/zked    #1148      +/-   ##
=============================================
+ Coverage      66.86%   68.59%   +1.73%     
=============================================
  Files            142      160      +18     
  Lines          10892    12359    +1467     
=============================================
+ Hits            7283     8478    +1195     
- Misses          2735     2888     +153     
- Partials         874      993     +119
Impacted Files Coverage Δ
testworld/httputils.go 91.21% <ø> (+8.2%) ⬆️
utils/tools.go 68.64% <ø> (+1.1%) ⬆️
testworld/payloads.go 90.95% <ø> (+8.76%) ⬆️
extensions/funding/validator.go 100% <ø> (ø)
extensions/funding/test_bootstrapper.go 0% <ø> (ø)
nft/invoice_unpaid.go 56.52% <ø> (+30.43%) ⬆️
utils/strings.go 100% <ø> (ø) ⬆️
extensions/funding/bootstrapper.go 100% <ø> (ø)
testworld/park.go 78.63% <ø> (+2.8%) ⬆️
utils/httputils/utils.go 100% <ø> (ø)
... and 105 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7f74a2e...09586a7. Read the comment docs.

@mikiquantum mikiquantum merged commit 2ef111c into centrifuge:feat/zked Jun 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants