This is an automated email from the ASF dual-hosted git repository.

xuanwo pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/incubator-opendal.git


The following commit(s) were added to refs/heads/main by this push:
     new a8d584894 chore: Fix bindings CI (#3722)
a8d584894 is described below

commit a8d5848949aa2620e650e5e339c772aae9b16c9c
Author: Xuanwo <[email protected]>
AuthorDate: Thu Dec 7 18:00:21 2023 +0800

    chore: Fix bindings CI (#3722)
    
    * chore: Fix bindings CI
    
    Signed-off-by: Xuanwo <[email protected]>
    
    * Fix test
    
    Signed-off-by: Xuanwo <[email protected]>
    
    ---------
    
    Signed-off-by: Xuanwo <[email protected]>
---
 bindings/cpp/tests/basic_test.cpp  |  8 --------
 bindings/haskell/test/BasicTest.hs | 24 +-----------------------
 bindings/lua/test/opendal_test.lua |  4 ----
 bindings/ocaml/test/test.ml        |  9 ---------
 4 files changed, 1 insertion(+), 44 deletions(-)

diff --git a/bindings/cpp/tests/basic_test.cpp 
b/bindings/cpp/tests/basic_test.cpp
index 8d3da07f8..c6fc37a0d 100644
--- a/bindings/cpp/tests/basic_test.cpp
+++ b/bindings/cpp/tests/basic_test.cpp
@@ -68,14 +68,6 @@ TEST_F(OpendalTest, BasicTest) {
   op.create_dir(dir_path);
   EXPECT_TRUE(op.is_exist(dir_path));
 
-  // copy
-  op.copy(file_path, file_path_copied);
-  EXPECT_TRUE(op.is_exist(file_path_copied));
-
-  // rename
-  op.rename(file_path_copied, file_path_renamed);
-  EXPECT_TRUE(op.is_exist(file_path_renamed));
-
   // stat
   auto metadata = op.stat(file_path);
   EXPECT_EQ(metadata.type, opendal::EntryMode::FILE);
diff --git a/bindings/haskell/test/BasicTest.hs 
b/bindings/haskell/test/BasicTest.hs
index 77e0a6586..1311f9a05 100644
--- a/bindings/haskell/test/BasicTest.hs
+++ b/bindings/haskell/test/BasicTest.hs
@@ -46,22 +46,11 @@ testRawOperation = do
   isExistOpRaw op "key2" ?= Right True
   createDirOpRaw op "dir1/" ?= Right ()
   isExistOpRaw op "dir1/" ?= Right True
-  copyOpRaw op "key1" "key3" ?= Right ()
-  isExistOpRaw op "key1" ?= Right True
-  isExistOpRaw op "key3" ?= Right True
-  renameOpRaw op "key2" "key4" ?= Right ()
-  isExistOpRaw op "key2" ?= Right False
-  isExistOpRaw op "key4" ?= Right True
   statOpRaw op "key1" >>= \case
     Right meta -> meta @?= except_meta
     Left _ -> assertFailure "should not reach here"
   deleteOpRaw op "key1" ?= Right ()
   isExistOpRaw op "key1" ?= Right False
-  Right lister <- listOpRaw op "/"
-  liftIO $ findLister lister "key3" ?= True
-  renameOpRaw op "key3" "/dir1/key5" ?= Right ()
-  Right lister2 <- scanOpRaw op "/"
-  liftIO $ findLister lister2 "dir1/key5" ?= True
   where
     except_meta =
       Metadata
@@ -89,20 +78,9 @@ testMonad = do
       isExistOp "key2" ?= True
       createDirOp "dir1/"
       isExistOp "dir1/" ?= True
-      copyOp "key1" "key3"
-      isExistOp "key1" ?= True
-      isExistOp "key3" ?= True
-      renameOp "key2" "key4"
-      isExistOp "key2" ?= False
-      isExistOp "key4" ?= True
       statOp "key1" ?= except_meta
       deleteOp "key1"
       isExistOp "key1" ?= False
-      lister <- listOp "/"
-      liftIO $ findLister lister "key3" ?= True
-      renameOp "key3" "/dir1/key5"
-      lister2 <- scanOp "/"
-      liftIO $ findLister lister2 "dir1/key5" ?= True
     except_meta =
       Metadata
         { mMode = File,
@@ -144,4 +122,4 @@ findLister lister key = do
   case res of
     Left _ -> return False
     Right Nothing -> return False
-    Right (Just k) -> if k == key then return True else findLister lister key
\ No newline at end of file
+    Right (Just k) -> if k == key then return True else findLister lister key
diff --git a/bindings/lua/test/opendal_test.lua 
b/bindings/lua/test/opendal_test.lua
index a6007bdc1..859ddc361 100644
--- a/bindings/lua/test/opendal_test.lua
+++ b/bindings/lua/test/opendal_test.lua
@@ -66,10 +66,6 @@ describe("opendal unit test", function()
       local res, err = op:read("test.txt")
       assert.is_nil(err)
       assert.are.equal(res, "hello world")
-      assert.is_nil(op:rename("test.txt", "test_1.txt"))
-      assert.equal(op:is_exist("test_1.txt"), true)
-      assert.equal(op:is_exist("test.txt"), false)
-      assert.equal(op:is_exist("test_1.txt"), true)
       assert.is_nil(op:delete("test_1.txt"))
       assert.equal(op:is_exist("test_1.txt"), false)
     end)
diff --git a/bindings/ocaml/test/test.ml b/bindings/ocaml/test/test.ml
index 0dc96a254..26a7b04d3 100644
--- a/bindings/ocaml/test/test.ml
+++ b/bindings/ocaml/test/test.ml
@@ -50,14 +50,6 @@ let test_block_write_and_read test_ctxt =
   assert_equal "helloworld"
     (data |> Array.to_seq |> Bytes.of_seq |> Bytes.to_string)
 
-let test_copy_and_read test_ctxt =
-  let bo = new_test_block_operator test_ctxt in
-  let data = "helloworld" in
-  ignore (test_check_result (Operator.write bo "foo" (Bytes.of_string data)));
-  ignore (test_check_result (Operator.copy bo "foo" "bar"));
-  let got_res = test_check_result (Operator.read bo "bar") in
-  assert_equal data (got_res |> Array.to_seq |> Bytes.of_seq |> 
Bytes.to_string)
-
 let test_operator_reader test_ctxt =
   let bo = new_test_block_operator test_ctxt in
   ignore
@@ -105,7 +97,6 @@ let suite =
          "test_new_block_operator" >:: test_new_block_operator;
          "test_create_dir_and_remove_all" >:: test_create_dir_and_remove_all;
          "test_block_write_and_read" >:: test_block_write_and_read;
-         "test_copy_and_read" >:: test_copy_and_read;
          "test_operator_reader" >:: test_operator_reader;
          "test_operator_stat" >:: test_operator_stat;
          "test_list" >:: test_list;

Reply via email to