Browse Source

libs: minor cleanup (#3794)

* more minor cleanup of libs

Remove unused `version.go`, `assert.go` and `libs/circle.yml`

* Update types/vote_set_test.go

Co-Authored-By: Anton Kaliaev <anton.kalyaev@gmail.com>

* spelling change
pull/3802/head
Marko 5 years ago
committed by Anton Kaliaev
parent
commit
e0b9298134
6 changed files with 5 additions and 40 deletions
  1. +1
    -0
      CHANGELOG_PENDING.md
  2. +0
    -21
      libs/circle.yml
  3. +0
    -14
      libs/test/assert.go
  4. +0
    -3
      libs/version/version.go
  5. +3
    -2
      types/vote_set_test.go
  6. +1
    -0
      types/vote_test.go

+ 1
- 0
CHANGELOG_PENDING.md View File

@ -23,6 +23,7 @@ program](https://hackerone.com/tendermint).
- [libs] Remove unused `db/debugDB` and `common/colors.go` & `errors/errors.go` files (@marbar3778)
- [libs] \#2432 Remove unused `common/heap.go` file (@marbar3778)
- [libs] Remove unused `date.go`, `io.go`. Remove `GoPath()`, `Prompt()` and `IsDirEmpty()` functions from `os.go` (@marbar3778)
- [libs] Remove unused `version.go`, `assert.go` and `libs/circle.yml`
- [libs] Remove unused `FailRand()` func and minor clean up to `fail.go`(@marbar3778)
- Blockchain Protocol


+ 0
- 21
libs/circle.yml View File

@ -1,21 +0,0 @@
machine:
environment:
GOPATH: "${HOME}/.go_workspace"
PROJECT_PARENT_PATH: "$GOPATH/src/github.com/$CIRCLE_PROJECT_USERNAME"
PROJECT_PATH: $GOPATH/src/github.com/$CIRCLE_PROJECT_USERNAME/$CIRCLE_PROJECT_REPONAME
hosts:
localhost: 127.0.0.1
dependencies:
override:
- mkdir -p "$PROJECT_PARENT_PATH"
- ln -sf "$HOME/$CIRCLE_PROJECT_REPONAME/" "$PROJECT_PATH"
post:
- go version
test:
override:
- cd $PROJECT_PATH && make get_tools && bash ./test.sh
post:
- cd "$PROJECT_PATH" && bash <(curl -s https://codecov.io/bash) -f coverage.txt
- cd "$PROJECT_PATH" && mv coverage.txt "${CIRCLE_ARTIFACTS}"

+ 0
- 14
libs/test/assert.go View File

@ -1,14 +0,0 @@
package test
import (
"testing"
)
func AssertPanics(t *testing.T, msg string, f func()) {
defer func() {
if err := recover(); err == nil {
t.Errorf("Should have panic'd, but didn't: %v", msg)
}
}()
f()
}

+ 0
- 3
libs/version/version.go View File

@ -1,3 +0,0 @@
package version
const Version = "0.9.0"

+ 3
- 2
types/vote_set_test.go View File

@ -4,9 +4,10 @@ import (
"bytes"
"testing"
"github.com/stretchr/testify/assert"
"github.com/tendermint/tendermint/crypto"
cmn "github.com/tendermint/tendermint/libs/common"
tst "github.com/tendermint/tendermint/libs/test"
tmtime "github.com/tendermint/tendermint/types/time"
)
@ -490,7 +491,7 @@ func TestMakeCommit(t *testing.T) {
}
// MakeCommit should fail.
tst.AssertPanics(t, "Doesn't have +2/3 majority", func() { voteSet.MakeCommit() })
assert.Panics(t, func() { voteSet.MakeCommit() }, "Doesn't have +2/3 majority")
// 7th voted for some other block.
{


+ 1
- 0
types/vote_test.go View File

@ -7,6 +7,7 @@ import (
"github.com/stretchr/testify/assert"
"github.com/stretchr/testify/require"
amino "github.com/tendermint/go-amino"
"github.com/tendermint/tendermint/crypto"
"github.com/tendermint/tendermint/crypto/ed25519"


Loading…
Cancel
Save