Verify that the file to be analyze exists before analyze it

Allow flag -no-array for backwards compatibility
Add tests for the cases that were causing issues

go/tests/rc/rc.go

@@ -92,7 +92,7 @@ passed to the program would not be allowed`,
 	flag.BoolVar(&noRelativeImports, "no-relative-imports", false, `Disallowes the use of relative imports`)
 	flag.BoolVar(&noFor, "no-for", false, `The "for" instruction is not allowed`)
 	flag.BoolVar(&casting, "cast", false, "Allowes casting")
-	flag.BoolVar(&noArrays, "no-arrays", false, "Deprecated: use -no-slices")
+	flag.BoolVar(&noArrays, "no-array", false, "Deprecated: use -no-slices")
 	flag.BoolVar(&noSlices, "no-slices", false, "Disallowes all slice types")
 	flag.BoolVar(&allowBuiltin, "allow-builtin", false, "Allowes all builtin functions and casting")
 	sort.Sort(sort.StringSlice(os.Args[1:]))
@@ -102,8 +102,8 @@ func main() {
 	flag.Parse()
 	filename := goFile(flag.Args())
 
-	if filename == "" {
-		fmt.Println("\tNo file to analyze")
+	if _, err := os.Stat(filename); err != nil {
+		fmt.Printf("\t%s\n", err)
 		os.Exit(1)
 	}
 
@@ -596,7 +596,7 @@ func allowFunction(functionPath string) error {
 
 func functionName(functionPath string) string {
 	segmentedPath := strings.Split(functionPath, ".")
-	return segmentedPath[len(segmentedPath)-1]
+	return strings.Split(segmentedPath[len(segmentedPath)-1], "#")[0]
 }
 
 func packageName(functionPath string) string {

go/tests/rc/rc_test.go

@@ -52,7 +52,7 @@ func TestFlags(t *testing.T) {
 	illegal-access    	z01.PrintRune        	tests/eightqueens.go:55:2
 	illegal-definition	printQueens          	tests/eightqueens.go:42:1
 `,
-		"-no-arrays tests/printalphabet/printalphabet.go": `Cheating:
+		"-no-array tests/printalphabet/printalphabet.go": `Cheating:
 	TYPE:             	NAME:                	LOCATION:
 	illegal-import    	fmt                  	tests/printalphabet/printalphabet.go:4:2
 	illegal-import    	github.com/01-edu/z01	tests/printalphabet/printalphabet.go:6:2
@@ -115,25 +115,23 @@ func TestFlags(t *testing.T) {
 	illegal-definition	Length          	tests/testingWrapping.go:7:1
 `,
 		`tests/testingWrapping.go len`: ``,
-		`tests/empty/empty len`: `	No file to analyze
+		`tests/empty/empty len`: `	stat : no such file or directory
 `,
 		`tests/empty/empty.go tests/empty/empty`: `	tests/empty/empty.go:1:1: expected ';', found 'EOF' (and 2 more errors)
+`,
+		`tests/abc/main.go --cast github.com/01-edu/z01.PrintRune#2 --no-lit=[b-mB-Y]`: ``,
+		`itoa.go len --cast`: `	stat itoa.go: no such file or directory
 `,
 	}
 	Compare(t, argsAndSolution)
 }
 
 func Compare(t *testing.T, argsAndSol map[string]string) {
-	for args, sol := range argsAndSol {
+	for args, expected := range argsAndSol {
 		a := strings.Split(args, " ")
-		out, err := z01.MainOut("../rc", a...)
-		if !EqualResult(sol, out) {
-			if err == nil {
-				t.Errorf("./rc %s prints %q\n instead of %q\n", args, out, sol)
-			}
-			if err != nil && !EqualResult(sol, err.Error()) {
-				t.Errorf("./rc %s prints %q\n instead of %q\n", args, err.Error(), sol)
-			}
+		_, err := z01.MainOut("../rc", a...)
+		if err != nil && !EqualResult(expected, err.Error()) {
+			t.Errorf("./rc %s prints %q\n instead of %q\n", args, err.Error(), expected)
 		}
 	}
 }
@@ -146,8 +144,8 @@ func EqualResult(sol, out string) bool {
 	sort.Sort(sort.StringSlice(solSli))
 	sort.Sort(sort.StringSlice(outSli))
 	// join
-	sol = strings.Join(solSli, "\n")
-	out = strings.Join(outSli, "\n")
+	sol = strings.Join(solSli, " ")
+	out = strings.Join(outSli, " ")
 	// compare
 	return sol == out
 }

go/tests/rc/tests/abc/main.go

@@ -0,0 +1,10 @@
+package main
+
+import "github.com/01-edu/z01"
+
+func main() {
+	for r := 'a'; r <= 'z'; r++ {
+		z01.PrintRune(r)
+	}
+	z01.PrintRune('\n')
+}