[Enhance] Auto run nondex

[MyEnthusiastic]

Background

https://campuswire.com/c/G7A0E96FD/feed/639
https://campuswire.com/c/G7A0E96FD/feed/637
https://campuswire.com/c/G7A0E96FD/feed/624

Change

Optimize : with -fae and -Dskip.rat to power the script to run more tests at a time

Simplified 1
Actually there is no need to go into each sub-module

Simplified 2

In Maven, earlier lifecycle phases are indeed executed automatically before the current one.

for all modules the dependency will be automatically "compiled" first, but the mvn install is important for the single module when we only want small piece of the system to run

Previously, Installing dependency is daunting, also simplification can help people customize, understand, and use more easily. So that every students in the following semester can use this to run their own repo 😆

Clean : misleading README to make it more straightforward, the name and format for the sh file

[darko-marinov]

1. Do the flags require explicit =true?

2. Which example needed that numUnapprovedLicenses?

3. Does the rest of script support the case where multiple modules have a test failure? (Maybe also expand -fae to the full name, so it's easier to read/maintain.)

[alexbadia1]

1. Do the flags require explicit =true?

The explicit true should not be required, but improves readability by making the purpose of the flags -Dlicense.skip=true -Drat.skip=true as boolean flags

2. Which example needed that numUnapprovedLicenses?

I don't know, but is a nice to have as a workaround for a project with many dependencies that have unapproved licenses.

3. Does the rest of script support the case where multiple modules have a test failure? (Maybe also expand -fae to the full name, so it's easier to read/maintain.)

Changing -fae to --fail-at-end might be a good idea for readability.

Can you clarify what you mean by "support" though. The only other maven commands run in the auto-run-nondex/runNondex.sh script are:

mvn install -DskipTests
mvn -Dexec.executable='echo' -Dexec.args='${project.artifactId}' exec:exec -q -fn | tee modnames

[MyEnthusiastic]

Simplify

[darko-marinov]

Please squash your changes into one commit. Can your new script be just an option in the old script, to run on all modules or only one?

[MyEnthusiastic]

Please squash your changes into one commit. Can your new script be just an option in the old script, to run on all modules or only one?

I squashed them --- thanks for the suggestion --- the old script can run all --- but it was very complex and need python dependency to convert to HTML and did many un-necessary works. The reason I add the new file is to provide a quick start :)

[alexbadia1]

Hi, the status of this PR is a bit unclear. Are there any specific concerns or changes needed before this PR can be accepted?

This PR also seems to be a duplicate of https://github.com/TestingResearchIllinois/idoft/pull/1069/commits, otherwise the changes look good to me.