strings: add a test case of growLen is negative

Before committing, the test coverage of strings/builder.go is 97.4% After committing, the test coverage of strings/builder.go is 100% Change-Id: I22643b1c4632b5ca7ef98362f32bb85faae80bad GitHub-Last-Rev: 2a55ca3e33d3aabd2ccc047de580abcfe05d6bb4 GitHub-Pull-Request: golang/go#55004 Reviewed-on: https://go-review.googlesource.com/c/go/+/430156 Reviewed-by: Ian Lance Taylor <[email protected]> TryBot-Result: Gopher Robot <[email protected]> Reviewed-by: Tobias Klauser <[email protected]> Run-TryBot: Ian Lance Taylor <[email protected]> Auto-Submit: Ian Lance Taylor <[email protected]> Reviewed-by: Jenny Rakoczy <[email protected]>
cloudflare · Sep 13, 2022 · 09f3ff1 · 09f3ff1
1 parent 77420fa
commit 09f3ff1
Showing 1 changed file with 9 additions and 0 deletions.
diff --git a/src/strings/builder_test.go b/src/strings/builder_test.go
@@ -109,6 +109,15 @@ func TestBuilderGrow(t *testing.T) {
  t.Errorf("growLen=%d: got %d allocs during Write; want %v", growLen, g, w)
  }
  }
+ // when growLen < 0, should panic
+ var a Builder
+ n := -1
+ defer func() {
+ if r := recover(); r == nil {
+ t.Errorf("a.Grow(%d) should panic()", n)
+ }
+ }()
+ a.Grow(n)
 }
 
 func TestBuilderWrite2(t *testing.T) {