This PR refactors the operator metrics logging tests in `test_stats.py`
to improve clarity, reliability, and maintainability.

- Replaced `test_op_metrics_logging` and `test_op_state_logging` with a
single, more focused test:
`test_executor_logs_metrics_on_operator_completion`
- Uses pytest's `caplog` fixture instead of mocking the logger (more
idiomatic)
- Tests the core behavior (operator completion metrics logged exactly
once) without depending on exact log message formatting
- Eliminates reliance on helper functions and complex string matching
- More descriptive test name following unit testing best practices
- Reduced test code complexity while maintaining coverage of critical
logging behavior

Signed-off-by: Balaji Veeramani <bveeramani@berkeley.edu>

…ject#57702)

This PR refactors the operator metrics logging tests in `test_stats.py`
to improve clarity, reliability, and maintainability.

- Replaced `test_op_metrics_logging` and `test_op_state_logging` with a
single, more focused test:
`test_executor_logs_metrics_on_operator_completion`
- Uses pytest's `caplog` fixture instead of mocking the logger (more
idiomatic)
- Tests the core behavior (operator completion metrics logged exactly
once) without depending on exact log message formatting
- Eliminates reliance on helper functions and complex string matching
- More descriptive test name following unit testing best practices
- Reduced test code complexity while maintaining coverage of critical
logging behavior

Signed-off-by: Balaji Veeramani <bveeramani@berkeley.edu>

…ject#57702)

This PR refactors the operator metrics logging tests in `test_stats.py`
to improve clarity, reliability, and maintainability.

- Replaced `test_op_metrics_logging` and `test_op_state_logging` with a
single, more focused test:
`test_executor_logs_metrics_on_operator_completion`
- Uses pytest's `caplog` fixture instead of mocking the logger (more
idiomatic)
- Tests the core behavior (operator completion metrics logged exactly
once) without depending on exact log message formatting
- Eliminates reliance on helper functions and complex string matching
- More descriptive test name following unit testing best practices
- Reduced test code complexity while maintaining coverage of critical
logging behavior

Signed-off-by: Balaji Veeramani <bveeramani@berkeley.edu>
Signed-off-by: xgui <xgui@anyscale.com>

This PR refactors the operator metrics logging tests in `test_stats.py`
to improve clarity, reliability, and maintainability.

- Replaced `test_op_metrics_logging` and `test_op_state_logging` with a
single, more focused test:
`test_executor_logs_metrics_on_operator_completion`
- Uses pytest's `caplog` fixture instead of mocking the logger (more
idiomatic)
- Tests the core behavior (operator completion metrics logged exactly
once) without depending on exact log message formatting
- Eliminates reliance on helper functions and complex string matching
- More descriptive test name following unit testing best practices
- Reduced test code complexity while maintaining coverage of critical
logging behavior

Signed-off-by: Balaji Veeramani <bveeramani@berkeley.edu>
Signed-off-by: elliot-barn <elliot.barnwell@anyscale.com>

…ject#57702)

This PR refactors the operator metrics logging tests in `test_stats.py`
to improve clarity, reliability, and maintainability.

- Replaced `test_op_metrics_logging` and `test_op_state_logging` with a
single, more focused test:
`test_executor_logs_metrics_on_operator_completion`
- Uses pytest's `caplog` fixture instead of mocking the logger (more
idiomatic)
- Tests the core behavior (operator completion metrics logged exactly
once) without depending on exact log message formatting
- Eliminates reliance on helper functions and complex string matching
- More descriptive test name following unit testing best practices
- Reduced test code complexity while maintaining coverage of critical
logging behavior

Signed-off-by: Balaji Veeramani <bveeramani@berkeley.edu>

…ject#57702)

This PR refactors the operator metrics logging tests in `test_stats.py`
to improve clarity, reliability, and maintainability.

- Replaced `test_op_metrics_logging` and `test_op_state_logging` with a
single, more focused test:
`test_executor_logs_metrics_on_operator_completion`
- Uses pytest's `caplog` fixture instead of mocking the logger (more
idiomatic)
- Tests the core behavior (operator completion metrics logged exactly
once) without depending on exact log message formatting
- Eliminates reliance on helper functions and complex string matching
- More descriptive test name following unit testing best practices
- Reduced test code complexity while maintaining coverage of critical
logging behavior

Signed-off-by: Balaji Veeramani <bveeramani@berkeley.edu>
Signed-off-by: Aydin Abiar <aydin@anyscale.com>

…ject#57702)

This PR refactors the operator metrics logging tests in `test_stats.py`
to improve clarity, reliability, and maintainability.

- Replaced `test_op_metrics_logging` and `test_op_state_logging` with a
single, more focused test:
`test_executor_logs_metrics_on_operator_completion`
- Uses pytest's `caplog` fixture instead of mocking the logger (more
idiomatic)
- Tests the core behavior (operator completion metrics logged exactly
once) without depending on exact log message formatting
- Eliminates reliance on helper functions and complex string matching
- More descriptive test name following unit testing best practices
- Reduced test code complexity while maintaining coverage of critical
logging behavior

Signed-off-by: Balaji Veeramani <bveeramani@berkeley.edu>
Signed-off-by: Future-Outlier <eric901201@gmail.com>