Add check for charge_full or charge_now >> charge_full_design.

client/common_lib/site_power_status.py

 import glob, logging, os, re, time
 from autotest_lib.client.bin import utils as bin_utils
 from autotest_lib.client.common_lib import error, utils
 
 
 class DevStat(object):
     """
     Device power status. This class implements generic status initialization
     and parsing routines.
     """
@@ skipping 72 matching lines @@
     def update(self):
         self.read_all_vals()
 
         self.charge_full = self.charge_full / 1000000
         self.charge_full_design = self.charge_full_design / 1000000
         self.charge_now = self.charge_now / 1000000
         self.current_now = self.current_now / 1000000
         self.voltage_min_design = self.voltage_min_design / 1000000
         self.voltage_now = self.voltage_now / 1000000
 
+        if self.charge_full > (self.charge_full_design * 1.5):
+            raise error.TestError('Unreasonable charge_full value')
+        if self.charge_now > (self.charge_full_design * 1.5):
+            raise error.TestError('Unreasonable charge_now value')
         self.energy =  self.voltage_now * self.charge_now
         self.energy_full = self.voltage_now * self.charge_full
         self.energy_full_design = self.voltage_now * self.charge_full_design
         self.energy_rate =  self.voltage_now * self.current_now
 
         self.remaining_time = 0
         if self.current_now:
             self.remaining_time =  self.energy / self.energy_rate
 
 
@@ skipping 282 matching lines @@
 
             active = int(utils.read_file(active_duration_path))
             connected = int(utils.read_file(connected_duration_path))
             logging.debug('device %s active for %.2f%%',
                           path, active * 100.0 / connected)
 
             total_active += active
             total_connected += connected
 
         return total_active, total_connected