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

Pass the remaining tests in data_create_unknown #212

Merged
merged 3 commits into from
Oct 23, 2023

Conversation

zpzigi754
Copy link
Collaborator

@zpzigi754 zpzigi754 commented Oct 20, 2023

This PR adds missing checks in data_create_unknown. All the remaining tests in ACS's data_create_unknown will be passed after this change. It also includes the commit resolving the ci storage issue.

[before]

   TOTAL TESTS     : 51

   TOTAL PASSED    : 28

   TOTAL SIM ERROR : 2

[after]

   TOTAL TESTS     : 51

   TOTAL PASSED    : 29

   TOTAL SIM ERROR : 1

This will pass the remaining tests in ACS's data_create_unknown.
[before]
```
...
[!] Setting islet features: ['--features', 'max_level_trace']
[!] Building realm management monitor...: islet
[!] Building tf-a-tests...
[!] Building bootloaders(bl1.bin, fip.bin)...
[-] Test failed! (There are no proper result logs)
Error: Process completed with exit code 1.
```

[after]
```
...
[!] Setting islet features: ['--features', 'max_level_error']
[!] Building realm management monitor...: islet
[!] Building tf-a-tests...
[!] Building bootloaders(bl1.bin, fip.bin)...
[!] Tests result: 6 passed, 0 failed.
[!] Setting islet features: ['--features', 'max_level_error']
[!] Building realm management monitor...: islet
[!] Building realm(rsi-test)...
[!] Building tf-a-tests...
[!] Building bootloaders(bl1.bin, fip.bin)...
[!] Tests result: 6 passed, 0 failed.
```
@zpzigi754 zpzigi754 merged commit bb9fd0a into islet-project:main Oct 23, 2023
7 checks passed
@zpzigi754 zpzigi754 deleted the fix-data-create-unknown branch October 23, 2023 02:21
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.

2 participants