fix(ci): fix test code that caused ci freeze (#69)

Fixes an issue in the test code that used a closed channel unintentionally.
JuliaComputing · Mar 15, 2024 · 48a9fff · 48a9fff
1 parent 1916168
commit 48a9fff
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 2 deletions.
diff --git a/test/client/openapigenerator_petstore_v3/petstore_test_storeapi.jl b/test/client/openapigenerator_petstore_v3/petstore_test_storeapi.jl
@@ -53,6 +53,9 @@ function test(uri)
     # a closed channel is equivalent of cancellation of the call,
     # no error should be thrown, but response can be nothing if call was interrupted immediately
     @test !isopen(response_channel)
+
+    # open a new channel to use
+    response_channel = Channel{Order}(1)
     try
         resp, http_resp = get_order_by_id(api, response_channel, Int64(5))
         @test (200 <= http_resp.status <= 206)

diff --git a/test/client/petstore_v2/petstore_test_storeapi.jl b/test/client/petstore_v2/petstore_test_storeapi.jl
@@ -53,6 +53,9 @@ function test(uri)
     # a closed channel is equivalent of cancellation of the call,
     # no error should be thrown, but response can be nothing if call was interrupted immediately
     @test !isopen(response_channel)
+
+    # open a new channel to use
+    response_channel = Channel{Order}(1)
     try
         resp, http_resp = get_order_by_id(api, response_channel, Int64(5))
         @test (200 <= http_resp.status <= 206)

diff --git a/test/client/petstore_v3/petstore_test_storeapi.jl b/test/client/petstore_v3/petstore_test_storeapi.jl
@@ -53,6 +53,9 @@ function test(uri)
     # a closed channel is equivalent of cancellation of the call,
     # no error should be thrown, but response can be nothing if call was interrupted immediately
     @test !isopen(response_channel)
+
+    # open a new channel to use
+    response_channel = Channel{Order}(1)
     try
         resp, http_resp = get_order_by_id(api, response_channel, Int64(5))
         @test (200 <= http_resp.status <= 206)

diff --git a/test/client/runtests.jl b/test/client/runtests.jl
@@ -30,8 +30,8 @@ function runtests(; skip_petstore=false, test_file_upload=false)
                         PetStoreV3Tests.runtests(; test_file_upload=test_file_upload)
                     end
                     @testset "V2" begin
-                    @info("Running petstore v2 tests")
-                    PetStoreV2Tests.runtests()
+                        @info("Running petstore v2 tests")
+                        PetStoreV2Tests.runtests()
                     end
                 else
                     @info("Skipping petstore tests in non Linux environment (can not run petstore docker on OSX or Windows)")