| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260 |
- """OA list_instance_history:按 appId + 全员权限收窄。"""
- from __future__ import annotations
- import unittest
- from datetime import datetime
- from sqlalchemy import create_engine
- from sqlalchemy.orm import sessionmaker
- from app.models.base import Base
- from app.models.models import OaApp, OaAppVersion, OaInstance, OaInstanceEvent, OaTask, SysUser
- from app.services import oa_engine
- class TestOaInstanceHistory(unittest.TestCase):
- def setUp(self):
- self.engine = create_engine("sqlite:///:memory:")
- Base.metadata.create_all(
- self.engine,
- tables=[
- OaApp.__table__,
- OaAppVersion.__table__,
- SysUser.__table__,
- OaInstance.__table__,
- OaTask.__table__,
- OaInstanceEvent.__table__,
- ],
- )
- Session = sessionmaker(bind=self.engine, autoflush=False, autocommit=False)
- self.db = Session()
- t1 = datetime(2026, 8, 10, 9, 0, 0)
- t2 = datetime(2026, 8, 20, 15, 30, 0)
- t3 = datetime(2026, 8, 25, 11, 0, 0)
- self.db.add_all(
- [
- OaApp(
- id="app-leave",
- app_name="请假申请",
- initiator_scope="all",
- admin_scope="all_oa_admin",
- status="1",
- tenant_no="zkxw",
- create_dept_id="",
- create_time=t1,
- update_time=t1,
- del_flag=0,
- ),
- OaAppVersion(
- id="v-leave",
- app_id="app-leave",
- version_no=1,
- status="published",
- is_latest_published="1",
- form_schema_json='{"rule":[{"type":"input","field":"leaveType","title":"请假类型"},{"type":"input","field":"days","title":"天数"},{"type":"upload","field":"voucher","title":"付款凭证"}]}',
- flow_tree_json="{}",
- advanced_settings_json="{}",
- tenant_no="zkxw",
- create_dept_id="",
- create_time=t1,
- update_time=t1,
- del_flag=0,
- ),
- SysUser(
- id="u-a",
- user_no="a",
- nick_name="张三",
- password="x",
- tenant_no="zkxw",
- create_dept_id="",
- create_time=t1,
- update_time=t1,
- del_flag=0,
- ),
- SysUser(
- id="u-b",
- user_no="b",
- nick_name="李四",
- password="x",
- tenant_no="zkxw",
- create_dept_id="",
- create_time=t1,
- update_time=t1,
- del_flag=0,
- ),
- OaInstance(
- id="inst-alice-done",
- app_id="app-leave",
- app_version_id="v-leave",
- title="请假-张三",
- initiator_user_id="u-a",
- form_data_json='{"leaveType":"年假","days":"2","voucher":"f33a569a9dd64f2c853899498cd47498.png"}',
- status="completed",
- tenant_no="zkxw",
- create_dept_id="",
- create_time=t1,
- update_time=t2,
- del_flag=0,
- ),
- OaInstance(
- id="inst-bob-run",
- app_id="app-leave",
- app_version_id="v-leave",
- title="请假-李四",
- initiator_user_id="u-b",
- form_data_json='{"leaveType":"事假","days":"1"}',
- status="running",
- tenant_no="zkxw",
- create_dept_id="",
- create_time=t3,
- update_time=t3,
- del_flag=0,
- ),
- ]
- )
- self.db.commit()
- def tearDown(self):
- self.db.close()
- self.engine.dispose()
- def test_requires_app_id(self):
- with self.assertRaises(oa_engine.OaEngineError):
- oa_engine.list_instance_history(
- self.db,
- tenant_no="zkxw",
- user_id="u-a",
- can_view_all=False,
- app_id="",
- )
- def test_self_scope_hides_others(self):
- data = oa_engine.list_instance_history(
- self.db,
- tenant_no="zkxw",
- user_id="u-a",
- can_view_all=False,
- app_id="app-leave",
- )
- self.assertEqual(data["total"], 1)
- self.assertFalse(data["canViewAll"])
- self.assertEqual(data["list"][0]["id"], "inst-alice-done")
- def test_all_scope_returns_both(self):
- data = oa_engine.list_instance_history(
- self.db,
- tenant_no="zkxw",
- user_id="u-a",
- can_view_all=True,
- app_id="app-leave",
- )
- self.assertEqual(data["total"], 2)
- self.assertTrue(data["canViewAll"])
- ids = {r["id"] for r in data["list"]}
- self.assertEqual(ids, {"inst-alice-done", "inst-bob-run"})
- def test_initiator_name_ignored_without_all(self):
- data = oa_engine.list_instance_history(
- self.db,
- tenant_no="zkxw",
- user_id="u-a",
- can_view_all=False,
- app_id="app-leave",
- initiator_name="李四",
- )
- self.assertEqual(data["total"], 1)
- self.assertEqual(data["list"][0]["id"], "inst-alice-done")
- def test_finish_time_only_on_terminal(self):
- data = oa_engine.list_instance_history(
- self.db,
- tenant_no="zkxw",
- user_id="u-a",
- can_view_all=True,
- app_id="app-leave",
- )
- by_id = {r["id"]: r for r in data["list"]}
- self.assertTrue(by_id["inst-alice-done"].get("finishTime"))
- self.assertFalse(by_id["inst-bob-run"].get("finishTime"))
- def test_finish_time_filter(self):
- data = oa_engine.list_instance_history(
- self.db,
- tenant_no="zkxw",
- user_id="u-a",
- can_view_all=True,
- app_id="app-leave",
- finish_time_from="2026-08-20",
- finish_time_to="2026-08-20",
- )
- self.assertEqual(data["total"], 1)
- self.assertEqual(data["list"][0]["id"], "inst-alice-done")
- def test_preview_fields_limit_six(self):
- data = oa_engine.list_instance_history(
- self.db,
- tenant_no="zkxw",
- user_id="u-a",
- can_view_all=False,
- app_id="app-leave",
- )
- pf = data["list"][0].get("previewFields") or []
- self.assertIsInstance(pf, list)
- self.assertLessEqual(len(pf), 6)
- if pf:
- self.assertGreaterEqual(len(pf), 1)
- def test_preview_fields_keep_upload_files(self):
- data = oa_engine.list_instance_history(
- self.db,
- tenant_no="zkxw",
- user_id="u-a",
- can_view_all=False,
- app_id="app-leave",
- )
- pf = data["list"][0].get("previewFields") or []
- voucher = next((f for f in pf if f.get("label") == "付款凭证"), None)
- self.assertIsNotNone(voucher)
- self.assertEqual(voucher.get("fieldType"), "upload")
- self.assertEqual(
- voucher.get("files"),
- [
- {
- "url": "f33a569a9dd64f2c853899498cd47498.png",
- "name": "f33a569a9dd64f2c853899498cd47498.png",
- }
- ],
- )
- def test_export_history_rows(self):
- data = oa_engine.export_instance_history(
- self.db,
- tenant_no="zkxw",
- user_id="u-a",
- can_view_all=False,
- app_id="app-leave",
- )
- self.assertEqual(data["total"], 1)
- self.assertGreaterEqual(len(data["businessColumns"]), 1)
- self.assertEqual(len(data["fixedColumns"]), 13)
- row = data["rows"][0]
- self.assertEqual(row["id"], "inst-alice-done")
- self.assertEqual(row["business"].get("leaveType"), "年假")
- self.assertEqual(row["fixed"].get("initiatorName"), "张三")
- self.assertIn("approvalRecord", row["fixed"])
- def test_export_selected_ids_only(self):
- data = oa_engine.export_instance_history(
- self.db,
- tenant_no="zkxw",
- user_id="u-a",
- can_view_all=True,
- app_id="app-leave",
- instance_ids=["inst-bob-run"],
- )
- self.assertEqual(data["total"], 1)
- self.assertEqual(data["rows"][0]["id"], "inst-bob-run")
- if __name__ == "__main__":
- unittest.main()
|