Use constructor-style for NoopSubmitter

cmd/check_container.go

@@ -144,7 +144,7 @@ func resolveSubmitter(pc pyxisClient, cfg certification.Config) resultSubmitter
 		}
 	}
 
-	return &noopSubmitter{emitLog: true}
+	return newNoopSubmitter(true, "", nil)
 }
 
 // getContainerPolicyExceptions will query Pyxis to determine if

cmd/check_container_test.go

@@ -268,12 +268,12 @@ var _ = Describe("Check Container Command", func() {
 			bf = bytes.NewBuffer([]byte{})
 			bufferLogger.SetOutput(bf)
 
-			noop = &noopSubmitter{log: bufferLogger}
+			noop = newNoopSubmitter(false, "", bufferLogger)
 		})
 
 		Context("and enabling log emitting", func() {
 			BeforeEach(func() {
-				noop.emitLog = true
+				noop.SetEmitLog(true)
 			})
 
 			It("should include the reason in the emitted log if specified", func() {
@@ -293,7 +293,7 @@ var _ = Describe("Check Container Command", func() {
 
 		Context("and disabling log emitting", func() {
 			It("should not emit logs when calling submit", func() {
-				noop.emitLog = false
+				noop.SetEmitLog(false)
 				err := noop.Submit(context.TODO())
 				Expect(err).ToNot(HaveOccurred())
 				Expect(bf.String()).To(BeEmpty())

cmd/check_operator.go

@@ -129,7 +129,7 @@ func checkOperatorRunE(cmd *cobra.Command, args []string) error {
 		checkOperator.eng,
 		checkOperator.formatter,
 		checkOperator.rw,
-		&noopSubmitter{}, // we do not submit these results.
+		newNoopSubmitter(false, "", nil), // we do not submit these results.
 	)
 }
 

cmd/preflight_check_test.go

@@ -58,7 +58,7 @@ var _ = Describe("Preflight Check Func", func() {
 
 			fmttr, _ = formatters.NewByName(formatters.DefaultFormat)
 			rw = &runtime.ResultWriterFile{}
-			rs = &noopSubmitter{}
+			rs = newNoopSubmitter(false, "", nil)
 
 			DeferCleanup(os.RemoveAll, localTempDir)
 			DeferCleanup(os.RemoveAll, localArtifactsDir)

cmd/types.go

@@ -184,7 +184,15 @@ type noopSubmitter struct {
 	log     *log.Logger
 }
 
-var _ resultSubmitter = &noopSubmitter{}
+func newNoopSubmitter(emitLog bool, reason string, log *log.Logger) *noopSubmitter {
+	return &noopSubmitter{
+		emitLog: emitLog,
+		reason:  reason,
+		log:     log,
+	}
+}
+
+var _ resultSubmitter = newNoopSubmitter(false, "", nil)
 
 func (s *noopSubmitter) Submit(ctx context.Context) error {
 	if s.emitLog {
@@ -198,3 +206,11 @@ func (s *noopSubmitter) Submit(ctx context.Context) error {
 
 	return nil
 }
+
+func (s *noopSubmitter) SetEmitLog(emitLog bool) {
+	s.emitLog = emitLog
+}
+
+func (s *noopSubmitter) SetReason(reason string) {
+	s.reason = reason
+}