wire: use return in tests instead of panic (google/go-cloud#169)

Fixes google/go-cloud#105

internal/wire/testdata/Chain/foo/wire.go

@@ -21,5 +21,6 @@ import (
 )
 
 func injectFooBar() FooBar {
-	panic(wire.Build(Set))
+	wire.Build(Set)
+	return 0
 }

internal/wire/testdata/Cleanup/foo/wire.go

@@ -21,5 +21,6 @@ import (
 )
 
 func injectBar() (*Bar, func()) {
-	panic(wire.Build(provideFoo, provideBar))
+	wire.Build(provideFoo, provideBar)
+	return nil, nil
 }

internal/wire/testdata/CopyOtherDecls/foo/foo.go

@@ -35,5 +35,6 @@ func provideMessage() string {
 }
 
 func injectedMessage() string {
-	panic(wire.Build(provideMessage))
+	wire.Build(provideMessage)
+	return ""
 }

internal/wire/testdata/Cycle/foo/wire.go

@@ -21,5 +21,6 @@ import (
 )
 
 func injectedBaz() Baz {
-	panic(wire.Build(provideFoo, provideBar, provideBaz))
+	wire.Build(provideFoo, provideBar, provideBaz)
+	return 0
 }

internal/wire/testdata/EmptyVar/foo/wire.go

@@ -21,5 +21,6 @@ import (
 )
 
 func injectedMessage() string {
-	panic(wire.Build(myFakeSet))
+	wire.Build(myFakeSet)
+	return ""
 }

internal/wire/testdata/ExportedValue/foo/wire.go

@@ -22,5 +22,6 @@ import (
 )
 
 func injectedMessage() string {
-	panic(wire.Build(bar.Value))
+	wire.Build(bar.Value)
+	return ""
 }

internal/wire/testdata/ExportedValueDifferentPackage/foo/wire.go

@@ -24,5 +24,6 @@ import (
 )
 
 func injectedFile() *os.File {
-	panic(wire.Build(bar.Value))
+	wire.Build(bar.Value)
+	return nil
 }

internal/wire/testdata/ImportedInterfaceBinding/bar/wire.go

@@ -17,10 +17,11 @@
 package main
 
 import (
-	"github.com/google/go-cloud/wire"
 	"foo"
+	"github.com/google/go-cloud/wire"
 )
 
 func injectFooer() foo.Fooer {
-	panic(wire.Build(Set))
+	wire.Build(Set)
+	return nil
 }

internal/wire/testdata/InjectInput/foo/wire.go

@@ -21,5 +21,6 @@ import (
 )
 
 func injectFooBar(foo Foo) FooBar {
-	panic(wire.Build(Set))
+	wire.Build(Set)
+	return 0
 }

internal/wire/testdata/InjectInputConflict/foo/wire.go

@@ -21,5 +21,6 @@ import (
 )
 
 func injectBar(foo Foo) Bar {
-	panic(wire.Build(Set))
+	wire.Build(Set)
+	return 0
 }

internal/wire/testdata/InjectWithPanic/foo/wire.go

@@ -21,6 +21,5 @@ import (
 )
 
 func injectedMessage() string {
-	wire.Build(provideMessage)
-	return ""
+	panic(wire.Build(provideMessage))
 }

internal/wire/testdata/InterfaceBinding/foo/wire.go

@@ -21,5 +21,6 @@ import (
 )
 
 func injectFooer() Fooer {
-	panic(wire.Build(Set))
+	wire.Build(Set)
+	return nil
 }

internal/wire/testdata/InterfaceBindingReuse/foo/wire.go

@@ -21,8 +21,10 @@ import (
 )
 
 func injectFooBar() FooBar {
-	panic(wire.Build(
+	wire.Build(
 		provideBar,
 		provideFooBar,
-		wire.Bind((*Fooer)(nil), (*Bar)(nil))))
+		wire.Bind((*Fooer)(nil), (*Bar)(nil)),
+	)
+	return FooBar{}
 }

internal/wire/testdata/NiladicIdentity/foo/wire.go

@@ -21,5 +21,6 @@ import (
 )
 
 func injectedMessage() string {
-	panic(wire.Build(provideMessage))
+	wire.Build(provideMessage)
+	return ""
 }

internal/wire/testdata/NiladicValue/foo/wire.go

@@ -21,5 +21,6 @@ import (
 )
 
 func injectedMessage() string {
-	panic(wire.Build(wire.Value("Hello, World!")))
+	wire.Build(wire.Value("Hello, World!"))
+	return ""
 }

internal/wire/testdata/NoImplicitInterface/foo/wire.go

@@ -21,5 +21,6 @@ import (
 )
 
 func injectFooer() Fooer {
-	panic(wire.Build(provideBar))
+	wire.Build(provideBar)
+	return nil
 }

internal/wire/testdata/NoInjectParamNames/foo/wire.go

@@ -26,5 +26,6 @@ import (
 // parameter names on the inject stub.
 
 func inject(stdcontext.Context, struct{}) (context, error) {
-	panic(wire.Build(provide))
+	wire.Build(provide)
+	return context{}, nil
 }

internal/wire/testdata/PartialCleanup/foo/wire.go

@@ -21,5 +21,6 @@ import (
 )
 
 func injectBaz() (Baz, func(), error) {
-	panic(wire.Build(provideFoo, provideBar, provideBaz))
+	wire.Build(provideFoo, provideBar, provideBaz)
+	return 0, nil, nil
 }

internal/wire/testdata/PkgImport/foo/wire.go

@@ -21,5 +21,6 @@ import (
 )
 
 func injectFooBar() FooBar {
-	panic(wire.Build(Set))
+	wire.Build(Set)
+	return 0
 }

internal/wire/testdata/ReturnError/foo/wire.go

@@ -21,5 +21,6 @@ import (
 )
 
 func injectFoo() (Foo, error) {
-	panic(wire.Build(Set))
+	wire.Build(Set)
+	return 0, nil
 }

internal/wire/testdata/Struct/foo/wire.go

@@ -21,5 +21,6 @@ import (
 )
 
 func injectFooBar() FooBar {
-	panic(wire.Build(Set))
+	wire.Build(Set)
+	return FooBar{}
 }

internal/wire/testdata/StructPointer/foo/wire.go

@@ -21,5 +21,6 @@ import (
 )
 
 func injectFooBar() *FooBar {
-	panic(wire.Build(Set))
+	wire.Build(Set)
+	return nil
 }

internal/wire/testdata/TwoDeps/foo/wire.go

@@ -21,5 +21,6 @@ import (
 )
 
 func injectFooBar() FooBar {
-	panic(wire.Build(Set))
+	wire.Build(Set)
+	return 0
 }

internal/wire/testdata/UnexportedValue/foo/wire.go

@@ -22,5 +22,6 @@ import (
 )
 
 func injectedMessage() string {
-	panic(wire.Build(bar.Value))
+	wire.Build(bar.Value)
+	return ""
 }

internal/wire/testdata/ValueChain/foo/wire.go

@@ -21,5 +21,6 @@ import (
 )
 
 func injectFooBar() FooBar {
-	panic(wire.Build(Set))
+	wire.Build(Set)
+	return 0
 }

internal/wire/testdata/ValueConversion/foo/wire.go

@@ -21,5 +21,6 @@ import (
 )
 
 func injectedMessage() Foo {
-	panic(wire.Build(wire.Value(Foo("Hello, World!"))))
+	wire.Build(wire.Value(Foo("Hello, World!")))
+	return ""
 }

internal/wire/testdata/VarValue/foo/wire.go

@@ -21,5 +21,6 @@ import (
 )
 
 func injectedMessage() string {
-	panic(wire.Build(wire.Value(msg)))
+	wire.Build(wire.Value(msg))
+	return ""
 }

internal/wire/testdata/Vendor/foo/wire.go

@@ -22,5 +22,6 @@ import (
 )
 
 func injectedMessage() string {
-	panic(wire.Build(bar.ProvideMessage))
+	wire.Build(bar.ProvideMessage)
+	return ""
 }