Include system error logs with KCL samples tests (#7904)

* Include system error logs with KCL samples tests

* Update snapshots

* Update snapshots

* Add comment to indicate the purpose of the error message

---------

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

Author: jacebrowning

.github/workflows/cargo-test.yml

@@ -86,7 +86,10 @@ jobs:
         run: |
           set -euo pipefail
           cd rust
-          cargo nextest run --workspace --features artifact-graph --retries=10 --no-fail-fast --profile ci simulation_tests::kcl_samples 2>&1 | tee /tmp/github-actions.log
+          cargo nextest run \
+          --workspace --features=artifact-graph --retries=10 --no-fail-fast --profile=ci \
+          simulation_tests::kcl_samples \
+          2>&1 | tee /tmp/github-actions.log
         env:
           KITTYCAD_API_TOKEN: ${{secrets.KITTYCAD_API_TOKEN_DEV}}
           ZOO_HOST: https://api.dev.zoo.dev
@@ -175,7 +178,6 @@ jobs:
         shell: bash
         run: |-
           cp nextest-archive.tar.zst rust/nextest-archive.tar.zst
-          ls -lah
           cd rust
           cargo nextest run \
            --retries=10 --no-fail-fast --profile=ci --archive-file nextest-archive.tar.zst \
@@ -229,7 +231,7 @@ jobs:
         run: |-
           cd rust/kcl-lib
           cargo nextest run \
-           --retries=10 --no-fail-fast --features artifact-graph --profile=ci \
+           --retries=10 --no-fail-fast --features=artifact-graph --profile=ci \
            internal \
            2>&1 | tee /tmp/github-actions.log
         env:

.github/workflows/kcl-language-server.yml

@@ -61,7 +61,6 @@ jobs:
           cd rust/kcl-language-server
           npm run build
           npm run test-compile
-          ls -la dist
           xvfb-run -a npm run test
         if: runner.os == 'Linux'
       - name: Run tests
@@ -317,7 +316,6 @@ jobs:
           #with:
           #name: release-aarch64-pc-windows-msvc
           #path: rust/build
-      - run: ls -al ./rust/build
       - name: Publish Release
         uses: ./.github/actions/github-release
         with:

e2e/playwright/snapshot-tests.spec.ts-snapshots/Draft-circle-should-look-right-1-Google-Chrome-linux.png

@@ -53,5 +53,13 @@ test-group = "after-engine"
 [profile.default.junit]
 path = "../../../../test-results/junit.xml"
 
+[[profile.default.overrides]]
+filter = 'test(kcl_samples)'
+junit.store-success-output = true
+
 [profile.ci.junit]
 path = "../../../../test-results/junit.xml"
+
+[[profile.ci.overrides]]
+filter = 'test(kcl_samples)'
+junit.store-success-output = true

rust/.config/nextest.toml

@@ -75,6 +75,9 @@ async fn unparse_test(test: &Test) {
 #[kcl_directory_test_macro::test_all_dirs("../public/kcl-samples")]
 async fn kcl_test_execute(dir_name: &str, dir_path: &Path) {
     if DISABLED_SAMPLES.contains(&dir_name) {
+        // NOTE: Do not remove this line. The error message is included in the
+        // JUnit XML report for the Test Analysis Bot to post-process results
+        // and determine whether the test should be marked as skipped.
         eprintln!("Skipping disabled sample: {}", dir_name);
         return;
     }