DMD-920 - Move formatJobTimePtr to internal/pkg/utils/timeutils

- Created new timeutils package with FormatISO8601Ptr function
- Updated processor.go to use the new utility function
- Moved test to timeutils package

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: vojtabiberle

internal/pkg/llm/twinformat/processor.go

@@ -6,11 +6,11 @@ import (
 	"time"
 
 	"github.com/keboola/keboola-sdk-go/v2/pkg/keboola"
-	"github.com/relvacode/iso8601"
 
 	"github.com/keboola/keboola-as-code/internal/pkg/filesystem"
 	"github.com/keboola/keboola-as-code/internal/pkg/log"
 	"github.com/keboola/keboola-as-code/internal/pkg/telemetry"
+	"github.com/keboola/keboola-as-code/internal/pkg/utils/timeutils"
 )
 
 // ProcessedData holds all processed data ready for generation.
@@ -425,7 +425,7 @@ func (p *Processor) processTransformations(ctx context.Context, configs []*Trans
 		jobKey := cfg.ComponentID + ":" + cfg.ID
 		if job, ok := jobMap[jobKey]; ok {
 			jobExec = &JobExecution{
-				LastRunTime:     formatJobTimePtr(job.StartTime),
+				LastRunTime:     timeutils.FormatISO8601Ptr(job.StartTime),
 				LastRunStatus:   job.Status,
 				JobReference:    job.ID.String(),
 				DurationSeconds: job.DurationSeconds,
@@ -622,14 +622,6 @@ func extractBucketName(bucketID string) string {
 	return bucket
 }
 
-// formatJobTimePtr formats a job time pointer for output.
-func formatJobTimePtr(t *iso8601.Time) string {
-	if t == nil {
-		return ""
-	}
-	return t.Time.UTC().Format(time.RFC3339)
-}
-
 // buildTableUID builds a table UID from bucket and table name.
 func buildTableUID(bucket, table string) string {
 	var b strings.Builder

internal/pkg/llm/twinformat/processor_test.go

@@ -72,31 +72,6 @@ func TestExtractBucketName(t *testing.T) {
 	}
 }
 
-func TestFormatJobTimePtr(t *testing.T) {
-	t.Parallel()
-
-	// Create a fixed time for testing
-	fixedTime := time.Date(2024, 1, 15, 10, 30, 0, 0, time.UTC)
-	iso8601Time := iso8601.Time{Time: fixedTime}
-
-	tests := []struct {
-		name     string
-		time     *iso8601.Time
-		expected string
-	}{
-		{name: "nil time", time: nil, expected: ""},
-		{name: "valid time", time: &iso8601Time, expected: "2024-01-15T10:30:00Z"},
-	}
-
-	for _, tc := range tests {
-		t.Run(tc.name, func(t *testing.T) {
-			t.Parallel()
-			result := formatJobTimePtr(tc.time)
-			assert.Equal(t, tc.expected, result)
-		})
-	}
-}
-
 func TestIsJobNewer(t *testing.T) {
 	t.Parallel()
 

internal/pkg/utils/timeutils/timeutils.go

@@ -0,0 +1,17 @@
+// Package timeutils provides time formatting utilities.
+package timeutils
+
+import (
+	"time"
+
+	"github.com/relvacode/iso8601"
+)
+
+// FormatISO8601Ptr formats an iso8601.Time pointer to RFC3339 string.
+// Returns empty string if the pointer is nil.
+func FormatISO8601Ptr(t *iso8601.Time) string {
+	if t == nil {
+		return ""
+	}
+	return t.Time.UTC().Format(time.RFC3339)
+}

internal/pkg/utils/timeutils/timeutils_test.go

@@ -0,0 +1,34 @@
+package timeutils
+
+import (
+	"testing"
+	"time"
+
+	"github.com/relvacode/iso8601"
+	"github.com/stretchr/testify/assert"
+)
+
+func TestFormatISO8601Ptr(t *testing.T) {
+	t.Parallel()
+
+	// Create a fixed time for testing
+	fixedTime := time.Date(2024, 1, 15, 10, 30, 0, 0, time.UTC)
+	iso8601Time := iso8601.Time{Time: fixedTime}
+
+	tests := []struct {
+		name     string
+		time     *iso8601.Time
+		expected string
+	}{
+		{name: "nil time", time: nil, expected: ""},
+		{name: "valid time", time: &iso8601Time, expected: "2024-01-15T10:30:00Z"},
+	}
+
+	for _, tc := range tests {
+		t.Run(tc.name, func(t *testing.T) {
+			t.Parallel()
+			result := FormatISO8601Ptr(tc.time)
+			assert.Equal(t, tc.expected, result)
+		})
+	}
+}