fix: list API compatibility issues with old versions (#4248)

infinyon · Nov 11, 2024 · 2f31d45 · 2f31d45
1 parent f86fe12
commit 2f31d45
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 0 deletions.
diff --git a/crates/fluvio-sc-schema/src/objects/list.rs b/crates/fluvio-sc-schema/src/objects/list.rs
@@ -108,6 +108,7 @@ where
 
 impl Request for ObjectApiListRequest {
     const API_KEY: u16 = AdminPublicApiKey::List as u16;
+    const MIN_API_VERSION: i16 = 15;
     const DEFAULT_API_VERSION: i16 = COMMON_VERSION;
     type Response = ObjectApiListResponse;
 }

diff --git a/tests/cli/cli-platform-cross-version.bats b/tests/cli/cli-platform-cross-version.bats
@@ -80,6 +80,24 @@ teardown_file() {
     assert_success
 }
 
+# List topics
+@test "List topics" {
+    run timeout 15s "$FLUVIO_CLIENT_BIN" topic list
+
+    assert_success
+    assert_line --partial --index 1 "$TOPIC_NAME"
+    assert [ ${#lines[@]} -eq 2 ]
+}
+
+# List partitions
+@test "List partitions" {
+    run timeout 15s "$FLUVIO_CLIENT_BIN" partition list
+
+    assert_success
+    assert_line --partial --index 1 "$TOPIC_NAME"
+    assert [ ${#lines[@]} -eq 2 ]
+}
+
 # Consume message and compare message
 # Warning: Adding anything extra to the `debug_msg` skews the message comparison
 @test "Consume message" {