Browse Source

[test] remove test_libs

Reasons:
1) all deps we're using should be passing tests (including external)
2) deps can require complicated setup for testing
3) the person responsible for releasing Tendermint should be cautious
when updating a dep
pull/1378/head
Anton Kaliaev 7 years ago
parent
commit
9c757108ca
No known key found for this signature in database GPG Key ID: 7B6881D965918214
5 changed files with 1 additions and 100 deletions
  1. +0
    -21
      .circleci/config.yml
  2. +1
    -6
      Makefile
  3. +0
    -35
      scripts/dep_utils/checkout.sh
  4. +0
    -4
      test/README.md
  5. +0
    -34
      test/test_libs.sh

+ 0
- 21
.circleci/config.yml View File

@ -130,19 +130,6 @@ jobs:
paths:
- "profiles/*"
#test_libs:
# <<: *defaults
# steps:
# - attach_workspace:
# at: /tmp/workspace
# - restore_cache:
# key: v1-pkg-cache
# - restore_cache:
# key: v1-tree-{{ .Environment.CIRCLE_SHA1 }}
# - run:
# name: Run tests
# command: bash test/test_libs.sh
test_persistence:
<<: *defaults
steps:
@ -205,14 +192,6 @@ workflows:
- test_cover:
requires:
- setup_dependencies
#- test_libs:
#- filters:
#- branches:
#- only:
#- - develop
#- - master
#- requires:
#- - setup_dependencies
- test_persistence:
requires:
- setup_abci


+ 1
- 6
Makefile View File

@ -119,11 +119,6 @@ test_integrations:
make test_persistence
make test_p2p
test_libs:
# checkout every github.com/tendermint dir and run its tests
# NOTE: on release-* or master branches only (set by Jenkins)
docker run --name run_libs -t tester bash test/test_libs.sh
test_release:
@go test -tags release $(PACKAGES)
@ -186,4 +181,4 @@ metalinter_all:
# To avoid unintended conflicts with file names, always add to .PHONY
# unless there is a reason not to.
# https://www.gnu.org/software/make/manual/html_node/Phony-Targets.html
.PHONY: check build build_race dist install check_tools get_tools update_tools get_vendor_deps draw_deps test_cover test_apps test_persistence test_p2p test test_race test_libs test_integrations test_release test100 vagrant_test fmt
.PHONY: check build build_race dist install check_tools get_tools update_tools get_vendor_deps draw_deps test_cover test_apps test_persistence test_p2p test test_race test_integrations test_release test100 vagrant_test fmt

+ 0
- 35
scripts/dep_utils/checkout.sh View File

@ -1,35 +0,0 @@
#! /bin/bash
set -ex
set +u
if [[ "$DEP" == "" ]]; then
DEP=$GOPATH/src/github.com/tendermint/tendermint/Gopkg.lock
fi
set -u
set -u
function getVendoredVersion() {
grep -A100 "$LIB" "$DEP" | grep revision | head -n1 | grep -o '"[^"]\+"' | cut -d '"' -f 2
}
# fetch and checkout vendored dep
lib=$1
echo "----------------------------------"
echo "Getting $lib ..."
go get -t "github.com/tendermint/$lib/..."
VENDORED=$(getVendoredVersion "$lib")
cd "$GOPATH/src/github.com/tendermint/$lib" || exit
MASTER=$(git rev-parse origin/master)
if [[ "$VENDORED" != "$MASTER" ]]; then
echo "... VENDORED != MASTER ($VENDORED != $MASTER)"
echo "... Checking out commit $VENDORED"
git checkout "$VENDORED" &> /dev/null
fi

+ 0
- 4
test/README.md View File

@ -19,7 +19,3 @@ and run the following tests in docker containers:
- send a tx on each node and ensure the state root is updated on all of them
- crash and restart nodes one at a time and ensure they can sync back up (via fastsync)
- crash and restart all nodes at once and ensure they can sync back up
If on a `release-x.x.x` branch, we also run
- `go test` for all our dependency libs (test/test_libs.sh)

+ 0
- 34
test/test_libs.sh View File

@ -1,34 +0,0 @@
#! /bin/bash
set -ex
export PATH="$GOBIN:$PATH"
# Get the parent directory of where this script is.
SOURCE="${BASH_SOURCE[0]}"
while [ -h "$SOURCE" ] ; do SOURCE="$(readlink "$SOURCE")"; done
DIR="$( cd -P "$( dirname "$SOURCE" )/.." && pwd )"
####################
# libs we depend on
####################
# All libs should define `make test` and `make get_vendor_deps`
LIBS=(tmlibs go-wire go-crypto abci)
for lib in "${LIBS[@]}"; do
# checkout vendored version of lib
bash scripts/dep_utils/checkout.sh "$lib"
echo "Testing $lib ..."
cd "$GOPATH/src/github.com/tendermint/$lib"
make get_tools
make get_vendor_deps
make test
if [[ "$?" != 0 ]]; then
echo "FAIL"
exit 1
fi
cd "$DIR"
done
echo ""
echo "PASS"

Loading…
Cancel
Save