From 57b9f8cf52e96b74a279531a0fc64d2615a281a0 Mon Sep 17 00:00:00 2001 From: Liu An Date: Tue, 12 Aug 2025 10:57:30 +0800 Subject: [PATCH] Fix: Update test assertions and simplify test cases (#9400) ### What problem does this PR solve? - Fix error message assertion in test_update_chunk.py to match new ownership validation - Simplify dataset listing test cases by removing lambda assertions for sorting - Fix actions: https://github.com/infiniflow/ragflow/actions/runs/16885465524/job/47831942553 ### Type of change - [x] Fix test cases --- .../test_update_chunk.py | 2 +- .../test_dataset_mangement/test_list_datasets.py | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/test/testcases/test_http_api/test_chunk_management_within_dataset/test_update_chunk.py b/test/testcases/test_http_api/test_chunk_management_within_dataset/test_update_chunk.py index d6d0278fe..1281df433 100644 --- a/test/testcases/test_http_api/test_chunk_management_within_dataset/test_update_chunk.py +++ b/test/testcases/test_http_api/test_chunk_management_within_dataset/test_update_chunk.py @@ -245,4 +245,4 @@ class TestUpdatedChunk: delete_documents(HttpApiAuth, dataset_id, {"ids": [document_id]}) res = update_chunk(HttpApiAuth, dataset_id, document_id, chunk_ids[0]) assert res["code"] == 102 - assert res["message"] == f"Can't find this chunk {chunk_ids[0]}" + assert res["message"] == f"You don't own the document {document_id}." diff --git a/test/testcases/test_http_api/test_dataset_mangement/test_list_datasets.py b/test/testcases/test_http_api/test_dataset_mangement/test_list_datasets.py index 96357bfd2..ae924d842 100644 --- a/test/testcases/test_http_api/test_dataset_mangement/test_list_datasets.py +++ b/test/testcases/test_http_api/test_dataset_mangement/test_list_datasets.py @@ -163,9 +163,9 @@ class TestDatasetsList: [ {"orderby": ""}, {"orderby": "unknown"}, - ({"orderby": "CREATE_TIME"}, lambda r: (is_sorted(r["data"], "create_time", True))), - ({"orderby": "UPDATE_TIME"}, lambda r: (is_sorted(r["data"], "update_time", True))), - ({"orderby": " create_time "}, lambda r: (is_sorted(r["data"], "update_time", True))), + {"orderby": "CREATE_TIME"}, + {"orderby": "UPDATE_TIME"}, + {"orderby": " create_time "}, ], ids=["empty", "unknown", "orderby_create_time_upper", "orderby_update_time_upper", "whitespace"], )