add github action for testing

add test around failure that led to v0.1.1

Author: Jeff Carter

.github/workflows/test.yml

@@ -0,0 +1,18 @@
+on: [push, pull_request]
+name: Test
+jobs:
+  test:
+    strategy:
+      matrix:
+        go-version: [1.14.x, 1.15.x]
+        os: [ubuntu-latest, macos-latest, windows-latest]
+    runs-on: ${{ matrix.os }}
+    steps:
+      - name: Install Go
+        uses: actions/setup-go@v2
+        with:
+          go-version: ${{ matrix.go-version }}
+      - name: Checkout code
+        uses: actions/checkout@v2
+      - name: Test
+        run: go test ./...

main.go

@@ -95,26 +95,26 @@ func Listers(include []string, exclude []string) []ListerName {
 	return ret
 }
 
-func GetByName(name ListerName) lister.Lister {
+func GetByName(name ListerName) (lister.Lister, error) {
 
 	for _, v := range lister.AllListers() {
 		if name == ListerName(reflect.TypeOf(v).Name()) {
-			return v
+			return v, nil
 		}
 	}
-	panic(fmt.Errorf("no lister found for %s", name))
+	return nil, fmt.Errorf("no lister found for %s", name)
 }
 
-func GetByType(kind resource.ResourceType) lister.Lister {
+func GetByType(kind resource.ResourceType) (lister.Lister, error) {
 
 	for _, v := range lister.AllListers() {
 		for _, t := range v.Types() {
 			if t == kind {
-				return v
+				return v, nil
 			}
 		}
 	}
-	panic(fmt.Errorf("no lister found for %s", kind))
+	return nil, fmt.Errorf("no lister found for %s", kind)
 }
 
 func Regions(prefixes ...string) ([]string, error) {
@@ -197,9 +197,13 @@ func List(ctx context2.AWSetsCtx, regions []string, listers []ListerName, cache
 						rg.Merge(group)
 					} else {
 						ctx.Logger.Infof("%d: not cached: %s - %s\n", id, job.region, job.lister)
-						idx := GetByName(job.lister)
+						lister, err := GetByName(job.lister)
+						if err != nil {
+							ctx.Logger.Errorf("failed to get lister by name: %v", err)
+							continue
+						}
 						ctxcp := ctx.Copy(job.region)
-						group, err := idx.List(ctxcp)
+						group, err := lister.List(ctxcp)
 						if err != nil {
 							// indicates service is not supported in a region, likely a better way to do this though
 							// eks returns "AccessDenied" if the service isn't in the region though

main_test.go

@@ -0,0 +1,40 @@
+package awspelunk
+
+import (
+	"strings"
+	"testing"
+)
+
+func Test_Types(t *testing.T) {
+	types := Types([]string{""}, []string{""})
+	if len(types) == 0 {
+		t.Fatalf("expected all types")
+	}
+	types = Types(nil, nil)
+	if len(types) == 0 {
+		t.Fatalf("expected all types")
+	}
+	types = Types([]string{""}, []string{"ec2"})
+	for _, rt := range types {
+		if strings.HasPrefix(rt.String(), "ec2") {
+			t.Fatalf("expected ec2* resource types to have been filtered out")
+		}
+	}
+	types = Types([]string{"ec2"}, []string{""})
+	for _, rt := range types {
+		if !strings.HasPrefix(rt.String(), "ec2") {
+			t.Fatalf("on expected ec2* resource types to be present")
+		}
+	}
+}
+
+func Test_Listers(t *testing.T) {
+	listers := Listers([]string{""}, []string{""})
+	if len(listers) == 0 {
+		t.Fatalf("expected all listers")
+	}
+	listers = Listers(nil, nil)
+	if len(listers) == 0 {
+		t.Fatalf("expected all listers")
+	}
+}