apps/testing:merge case folder to the new mm folder#2965
Conversation
|
[Experimental Bot, please feedback here] No, this PR description does not fully meet the NuttX requirements. Here's why and how to fix it: Missing Information in Summary:
Missing Information in Impact:
Missing Information in Testing:
Example of an Improved PR Description: ... output showing successful test run from apps/testing/mm/memtester ... ... output showing successful test run from apps/testing/mm/memtester ... By providing more detail and explicitly addressing all the points in the template, the PR will be much clearer and easier for reviewers to evaluate. |
cederom
left a comment
There was a problem hiding this comment.
Thank you @txy-21 :-) Here goes my conditional go assuming all @xiaoxiang781216 remarks are adressed :-)
e39c35f to
e1eabc3
Compare
e1eabc3 to
d5d3991
Compare
|
@txy-21 please update nuttx board's defconfig too |
1.rename original mm folder to heaptest and move it to mm folder 2.move the following folders into the new mm folder: cachetest, heaptest, iob, kasantest, memstress, memtester, ramtest, stressapptest Signed-off-by: tengshuangshuang <tengshuangshuang@xiaomi.com>
d5d3991 to
04e954b
Compare
Summary
This is one step that merge test case as discussed in #2931.
Create mm folder and move folders to the new mm folder.
Impact
Only affect apps/testing. The changes are as follows:
1.rename original mm folder to mmtools and move it to the new mm folder.
2.move the following folders into the new mm folder:
cachetest, iob, kasantest, memstress, memtester, open-memstream, ramtest, stressapptest
Testing
CI test.
depends on apache/nuttx#15659