From d5c8377eced5711675e7bdb4b599d29ca6df2cf2 Mon Sep 17 00:00:00 2001 From: wangyidong Date: Thu, 12 Jun 2025 11:05:09 +0800 Subject: [PATCH] * [unittest] Fix upgrade processoldmetrics model method test. --- module/upgrade/test/model/processoldmetrics.php | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/module/upgrade/test/model/processoldmetrics.php b/module/upgrade/test/model/processoldmetrics.php index 7fac9f342e..1f6a82445c 100644 --- a/module/upgrade/test/model/processoldmetrics.php +++ b/module/upgrade/test/model/processoldmetrics.php @@ -30,7 +30,8 @@ $result = $upgrade->processOldMetrics(); $oldMetrics = $tester->dao->select('*')->from(TABLE_BASICMEAS)->where('deleted')->eq('0')->orderBy('order_asc')->fetchAll(); $check = true; -foreach($oldMetrics as $oldMetric) { +foreach($oldMetrics as $oldMetric) +{ $metric = $tester->dao->select('*')->from(TABLE_METRIC)->where('fromID')->eq($oldMetric->id)->fetch(); if(!$metric) { @@ -38,19 +39,19 @@ foreach($oldMetrics as $oldMetric) { break; } - if($metric->scope != ($scopeMap[$oldMetric->scope] ? $scopeMap[$oldMetric->scope] : 'other')) + if($metric->scope != zget($scopeMap, $oldMetric->scope, 'other')) { $check = false; break; } - if($metric->purpose != ($purposeMap[$oldMetric->purpose] ? $purposeMap[$oldMetric->purpose] : 'other')) + if($metric->purpose != zget($purposeMap, $oldMetric->purpose, 'other')) { $check = false; break; } - if($metric->object != ($objectMap[$oldMetric->object] ? $objectMap[$oldMetric->object] : 'other')) + if($metric->object != zget($objectMap, $oldMetric->object, 'other')) { $check = false; break;