cederom commented on PR #2931: URL: https://github.com/apache/nuttx-apps/pull/2931#issuecomment-2593427770
> @raiden00pl: if we decide to separate testing utilities and test scenarios, then this PR doesn't make much sense. The problem is how to split these applications, `testing`, `tests` and `testapp` ? isn't it too many directories starting with `test*`? Lets use small measurable steps approach and do not put too much changes into a single PR. The idea to groups tests by function is nice. The idea to split testing utilities, apps, and scenarios is also nice and it could be second step in testing cleanup? > Or maybe we use something like: `testing/tests` for unit-tests and integration tests, `testing/utils` for testing frameworks and similar useful for user code, `testing/apps` for all the rest small apps. Yes this seems best coherent and self-explanatory approach I like it :-) I am for accepting this PR as it passes the Ci tests and follows the previous discussions. Then moving forward with further cleanup in next PRs. But if author is willing to update the PR and further split `testing/` into `testing/tests`, `testing/apps`, `testing/utils` then it would be even better, what do you think @txy-21 ? :-) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
