提交

backend/tests/test_collection_protocol_conventions.py

@@ -195,6 +195,60 @@ class TestListReceiptConfirmSortOrder(unittest.TestCase):
         )
 
 
+class TestListReceiptConfirmedSortOrder(unittest.TestCase):
+    """list_receipt_confirmed_details should sort by update_time DESC."""
+
+    @patch("app.crud.collection_protocol.BizProtocolPaymentDetail")
+    @patch("app.crud.collection_protocol.BizProtocolPayment")
+    @patch("app.crud.collection_protocol.BizCollectionProtocol")
+    def test_receipt_confirmed_sorts_by_update_time_desc(self, MockProtocol, MockPayment, MockDetail):
+        from app.crud.collection_protocol import list_receipt_confirmed_details
+
+        mock_db = MagicMock()
+        mock_query = MagicMock()
+        mock_query.join.return_value = mock_query
+        mock_query.filter.return_value = mock_query
+        mock_query.order_by.return_value = mock_query
+        mock_query.with_entities.return_value = mock_query
+        mock_query.scalar.return_value = 0
+        mock_query.offset.return_value = mock_query
+        mock_query.limit.return_value = mock_query
+        mock_query.all.return_value = []
+        mock_db.query.return_value = mock_query
+
+        list_receipt_confirmed_details(mock_db, tenant_no="t1")
+
+        order_by_calls = mock_query.order_by.call_args_list
+        self.assertGreaterEqual(len(order_by_calls), 1)
+
+
+class TestListRefundConfirmedSortOrder(unittest.TestCase):
+    """list_refund_confirmed_details should sort by update_time DESC."""
+
+    @patch("app.crud.collection_protocol.BizProtocolPaymentDetail")
+    @patch("app.crud.collection_protocol.BizProtocolPayment")
+    @patch("app.crud.collection_protocol.BizCollectionProtocol")
+    def test_refund_confirmed_sorts_by_update_time_desc(self, MockProtocol, MockPayment, MockDetail):
+        from app.crud.collection_protocol import list_refund_confirmed_details
+
+        mock_db = MagicMock()
+        mock_query = MagicMock()
+        mock_query.join.return_value = mock_query
+        mock_query.filter.return_value = mock_query
+        mock_query.order_by.return_value = mock_query
+        mock_query.with_entities.return_value = mock_query
+        mock_query.scalar.return_value = 0
+        mock_query.offset.return_value = mock_query
+        mock_query.limit.return_value = mock_query
+        mock_query.all.return_value = []
+        mock_db.query.return_value = mock_query
+
+        list_refund_confirmed_details(mock_db, tenant_no="t1")
+
+        order_by_calls = mock_query.order_by.call_args_list
+        self.assertGreaterEqual(len(order_by_calls), 1)
+
+
 # ---------------------------------------------------------------------------
 # M1: Sort order — list_refund_confirm_details should include order_num ASC
 # ---------------------------------------------------------------------------