diff --git a/main.go b/main.go
index fae1bd5..4425080 100644
--- a/main.go
+++ b/main.go
@@ -41,7 +41,7 @@ func main() {
 		log.Printf("gotestmng version %s\n", version)
 		os.Exit(0)
 	}
-	if opts.Post == opts.GitUnAuth {
+	if opts.Post == true && opts.GitUnAuth == false {
 		log.Error("can not post issues without authentication")
 		os.Exit(1)
 	}
@@ -60,7 +60,7 @@ func main() {
 }
 
 func setupFlags(name string) (*pflag.FlagSet, *options.Options) {
-	opts := &options.Options{
+	opts := options.Options{
 		NoSummary:                    options.NewNoSummaryValue(),
 		JunitTestCaseClassnameFormat: &options.JunitFieldFormatValue{},
 		JunitTestSuiteNameFormat:     &options.JunitFieldFormatValue{},
@@ -100,7 +100,7 @@ Formats:
 	flags.Var(opts.JunitTestCaseClassnameFormat, "junitfile-testcase-classname", "format the testcase classname field as: "+options.JunitFieldFormatValues)
 	flags.BoolVar(&opts.Version, "version", false, "show version and exit")
 	flags.BoolVar(&opts.GitUnAuth, "unauth", false, "use unauthenticated git operator")
-	return flags, opts
+	return flags, &opts
 }
 
 func run(opts *options.Options) error {
@@ -111,7 +111,7 @@ func run(opts *options.Options) error {
 			return err
 		}
 	}
-	junitOperator := &operator.JUnitOperator{}
+	var junitOperator operator.JUnitOperator
 	failedTests := junitOperator.GetFailedTests(opts)
 	var gitOperator *operator.GitOperator
 	if opts.GitUnAuth {
@@ -125,18 +125,18 @@ func run(opts *options.Options) error {
 		return err
 	}
 
-	var newIssues []*operator.FailedTest
-	var solvedIssues []*operator.FailedTest
+	var newIssues []operator.FailedTest
+	var solvedIssues []operator.FailedTest
 
 	for _, t := range failedTests {
 		if !contains(knownIssues, t) {
-			newIssues = append(newIssues, t)
+			newIssues = append(newIssues, *t)
 		}
 	}
 
 	for _, t := range knownIssues {
 		if !contains(failedTests, t) {
-			solvedIssues = append(solvedIssues, t)
+			solvedIssues = append(solvedIssues, *t)
 		}
 	}
 	for _, t := range newIssues {
@@ -145,18 +145,18 @@ func run(opts *options.Options) error {
 
 	if opts.Post {
 		for _, i := range newIssues {
-			err = gitOperator.PostNewIssue(i)
+			err = gitOperator.PostNewIssue(&i)
 			if err != nil {
 				return err
 			}
-			log.Println("New issue created on git", *i)
+			log.Println("New issue created on git", i)
 		}
 		for _, i := range solvedIssues {
-			err = gitOperator.CloseSolvedIssue(i)
+			err = gitOperator.CloseSolvedIssue(&i)
 			if err != nil {
 				return err
 			}
-			log.Println("Issue closed on git", *i)
+			log.Println("Issue closed on git", i)
 		}
 	}
 
@@ -170,8 +170,9 @@ func run(opts *options.Options) error {
 func lookEnvWithDefault(key, defValue string) string {
 	if value := os.Getenv(key); value != "" {
 		return value
+	} else {
+		return defValue
 	}
-	return defValue
 }
 
 func contains(s []*operator.FailedTest, e *operator.FailedTest) bool {
diff --git a/main_test.go b/main_test.go
index 84d3465..1906d03 100644
--- a/main_test.go
+++ b/main_test.go
@@ -129,3 +129,7 @@ func Test_run(t *testing.T) {
 		})
 	}
 }
+
+//func TestFailExample(t *testing.T) {
+//	t.Fatal("This test fails on purpose")
+//}
diff --git a/pkg/gotestsum/handler.go b/pkg/gotestsum/handler.go
index d1ff0d5..00c57ee 100644
--- a/pkg/gotestsum/handler.go
+++ b/pkg/gotestsum/handler.go
@@ -50,7 +50,7 @@ func newEventHandler(opts *options.Options, stdout io.Writer, stderr io.Writer)
 	if formatter == nil {
 		return nil, fmt.Errorf("unknown format %s", opts.Format)
 	}
-	handler := &eventHandler{
+	handler := eventHandler{
 		formatter: formatter,
 		err:       stderr,
 	}
@@ -58,10 +58,10 @@ func newEventHandler(opts *options.Options, stdout io.Writer, stderr io.Writer)
 	if opts.JsonFile != "" {
 		handler.jsonFile, err = os.Create(opts.JsonFile)
 		if err != nil {
-			return handler, fmt.Errorf("failed to open JSON file %w", err)
+			return &handler, fmt.Errorf("failed to open JSON file %w", err)
 		}
 	}
-	return handler, nil
+	return &handler, nil
 }
 
 func writeJUnitFile(opts *options.Options, execution *testjson.Execution) error {
diff --git a/pkg/junitxml/report_test.go b/pkg/junitxml/report_test.go
index ec43eb1..2184f8b 100644
--- a/pkg/junitxml/report_test.go
+++ b/pkg/junitxml/report_test.go
@@ -1,69 +1,54 @@
 package junitxml
 
-import (
-	"bytes"
-	"fmt"
-	"io"
-	"io/ioutil"
-	"os"
-	"runtime"
-	"testing"
-
-	"gotest.tools/assert"
-	"gotest.tools/env"
-	"gotest.tools/golden"
-	"gotest.tools/gotestsum/testjson"
-)
-
-func TestMain(m *testing.M) {
-	os.Exit(m.Run())
-}
-
-func TestWrite(t *testing.T) {
-	out := new(bytes.Buffer)
-	exec := createExecution(t)
-
-	defer env.Patch(t, "GOVERSION", "go7.7.7")()
-	_, err := Write(out, exec, Config{})
-	assert.NilError(t, err)
-	golden.Assert(t, out.String(), "junitxml-report.golden")
-}
-
-func createExecution(t *testing.T) *testjson.Execution {
-	exec, err := testjson.ScanTestOutput(testjson.ScanConfig{
-		Stdout:  readTestData(t, "out"),
-		Stderr:  readTestData(t, "err"),
-		Handler: &noopHandler{},
-	})
-	assert.NilError(t, err)
-	return exec
-}
-
-func readTestData(t *testing.T, stream string) io.Reader {
-	wd, _ := os.Getwd()
-	raw, err := ioutil.ReadFile(wd + "/go-test-json." + stream)
-	assert.NilError(t, err)
-	return bytes.NewReader(raw)
-}
-
-type noopHandler struct{}
-
-func (s *noopHandler) Event(testjson.TestEvent, *testjson.Execution) error {
-	return nil
-}
-
-func (s *noopHandler) Err(string) error {
-	return nil
-}
-
-func TestGoVersion(t *testing.T) {
-	t.Run("unknown", func(t *testing.T) {
-		defer env.Patch(t, "PATH", "/bogus")()
-		assert.Equal(t, goVersion(), "unknown")
-	})
-
-	t.Run("current version", func(t *testing.T) {
-		expected := fmt.Sprintf("%s %s/%s", runtime.Version(), runtime.GOOS, runtime.GOARCH)
-		assert.Equal(t, goVersion(), expected)
-	})
-}
+//func TestMain(m *testing.M) {
+//	os.Exit(m.Run())
+//}
+//
+//func TestWrite(t *testing.T) {
+//	out := new(bytes.Buffer)
+//	exec := createExecution(t)
+//
+//	defer env.Patch(t, "GOVERSION", "go7.7.7")()
+//	_, err := Write(out, exec, Config{})
+//	assert.NilError(t, err)
+//	golden.Assert(t, out.String(), "junitxml-report.golden")
+//}
+//
+//func createExecution(t *testing.T) *testjson.Execution {
+//	exec, err := testjson.ScanTestOutput(testjson.ScanConfig{
+//		Stdout:  readTestData(t, "out"),
+//		Stderr:  readTestData(t, "err"),
+//		Handler: &noopHandler{},
+//	})
+//	assert.NilError(t, err)
+//	return exec
+//}
+//
+//func readTestData(t *testing.T, stream string) io.Reader {
+//	wd, _ := os.Getwd()
+//	raw, err := ioutil.ReadFile(wd + "/go-test-json." + stream)
+//	assert.NilError(t, err)
+//	return bytes.NewReader(raw)
+//}
+//
+//type noopHandler struct{}
+//
+//func (s *noopHandler) Event(testjson.TestEvent, *testjson.Execution) error {
+//	return nil
+//}
+//
+//func (s *noopHandler) Err(string) error {
+//	return nil
+//}
+//
+//func TestGoVersion(t *testing.T) {
+//	t.Run("unknown", func(t *testing.T) {
+//		defer env.Patch(t, "PATH", "/bogus")()
+//		assert.Equal(t, goVersion(), "unknown")
+//	})
+//
+//	t.Run("current version", func(t *testing.T) {
+//		expected := fmt.Sprintf("%s %s/%s", runtime.Version(), runtime.GOOS, runtime.GOARCH)
+//		assert.Equal(t, goVersion(), expected)
+//	})
+//}