Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(config): improve logging setup and tests selection in the framework #22

Merged
merged 4 commits into from
Mar 6, 2024

Conversation

Diogo21Costa
Copy link
Member

PR summary

This PR introduces two major changes:

  1. Enable the definition of an array of strings for both the list of suites and tests (commit 01df220).
  2. Remove the "echo" and "log_level" configurations from config.dts; configuration is now performed via the command line (commits bfd1cfa and 47b26b7).

Old config.dts format:

/dts-v1/;
/ {
    platform = "qemu-aarch64-virt";
    echo = "full";  // fulll, tf, none

    test_config {
        recipe_test {
            nix_file = "default.nix";
            suites = "SUITE_1 SUITE_2";
            tests = "";
            log_level = "1"; //0, 1, 2
        };
    };
};

New config.dts format:

/dts-v1/;
/ {
    platform = "qemu-aarch64-virt";

    test_config {
        recipe_test {
            nix_file = "default.nix";
            suites = "SUITE_1", "SUITE_2";
            tests = "";
        };
    };
};

Old python framework execution command:

python3 test_framework.py\
  -dts_path $ROOT_DIR/tests/configs/mix_suite.dts\
  -bao_test_src_path $ROOT_DIR/bao-tests/src\
  -tests_src_path $ROOT_DIR/tests/src\

New python framework execution command:

python3 test_framework.py\
  -dts_path $ROOT_DIR/tests/configs/mix_suite.dts\
  -bao_test_src_path $ROOT_DIR/bao-tests/src\
  -tests_src_path $ROOT_DIR/tests/src\
  -echo=full\
  -log_level=1

@Diogo21Costa Diogo21Costa changed the title Fix/config fix(config): improve logging setup and tests selection in the framework Feb 16, 2024
@miguelafsilva5 miguelafsilva5 self-assigned this Mar 4, 2024
@ESCristiano ESCristiano self-requested a review March 5, 2024 02:32
@miguelafsilva5 miguelafsilva5 merged commit 58442ab into main Mar 6, 2024
5 checks passed
@miguelafsilva5 miguelafsilva5 deleted the fix/config branch March 6, 2024 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants