improve tests

track_h264_test.go

@@ -7,6 +7,23 @@ import (
 	"github.com/stretchr/testify/require"
 )
 
+func TestTrackH264Attributes(t *testing.T) {
+	track := &TrackH264{
+		PayloadType: 96,
+		SPS:         []byte{0x01, 0x02},
+		PPS:         []byte{0x03, 0x04},
+	}
+	require.Equal(t, 90000, track.ClockRate())
+	require.Equal(t, "", track.GetControl())
+	require.Equal(t, []byte{0x01, 0x02}, track.SafeSPS())
+	require.Equal(t, []byte{0x03, 0x04}, track.SafePPS())
+
+	track.SafeSetSPS([]byte{0x07, 0x08})
+	track.SafeSetPPS([]byte{0x09, 0x0A})
+	require.Equal(t, []byte{0x07, 0x08}, track.SafeSPS())
+	require.Equal(t, []byte{0x09, 0x0A}, track.SafePPS())
+}
+
 func TestTrackH264GetSPSPPSErrors(t *testing.T) {
 	for _, ca := range []struct {
 		name string
@@ -165,22 +182,6 @@ func TestTrackH264GetSPSPPSErrors(t *testing.T) {
 	}
 }
 
-func TestTrackH264Params(t *testing.T) {
-	track := &TrackH264{
-		PayloadType: 96,
-		SPS:         []byte{0x01, 0x02},
-		PPS:         []byte{0x03, 0x04},
-	}
-	require.Equal(t, "", track.GetControl())
-	require.Equal(t, []byte{0x01, 0x02}, track.SafeSPS())
-	require.Equal(t, []byte{0x03, 0x04}, track.SafePPS())
-
-	track.SafeSetSPS([]byte{0x07, 0x08})
-	track.SafeSetPPS([]byte{0x09, 0x0A})
-	require.Equal(t, []byte{0x07, 0x08}, track.SafeSPS())
-	require.Equal(t, []byte{0x09, 0x0A}, track.SafePPS())
-}
-
 func TestTrackH264Clone(t *testing.T) {
 	track := &TrackH264{
 		PayloadType: 96,

track_h265_test.go

@@ -7,13 +7,14 @@ import (
 	"github.com/stretchr/testify/require"
 )
 
-func TestTrackH265Params(t *testing.T) {
+func TestTrackH265Attributes(t *testing.T) {
 	track := &TrackH265{
 		PayloadType: 96,
 		VPS:         []byte{0x01, 0x02},
 		SPS:         []byte{0x03, 0x04},
 		PPS:         []byte{0x05, 0x06},
 	}
+	require.Equal(t, 90000, track.ClockRate())
 	require.Equal(t, "", track.GetControl())
 	require.Equal(t, []byte{0x01, 0x02}, track.SafeVPS())
 	require.Equal(t, []byte{0x03, 0x04}, track.SafeSPS())

track_jpeg_test.go

@@ -1,4 +1,4 @@
-package gortsplib
+package gortsplib //nolint:dupl
 
 import (
 	"testing"
@@ -7,6 +7,12 @@ import (
 	"github.com/stretchr/testify/require"
 )
 
+func TestTrackJPEGAttributes(t *testing.T) {
+	track := &TrackJPEG{}
+	require.Equal(t, 90000, track.ClockRate())
+	require.Equal(t, "", track.GetControl())
+}
+
 func TestTrackJPEGClone(t *testing.T) {
 	track := &TrackJPEG{}
 

track_mpeg2audio_test.go

@@ -1,4 +1,4 @@
-package gortsplib
+package gortsplib //nolint:dupl
 
 import (
 	"testing"
@@ -7,8 +7,9 @@ import (
 	"github.com/stretchr/testify/require"
 )
 
-func TestTrackMPEG2AudioNew(t *testing.T) {
+func TestTrackMPEG2AudioAttributes(t *testing.T) {
 	track := &TrackMPEG2Audio{}
+	require.Equal(t, 90000, track.ClockRate())
 	require.Equal(t, "", track.GetControl())
 }
 

track_mpeg2video_test.go

@@ -1,4 +1,4 @@
-package gortsplib
+package gortsplib //nolint:dupl
 
 import (
 	"testing"
@@ -7,8 +7,9 @@ import (
 	"github.com/stretchr/testify/require"
 )
 
-func TestTrackMPEG2VideoNew(t *testing.T) {
+func TestTrackMPEG2VideoAttributes(t *testing.T) {
 	track := &TrackMPEG2Video{}
+	require.Equal(t, 90000, track.ClockRate())
 	require.Equal(t, "", track.GetControl())
 }
 

track_mpeg4audio_test.go

@@ -9,6 +9,22 @@ import (
 	"github.com/aler9/gortsplib/pkg/mpeg4audio"
 )
 
+func TestTrackMPEG4AudioAttributes(t *testing.T) {
+	track := &TrackMPEG4Audio{
+		PayloadType: 96,
+		Config: &mpeg4audio.Config{
+			Type:         2,
+			SampleRate:   48000,
+			ChannelCount: 2,
+		},
+		SizeLength:       13,
+		IndexLength:      3,
+		IndexDeltaLength: 3,
+	}
+	require.Equal(t, 48000, track.ClockRate())
+	require.Equal(t, "", track.GetControl())
+}
+
 func TestTrackMPEG4AudioClone(t *testing.T) {
 	track := &TrackMPEG4Audio{
 		PayloadType: 96,

track_opus_test.go

@@ -7,6 +7,16 @@ import (
 	"github.com/stretchr/testify/require"
 )
 
+func TestTrackOpusAttributes(t *testing.T) {
+	track := &TrackOpus{
+		PayloadType:  96,
+		SampleRate:   48000,
+		ChannelCount: 2,
+	}
+	require.Equal(t, 48000, track.ClockRate())
+	require.Equal(t, "", track.GetControl())
+}
+
 func TestTracOpusClone(t *testing.T) {
 	track := &TrackOpus{
 		PayloadType:  96,

track_pcma_test.go

@@ -1,4 +1,4 @@
-package gortsplib
+package gortsplib //nolint:dupl
 
 import (
 	"testing"
@@ -7,8 +7,9 @@ import (
 	"github.com/stretchr/testify/require"
 )
 
-func TestTrackPCMANew(t *testing.T) {
+func TestTrackPCMAAttributes(t *testing.T) {
 	track := &TrackPCMA{}
+	require.Equal(t, 8000, track.ClockRate())
 	require.Equal(t, "", track.GetControl())
 }
 

track_pcmu_test.go

@@ -1,4 +1,4 @@
-package gortsplib
+package gortsplib //nolint:dupl
 
 import (
 	"testing"
@@ -7,8 +7,9 @@ import (
 	"github.com/stretchr/testify/require"
 )
 
-func TestTrackPCMUNew(t *testing.T) {
+func TestTrackPCMUAttributes(t *testing.T) {
 	track := &TrackPCMU{}
+	require.Equal(t, 8000, track.ClockRate())
 	require.Equal(t, "", track.GetControl())
 }
 

track_vp8_test.go

@@ -7,7 +7,13 @@ import (
 	"github.com/stretchr/testify/require"
 )
 
-func TestTracVP8Clone(t *testing.T) {
+func TestTrackVP8ttributes(t *testing.T) {
+	track := &TrackVP8{}
+	require.Equal(t, 90000, track.ClockRate())
+	require.Equal(t, "", track.GetControl())
+}
+
+func TestTrackVP8Clone(t *testing.T) {
 	maxFR := 123
 	maxFS := 456
 	track := &TrackVP8{

track_vp9_test.go

@@ -7,7 +7,13 @@ import (
 	"github.com/stretchr/testify/require"
 )
 
-func TestTracVP9Clone(t *testing.T) {
+func TestTrackVP9ttributes(t *testing.T) {
+	track := &TrackVP9{}
+	require.Equal(t, 90000, track.ClockRate())
+	require.Equal(t, "", track.GetControl())
+}
+
+func TestTrackVP9Clone(t *testing.T) {
 	maxFR := 123
 	maxFS := 456
 	profileID := 789