gitaly-lfs-smudge: Split out a new `run()` function

We're about to move loading the configuration out of `smudge()` so that
it doesn't need to depend on any global state anymore. Prepare for this
by pulling out a separate `run()` function so that we can continue to
log any errors and exit with an error code in a single location, only.

1 files changed, 9 insertions, 1 deletions

diff --git a/cmd/gitaly-lfs-smudge/main.go b/cmd/gitaly-lfs-smudge/main.go
index 2d39f2619..c29d03d25 100644
--- a/cmd/gitaly-lfs-smudge/main.go
+++ b/cmd/gitaly-lfs-smudge/main.go
@@ -41,7 +41,7 @@ func main() {
 	}
 	defer closer.Close()
 
-	if err := smudge(os.Stdout, os.Stdin, &envConfig{}); err != nil {
+	if err := run(os.Stdout, os.Stdin, &envConfig{}); err != nil {
 		log.WithError(err).Error(err)
 		os.Exit(1)
 	}
@@ -61,3 +61,11 @@ func initLogging(p configProvider) (io.Closer, error) {
 		log.WithOutputName(filepath),
 	)
 }
+
+func run(out io.Writer, in io.Reader, cfgProvider configProvider) error {
+	if err := smudge(out, in, cfgProvider); err != nil {
+		return fmt.Errorf("running smudge filter: %w", err)
+	}
+
+	return nil
+}