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

feat(framework): add config and build capabilities to test framework #11

Merged
merged 9 commits into from
Oct 9, 2023

Conversation

Diogo21Costa
Copy link
Member

PR Description

This pull request implements configuration and build capabilities into the test framework.

Types of Changes

  • feat: Configuration Parsing
    • Logical Unit: test_framework.py

The commit ae1c283 introduces the ability to parse a .dts file, enabling the selection of test configurations, definition of Nix build recipes, and specifying test verbosity.

  • feat: System Build (Codegen Integration)
    • Logical Unit: test_framework.py

The commit 3d495f3 introduces the integration of the codegen Python script, as introduced in a88f836.

  • feat: System Build (Nix Build)
    • Logical Unit: test_framework.py

The commit b1669ff enables the use of Nix scripts for building the system under test. Additionally, commit 0f7dff8 introduces the capability to launch the target platform, currently limited to the qemu platform.

Checklist:

  • I have run the CI checkers before submitting the PR to avoid unnecessary runs of the workflow.

ESCristiano
ESCristiano previously approved these changes Sep 30, 2023
@miguelafsilva5 miguelafsilva5 self-assigned this Oct 2, 2023
@miguelafsilva5
Copy link
Member

@Diogo21Costa Is there a .dts template anywhere? It might be a good idea to provide one for reference.

@miguelafsilva5
Copy link
Member

I think the parent directory should be cleaner. Moving the codegen.py to the framework folder and the template.c inside the src folder would be good. This way every directory would only contain files from the same type.

@Diogo21Costa
Copy link
Member Author

@Diogo21Costa Is there a .dts template anywhere? It might be a good idea to provide one for reference.

@miguelafsilva5 That's a good point. Added some pointers to use the framework in the README.me which also includes a template for the config.dts file in 127bbae.

@Diogo21Costa
Copy link
Member Author

I think the parent directory should be cleaner. Moving the codegen.py to the framework folder and the template.c inside the src folder would be good. This way every directory would only contain files from the same type.

@miguelafsilva5 Fully agree, thanks for your feedback. Addressed in ca284f6.

@miguelafsilva5 miguelafsilva5 merged commit c6b73d1 into main Oct 9, 2023
5 checks passed
@miguelafsilva5 miguelafsilva5 deleted the feat/config-build branch October 9, 2023 16:30
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