Remove unused function argument

internal/exporter/exporter.go

@@ -121,7 +121,7 @@ func (e *exporter) export(ctx context.Context) error {
 		return nil
 	}
 
-	batch := e.mapper.Map(metricFamilies, time.Now())
+	batch := e.mapper.Map(metricFamilies)
 	if err := e.client.UploadBatch(ctx, batch); err != nil {
 		return fmt.Errorf("error whlie sending %d metrics to castai %w", len(batch.Metrics), err)
 	}

internal/exporter/exporter_test.go

@@ -81,7 +81,7 @@ func TestExporter_Running(t *testing.T) {
 	batch := &pb.MetricsBatch{}
 
 	scraper.EXPECT().Scrape(ctx, []string{"http://192.168.1.1:9400/metrics"}).Times(1).Return(metricFamilies, nil)
-	mapper.EXPECT().Map(metricFamilies, mock.Anything).Times(1).Return(batch, nil)
+	mapper.EXPECT().Map(metricFamilies).Times(1).Return(batch, nil)
 	client.EXPECT().UploadBatch(mock.Anything, batch).Times(1).Return(nil, nil)
 
 	go func() {

internal/exporter/mapper.go

@@ -1,13 +1,11 @@
 package exporter
 
 import (
-	"time"
-
 	"github.com/castai/gpu-metrics-exporter/pb"
 )
 
 type MetricMapper interface {
-	Map(metrics []MetricFamilyMap, ts time.Time) *pb.MetricsBatch
+	Map(metrics []MetricFamilyMap) *pb.MetricsBatch
 }
 
 type metricMapper struct{}
@@ -16,7 +14,7 @@ func NewMapper() MetricMapper {
 	return &metricMapper{}
 }
 
-func (p metricMapper) Map(metricFamilyMaps []MetricFamilyMap, ts time.Time) *pb.MetricsBatch {
+func (p metricMapper) Map(metricFamilyMaps []MetricFamilyMap) *pb.MetricsBatch {
 	metrics := &pb.MetricsBatch{}
 	metricsMap := make(map[string]*pb.Metric)
 	for _, familyMap := range metricFamilyMaps {

internal/exporter/mapper_test.go

@@ -2,7 +2,6 @@ package exporter_test
 
 import (
 	"testing"
-	"time"
 
 	dto "github.com/prometheus/client_model/go"
 	"github.com/stretchr/testify/require"
@@ -28,18 +27,16 @@ func TestMetricMapper_Map(t *testing.T) {
 	mapper := exporter.NewMapper()
 
 	t.Run("empty input yields empty MetricsBatch", func(t *testing.T) {
-		ts := time.Now()
 		metricFamilyMaps := []exporter.MetricFamilyMap{}
 
-		got := mapper.Map(metricFamilyMaps, ts)
+		got := mapper.Map(metricFamilyMaps)
 		expected := &pb.MetricsBatch{}
 
 		r := require.New(t)
 		r.Equal(expected, got)
 	})
 
 	t.Run("metric familiy which is not enabled is skipped", func(t *testing.T) {
-		ts := time.Now()
 		metricFamilyMaps := []exporter.MetricFamilyMap{
 			{
 				"test_gauge": {
@@ -56,15 +53,14 @@ func TestMetricMapper_Map(t *testing.T) {
 			},
 		}
 
-		got := mapper.Map(metricFamilyMaps, ts)
+		got := mapper.Map(metricFamilyMaps)
 		expected := &pb.MetricsBatch{}
 
 		r := require.New(t)
 		r.Equal(expected, got)
 	})
 
 	t.Run("enabled metric family is included", func(t *testing.T) {
-		ts := time.Now()
 		metricFamilyMaps := []exporter.MetricFamilyMap{
 			{
 				"test_gauge": {
@@ -92,7 +88,7 @@ func TestMetricMapper_Map(t *testing.T) {
 			},
 		}
 
-		got := mapper.Map(metricFamilyMaps, ts)
+		got := mapper.Map(metricFamilyMaps)
 		expected := &pb.MetricsBatch{
 			Metrics: []*pb.Metric{
 				{