Skip to content

Switch test fixtures to EVGEN-relative paths under /volatile/eic/EPIC/EVGEN - #113

Draft
wdconinc with Copilot wants to merge 2 commits into
mainfrom
copilot/switch-test-files-to-relative-paths
Draft

wdconinc with Copilot wants to merge 2 commits into
mainfrom
copilot/switch-test-files-to-relative-paths

Conversation

Copilot AI commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

This updates the test suite to consistently target EVGEN data using server-relative paths rooted at /volatile/eic/EPIC/EVGEN. It removes mixed RECO/legacy absolute path assumptions that could resolve outside the intended test base.

  • Base directory defaults

    • test/xrootd.test.ts: default XROOTD_BASE_DIR changed to /volatile/eic/EPIC/EVGEN.
    • test/root-analysis.test.ts: XRootDClient base dir updated from /work/eic2/EPIC to /volatile/eic/EPIC/EVGEN.
  • Path normalization to EVGEN-relative usage

    • Converted hardcoded file/dataset test inputs to EVGEN-relative paths (e.g., SIDIS/...) instead of absolute/base-prefixed paths.
    • Updated large-directory scenario from RECO-specific path to EVGEN path (SIDIS).
  • Fixture naming alignment

    • Replaced RECO-style default sample file names (*.eicrecon.tree.edm4eic.root) with EVGEN-style names (*.hepmc3.tree.root) in test defaults/examples where applicable.
  • Test docs updates

    • test/README.md now documents EVGEN as the default base path and updates troubleshooting xrdfs examples accordingly, while keeping custom path examples generic.
// before
const TEST_BASE_DIR = process.env.XROOTD_BASE_DIR || '/volatile/eic/EPIC';
arguments: { path: `${TEST_BASE_DIR}/nonexistent-test-file.root` }

// after
const TEST_BASE_DIR = process.env.XROOTD_BASE_DIR || '/volatile/eic/EPIC/EVGEN';
arguments: { path: 'nonexistent-test-file.root' }

Copilot AI and others added 2 commits August 24, 2026 00:37
Co-authored-by: wdconinc <4656391+wdconinc@users.noreply.github.com>
Co-authored-by: wdconinc <4656391+wdconinc@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants