Commit Graph

25 Commits

Author SHA1 Message Date
Michał Iwanicki
5a7bbdb3d3 common-mock-func: futility: fix check_vboot_keys function
Fix futility mocking & add GBB region only if we don't already use
WP_RO otherwise flashrom will complain on write as those 2 regions
overlap

Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
2025-08-27 14:46:34 +02:00
Michał Iwanicki
8e7a660783 dts-scripts: Enhance mocking for purposes of profile comparison
common-mock-func: set dmidecode to return 0 by default

if test variable isn't defined this likely means that this value
doesn't matter for test purposes.

Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>

common-mock-func: cbmem shouldn't exit in failure in normal check

Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>

common-mock-func: cbfstool: allow specifying fmap regions in binary

Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>

dts-functions: set_flashrom_update_params: don't use $1 directly

It's hard to know exactly what this argument is for otherwise as this
function isn't documented

Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>

deploy: don't use mktemp so profile arguments are always the same

Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>

dts-boot: create TEMP_DIR

Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>

common-mock: cbfstool: return 1 if not coreboot

Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>

cbfstool: don't mock when we check update binary config

Except in some cases where we want to possibly test for identical
config between update binary and current firmware.

Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>

dasharo-deploy: add 2>>ERR_LOG_FILE redirection

Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>

common-mock: cbmem: check if coreboot

Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>

dts-functions: download_keys: use 'mkdir -p' so it doesn't error out

Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>

common-mock: add cbmem_common_mock which returns error if not coreboot

Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>

common-mock: add msrtool and mei-amt-check common mock

Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>

common-mock: ifdtool: call original ifdtool if used with update binary

Fixes situation on QEMU that results in user not being asked whether to
continue flashing without ME even though we want to test with ME enabled

Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>

common-mock: flashrom: verify internal chip use

Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>

common-mock: add bootsplash presence mocking

Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>

common-mock: both flashrom and cbfstool allow writing over existing file

Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>

common-mock: bootsplash region might exist without custom bootsplash

Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>

check_if_me_disabled: Don't use mocked functions with 'grep -q'

Otherwise grep will break pipe immediately after matching searched
string which will result in mocked function being immediately stopped
even before finishing. It also results in e.g. tool wrapper not writing
used command to profile

Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>

common-mock: Add separate ME disabled and ME HAP disabled

Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>

common-mock: add cbfstool smmstore mocking

Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>

common-mock: simplify fuse mock

Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>

dts-functions: check_if_boot_guard_enabled: hide rdmsr output

remove non-existent command, likely pasted by mistake

Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>

include: hal: add cbfstool_add_firmware_section_mock

Signed-off-by: Daniil Klimuk <daniil.klimuk@3mdeb.com>

use dont_mock instead of cbfstool_add_firmware_section_mock

Signed-off-by: Daniil Klimuk <daniil.klimuk@3mdeb.com>

common-mock: rename regions to _regions

Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
2025-08-27 14:45:01 +02:00
Daniil Klimuk
4bec2898cd Style and typos fixes
Signed-off-by: Daniil Klimuk <daniil.klimuk@3mdeb.com>
2025-08-06 15:15:24 +02:00
Daniil Klimuk
3b792fbf85 include: hal: dts-hal: check_if_seabios: redirect cbfstool output to /dev/null
Signed-off-by: Daniil Klimuk <daniil.klimuk@3mdeb.com>
2025-08-06 15:15:24 +02:00
Daniil Klimuk
d985b69e3b include: ha: dts-hal: check for payload functions should have only one
goal

This functions should check only for payloads. We do have another
function called "check_if_dasharo" for checking if we have Dasharo
firmware flashed.

Change the function names to adhere to the "check_if_dasharo" function.

