diff options
author | Jack Rosenthal <jrosenth@chromium.org> | 2021-11-04 12:11:58 -0600 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2021-11-05 04:22:34 +0000 |
commit | 252457d4b21f46889eebad61d4c0a65331919cec (patch) | |
tree | 01856c4d31d710b20e85a74c8d7b5836e35c3b98 /test/battery_get_params_smart.c | |
parent | 08f5a1e6fc2c9467230444ac9b582dcf4d9f0068 (diff) | |
download | chrome-ec-stabilize-14469.8.B-ish.tar.gz |
ish: Trim down the release branchstabilize-wristpin-14469.59.B-ishstabilize-voshyr-14637.B-ishstabilize-quickfix-14695.187.B-ishstabilize-quickfix-14695.124.B-ishstabilize-quickfix-14526.91.B-ishstabilize-14695.85.B-ishstabilize-14695.107.B-ishstabilize-14682.B-ishstabilize-14633.B-ishstabilize-14616.B-ishstabilize-14589.B-ishstabilize-14588.98.B-ishstabilize-14588.14.B-ishstabilize-14588.123.B-ishstabilize-14536.B-ishstabilize-14532.B-ishstabilize-14528.B-ishstabilize-14526.89.B-ishstabilize-14526.84.B-ishstabilize-14526.73.B-ishstabilize-14526.67.B-ishstabilize-14526.57.B-ishstabilize-14498.B-ishstabilize-14496.B-ishstabilize-14477.B-ishstabilize-14469.9.B-ishstabilize-14469.8.B-ishstabilize-14469.58.B-ishstabilize-14469.41.B-ishstabilize-14442.B-ishstabilize-14438.B-ishstabilize-14411.B-ishstabilize-14396.B-ishstabilize-14395.B-ishstabilize-14388.62.B-ishstabilize-14388.61.B-ishstabilize-14388.52.B-ishstabilize-14385.B-ishstabilize-14345.B-ishstabilize-14336.B-ishstabilize-14333.B-ishrelease-R99-14469.B-ishrelease-R98-14388.B-ishrelease-R102-14695.B-ishrelease-R101-14588.B-ishrelease-R100-14526.B-ishfirmware-cherry-14454.B-ishfirmware-brya-14505.B-ishfirmware-brya-14505.71.B-ishfactory-kukui-14374.B-ishfactory-guybrush-14600.B-ishfactory-cherry-14455.B-ishfactory-brya-14517.B-ish
In the interest of making long-term branch maintenance incur as little
technical debt on us as possible, we should not maintain any files on
the branch we are not actually using.
This has the added effect of making it extremely clear when merging CLs
from the main branch when changes have the possibility to affect us.
The follow-on CL adds a convenience script to actually pull updates from
the main branch and generate a CL for the update.
BUG=b:204206272
BRANCH=ish
TEST=make BOARD=arcada_ish && make BOARD=drallion_ish
Signed-off-by: Jack Rosenthal <jrosenth@chromium.org>
Change-Id: I17e4694c38219b5a0823e0a3e55a28d1348f4b18
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3262038
Reviewed-by: Jett Rink <jettrink@chromium.org>
Reviewed-by: Tom Hughes <tomhughes@chromium.org>
Diffstat (limited to 'test/battery_get_params_smart.c')
-rw-r--r-- | test/battery_get_params_smart.c | 98 |
1 files changed, 0 insertions, 98 deletions
diff --git a/test/battery_get_params_smart.c b/test/battery_get_params_smart.c deleted file mode 100644 index 3163fb587e..0000000000 --- a/test/battery_get_params_smart.c +++ /dev/null @@ -1,98 +0,0 @@ -/* Copyright 2014 The Chromium OS Authors. All rights reserved. - * Use of this source code is governed by a BSD-style license that can be - * found in the LICENSE file. - * - * Test the logic of battery_get_params() to be sure it sets the correct flags - * when i2c reads fail. - */ - -#include "battery.h" -#include "battery_smart.h" -#include "common.h" -#include "console.h" -#include "i2c.h" -#include "test_util.h" -#include "util.h" - -/* Test state */ -static int fail_on_first, fail_on_last; -static int read_count, write_count; -struct batt_params batt; - - -void battery_compensate_params(struct batt_params *batt) -{ -} - -void board_battery_compensate_params(struct batt_params *batt) -{ -} - -static void reset_and_fail_on(int first, int last) -{ - /* We're not initializing the fake battery, so everything reads zero */ - memset(&batt, 0, sizeof(typeof(batt))); - read_count = write_count = 0; - fail_on_first = first; - fail_on_last = last; -} - -/* Mocked functions */ -int sb_read(int cmd, int *param) -{ - read_count++; - if (read_count >= fail_on_first && read_count <= fail_on_last) - return EC_ERROR_UNKNOWN; - - return i2c_read16(I2C_PORT_BATTERY, BATTERY_ADDR_FLAGS, - cmd, param); -} -int sb_write(int cmd, int param) -{ - write_count++; - return i2c_write16(I2C_PORT_BATTERY, BATTERY_ADDR_FLAGS, - cmd, param); -} - - -/* Tests */ -static int test_param_failures(void) -{ - int i, num_reads; - - /* No failures */ - reset_and_fail_on(0, 0); - battery_get_params(&batt); - TEST_ASSERT(batt.flags & BATT_FLAG_RESPONSIVE); - TEST_ASSERT(!(batt.flags & BATT_FLAG_BAD_ANY)); - num_reads = read_count; - - /* Just a single failure */ - for (i = 1; i <= num_reads; i++) { - reset_and_fail_on(i, i); - battery_get_params(&batt); - TEST_ASSERT(batt.flags & BATT_FLAG_BAD_ANY); - TEST_ASSERT(batt.flags & BATT_FLAG_RESPONSIVE); - } - - /* Once it fails, it keeps failing */ - for (i = 1; i <= num_reads; i++) { - reset_and_fail_on(i, num_reads); - battery_get_params(&batt); - TEST_ASSERT(batt.flags & BATT_FLAG_BAD_ANY); - if (i == 1) - /* If every read fails, it's not responsive */ - TEST_ASSERT(!(batt.flags & BATT_FLAG_RESPONSIVE)); - else - TEST_ASSERT(batt.flags & BATT_FLAG_RESPONSIVE); - } - - return EC_SUCCESS; -} - -void run_test(int argc, char **argv) -{ - RUN_TEST(test_param_failures); - - test_print_result(); -} |