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 random for test/ #289

Merged
merged 5 commits into from
Nov 9, 2014
Merged

Update random for test/ #289

merged 5 commits into from
Nov 9, 2014

Conversation

jbenet
Copy link
Member

@jbenet jbenet commented Nov 9, 2014

The tests inside test/ used github.com/jbenet/go-random
I updated it (to make it faster) changing what the output
would be, breaking tests. This PR:

  • updates the expected hashes using new random
  • vendors go-random so that it doesn't happen again
  • moves things in test/ to test/bin and test/lib

I moved installed things into own dirs bin and lib.

@chriscool sorry to move things around again, the top level
test dir was getting a bit clutterd.
There is a dummy test import: test/lib/random-dep.go
Because godep doen't yet vendor binaries nicely.
@jbenet jbenet added the status/in-progress In progress label Nov 9, 2014
jbenet added a commit that referenced this pull request Nov 9, 2014
@jbenet jbenet merged commit 7ba3cad into master Nov 9, 2014
@jbenet jbenet removed the status/in-progress In progress label Nov 9, 2014
@jbenet jbenet deleted the update-random-sharness-tests branch November 9, 2014 04:21
ariescodescream pushed a commit to ariescodescream/go-ipfs that referenced this pull request Oct 23, 2021
@lidel lidel mentioned this pull request Nov 24, 2022
12 tasks
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.

2 participants