ReactiveX / RxPY / 25215980649

94%

push

github

test: fix pyright strict-mode errors in tests/test_observable/ (batch 3) (#785)

* test: fix pyright strict-mode errors in tests/test_observable/ (batch 3)

Fix pyright errors in 25 test files in tests/test_observable/:
- Add 'from typing import NoReturn' import
- Annotate '_raise' functions with 'ex: Exception -> NoReturn' signature
- Fix callsites passing str to '_raise()' to wrap in Exception()
- Fix subscribe functions returning 'lambda: None' to return Disposable()
- Fix 'return dispose' to 'return Disposable(dispose)'
- Fix 'o.on_error(str)' calls to wrap strings in Exception()
- Fix 'do_action(on_next=action)' with zero-param action to accept one param
- Remove non-None return values from 'finally_action' callbacks

These changes allow tests/test_observable/ to eventually be removed
from the pyright exclude list in pyproject.toml.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* ci: trigger build

* style: fix ruff import order and line length in test_observable batch 3

Pre-commit hook failures from PR #785:
- Reorder `from typing import NoReturn` after `from datetime` in 3 files
- Wrap two long lines exceeding 88 chars in test_fromiterable.py and
  test_minby.py

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Dag Brattli <dag@brattli.net>
Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

62 of 65 new or added lines in 24 files covered. (95.38%)

25388 of 27146 relevant lines covered (93.52%)

0.94 hits per line