From bb6f470df5ec0dc334b76973386de47ec0f2f91a Mon Sep 17 00:00:00 2001 From: Dennis Date: Thu, 20 Aug 2026 23:55:37 +0800 Subject: [PATCH] test: address shortcut regression review --- .changes/1074-shortcut-regression-fixes.md | 7 +++++++ .../todo/todo_additional_cross_platform_coverage_test.go | 8 ++++++++ 2 files changed, 15 insertions(+) create mode 100644 .changes/1074-shortcut-regression-fixes.md diff --git a/.changes/1074-shortcut-regression-fixes.md b/.changes/1074-shortcut-regression-fixes.md new file mode 100644 index 00000000..55390ec8 --- /dev/null +++ b/.changes/1074-shortcut-regression-fixes.md @@ -0,0 +1,7 @@ +--- +category: Fixed +--- + +- **Calendar empty windows** (#1074) — returns a legitimate empty result when the service emits its exact exhausted empty-event sentinel. +- **Task update verification** (#1074) — compares due-time readback as exact milliseconds so committed updates are no longer reported as failures. +- **Comment reaction validation** (#1074) — narrows accepted reaction input to reviewed DingTalk emoji names and rejects Unicode emoji and unsupported names such as `like` and `heart` before the RPC. diff --git a/internal/shortcut/todo/todo_additional_cross_platform_coverage_test.go b/internal/shortcut/todo/todo_additional_cross_platform_coverage_test.go index 7bbfc282..caae2acc 100644 --- a/internal/shortcut/todo/todo_additional_cross_platform_coverage_test.go +++ b/internal/shortcut/todo/todo_additional_cross_platform_coverage_test.go @@ -8,6 +8,7 @@ import ( "encoding/json" "errors" "io" + "math" "strconv" "strings" "testing" @@ -331,7 +332,14 @@ func TestCrossPlatformCoverageTodoCreateAndUpdate(t *testing.T) { {value: float64(40), want: 40, ok: true}, {value: json.Number("40"), want: 40, ok: true}, {value: " 40 ", want: 40, ok: true}, + {value: float64(math.MinInt64), want: math.MinInt64, ok: true}, {value: 1786932000000.5}, + {value: math.NaN()}, + {value: math.Inf(1)}, + {value: math.Inf(-1)}, + {value: float64(math.MaxInt64)}, + {value: 1e300}, + {value: -1e300}, {value: json.Number("not-a-number")}, {value: "not-a-number"}, {value: true},