Refactor video processing to extract variant generation logic into a separate function

Author: Prateet-Github

internal/handlers/variants.go

@@ -0,0 +1,37 @@
+package handlers
+
+import (
+	"context"
+	"log"
+	"path/filepath"
+
+	"github.com/Prateet-Github/worker-go/internal/ffmpeg"
+)
+
+func (h *VideoHandler) generateVariants(
+	ctx context.Context,
+	inputPath string,
+	outputDir string,
+) error {
+
+	for _, rendition := range ffmpeg.Renditions {
+
+		renditionDir := filepath.Join(
+			outputDir,
+			rendition.Name,
+		)
+
+		log.Printf("Generating %s...", rendition.Name)
+
+		if err := h.ffmpeg.GenerateVariant(
+			ctx,
+			inputPath,
+			renditionDir,
+			rendition,
+		); err != nil {
+			return err
+		}
+	}
+
+	return nil
+}

internal/handlers/video.go

@@ -82,33 +82,12 @@ func (h *VideoHandler) ProcessVideo(
 
 	log.Println("Generating HLS...")
 
-	// if err := h.ffmpeg.GenerateVariant(
-	// 	ctx,
-	// 	inputPath,
-	// 	outputDir,
-	// 	ffmpeg.Renditions[1],
-	// ); err != nil {
-	// 	log.Printf("GenerateHLS failed: %v", err)
-	// 	return err
-	// }
-
-	for _, rendition := range ffmpeg.Renditions {
-
-		renditionDir := filepath.Join(
-			outputDir,
-			rendition.Name,
-		)
-
-		log.Printf("Generating %s...", rendition.Name)
-
-		if err := h.ffmpeg.GenerateVariant(
-			ctx,
-			inputPath,
-			renditionDir,
-			rendition,
-		); err != nil {
-			return err
-		}
+	if err := h.generateVariants(
+		ctx,
+		inputPath,
+		outputDir,
+	); err != nil {
+		return err
 	}
 
 	log.Println("HLS generated")