From bbcea7392a33c9929dbd8800fa9e5b50778e7c4f Mon Sep 17 00:00:00 2001 From: jvoisin Date: Wed, 18 Jun 2025 16:11:32 +0200 Subject: [PATCH] refactor(tests): use b.Loop() instead of for range b.N See https://tip.golang.org/doc/go1.24#new-benchmark-function --- internal/reader/parser/parser_test.go | 2 +- internal/reader/readingtime/readingtime_test.go | 2 +- internal/reader/sanitizer/sanitizer_test.go | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/internal/reader/parser/parser_test.go b/internal/reader/parser/parser_test.go index 9ab55a0c..61473fe4 100644 --- a/internal/reader/parser/parser_test.go +++ b/internal/reader/parser/parser_test.go @@ -22,7 +22,7 @@ func BenchmarkParse(b *testing.B) { } testCases[filename][1] = string(data) } - for range b.N { + for b.Loop() { for _, v := range testCases { ParseFeed(v[0], strings.NewReader(v[1])) } diff --git a/internal/reader/readingtime/readingtime_test.go b/internal/reader/readingtime/readingtime_test.go index a70d1c6c..692d4572 100644 --- a/internal/reader/readingtime/readingtime_test.go +++ b/internal/reader/readingtime/readingtime_test.go @@ -80,7 +80,7 @@ func TestEstimateReadingTime(t *testing.T) { } func BenchmarkEstimateReadingTime(b *testing.B) { - for range b.N { + for b.Loop() { for _, sample := range samples { EstimateReadingTime(sample, 200, 500) } diff --git a/internal/reader/sanitizer/sanitizer_test.go b/internal/reader/sanitizer/sanitizer_test.go index 53819bdf..5ffb6a39 100644 --- a/internal/reader/sanitizer/sanitizer_test.go +++ b/internal/reader/sanitizer/sanitizer_test.go @@ -25,7 +25,7 @@ func BenchmarkSanitize(b *testing.B) { } testCases[filename][1] = string(data) } - for range b.N { + for b.Loop() { for _, v := range testCases { SanitizeHTMLWithDefaultOptions(v[0], v[1]) }