forked from go-zookeeper/zk
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
more travis tweaks adding link for pre-build process goveralls is out for now, also test fixes for older jar strategy bump them timeouts adding coveralls but ignore if it fails Adding back the coveralls env var
- Loading branch information
Showing
3 changed files
with
22 additions
and
8 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,6 @@ | ||
.vscode/ | ||
.DS_Store | ||
profile.cov | ||
zookeeper | ||
zookeeper-*/ | ||
zookeeper-*.tar.gz |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,18 +1,30 @@ | ||
# make file to hold the logic of build and test setup | ||
ZK_VERSION ?= 3.5.4-beta | ||
ZK_VERSION ?= 3.4.12 | ||
|
||
ZK = zookeeper-$(ZK_VERSION) | ||
ZK_URL = "https://archive.apache.org/dist/zookeeper/$(ZK)/$(ZK).tar.gz" | ||
|
||
$(ZK): | ||
wget $(ZK_URL) | ||
tar -zxf $(ZK).tar.gz | ||
rm $(ZK).tar.gz | ||
# we link to a standard directory path so then the tests dont need to find based on version | ||
ln -s $(ZK) zookeeper | ||
# in older versions we use the zk fatjar and need to remove the signature from the jar to run it. | ||
-zip -d zookeeper/contrib/fatjar/zookeeper-${zk_version}-fatjar.jar 'META-INF/*.SF' 'META-INF/*.DSA' | ||
|
||
|
||
.PHONY: install-test-deps | ||
install-test-deps: | ||
go get github.com/mattn/goveralls | ||
go get golang.org/x/tools/cmd/cover | ||
|
||
.PHONY: travis | ||
travis: $(ZK) install-test-deps | ||
travis: $(ZK) install-test-deps | ||
|
||
.PHONY: test | ||
test: | ||
go test -timeout 120s -v ./... | ||
go test -timeout 300s -v -i -race ./... | ||
go test -timeout 300s -v -race -covermode atomic -coverprofile=profile.cov ./zk | ||
# ignore if we fail to publish coverage | ||
-goveralls -coverprofile=profile.cov -service=travis-ci |