[go] time: skip test that will fail with GO111MODULE=off

0 views
Skip to first unread message

Kevin Burke (Gerrit)

unread,
Dec 23, 2025, 3:51:48 PM (2 days ago) Dec 23
to goph...@pubsubhelper.golang.org, golang-...@googlegroups.com, Go LUCI, Roland Shoemaker, Nicholas Husin, Michael Knyszek, Russ Cox, Gopher Robot, golang-co...@googlegroups.com

Kevin Burke submitted the change

Change information

Commit message:
time: skip test that will fail with GO111MODULE=off

The test is designed to ensure that behavior introduced in Go 1.23
to garbage collect async timed channels is working correctly. If
GO111MODULE=off is set (or GODEBUG=asynctimerchan=1) Go reverts to the
Go 1.20 behavior of not garbage collecting these channels, which fails
the test.

Instead of running a test in conditions where we know it will fail,
just skip the test. A more comprehensive test does not make sense
right now because this code may go away soon.

Fixes #76948.
Change-Id: Ib186abd2ea583a06b5c246bfd6df932522cf7f1e
Reviewed-by: Nicholas Husin <hu...@google.com>
Reviewed-by: Michael Knyszek <mkny...@google.com>
Files:
  • M src/time/export_test.go
  • M src/time/tick_test.go
Change size: XS
Delta: 2 files changed, 5 insertions(+), 0 deletions(-)
Branch: refs/heads/master
Submit Requirements:
  • requirement satisfiedCode-Review: +2 by Michael Knyszek, +1 by Nicholas Husin
  • requirement satisfiedTryBots-Pass: LUCI-TryBot-Result+1 by Go LUCI
Open in Gerrit
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: merged
Gerrit-Project: go
Gerrit-Branch: master
Gerrit-Change-Id: Ib186abd2ea583a06b5c246bfd6df932522cf7f1e
Gerrit-Change-Number: 732100
Gerrit-PatchSet: 2
Gerrit-Owner: Kevin Burke <ke...@burke.dev>
Gerrit-Reviewer: Kevin Burke <ke...@burke.dev>
Gerrit-Reviewer: Michael Knyszek <mkny...@google.com>
Gerrit-Reviewer: Nicholas Husin <hu...@google.com>
Gerrit-Reviewer: Roland Shoemaker <rol...@golang.org>
Gerrit-Reviewer: Russ Cox <r...@golang.org>
Gerrit-CC: Gopher Robot <go...@golang.org>
open
diffy
satisfied_requirement
Reply all
Reply to author
Forward
0 new messages