Merge pull request #87 from dnephin/add-watcher

Support running unit tests when files are saved

Makefile

@@ -19,7 +19,6 @@ test:
 lint:
 	@gometalinter --config gometalinter.json ./...
 
-
 .PHONY: binary
 binary:
 	@./scripts/build/binary
@@ -33,6 +32,10 @@ cross:
 dynbinary:
 	@./scripts/build/dynbinary
 
+.PHONY: watch
+watch:
+	@./scripts/test/watch
+
 # download dependencies (vendor/) listed in vendor.conf
 .PHONY: vendor
 vendor: vendor.conf

README.md

@@ -28,7 +28,6 @@ Run all linting:
 ```
 $ make -f docker.Makefile lint
 ```
-`
 
 ### In-container development environment
 

docker.Makefile

@@ -46,6 +46,10 @@ test: build_docker_image
 cross: build_cross_image
 	@docker run --rm $(MOUNTS) $(CROSS_IMAGE_NAME) make cross
 
+.PHONY: watch
+watch: build_docker_image
+	@docker run --rm $(MOUNTS) $(DEV_DOCKER_IMAGE_NAME) make watch
+
 # start container in interactive mode for in-container development
 .PHONY: dev
 dev: build_docker_image

dockerfiles/Dockerfile.dev

@@ -15,6 +15,10 @@ RUN     go get github.com/jteeuwen/go-bindata/go-bindata && \
         cp /go/bin/go-bindata /usr/bin && \
         rm -rf /go/src/* /go/pkg/* /go/bin/*
 
+RUN     go get github.com/dnephin/filewatcher && \
+        cp /go/bin/filewatcher /usr/bin/ && \
+        rm -rf /go/src/* /go/pkg/* /go/bin/*
+
 ENV     CGO_ENABLED=0
 WORKDIR /go/src/github.com/docker/cli
 CMD     sh

scripts/test/watch

@@ -0,0 +1,12 @@
+#!/bin/sh
+
+set -e
+
+filewatcher \
+    -L 5 \
+    -x '**/*.swp' \
+    -x .git \
+    -x build \
+    -x .idea \
+    -- \
+    sh -c 'go test -timeout 10s -v ./${dir} || ( echo; echo; exit 1 )'