plugin: add sharness tests

License: MIT
Signed-off-by: Łukasz Magiera <[email protected]>

Makefile

@@ -7,4 +7,5 @@ PROTOC = protoc --gogo_out=. --proto_path=.:/usr/local/opt/protobuf/include:$(di
 # enable second expansion
 .SECONDEXPANSION:
 
+include config.mk
 include Rules.mk

cmd/ipfs/Rules.mk

@@ -12,10 +12,10 @@ PATH := $(realpath $(d)):$(PATH)
 # DEPS_OO_$(d) += merkledag/pb/merkledag.pb.go namesys/pb/namesys.pb.go
 # DEPS_OO_$(d) += pin/internal/pb/header.pb.go unixfs/pb/unixfs.pb.go
 
-CONFIG_COMMIT ?= github.com/ipfs/go-ipfs/repo/config.CurrentCommit=$(shell git rev-parse --short HEAD)
-CONFIG_PLUGIN_PATH ?= github.com/ipfs/go-ipfs/repo/config.SystemPluginPath=${PLUGIN_PATH}
-
-$(d)_flags =-ldflags="-X ${CONFIG_COMMIT} -X ${CONFIG_PLUGIN_PATH}"
+$(d)_flags  =-ldflags="
+$(d)_flags += -X github.com/ipfs/go-ipfs/repo/config.CurrentCommit=${CONFIG_COMMIT}
+$(d)_flags += -X github.com/ipfs/go-ipfs/repo/config.SystemPluginPath=${CONFIG_PLUGIN_PATH}
+$(d)_flags += "
 
 $(IPFS_BIN_$(d)): GOFLAGS += $(cmd/ipfs_flags)
 

config.mk

@@ -0,0 +1,2 @@
+CONFIG_COMMIT := $(shell git rev-parse --short HEAD)
+CONFIG_PLUGIN_PATH = ${PLUGIN_PATH}

test/sharness/Rules.mk

@@ -7,9 +7,9 @@ T_$(d) = $(sort $(wildcard $(d)/t[0-9][0-9][0-9][0-9]-*.sh))
 DEPS_$(d) := test/bin/random test/bin/multihash test/bin/pollEndpoint \
 	   test/bin/iptb test/bin/go-sleep test/bin/random-files \
 	   test/bin/go-timeout test/bin/hang-fds test/bin/ma-pipe-unidir
-DEPS_$(d) += cmd/ipfs/ipfs
 DEPS_$(d) += $(d)/clean-test-results
 DEPS_$(d) += $(SHARNESS_$(d))
+DEPS_$(d) += $(d)/sharness-ipfs
 
 ifeq ($(OS),Linux)
 PLUGINS_DIR_$(d) := $(d)/plugins/
@@ -49,13 +49,16 @@ $(SHARNESS_$(d)): $(d) ALWAYS
 $(d)/deps: $(SHARNESS_$(d)) $$(DEPS_$(d)) # use second expansion so coverage can inject dependency
 .PHONY: $(d)/deps
 
+$(d)/sharness-ipfs: PLUGIN_PATH=./system_plugins
+$(d)/sharness-ipfs: cmd/ipfs/ipfs
+.PHONY: $(d)/sharness-ipfs
+
 test_sharness_deps: $(d)/deps
 .PHONY: test_sharness_deps
 
 test_sharness_short: $(d)/aggregate
 .PHONY: test_sharness_short
 
-
 test_sharness_expensive: export TEST_EXPENSIVE=1
 test_sharness_expensive: test_sharness_short
 .PHONY: test_sharness_expensive

test/sharness/t0280-plugin-git.sh

@@ -17,11 +17,6 @@ fi
 
 test_init_ipfs
 
-test_expect_success "copy plugin" '
-  mkdir -p "$IPFS_PATH/plugins" &&
-  cp ../plugins/git.so "$IPFS_PATH/plugins/"
-'
-
 # from https://github.com/ipfs/go-ipld-git/blob/master/make-test-repo.sh
 test_expect_success "prepare test data" '
   tar xzf ../t0280-plugin-git-data/git.tar.gz
@@ -46,6 +41,33 @@ test_dag_git() {
   '
 }
 
+test_expect_success "copy plugin to local plugin directory" '
+  mkdir -p "$IPFS_PATH/plugins" &&
+  chmod +x ../plugins/git.so &&
+  cp ../plugins/git.so "$IPFS_PATH/plugins/"
+'
+
+# should work offline
+#test_dag_git
+
+# should work online
+test_launch_ipfs_daemon
+test_dag_git
+test_kill_ipfs_daemon
+
+test_expect_success "copy plugin to global plugin directory" '
+  mkdir -p "./system_plugins" &&
+  cp ../plugins/git.so "./system_plugins/"
+'
+
+test_expect_success "fail with duplicate plugin" '
+  echo 123 | test_must_fail ipfs dag put
+'
+
+test_expect_success "remove local plugin" '
+  rm -r "$IPFS_PATH/plugins/"
+'
+
 # should work offline
 #test_dag_git