Cleanup (#202)

* DummySegment and DummySubSegment creation

* Revert "DummySegment and DummySubSegment creation"

This reverts commit 4edb4b5.

* Revert "Revert "DummySegment and DummySubSegment creation""

This reverts commit 0442d84.

* ran gofmt command to remove golint warnings

* remove redundent declaration

* Added Dummysegment creation logic

* Added reviewdog suggestions

* modified code based on feedback

* clean up before release

Author: bhautikpip

xray/client_test.go

@@ -336,14 +336,6 @@ func TestRoundTripReuseDatarace(t *testing.T) {
 		}()
 	}
 	wg.Wait()
-
-	// FIXME: @shogo82148 sometimes fail with context deadline exceeded
-	// for i := 0; i < n; i++ {
-	// 	_, err := td.Recv()
-	// 	if !assert.NoError(t, err) {
-	// 		return
-	// 	}
-	// }
 }
 
 func TestRoundTripReuseTLSDatarace(t *testing.T) {
@@ -373,14 +365,6 @@ func TestRoundTripReuseTLSDatarace(t *testing.T) {
 		}()
 	}
 	wg.Wait()
-
-	// FIXME: @shogo82148 sometimes fail with context deadline exceeded
-	// for i := 0; i < n; i++ {
-	// 	_, err := td.Recv()
-	// 	if !assert.NoError(t, err) {
-	// 		return
-	// 	}
-	// }
 }
 
 func TestRoundTripReuseHTTP2Datarace(t *testing.T) {
@@ -423,14 +407,6 @@ func TestRoundTripReuseHTTP2Datarace(t *testing.T) {
 		}()
 	}
 	wg.Wait()
-
-	// FIXME: @shogo82148 sometimes fail with context deadline exceeded
-	// for i := 0; i < n; i++ {
-	// 	_, err := td.Recv()
-	// 	if !assert.NoError(t, err) {
-	// 		return
-	// 	}
-	// }
 }
 
 // Benchmarks