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

Update Sonar to Hypercore v10 and Hyperswarm v4 #87

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Frando
Copy link
Member

@Frando Frando commented Oct 28, 2022

This moves Sonar to Hypercore v10 and Hyperswarm v4.

It depends on several not-yet-released dependencies.

Core tests pass!

Open TODOs are:

  • Tests take longer than before. Likely we await an event that is emitted only after connections are established or similar. Hyperswarm v4 should not take longer for connections than v2. Needs to be investigated.
  • Review non-core tests
  • Release all dependency upgrade
  • Work with @RangerMauve on hyper-sdk upgrade branch

@Frando
Copy link
Member Author

Frando commented Oct 28, 2022

CI fails with a V8 error. This needs to be investigated but might be unrelated? Locally, all tests pass.

Log of local test run on hypercore10 branch
~/Code/sonar/sonar hypercore10*
(base) $ yarn test
yarn run v1.22.19
$ tape packages/*/test/*.js
TAP version 13
# open close
ok 1 opened property is set
# put and get 1
ok 2 should be strictly equal
ok 3 should be strictly equal
ok 4 should be strictly equal
# batch and query
ok 5 hello search
ok 6 hello results ok
ok 7 moon search
ok 8 moon results ok
ok 9 should be strictly equal
# close collection
ok 10 opened property set
ok 11 closed property set
# query empty collection
ok 12 empty result
# basic put and get
ok 13 should be strictly equal
ok 14 should be strictly equal
ok 15 should be strictly equal
ok 16 should be deeply equivalent
# basic replication
ok 17 c1 len ok
ok 18 c2 val ok
ok 19 c2 len ok
ok 20 c2 val ok
ok 21 c2 len ok
ok 22 c2 val updated
ok 23 should be strictly equal
ok 24 c1 query correct
# open and close and open
ok 25 should be strictly equal
ok 26 should be strictly equal
ok 27 should be deeply equivalent
ok 28 should be strictly equal
ok 29 should be deeply equivalent
# replication with 3 workspaces
ok 30 should be strictly equal
ok 31 should be strictly equal
ok 32 should be strictly equal
# put query without replication
ok 33 should be strictly equal
ok 34 should be strictly equal
# simple replication
ok 35 init collection1 ok (result len)
ok 36 init collection1 ok (value)
ok 37 init collection2 ok (result len)
ok 38 init collection2 ok (value)
ok 39 end collection1 ok (result len)
ok 40 end collection1 ok (value)
ok 41 end collection2 ok (result len)
ok 42 end collection2 ok (value)
# subscription stream
ok 43 record arrived in subscription stream
# relations
ok 44 should be deeply equivalent
# basic
[2022-10-28T12:20:37 WARN] Authentication is disabled.
ok 45 key is 32 byte hex string
ok 46 get collection: key correct
ok 47 get collection: root key correct
ok 48 query result is array
ok 49 query length 1
ok 50 query id matches put

1..50

tests 50

pass 50

ok

Done in 167.09s.

@arso-project arso-project deleted a comment from netlify bot Oct 28, 2022
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.

1 participant