Skip to content

Commit

Permalink
Merge pull request #1510 from ipfs/pinfail-debug
Browse files Browse the repository at this point in the history
print output on test failure for t0081
  • Loading branch information
jbenet committed Jul 28, 2015
2 parents 870d850 + 138198e commit f417cb5
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 6 deletions.
5 changes: 0 additions & 5 deletions core/commands/pin.go
Original file line number Diff line number Diff line change
Expand Up @@ -60,11 +60,6 @@ on disk.
recursive = false
}

go func() {
<-req.Context().Done()
log.Error("CONTEXT IS OVER!")
}()

added, err := corerepo.Pin(n, req.Context(), req.Arguments(), recursive)
if err != nil {
res.SetError(err, cmds.ErrNormal)
Expand Down
3 changes: 2 additions & 1 deletion test/sharness/t0081-repo-pinning.sh
Original file line number Diff line number Diff line change
Expand Up @@ -243,7 +243,8 @@ test_expect_success "some are no longer there" '
test_expect_success "recursive pin fails without objects" '
ipfs pin rm "$HASH_DIR1" &&
test_must_fail ipfs pin add -r "$HASH_DIR1" --timeout=500ms 2>err_expected8 &&
grep "context deadline exceeded" err_expected8
grep "context deadline exceeded" err_expected8 ||
test_fsh cat err_expected8
'

# test_kill_ipfs_daemon
Expand Down

0 comments on commit f417cb5

Please sign in to comment.