[chore] remove test only request impl for profile, use requesttest (#…

…12248)

Signed-off-by: Bogdan Drutu <[email protected]>

exporter/exporterhelper/xexporterhelper/profiles_batch_test.go

@@ -12,6 +12,7 @@ import (
 
 	"go.opentelemetry.io/collector/exporter/exporterbatcher"
 	"go.opentelemetry.io/collector/exporter/exporterhelper"
+	"go.opentelemetry.io/collector/exporter/internal/requesttest"
 	"go.opentelemetry.io/collector/pdata/pprofile"
 	"go.opentelemetry.io/collector/pdata/testdata"
 )
@@ -26,9 +27,8 @@ func TestMergeProfiles(t *testing.T) {
 }
 
 func TestMergeProfilesInvalidInput(t *testing.T) {
-	pr1 := &dummyRequest{}
 	pr2 := newProfilesRequest(testdata.GenerateProfiles(3), nil)
-	_, err := pr2.MergeSplit(context.Background(), exporterbatcher.MaxSizeConfig{}, pr1)
+	_, err := pr2.MergeSplit(context.Background(), exporterbatcher.MaxSizeConfig{}, &requesttest.FakeRequest{Items: 1})
 	assert.Error(t, err)
 }
 
@@ -125,13 +125,6 @@ func TestMergeSplitProfiles(t *testing.T) {
 	}
 }
 
-func TestMergeSplitProfilesInvalidInput(t *testing.T) {
-	r1 := &dummyRequest{}
-	r2 := newProfilesRequest(testdata.GenerateProfiles(3), nil)
-	_, err := r2.MergeSplit(context.Background(), exporterbatcher.MaxSizeConfig{}, r1)
-	assert.Error(t, err)
-}
-
 func TestExtractProfiles(t *testing.T) {
 	for i := 0; i < 10; i++ {
 		ld := testdata.GenerateProfiles(10)
@@ -140,20 +133,3 @@ func TestExtractProfiles(t *testing.T) {
 		assert.Equal(t, 10-i, ld.SampleCount())
 	}
 }
-
-// dummyRequest implements Request. It is for checking that merging two request types would fail
-type dummyRequest struct{}
-
-func (req *dummyRequest) Export(_ context.Context) error {
-	return nil
-}
-
-func (req *dummyRequest) ItemsCount() int {
-	return 1
-}
-
-func (req *dummyRequest) MergeSplit(_ context.Context, _ exporterbatcher.MaxSizeConfig, _ exporterhelper.Request) (
-	[]exporterhelper.Request, error,
-) {
-	return nil, nil
-}