Introduce analysistest testdata convention.

CONTRIBUTING.md

@@ -26,3 +26,19 @@ information on using pull requests.
 
 This project follows [Google's Open Source Community
 Guidelines](https://opensource.google/conduct/).
+
+## Project Conventions
+
+### Use Go Modules in `testdata`
+
+The analyzer testing package `analysistest` executes a build of the Go code it tests.
+This treats the `testdata` folder as a synthetic GOPATH, and Go code tested within will look for imported packages in `testdata/src/...`.
+Unfortunately, this tends to break IDE linking, making test development difficult.
+
+To ameliorate this issue, we group a given test's `testdata` into a synthetic root package and use Go modules to identify it for the IDE.
+This allows your IDE to correctly link imports while adhering to the expectation of `analysistest`.
+
+Specifically, `testdata` should be grouped under `testdata/src/NAME_analysistest`, where `NAME` is the name of the analyzer being tested.
+Initialize a trivial Go module file with `go mod init NAME_analysistest`.
+Any imports used within this `testdata` will begin with `NAME_analysistest`.
+See [`levee_analysistest`](internal/pkg/levee/testdata/src/levee_analysistest) for an example. 

internal/pkg/config/testdata/src/example.com/crosspkg/crosspkg.go → internal/pkg/config/testdata/src/config_analysistest/example/crosspkg/crosspkg.go

@@ -15,9 +15,9 @@
 package crosspkg
 
 import (
-	"example.com/core"
-	"example.com/notcore"
-	necore "notexample.com/core"
+	"config_analysistest/example/core"
+	"config_analysistest/example/notcore"
+	necore "config_analysistest/notexample/core"
 )
 
 func CoreCalls() {

internal/pkg/config/testdata/src/config_analysistest/go.mod

@@ -0,0 +1,3 @@
+module config_analysistest
+
+go 1.15

internal/pkg/config/testdata/test-config.yaml

@@ -13,15 +13,15 @@
 # limitations under the License.
 ---
 sinks:
-  - packageRE: "^example.com/core"
-    methodRE: "^Sink$"
-  - packageRE: "^example.com/core"
-    methodRE: "^Do$"
-    receiverRE: "^Sinker$"
+  - package: "config_analysistest/example/core"
+    method: "Sink"
+  - package: "config_analysistest/example/core"
+    method: "Do"
+    receiver: "Sinker"
 fieldTags:
   - key: example
     value: sensitive
 exclude:
-  - packageRE: "^example.com/exclusion$"
-    methodRE: "^Foo$"
-  - packageRE: "^notexample.com/exclusion$"
+  - package: "config_analysistest/example/exclusion"
+    method: "Foo"
+  - package: "config_analysistest/notexample/exclusion"

internal/pkg/fieldpropagator/testdata/src/fieldpropagator_analysistest/go.mod

@@ -0,0 +1,3 @@
+module fieldpropagator_analysistest
+
+go 1.15

internal/pkg/fieldpropagator/testdata/test-config.yaml

@@ -16,12 +16,12 @@ Sources:
   - PackageRE: source
     TypeRE: "^Source$"
     FieldRE: "^data"
-  - Package: proto
+  - PackageRE: proto
     Type: KeyPair
     Field: Private
-  - Package: proto
+  - PackageRE: proto
     Type: TLS
     Field: Cert
-  - Package: proto
+  - PackageRE: proto
     Type: BasicAuth
     Field: Password

internal/pkg/fieldtags/analyzer_test.go

@@ -31,7 +31,7 @@ func TestFieldTagsAnalysis(t *testing.T) {
 		t.Error(err)
 	}
 
-	results := analysistest.Run(t, testdata, Analyzer, "example.com/core", "example.com/crosspkg")
+	results := analysistest.Run(t, testdata, Analyzer, "fieldtags_analysistest/core", "fieldtags_analysistest/crosspkg")
 
 	if len(results) != 2 {
 		t.Fatalf("expected 2 results, got %d", len(results))

internal/pkg/fieldtags/testdata/src/example.com/crosspkg/crosspkg.go → internal/pkg/fieldtags/testdata/src/fieldtags_analysistest/crosspkg/crosspkg.go

@@ -14,7 +14,7 @@
 
 package crosspkg
 
-import "example.com/core"
+import "fieldtags_analysistest/core"
 
 type PersonWrapper struct {
 	p          core.Person

internal/pkg/fieldtags/testdata/src/fieldtags_analysistest/go.mod

@@ -0,0 +1,3 @@
+module fieldtags_analysistest
+
+go 1.15

internal/pkg/levee/levee_test.go

@@ -32,7 +32,7 @@ func TestLevee(t *testing.T) {
 	if *debugging {
 		Analyzer.Requires = append(Analyzer.Requires, debug.Analyzer)
 	}
-	analysistest.Run(t, dataDir, Analyzer, "./src/example.com/...")
+	analysistest.Run(t, dataDir, Analyzer, "./src/levee_analysistest/example/...")
 }
 
 func TestLeveeDoesNotCreateReportsForPanicIfPanicingOnTaintedValuesIsAllowed(t *testing.T) {
@@ -43,21 +43,21 @@ func TestLeveeDoesNotCreateReportsForPanicIfPanicingOnTaintedValuesIsAllowed(t *
 	if *debugging {
 		Analyzer.Requires = append(Analyzer.Requires, debug.Analyzer)
 	}
-	analysistest.Run(t, dataDir, Analyzer, "./src/nopanic.com/...")
+	analysistest.Run(t, dataDir, Analyzer, "./src/levee_analysistest/nopanic.com/...")
 }
 
 func TestFormattingWithCustomReportMessage(t *testing.T) {
 	dataDir := analysistest.TestData()
 	if err := Analyzer.Flags.Set("config", dataDir+"/with-custom-message.yaml"); err != nil {
 		t.Error(err)
 	}
-	analysistest.Run(t, dataDir, Analyzer, "./src/custom.message.com/withcustom")
+	analysistest.Run(t, dataDir, Analyzer, "./src/levee_analysistest/custom.message.com/withcustom")
 }
 
 func TestFormattingWithoutCustomReportMessage(t *testing.T) {
 	dataDir := analysistest.TestData()
 	if err := Analyzer.Flags.Set("config", dataDir+"/no-custom-message.yaml"); err != nil {
 		t.Error(err)
 	}
-	analysistest.Run(t, dataDir, Analyzer, "./src/custom.message.com/nocustom")
+	analysistest.Run(t, dataDir, Analyzer, "./src/levee_analysistest/custom.message.com/nocustom")
 }

internal/pkg/levee/testdata/allowpanicontaintedvalues-config.yaml

@@ -14,9 +14,9 @@
 ---
 AllowPanicOnTaintedValues: true
 Sources:
-  - Package: "nopanic.com/core"
+  - Package: "levee_analysistest/nopanic.com/core"
     Type: "Source"
     Field: "Data"
 Sinks:
-  - Package: "nopanic.com/core"
+  - Package: "levee_analysistest/nopanic.com/core"
     Method: "Sink"

internal/pkg/levee/testdata/src/example.com/tests/arguments/tests.go → internal/pkg/levee/testdata/src/levee_analysistest/example/tests/arguments/tests.go

@@ -15,7 +15,7 @@
 package arguments
 
 import (
-	"example.com/core"
+	"levee_analysistest/example/core"
 )
 
 func TestSourceFromParamByReference(s *core.Source) {

internal/pkg/levee/testdata/src/example.com/tests/binop/tests.go → internal/pkg/levee/testdata/src/levee_analysistest/example/tests/binop/tests.go

@@ -17,7 +17,7 @@ package binop
 import (
 	"fmt"
 
-	"example.com/core"
+	"levee_analysistest/example/core"
 )
 
 func TestConcatenatingTaintedAndNonTaintedStrings(prefix string) {

internal/pkg/levee/testdata/src/example.com/tests/booleans/tests.go → internal/pkg/levee/testdata/src/levee_analysistest/example/tests/booleans/tests.go

@@ -15,7 +15,7 @@
 package inlining
 
 import (
-	"example.com/core"
+	"levee_analysistest/example/core"
 )
 
 func HasSecret(s core.Source) bool {

internal/pkg/levee/testdata/src/example.com/tests/callorder/beforesource.go → internal/pkg/levee/testdata/src/levee_analysistest/example/tests/callorder/beforesource.go

@@ -15,7 +15,7 @@
 package callorder
 
 import (
-	"example.com/core"
+	"levee_analysistest/example/core"
 )
 
 // This type should *not* be identified as a Source.

internal/pkg/levee/testdata/src/example.com/tests/callorder/colocation.go → internal/pkg/levee/testdata/src/levee_analysistest/example/tests/callorder/colocation.go

@@ -15,7 +15,7 @@
 package callorder
 
 import (
-	"example.com/core"
+	"levee_analysistest/example/core"
 )
 
 func TestTaintedColocatedArgumentDoesNotReachSinkThatPrecedesColocation() {

internal/pkg/levee/testdata/src/example.com/tests/callorder/multiblock.go → internal/pkg/levee/testdata/src/levee_analysistest/example/tests/callorder/multiblock.go

@@ -18,7 +18,7 @@ import (
 	"fmt"
 	"io"
 
-	"example.com/core"
+	"levee_analysistest/example/core"
 )
 
 func TestSinkInIfBeforeTaint(s core.Source, w io.Writer) {

internal/pkg/levee/testdata/src/example.com/tests/callorder/singleblock.go → internal/pkg/levee/testdata/src/levee_analysistest/example/tests/callorder/singleblock.go

@@ -18,7 +18,7 @@ import (
 	"fmt"
 	"io"
 
-	"example.com/core"
+	"levee_analysistest/example/core"
 )
 
 func TestTaintBeforeSinking(s core.Source, w io.Writer) {

internal/pkg/levee/testdata/src/example.com/tests/closures/tests.go → internal/pkg/levee/testdata/src/levee_analysistest/example/tests/closures/tests.go

@@ -15,7 +15,7 @@
 package closures
 
 import (
-	"example.com/core"
+	"levee_analysistest/example/core"
 )
 
 func TesCapturedSourceReachesSinkInClosure() func() {

internal/pkg/levee/testdata/src/example.com/tests/collections/arrays.go → internal/pkg/levee/testdata/src/levee_analysistest/example/tests/collections/arrays.go

@@ -15,7 +15,7 @@
 package collections
 
 import (
-	"example.com/core"
+	"levee_analysistest/example/core"
 )
 
 func TestArrayLiteralContainingSourceIsTainted(s core.Source) {

internal/pkg/levee/testdata/src/example.com/tests/collections/chans.go → internal/pkg/levee/testdata/src/levee_analysistest/example/tests/collections/chans.go

@@ -15,7 +15,7 @@
 package collections
 
 import (
-	"example.com/core"
+	"levee_analysistest/example/core"
 )
 
 func TestSourceReceivedFromChannelIsTainted(sources <-chan core.Source) {

internal/pkg/levee/testdata/src/example.com/tests/collections/maps.go → internal/pkg/levee/testdata/src/levee_analysistest/example/tests/collections/maps.go

@@ -15,7 +15,7 @@
 package collections
 
 import (
-	"example.com/core"
+	"levee_analysistest/example/core"
 )
 
 func TestMapLiteralContainingSourceKeyIsTainted(s core.Source) {

internal/pkg/levee/testdata/src/example.com/tests/collections/slices.go → internal/pkg/levee/testdata/src/levee_analysistest/example/tests/collections/slices.go

@@ -15,7 +15,7 @@
 package collections
 
 import (
-	"example.com/core"
+	"levee_analysistest/example/core"
 )
 
 func TestSlices(s core.Source) {

internal/pkg/levee/testdata/src/example.com/tests/collections/sources.go → internal/pkg/levee/testdata/src/levee_analysistest/example/tests/collections/sources.go

@@ -15,7 +15,7 @@
 package collections
 
 import (
-	"example.com/core"
+	"levee_analysistest/example/core"
 )
 
 func TestSourceMapIsSource() {

internal/pkg/levee/testdata/src/example.com/tests/colocation/tests.go → internal/pkg/levee/testdata/src/levee_analysistest/example/tests/colocation/tests.go

@@ -18,7 +18,7 @@ import (
 	"encoding/json"
 	"reflect"
 
-	"example.com/core"
+	"levee_analysistest/example/core"
 )
 
 func colocateString(core.Source, string) {}

internal/pkg/levee/testdata/src/example.com/tests/declarations/tests.go → internal/pkg/levee/testdata/src/levee_analysistest/example/tests/declarations/tests.go

@@ -16,7 +16,7 @@
 package declarations
 
 import (
-	"example.com/core"
+	"levee_analysistest/example/core"
 )
 
 func TestSourceDeclaredInBody() {

internal/pkg/levee/testdata/src/example.com/tests/eface/tests.go → internal/pkg/levee/testdata/src/levee_analysistest/example/tests/eface/tests.go

@@ -15,7 +15,7 @@
 package eface
 
 import (
-	"example.com/core"
+	"levee_analysistest/example/core"
 )
 
 func TestEfaceSource(s core.Source) {

internal/pkg/levee/testdata/src/example.com/tests/embedding/tests.go → internal/pkg/levee/testdata/src/levee_analysistest/example/tests/embedding/tests.go

@@ -15,7 +15,7 @@
 package embedding
 
 import (
-	"example.com/core"
+	"levee_analysistest/example/core"
 )
 
 type EmbedsSource struct {

internal/pkg/levee/testdata/src/example.com/tests/excludedpackage/tests.go → internal/pkg/levee/testdata/src/levee_analysistest/example/tests/excludedpackage/tests.go

@@ -15,7 +15,7 @@
 package excludedpackage
 
 import (
-	"example.com/core"
+	"levee_analysistest/example/core"
 )
 
 func Oops(s core.Source) {

internal/pkg/levee/testdata/src/example.com/tests/extracts/tests.go → internal/pkg/levee/testdata/src/levee_analysistest/example/tests/extracts/tests.go

@@ -15,7 +15,7 @@
 package extracts
 
 import (
-	"example.com/core"
+	"levee_analysistest/example/core"
 )
 
 func CreateSource() (core.Source, error) {

internal/pkg/levee/testdata/src/example.com/tests/fields/tests.go → internal/pkg/levee/testdata/src/levee_analysistest/example/tests/fields/tests.go

@@ -18,7 +18,7 @@ package fields
 import (
 	"strconv"
 
-	"example.com/core"
+	"levee_analysistest/example/core"
 )
 
 func TestFieldAccessors(s core.Source, ptr *core.Source) {

internal/pkg/levee/testdata/src/example.com/tests/includedpackage/tests.go → internal/pkg/levee/testdata/src/levee_analysistest/example/tests/includedpackage/tests.go

@@ -15,7 +15,7 @@
 package includedpackage
 
 import (
-	"example.com/core"
+	"levee_analysistest/example/core"
 )
 
 func Oops(s core.Source) {

internal/pkg/levee/testdata/src/example.com/tests/inlining/tests.go → internal/pkg/levee/testdata/src/levee_analysistest/example/tests/inlining/tests.go

@@ -15,7 +15,7 @@
 package inlining
 
 import (
-	"example.com/core"
+	"levee_analysistest/example/core"
 )
 
 func NewSource() *core.Source {

internal/pkg/levee/testdata/src/example.com/tests/loops/tests.go → internal/pkg/levee/testdata/src/levee_analysistest/example/tests/loops/tests.go

@@ -15,7 +15,7 @@
 package loops
 
 import (
-	"example.com/core"
+	"levee_analysistest/example/core"
 )
 
 func TestTaintInThenBlockInLoopSinkAfterLoop() {

internal/pkg/levee/testdata/src/example.com/tests/makeinterface/tests.go → internal/pkg/levee/testdata/src/levee_analysistest/example/tests/makeinterface/tests.go

@@ -15,7 +15,7 @@
 package deletethis
 
 import (
-	"example.com/core"
+	"levee_analysistest/example/core"
 )
 
 func TestTaintingInterfaceValueDoesNotTaintContainedValue(s *core.Source, str string) {

internal/pkg/levee/testdata/src/example.com/tests/method/tests.go → internal/pkg/levee/testdata/src/levee_analysistest/example/tests/method/tests.go

@@ -15,7 +15,7 @@
 package method
 
 import (
-	"example.com/core"
+	"levee_analysistest/example/core"
 )
 
 func TestMethodCallOnStaticallyKnownReceiverPropagatesTaint(s core.Source) {

internal/pkg/levee/testdata/src/example.com/tests/namedreturn/tests.go → internal/pkg/levee/testdata/src/levee_analysistest/example/tests/namedreturn/tests.go

@@ -15,7 +15,7 @@
 package namedreturn
 
 import (
-	"example.com/core"
+	"levee_analysistest/example/core"
 )
 
 func TestNamedReturnValue() (s core.Source) {

internal/pkg/levee/testdata/src/example.com/tests/panic/tests.go → internal/pkg/levee/testdata/src/levee_analysistest/example/tests/panic/tests.go

@@ -15,7 +15,7 @@
 package panic
 
 import (
-	"example.com/core"
+	"levee_analysistest/example/core"
 )
 
 func TestPanicIsASink(source core.Source) {