issue/877 - return saved_file in TestManager.test method

[baominghelly]

Description

return saved_file in TestManager.test method

Test evidence

test command

python test/infinicore/nn/rope.py --nvidia
python test/infinicore/tensor/narrow.py --nvidia
python test/infinicore/ops/add.py --nvidia --save
python test/infinicore/run.py --ops add acos acosh --nvidia --bench
python test/infinicore/run.py --ops add acos acosh --nvidia --bench --save
python test/infinicore/run.py --ops add acos acosh --nvidia --bench --save infinimetrics.json

test output

output.log

[Copilot]

Pull request overview

This PR modifies the test framework to track and return the paths of saved report files throughout the execution chain. The main change is updating TestManager.test() to return a tuple of (success, saved_files) instead of just a boolean success value.

Key Changes

• Updated TestManager.test() to collect and return saved report file paths alongside the success status
• Modified the entire call chain to propagate saved file paths from JSON save operations up through the test execution stack
• Added saved_file field to OperatorResult dataclass to store report paths

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
test/infinicore/run.py	Updated callers to unpack tuple return from test_manager.test(); minor style improvements (quote consistency, whitespace)
test/infinicore/framework/utils/json_utils.py	Modified save_json_report to return the saved file path or None on failure
test/infinicore/framework/test_manager.py	Added saved_files list collection and modified return statement to return tuple (all_passed, saved_files)
test/infinicore/framework/runner.py	Added saved_file instance variable and modified _save_report to return and store the saved path
test/infinicore/framework/results.py	Added saved_file field to OperatorResult; updated TestSummary.save_report to return the saved path
test/infinicore/framework/executor.py	Added assignment of runner.saved_file to result.saved_file after test execution

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The docstring for the test method should be updated to document the return value. Since the method now returns a tuple of (success: bool, saved_files: list), this should be documented in the docstring with a Returns section explaining both values.

[Copilot]

The saved_file field is declared as type str with default value "", but runner.saved_file can be None (when save is not requested or when saving fails). Consider either changing the type annotation to Optional[str] or ensuring runner.saved_file is always a string (e.g., runner.saved_file or "").

Suggested change

	result.saved_file = runner.saved_file
	result.saved_file = runner.saved_file or ""