import-boss: add tests

.travis.yml

@@ -13,4 +13,4 @@ jobs:
         - go test -v ./...
     - stage: Verify examples
       script:
-        - go run ./examples/import-boss/main.go -i k8s.io/gengo/... --verify-only
+        - go run ./examples/import-boss/main.go -i $(go list k8s.io/gengo/... | grep -v import-boss/tests | paste -sd',' -) --verify-only

examples/import-boss/Makefile

@@ -0,0 +1,28 @@
+TOOL=import-boss
+
+all: test
+
+test: build test_rules test_inverse
+
+build:
+	@go build -o /tmp/$(TOOL)
+
+test_rules:
+	  /tmp/$(TOOL) --logtostderr --v=4 -i $$(go list ./tests/rules/a)
+	! /tmp/$(TOOL) --logtostderr --v=4 -i $$(go list ./tests/rules/b)
+	  /tmp/$(TOOL) --logtostderr --v=4 -i ./tests/rules/c
+  	! /tmp/$(TOOL) --logtostderr --v=4 -i ./tests/rules/nested
+	  /tmp/$(TOOL) --logtostderr --v=4 -i ./tests/rules/nested/nested
+	! /tmp/$(TOOL) --logtostderr --v=4 -i ./tests/rules/nested/nested/nested
+	! /tmp/$(TOOL) --logtostderr --v=4 -i ./tests/rules/nested/nested/nested/inherit
+
+test_inverse:
+	  /tmp/$(TOOL) --logtostderr --v=4 -i $$(go list ./tests/inverse/a ./tests/inverse/lib/... | grep -v quarantine | paste -sd',' -)
+	! /tmp/$(TOOL) --logtostderr --v=4 -i $$(go list ./tests/inverse/b ./tests/inverse/lib/... | grep -v quarantine | paste -sd',' -)
+	  /tmp/$(TOOL) --logtostderr --v=4 -i $$(go list ./tests/inverse/c ./tests/inverse/lib/... | grep -v quarantine | paste -sd',' -)
+	! /tmp/$(TOOL) --logtostderr --v=4 -i $$(go list ./tests/inverse/d ./tests/inverse/lib/... | grep -v quarantine | paste -sd',' -)
+	! /tmp/$(TOOL) --logtostderr --v=4 -i $$(go list ./tests/inverse/lib/... | paste -sd',' -)
+
+	! /tmp/$(TOOL) --logtostderr --v=4 -i $$(go list ./tests/inverse/lib/... | paste -sd',' -)
+
+.PHONY: build test test_rules test_inverse

examples/import-boss/tests/inverse/a/doc.go

@@ -0,0 +1,6 @@
+// a only imports public packages.
+package a
+
+import (
+	_ "k8s.io/gengo/examples/import-boss/tests/inverse/lib/public"
+)

examples/import-boss/tests/inverse/b/doc.go

@@ -0,0 +1,7 @@
+// b only imports public and private packages. The latter it shouldn't.
+package b
+
+import (
+	_ "k8s.io/gengo/examples/import-boss/tests/inverse/lib/private"
+	_ "k8s.io/gengo/examples/import-boss/tests/inverse/lib/public"
+)

examples/import-boss/tests/inverse/c/doc.go

@@ -0,0 +1,7 @@
+// c imports the library root, which in turn imports the public and private packages. This is fine because
+// the private package is not directly imported.
+package c
+
+import (
+	_ "k8s.io/gengo/examples/import-boss/tests/inverse/lib"
+)

examples/import-boss/tests/inverse/d/doc.go

@@ -0,0 +1,6 @@
+// c imports non-prod code. It shouldn't.
+package d
+
+import (
+	_ "k8s.io/gengo/examples/import-boss/tests/inverse/lib/nonprod"
+)

examples/import-boss/tests/inverse/lib/doc.go

@@ -0,0 +1,6 @@
+package lib
+
+import (
+	_ "k8s.io/gengo/examples/import-boss/tests/inverse/lib/private"
+	_ "k8s.io/gengo/examples/import-boss/tests/inverse/lib/public"
+)

examples/import-boss/tests/inverse/lib/lib_test.go

@@ -0,0 +1,5 @@
+package lib
+
+import (
+	_ "k8s.io/gengo/examples/import-boss/tests/inverse/lib/nonprod"
+)

examples/import-boss/tests/inverse/lib/nonprod/.import-restrictions

@@ -0,0 +1,9 @@
+{
+   "InverseRules": [{
+       "Transitive": true,
+       "SelectorRegexp": "k8s[.]io/gengo",
+       "AllowedPrefixes": [
+         "k8s.io/gengo/examples/import-boss/tests/inverse/lib"
+       ]
+   }]
+}

examples/import-boss/tests/inverse/lib/nonprod/doc.go

