test: refactoring benchmark

bzz · bzz · commit 3f8805029ce5 · 2019-03-13T17:57:09.000+01:00
Signed-off-by: Alexander Bezzubov &lt;bzz@apache.org&gt;
diff --git a/driver/normalizer/strconv_test.go b/driver/normalizer/strconv_test.go
@@ -72,8 +72,8 @@ func printDebug(t *testing.T, quoted, actual string) {
 
 func BenchmarkReplacingNullEscape_Iterative(b *testing.B) {
 	b.ReportAllocs()
-	for n := 0; n < b.N; n++ {
-		for _, test := range testCasesUnquote {
+	for _, test := range testCasesUnquote {
+		for n := 0; n < b.N; n++ {
 			replaceEscapedMaybeIter(test.quoted, '0', '\x00')
 		}
 	}
@@ -118,8 +118,8 @@ func replaceEscapedMaybeIter(s string, old, new rune) string {
 
 func BenchmarkReplacingNullEscape_Regexp(b *testing.B) {
 	b.ReportAllocs()
-	for n := 0; n < b.N; n++ {
-		for _, test := range testCasesUnquote {
+	for _, test := range testCasesUnquote {
+		for n := 0; n < b.N; n++ {
 			replaceEscapedMaybeRegexp(test.quoted)
 		}
 	}
@@ -135,8 +135,8 @@ func replaceEscapedMaybeRegexp(s string) string {
 
 func BenchmarkReplacingNullEscape_Simple(b *testing.B) {
 	b.ReportAllocs()
-	for n := 0; n < b.N; n++ {
-		for _, test := range testCasesUnquote {
+	for _, test := range testCasesUnquote {
+		for n := 0; n < b.N; n++ {
 			replaceEscapedMaybe(test.quoted, "\\0", "\x00")
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -72,8 +72,8 @@ func printDebug(t *testing.T, quoted, actual string) {`
`72`	`72`
`73`	`73`	`func BenchmarkReplacingNullEscape_Iterative(b *testing.B) {`
`74`	`74`	`b.ReportAllocs()`
`75`		`- for n := 0; n < b.N; n++ {`
`76`		`- for _, test := range testCasesUnquote {`
	`75`	`+ for _, test := range testCasesUnquote {`
	`76`	`+ for n := 0; n < b.N; n++ {`
`77`	`77`	`replaceEscapedMaybeIter(test.quoted, '0', '\x00')`
`78`	`78`	`}`
`79`	`79`	`}`
`@@ -118,8 +118,8 @@ func replaceEscapedMaybeIter(s string, old, new rune) string {`
`118`	`118`
`119`	`119`	`func BenchmarkReplacingNullEscape_Regexp(b *testing.B) {`
`120`	`120`	`b.ReportAllocs()`
`121`		`- for n := 0; n < b.N; n++ {`
`122`		`- for _, test := range testCasesUnquote {`
	`121`	`+ for _, test := range testCasesUnquote {`
	`122`	`+ for n := 0; n < b.N; n++ {`
`123`	`123`	`replaceEscapedMaybeRegexp(test.quoted)`
`124`	`124`	`}`
`125`	`125`	`}`
`@@ -135,8 +135,8 @@ func replaceEscapedMaybeRegexp(s string) string {`
`135`	`135`
`136`	`136`	`func BenchmarkReplacingNullEscape_Simple(b *testing.B) {`
`137`	`137`	`b.ReportAllocs()`
`138`		`- for n := 0; n < b.N; n++ {`
`139`		`- for _, test := range testCasesUnquote {`
	`138`	`+ for _, test := range testCasesUnquote {`
	`139`	`+ for n := 0; n < b.N; n++ {`
`140`	`140`	`replaceEscapedMaybe(test.quoted, "\\0", "\x00")`
`141`	`141`	`}`
`142`	`142`	`}`