ChainSafe / ChainSafe/forest

Refactor CI harness script to standardize `TMP_DIR` and token handling

Open
#6,005 0 comments 0 reactions 0 assignees View on GitHub
Type: Task
Dominant language
Rust
Stars
697
Forks
200
Avg merge
1d 5h
Merged PRs (30d)
65

Description

# Summary

Currently, our harness test script initializes `data_dir` directories and tokens inconsistently, making CI tests harder to maintain.

We should refactor `harness.sh` to:

- [ ] Always use `$TMP_DIR` for `data_dir`, logs, and tokens
- [ ] Standardize environment variable initialization across all functions (`forest_init*`, `forest_run_node_*`)
- [ ] Reduce redundant path handling and improve readability

# Completion Criteria

- All `forest_init_*` functions use `$TMP_DIR` for data, logs, and token
- Admin token is exported consistently
- Tests run without modifying paths or tokens

# Additional Links & Resources

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.