| commit | 6193cb187e4a14df44699ecaa3497988b2a23fd6 | [log] [tgz] |
|---|---|---|
| author | Ben Clayton <bclayton@google.com> | Tue Mar 12 08:55:15 2019 +0000 |
| committer | Ben Clayton <bclayton@google.com> | Tue Mar 12 13:46:53 2019 +0000 |
| tree | c9c9cc7be17186431522c25cc31321f236ece074 | |
| parent | abb57857c5369cecdf048de8b9dc8f530fb37849 [diff] |
tests/regres/export_to_sheets: Fix off-by-one in loop We weren't looking at the most recent change. Kinda important when that's typically the new test list you're trying to export. Change-Id: I0e312dbde0ae54feb0641a658eebc73bbdb47eaa Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26928 Reviewed-by: Nicolas Capens <nicolascapens@google.com> Tested-by: Ben Clayton <bclayton@google.com>
diff --git a/tests/regres/export_to_sheets.go b/tests/regres/export_to_sheets.go index 70f9f4e..e982a5c 100644 --- a/tests/regres/export_to_sheets.go +++ b/tests/regres/export_to_sheets.go
@@ -124,7 +124,7 @@ rowInsertionPoint := 1 + len(hashValues) - for i := len(changes) - 1; i > 0; i-- { + for i := len(changes) - 1; i >= 0; i-- { change := changes[i] if !strings.HasPrefix(change.Subject, consts.TestListUpdateCommitSubjectPrefix) { continue