@@ -104,7 +100,6 @@
-
diff --git a/tests/frontend/chat.test.cjs b/tests/frontend/chat.test.cjs
index 367d8c0..7df3fcd 100644
--- a/tests/frontend/chat.test.cjs
+++ b/tests/frontend/chat.test.cjs
@@ -116,7 +116,6 @@ async function customerPage(pageOptions = {}){
const data = url === '/api/new_chat' ? {id:nextConvId++, welcome:'welcome'}
: url === '/api/visitor/session' ? {ok:true,key:'1'.repeat(64)}
: url === '/api/customer/session' ? {ok:true,authenticated:Boolean(customerState),user:customerState}
- : url === '/api/customer/session?overview=1' ? {ok:true,authenticated:Boolean(customerState),user:customerState,overview:pageOptions.overview || {total:0,processing:0,exceptions:0,stages:[],orders:[]}}
: url === '/api/visitor/human/request' ? (pageOptions.serviceClosed ? {ok:false,closed:true} : {ok:true, session_id:7})
: url.startsWith('/api/visitor/human/poll') ? state
: ['/api/rate', '/api/feedback', '/api/visitor/human/send', '/api/visitor/human/close', '/api/ticket'].includes(url) || url.startsWith('/api/del_history') ? {ok:true}
@@ -124,7 +123,7 @@ async function customerPage(pageOptions = {}){
return {ok:true, json:async () => data};
},
});
- for(const file of ['support.js', 'visitor.js', 'customer.js', 'overview.js', 'order_lookup.js', 'drafts.js', 'voice.js', 'chat.js']){
+ for(const file of ['support.js', 'visitor.js', 'customer.js', 'order_lookup.js', 'drafts.js', 'voice.js', 'chat.js']){
vm.runInContext(fs.readFileSync(path.join(__dirname, '../../static/js', file), 'utf8'), context, {filename:file});
if(file === 'voice.js'){
const createVoice=context.ChatVoice.create;
@@ -221,53 +220,16 @@ test('the homepage and exception shortcut match the US service in both languages
}
});
-test('authenticated entry loads one overview with actionable counts and safe exception text', async () => {
- const overview={total:9,processing:3,exceptions:2,stages:[{code:'IN_TRANSIT',count:3}],orders:[
- {order_no:'ORDER20260001',inbound_code:'INB20260001',stage:'IN_TRANSIT',exception_reason:'
'}
- ]};
- const app=await customerPage({human:false,customer:{id:1,name:'admin'},overview});
- const panel=app.element('customer-overview');
- assert.equal(panel.hidden,false);assert.equal(panel.open,true);
- assert.match(app.element('customer-overview-summary').textContent,/履约中 3 · 异常订单 2/);
- assert.equal(app.calls.filter(call=>call.url==='/api/customer/session?overview=1').length,1);
- const body=app.element('customer-overview-body');
- assert.equal(body.querySelector('.overview-reason').innerHTML,'');
- assert.equal(body.querySelector('.overview-reason').textContent,'');
- await app.click(body.querySelector('.overview-order'));
- assert.equal(app.element('order-lookup').hidden,false);
- assert.equal(app.element('order-reference').value,'INB20260001');
- await app.click(app.element('order-lookup-cancel'));
- await app.click(body.querySelectorAll('.overview-metric')[1]);
- assert.equal(app.calls.find(call=>call.url==='/stream_chat').body.msg,'列出履约中的订单');
- assert.equal(panel.open,false);
-});
-
-test('guest entry skips private overview and a late refresh cannot restore it after logout', async () => {
- const guest=await customerPage({human:false});
- assert.equal(guest.element('customer-overview').hidden,true);
- assert.equal(guest.calls.filter(call=>call.url==='/api/customer/session?overview=1').length,0);
+test('login keeps the chat layout and an overview is requested only by a chat question', async () => {
const app=await customerPage({human:false,customer:{id:1,name:'admin'}});
- let resolve;
- app.hooks['/api/customer/session?overview=1']=()=>new Promise(done=>{resolve=done;});
- app.element('customer-overview-body').querySelector('.overview-refresh').click();
- app.hooks['/api/customer/logout']=()=>response({ok:true,authenticated:false,user:null});
- await app.click(app.element('customer-account'));
- resolve(response({ok:true,authenticated:true,user:{id:1,name:'admin'},overview:{total:99,processing:99,exceptions:99,stages:[],orders:[]}}));
+ assert.equal(app.calls.filter(call=>call.url.includes('overview=1') || call.url==='/stream_chat').length,0);
+ const template=fs.readFileSync(path.join(__dirname,'../../templates/index.html'),'utf8');
+ assert.doesNotMatch(template,/customer-overview|js\/overview\.js/);
+ await app.context.window.fillInput('我的订单概览');
await flush();
- assert.equal(app.element('customer-overview').hidden,true);
- assert.equal(app.element('customer-overview-body').textContent,'');
- assert.doesNotMatch(app.element('customer-overview-summary').textContent,/99/);
-});
-
-test('overview failure can retry and account data stays out of its error state', async () => {
- const app=await customerPage({human:false,customer:{id:1,name:'admin'}});
- app.hooks['/api/customer/session?overview=1']=()=>response({ok:false},503);
- await app.element('customer-overview-body').querySelector('.overview-refresh').click();
- assert.match(app.element('customer-overview-summary').textContent,/暂时无法加载/);
- assert.equal(app.element('customer-overview-body').querySelector('.overview-metric'),null);
- delete app.hooks['/api/customer/session?overview=1'];
- await app.element('customer-overview-body').querySelector('.overview-refresh').click();
- assert.match(app.element('customer-overview-summary').textContent,/履约中 0/);
+ assert.equal(app.calls.filter(call=>call.url==='/stream_chat').length,1);
+ assert.equal(app.calls.find(call=>call.url==='/stream_chat').body.msg,'我的订单概览');
+ assert.match(app.element('chatDom').textContent,/测试 AI 回复/);
});
test('voice errors use a floating notice without changing the composer status', async () => {
diff --git a/tests/test_order_statistics.py b/tests/test_order_statistics.py
index 4161e2d..68fa9d5 100644
--- a/tests/test_order_statistics.py
+++ b/tests/test_order_statistics.py
@@ -57,25 +57,28 @@ class OrderStatisticsTests(unittest.TestCase):
self.assertTrue(self.ask(q)['login_required'])
read.assert_not_called()
- def test_session_overview_is_opt_in_scoped_prioritized_and_not_cached(self):
+ def test_overview_is_requested_in_chat_and_saved_in_customer_protected_history(self):
with patch('logistics_chat.order_listing.read_overview') as read:
- guest=self.client.get('/api/customer/session?overview=1&tenant_id=b',headers={'X-Tenant-Id':'b'})
- self.assertFalse(guest.json['authenticated'])
- self.assertNotIn('overview',guest.json)
+ for question in ('我的订单概览','查看订单情况','Order overview'):
+ self.assertTrue(self.ask(question)['login_required'])
read.assert_not_called()
self.login()
- self.assertNotIn('overview',self.client.get('/api/customer/session').json)
- response=self.client.get('/api/customer/session?overview=1&tenant_id=b',headers={'X-Tenant-Id':'b'})
- overview=response.json['overview']
+ with patch('logistics_chat.order_listing.read_overview') as read:
+ self.assertNotIn('overview',self.client.get('/api/customer/session?overview=1').json)
+ read.assert_not_called()
+ cid=self.client.post('/api/new_chat').json['id']
+ text=self.ask('我的订单概览',cid)['t']
+ self.assertIn('共有 16 票订单，履约中 12 票，异常订单 7 票',text)
+ self.assertIn('船舶已到目的港：10票',text)
+ self.assertEqual(text.count('- 入仓码：'),5)
+ self.assertEqual(text.count('异常原因：'),5)
+ response=self.client.get('/api/get_messages',query_string={'conv_id':cid})
self.assertIn('no-store',response.headers['Cache-Control'])
- self.assertEqual((overview['total'],overview['processing'],overview['exceptions']),(16,12,7))
- self.assertEqual(sum(stage['count'] for stage in overview['stages']),12)
- self.assertEqual(len(overview['orders']),5)
- self.assertTrue(all(order['exception_reason'] for order in overview['orders']))
- self.assertNotIn('SECRET',response.get_data(as_text=True))
- self.assertNotIn('tenant_id',response.get_data(as_text=True))
+ self.assertTrue(any(row['role']=='assistant' and row['content']==text for row in response.json))
self.client.post('/api/customer/logout')
- self.assertNotIn('overview',self.client.get('/api/customer/session?overview=1').json)
+ self.assertEqual(self.client.get('/api/get_messages',query_string={'conv_id':cid}).json,[])
+ self.login()
+ self.assertTrue(any(row['content']==text for row in self.client.get('/api/get_messages',query_string={'conv_id':cid}).json))
def test_overview_keeps_old_exception_ahead_of_new_processing_orders_and_refreshes(self):
self.login()
@@ -83,23 +86,24 @@ class OrderStatisticsTests(unittest.TestCase):
c=db.connect()
c.execute("UPDATE order_test_data SET exception_reason=NULL WHERE tenant_id='a' AND order_no<>'STAT20260000'")
c.commit();c.close()
- overview=self.client.get('/api/customer/session?overview=1').json['overview']
+ from logistics_chat.order_listing import read_overview
+ with self.app.app_context():overview=read_overview(('a',))
self.assertEqual(overview['orders'][0]['order_no'],'STAT20260000')
self.assertEqual(len({order['order_no'] for order in overview['orders']}),5)
with self.app.app_context():
c=db.connect()
order_data.import_records(c,[{'tenant_id':'a','order_no':'NEW20260001','status':'PROCESSING','exception_reason':'资料待核实'}])
c.commit();c.close()
- refreshed=self.client.get('/api/customer/session?overview=1').json['overview']
- self.assertEqual((refreshed['total'],refreshed['processing'],refreshed['exceptions']),(17,13,2))
+ refreshed=self.ask('订单概览')['t']
+ self.assertIn('共有 17 票订单，履约中 13 票，异常订单 2 票',refreshed)
def test_overview_read_failure_keeps_login_available_and_no_facts_leak(self):
self.login()
with patch('logistics_chat.order_listing.read_overview',side_effect=sqlite3.OperationalError('private details')):
- result=self.client.get('/api/customer/session?overview=1').json
- self.assertTrue(result['authenticated'])
- self.assertEqual(result['overview_error'],'unavailable')
- self.assertNotIn('overview',result)
+ result=self.ask('订单概览')
+ self.assertTrue(self.client.get('/api/customer/session').json['authenticated'])
+ self.assertIn('暂时无法读取',result['t'])
+ self.assertNotIn('共有',result['t'])
self.assertNotIn('private details',str(result))
def test_summaries_show_at_most_five_recent_rows_and_keep_intersecting_filters(self):