Signed-off-by: Daniil Klimuk <daniil.klimuk@3mdeb.com>
2025-08-06 15:15:23 +02:00
Daniil Klimuk
f559d8dc4b Fix pre-commit
Signed-off-by: Daniil Klimuk <daniil.klimuk@3mdeb.com>
2025-08-06 15:15:23 +02:00
Daniil Klimuk
0ac2105879 add transition to Dasharo (Slim Bootloader+UEFI)
Signed-off-by: Daniil Klimuk <daniil.klimuk@3mdeb.com>
2025-08-06 15:15:22 +02:00
Danik Klimuk
7eb3e0c844 include: common-mock-func.sh: fix cbfstoll config file mocking
Co-authored-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
2025-07-16 16:14:19 +02:00
Daniil Klimuk
56e10467e1 scripts: dasharo-deploy.sh: mock SeaBIOS payload presence
Signed-off-by: Daniil Klimuk <daniil.klimuk@3mdeb.com>
2025-07-16 16:14:13 +02:00
Michał Iwanicki
837b887a64 dts-hal.sh: generate tool call profile
Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
2025-07-15 16:38:06 +02:00
Danik Klimuk
28408b0d18 Merge pull request #89 from Dasharo/fix-hcl-report
include: hal: dts-hal.sh: add LSPCI and LSUSB
2025-06-12 16:53:40 +02:00
Daniil Klimuk
db5e4f2236 include: hal: common-mock-func.sh: add mocking EFI pressence in SYSFS
Signed-off-by: Daniil Klimuk <daniil.klimuk@3mdeb.com>
2025-06-11 13:30:53 +02:00
Daniil Klimuk
37197e1389 include: hal: dts-hal.sh: add LSPCI and LSUSB
Signed-off-by: Daniil Klimuk <daniil.klimuk@3mdeb.com>
2025-06-11 10:38:34 +02:00
Pawel Langowski
39021f596c Fix files to conform to shfmt
Signed-off-by: Pawel Langowski <pawel.langowski@3mdeb.com>
2025-04-15 14:48:13 +02:00
Daniil Klimuk
e69ca6c8a4 reports: dasharo-hcl-report: mock checking for sound card presence
This loop takes too much time during tests (5s * 12 = 1m). The sound
card presence should be mocked to true by default to optimize HCL report
execution time.

Signed-off-by: Daniil Klimuk <daniil.klimuk@3mdeb.com>
2025-01-15 14:29:12 +01:00
Daniil Klimuk
0f71309737 include: hal: dts-hal: do not send mocking fun name to FSREAD_TOOL
Signed-off-by: Daniil Klimuk <daniil.klimuk@3mdeb.com>
2025-01-15 14:29:12 +01:00
Michał Iwanicki
9e678e52eb logging: redirect more stderr to ERR_LOG_FILE
Signed-off-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
2025-01-10 10:56:53 +01:00
Daniil Klimuk
8e05b0f2e2 delete explicit redirection to stdout (1>&1)
Reviewed-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
Signed-off-by: Daniil Klimuk <daniil.klimuk@3mdeb.com>
2024-11-18 16:52:46 +01:00
Daniil Klimuk
4934117d74 include: dts-functions.sh: fix AC adapter mocking
Reviewed-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
Signed-off-by: Daniil Klimuk <daniil.klimuk@3mdeb.com>
2024-11-18 16:52:46 +01:00
Daniil Klimuk
09258fb168 include: hal: dts-hal.sh: use $@ instead of $*
Reviewed-by: Michał Iwanicki <michal.iwanicki@3mdeb.com>
Signed-off-by: Daniil Klimuk <daniil.klimuk@3mdeb.com>
2024-11-18 16:52:46 +01:00
Daniil Klimuk
821e7082b3 include: hal: dts-hal: add usage comment for tool_wrapper
Signed-off-by: Daniil Klimuk <daniil.klimuk@3mdeb.com>
2024-11-18 16:52:46 +01:00
Daniil Klimuk
1fde2de736 include: dts-functions.sh: check_if_boot_guard_enabled add mocks
Signed-off-by: Daniil Klimuk <daniil.klimuk@3mdeb.com>
2024-11-18 16:52:45 +01:00
Daniil Klimuk
ef75759a38 include: dts-functions.sh: check_if_fused: add mocks
Signed-off-by: Daniil Klimuk <daniil.klimuk@3mdeb.com>
2024-11-18 16:52:45 +01:00
Daniil Klimuk
5d310535f5 fix sourcing libs
Cause of this patch: After I have added and integrated HAL - I have
added following line in dts-boot:

source $DTS_HAL

This was needed because I have used some fuctions from the HAL in the
script.

Inside DTS_HAL I had following line:

source $DTS_ENV

Because I have used some vars from the DTS_ENV in the HAL.

The problem was, that I had another line in DTS_ENV:

source $DTS_HAL

So, I got following boot workflow:

Boot
|
v
dts-boot
|   /-------\
v   v       |
DTS_HAL->DTS_ENV
           .
............
v
dts

Instead of sourcing some scripts and then launching dts script - I got a
loop betwee DTS_HAL and DTS_ENV. Therefore I decided to clean up
sourcing a bit.

Signed-off-by: Daniil Klimuk <daniil.klimuk@3mdeb.com>
2024-11-18 16:17:36 +01:00
Daniil Klimuk
d7780f288b include: hal: add
Add HAL and mocking functions.

Signed-off-by: Daniil Klimuk <daniil.klimuk@3mdeb.com>
2024-11-18 16:17:35 +01:00