Merge pull request #113 from Revolyssup/master

Adding few extra checks in component generation

utils/manifests/error.go

@@ -8,6 +8,7 @@ const (
 	ErrGetAPIVersionCode  = "1003"
 	ErrGetAPIGroupCode    = "1004"
 	ErrPopulatingYamlCode = "1005"
+	ErrAbsentFilterCode   = "1006"
 )
 
 func ErrGetCrdNames(err error) error {
@@ -27,3 +28,6 @@ func ErrGetAPIGroup(err error) error {
 func ErrPopulatingYaml(err error) error {
 	return errors.New(ErrPopulatingYamlCode, errors.Alert, []string{"Error populating yaml"}, []string{err.Error()}, []string{"Yaml could not be populated with the returned manifests"}, []string{""})
 }
+func ErrAbsentFilter(err error) error {
+	return errors.New(ErrAbsentFilterCode, errors.Alert, []string{"Error with passed filters"}, []string{err.Error()}, []string{"ItrFilter or ItrSpecFilter is either not passed or empty"}, []string{"Pass the correct ItrFilter and ItrSpecFilter"})
+}

utils/manifests/utils.go

@@ -3,6 +3,7 @@ package manifests
 import (
 	"bytes"
 	"encoding/json"
+	"errors"
 	"os"
 	"os/exec"
 	"path/filepath"
@@ -63,6 +64,10 @@ func getDefinitions(crd string, resource int, cfg Config, filepath string, binPa
 }
 
 func getSchema(crd string, fp string, binPath string, cfg Config) (string, error) {
+	//few checks to avoid index out of bound panic
+	if len(cfg.Filter.ItrSpecFilter) == 0 {
+		return "", ErrAbsentFilter(errors.New("Empty ItrSpecFilter"))
+	}
 	inputFormat := "yaml"
 	if cfg.Filter.IsJson {
 		inputFormat = "json"
@@ -165,6 +170,11 @@ func getCrdnames(s string) []string {
 }
 
 func getApiVersion(binPath string, fp string, crd string, inputFormat string, cfg Config) (string, error) {
+	//few checks to avoid index out of bound panic
+	if len(cfg.Filter.ItrFilter) == 0 {
+		return "", ErrAbsentFilter(errors.New("Empty ItrFilter"))
+	}
+
 	var (
 		out bytes.Buffer
 		er  bytes.Buffer
@@ -199,6 +209,10 @@ func getApiVersion(binPath string, fp string, crd string, inputFormat string, cf
 	return s, nil
 }
 func getApiGrp(binPath string, fp string, crd string, inputFormat string, cfg Config) (string, error) {
+	//few checks to avoid index out of bound panic
+	if len(cfg.Filter.ItrFilter) == 0 {
+		return "", ErrAbsentFilter(errors.New("Empty ItrFilter"))
+	}
 	var (
 		out bytes.Buffer
 		er  bytes.Buffer