refactor(ai-proxy): simplify complex ternary

  operators with if-else statements

  - Replace complicated nested ternary operators in TEST 17 with
  clear if-else blocks
  - Improve code readability and maintainability in
  t/plugin/ai-proxy.t
  - Reindex test file for proper formatting
  - All linting checks pass

Signed-off-by: Orician <[email protected]>

Author: yixianOu

t/plugin/ai-proxy-multi.t

@@ -128,47 +128,6 @@ run_tests();
 
 __DATA__
 
-=== TEST 0: schema accepts 'logging' and ignores unknown 'logging_schema'
---- config
-    location /t {
-        content_by_lua_block {
-            local plugin = require("apisix.plugins.ai-proxy-multi")
-
-            local ok1, err1 = plugin.check_schema({
-                instances = {
-                    {
-                        name = "openai-1",
-                        provider = "openai",
-                        weight = 1,
-                        auth = { header = { apikey = "token" } },
-                        options = { model = "gpt-4" },
-                    },
-                },
-                logging = { summaries = true },
-            })
-
-            local ok2, err2 = plugin.check_schema({
-                instances = {
-                    {
-                        name = "openai-1",
-                        provider = "openai",
-                        weight = 1,
-                        auth = { header = { apikey = "token" } },
-                        options = { model = "gpt-4" },
-                    },
-                },
-                logging_schema = { summaries = true },
-            })
-
-            -- Unknown field 'logging_schema' should be ignored if additionalProperties are allowed
-            ngx.say((ok1 and "ok" or ("bad:" .. (err1 or ""))), ":", (ok2 and "ok" or "invalid"))
-        }
-    }
---- request
-GET /t
---- response_body
-ok:ok
-
 === TEST 1: minimal viable configuration
 --- config
     location /t {
@@ -694,3 +653,46 @@ qr/6data: \[DONE\]\n\n/
 --- error_code: 400
 --- response_body eval
 qr/.invalid endpoint.*/
+
+
+
+=== TEST 16: schema accepts 'logging' and ignores unknown 'logging_schema'
+--- config
+    location /t {
+        content_by_lua_block {
+            local plugin = require("apisix.plugins.ai-proxy-multi")
+
+            local ok1, err1 = plugin.check_schema({
+                instances = {
+                    {
+                        name = "openai-1",
+                        provider = "openai",
+                        weight = 1,
+                        auth = { header = { apikey = "token" } },
+                        options = { model = "gpt-4" },
+                    },
+                },
+                logging = { summaries = true },
+            })
+
+            local ok2, err2 = plugin.check_schema({
+                instances = {
+                    {
+                        name = "openai-1",
+                        provider = "openai",
+                        weight = 1,
+                        auth = { header = { apikey = "token" } },
+                        options = { model = "gpt-4" },
+                    },
+                },
+                logging_schema = { summaries = true },
+            })
+
+            -- Unknown field 'logging_schema' should be ignored if additionalProperties are allowed
+            ngx.say((ok1 and "ok" or ("bad:" .. (err1 or ""))), ":", (ok2 and "ok" or "invalid"))
+        }
+    }
+--- request
+GET /t
+--- response_body
+ok:ok

t/plugin/ai-proxy.t

@@ -181,36 +181,6 @@ run_tests();
 
 __DATA__
 
-=== TEST 0: schema accepts 'logging' and ignores unknown 'logging_schema'
---- config
-    location /t {
-        content_by_lua_block {
-            local plugin = require("apisix.plugins.ai-proxy")
-
-            local ok1, err1 = plugin.check_schema({
-                provider = "openai",
-                auth = { header = { apikey = "token" } },
-                options = { model = "gpt-4" },
-                logging = { summaries = true, payloads = false },
-            })
-
-            local ok2, err2 = plugin.check_schema({
-                provider = "openai",
-                auth = { header = { apikey = "token" } },
-                options = { model = "gpt-4" },
-                logging_schema = { summaries = true },
-            })
-
-            -- APISIX schema allows additional properties unless explicitly disallowed.
-            -- So unknown field 'logging_schema' should be ignored and still pass.
-            ngx.say((ok1 and "ok" or ("bad:" .. (err1 or ""))), ":", (ok2 and "ok" or "invalid"))
-        }
-    }
---- request
-GET /t
---- response_body
-ok:ok
-
 === TEST 1: minimal viable configuration
 --- config
     location /t {
@@ -701,3 +671,49 @@ POST /embeddings
 --- error_code: 200
 --- response_body_like eval
 qr/.*text-embedding-ada-002*/
+
+
+
+=== TEST 17: schema accepts 'logging' and ignores unknown 'logging_schema'
+--- config
+    location /t {
+        content_by_lua_block {
+            local plugin = require("apisix.plugins.ai-proxy")
+
+            local ok1, err1 = plugin.check_schema({
+                provider = "openai",
+                auth = { header = { apikey = "token" } },
+                options = { model = "gpt-4" },
+                logging = { summaries = true, payloads = false },
+            })
+
+            local ok2, err2 = plugin.check_schema({
+                provider = "openai",
+                auth = { header = { apikey = "token" } },
+                options = { model = "gpt-4" },
+                logging_schema = { summaries = true },
+            })
+
+            -- APISIX schema allows additional properties unless explicitly disallowed.
+            -- So unknown field 'logging_schema' should be ignored and still pass.
+            local msg1
+            if ok1 then
+                msg1 = "ok"
+            else
+                msg1 = "bad:" .. (err1 or "")
+            end
+
+            local msg2
+            if ok2 then
+                msg2 = "ok"
+            else
+                msg2 = "invalid"
+            end
+
+            ngx.say(msg1, ":", msg2)
+        }
+    }
+--- request
+GET /t
+--- response_body
+ok:ok