checkstyle: skip artifact copy and consolidate into single pytest run

checkstyle ran three separate cmd_test invocations (style, doctests,
clippy), each launching a container, copying 2.3GB of artifacts, and
starting pytest. Style checks don't need artifacts at all.

- Add FC_TEST_SKIP_ARTIFACT_COPY=1 support to test.sh
- Set it in cmd_checkstyle to skip the artifact copy
- Merge the three cmd_test calls into a single pytest invocation

Signed-off-by: Riccardo Mancini <mancio@amazon.com>

Author: Manciukic

tools/devtool

@@ -1011,7 +1011,7 @@ cmd_test() {
     say "$(sed '/^processor.*: 0$/,/^processor.*: 1$/!d; /^processor.*: 1$/d' /proc/cpuinfo)"
     say "RPM firmware versions: $(rpm -q microcode_ctl amd-ucode-firmware linux-firmware)"
 
-    env |grep -P "^(AWS_EMF_|BUILDKITE|CODECOV_)" > env.list
+    env |grep -P "^(AWS_EMF_|BUILDKITE|CODECOV_|FC_TEST_)" > env.list
     if [[ $performance_tweaks -eq 1 ]]; then
       if [[ "$(uname --machine)" == "x86_64" ]]; then
         say "Detected CI and performance tests, tuning CPU frequency scaling and idle states for reduced variability"
@@ -1230,9 +1230,13 @@ cmd_checkstyle() {
       cmd_sh "git-secrets --register-aws && git-secrets --scan"
     fi
 
-    cmd_test --no-build --no-kvm-check --no-build-dir-check --no-artifacts-check -- -n 4 --dist worksteal integration_tests/style || exit 1
-    cmd_test --no-build --no-kvm-check --no-build-dir-check --no-artifacts-check -- -n 4 --doctest-modules framework || exit 1
-    cmd_test --no-build --no-kvm-check --no-build-dir-check --no-artifacts-check -- integration_tests/build/test_clippy.py || exit 1
+    export FC_TEST_SKIP_ARTIFACT_COPY=1
+    cmd_test --no-build --no-kvm-check --no-build-dir-check --no-artifacts-check -- \
+        -n 4 --dist worksteal \
+        integration_tests/style \
+        integration_tests/build/test_clippy.py \
+        --doctest-modules framework \
+        || exit 1
 }
 
 cmd_checkbuild() {

tools/test.sh

@@ -31,7 +31,10 @@ if [ -f $CGROUP/cgroup.controllers -a -e $CGROUP/cgroup.type ]; then
         > $CGROUP/cgroup.subtree_control
 fi
 
-if [ -f build/current_artifacts ]; then
+if [ "${FC_TEST_SKIP_ARTIFACT_COPY:-}" = "1" ]; then
+  mkdir -p /srv/test_artifacts
+  say "Skipping artifact copy (FC_TEST_SKIP_ARTIFACT_COPY=1)"
+elif [ -f build/current_artifacts ]; then
   say "Copy artifacts to /srv/test_artifacts, so hardlinks work"
   cp -ruvfL $(cat build/current_artifacts) /srv/test_artifacts
 else