From 3ef9e6ed407d8382e906055d289abbe6cc81da7a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Viktor=20S=C3=B6derqvist?= Date: Sun, 12 Jan 2025 17:25:41 +0100 Subject: [PATCH 1/2] Test coverage for ECHO for reply schema validation MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Viktor Söderqvist --- tests/unit/other.tcl | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/tests/unit/other.tcl b/tests/unit/other.tcl index bb08c67471..225287aa3a 100644 --- a/tests/unit/other.tcl +++ b/tests/unit/other.tcl @@ -30,6 +30,10 @@ start_server {tags {"other"}} { } } + test {Coverage: ECHO} { + assert_equal bang [r ECHO bang] + } + test {SAVE - make sure there are all the types as values} { # Wait for a background saving in progress to terminate waitForBgsave r From 2d130e05ad8c26d15eb2beb1ccbecfc83ffaf888 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Viktor=20S=C3=B6derqvist?= Date: Sun, 12 Jan 2025 22:26:47 +0100 Subject: [PATCH 2/2] Temporarily include reply-schema-validator job in run-extra-tests label MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Viktor Söderqvist --- .github/workflows/daily.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/daily.yml b/.github/workflows/daily.yml index e1d577b51b..c0e7e4b446 100644 --- a/.github/workflows/daily.yml +++ b/.github/workflows/daily.yml @@ -1220,7 +1220,7 @@ jobs: if: | (github.event_name == 'workflow_dispatch' || (github.event_name == 'schedule' && github.repository == 'valkey-io/valkey') || - (github.event_name == 'pull_request' && github.event.pull_request.base.ref != 'unstable')) && + (github.event_name == 'pull_request' && (contains(github.event.pull_request.labels.*.name, 'run-extra-tests') || github.event.pull_request.base.ref != 'unstable'))) && !contains(github.event.inputs.skipjobs, 'reply-schema') steps: - name: prep