[minor] Fix bug in conditional for getting ResultSinkCtx
Change-Id: Ibe667fd6d9e4d1b24a8f09126533bc89e02fe60c
Reviewed-on: https://fuchsia-review.googlesource.com/c/shac-project/shac/+/891373
Commit-Queue: Oliver Newman <olivernewman@google.com>
Reviewed-by: Marc-Antoine Ruel <maruel@google.com>
Reviewed-by: Oliver Newman <olivernewman@google.com>
diff --git a/internal/reporting/resultdb.go b/internal/reporting/resultdb.go
index 8dd9d63..66ea6e9 100644
--- a/internal/reporting/resultdb.go
+++ b/internal/reporting/resultdb.go
@@ -251,7 +251,7 @@
}
// We are clearly running inside a LUCI_CONTEXT luciexe environment but rdb
// stream was not started. Hard fail since it means we need to fix the recipe.
- if ctx.ResultDB.CurrentInvocation.Name != "" && (ctx.ResultSink.AuthToken == "" || ctx.ResultSink.ResultSinkAddr == "") {
+ if ctx.ResultDB.CurrentInvocation.Name == "" || ctx.ResultSink.AuthToken == "" || ctx.ResultSink.ResultSinkAddr == "" {
return nil, fmt.Errorf("resultdb is enabled but not resultsink for invocation %s. Make sure shac is run under \"rdb stream\"", ctx.ResultDB.CurrentInvocation.Name)
}
return &ctx.ResultSink, nil