@@ -0,0 +1,2 @@
+// nonprod is non-production code that should not be linked into any outside package.
+package nonprod

examples/import-boss/tests/inverse/lib/private/.import-restrictions

@@ -0,0 +1,11 @@
+{
+   "InverseRules": [{
+       "SelectorRegexp": "k8s[.]io/gengo",
+       "AllowedPrefixes": [
+         "k8s.io/gengo/examples/import-boss/tests/inverse/lib"
+       ],
+       "ForbiddenPrefixes": [
+         "k8s.io/gengo/examples/import-boss/tests/inverse/lib/quarantine"
+       ]
+   }]
+}

examples/import-boss/tests/inverse/lib/private/doc.go

@@ -0,0 +1 @@
+package private

examples/import-boss/tests/inverse/lib/public/doc.go

@@ -0,0 +1 @@
+package public

examples/import-boss/tests/inverse/lib/quarantine/doc.go

@@ -0,0 +1,6 @@
+// quarantine is inside the library, but should not import the private package. But it does!
+package quarantine
+
+import (
+	_ "k8s.io/gengo/examples/import-boss/tests/inverse/lib/private"
+)

examples/import-boss/tests/rules/a/.import-restrictions

@@ -0,0 +1,11 @@
+{
+   "Rules": [{
+       "SelectorRegexp": "k8s[.]io/gengo",
+       "AllowedPrefixes": [
+         "k8s.io/gengo/examples/import-boss/tests/rules/b"
+       ],
+       "ForbiddenPrefixes": [
+         "k8s.io/gengo/examples/import-boss/tests/rules/c"
+       ]
+   }]
+}

examples/import-boss/tests/rules/a/doc.go

@@ -0,0 +1,5 @@
+package a
+
+import (
+	_ "k8s.io/gengo/examples/import-boss/tests/rules/b"
+)

examples/import-boss/tests/rules/b/.import-restrictions

@@ -0,0 +1,11 @@
+{
+   "Rules": [{
+       "SelectorRegexp": "k8s[.]io/gengo",
+       "AllowedPrefixes": [
+         ""
+       ],
+       "ForbiddenPrefixes": [
+         "k8s.io/gengo/examples/import-boss/tests/rules/c"
+       ]
+   }]
+}

examples/import-boss/tests/rules/b/doc.go

@@ -0,0 +1,6 @@
+// b only public and private packages. The latter it shouldn't.
+package b
+
+import (
+	_ "k8s.io/gengo/examples/import-boss/tests/rules/c"
+)

examples/import-boss/tests/rules/c/doc.go

@@ -0,0 +1 @@
+package c

examples/import-boss/tests/rules/nested/.import-restrictions

@@ -0,0 +1,10 @@
+{
+   "Rules": [{
+       "SelectorRegexp": "k8s[.]io/gengo",
+       "AllowedPrefixes": [
+       ],
+       "ForbiddenPrefixes": [
+         "k8s.io/gengo/examples/import-boss/tests/rules/c"
+       ]
+   }]
+}

examples/import-boss/tests/rules/nested/doc.go

@@ -0,0 +1,5 @@
+package nested
+
+import (
+	_ "k8s.io/gengo/examples/import-boss/tests/rules/c"
+)

examples/import-boss/tests/rules/nested/nested/.import-restrictions

@@ -0,0 +1,10 @@
+{
+   "Rules": [{
+       "SelectorRegexp": "k8s[.]io/gengo",
+       "AllowedPrefixes": [
+           "k8s.io/gengo/examples/import-boss/tests/rules/c"
+       ],
+       "ForbiddenPrefixes": [
+       ]
+   }]
+}

examples/import-boss/tests/rules/nested/nested/doc.go

@@ -0,0 +1,5 @@
+package nested
+
+import (
+	_ "k8s.io/gengo/examples/import-boss/tests/rules/c"
+)

examples/import-boss/tests/rules/nested/nested/nested/.import-restrictions

@@ -0,0 +1,10 @@
+{
+   "Rules": [{
+       "SelectorRegexp": "k8s[.]io/gengo",
+       "AllowedPrefixes": [
+       ],
+       "ForbiddenPrefixes": [
+            "k8s.io/gengo/examples/import-boss/tests/rules/c"
+       ]
+   }]
+}

examples/import-boss/tests/rules/nested/nested/nested/doc.go

@@ -0,0 +1,5 @@
+package nested
+
+import (
+	_ "k8s.io/gengo/examples/import-boss/tests/rules/c"
+)

examples/import-boss/tests/rules/nested/nested/nested/inherit/doc.go

@@ -0,0 +1,5 @@
+package inherit
+
+import (
+	_ "k8s.io/gengo/examples/import-boss/tests/rules/c"
+)