Add test coverage for sys.modules fallback path in get_scope

[Copilot]

Summary

get_scope's fallback branch (module = sys.modules.get(module_name), line 53) was unreachable by existing tests because inspect.getmodule internally calls sys.modules.get(object.__module__) — meaning if the module is in sys.modules, it never returns None and the fallback is skipped entirely. This PR adds test coverage for that path and fixes repository hygiene by removing a generated file from version control.

Related Issue(s)

Backward Compatibility

No change to production code — test-only addition.

Implementation Notes

• tests/unit/models/parsers/test_object_scope.py

  • Added test_sys_modules_fallback_when_getmodule_returns_none: patches inspect.getmodule to return None (simulating C-extension types or other objects where module resolution fails), then verifies the fallback correctly resolves the module via sys.modules.get(module_name) and returns a valid ScopeProxy
  • Added from unittest.mock import patch import

• Removed accidentally committed flowrep/_version.py (a setuptools-scm generated file that explicitly states "don't track in version control") and added */_version.py to .gitignore to prevent recurrence.

The fallback path exists for objects (e.g. C-extension callables) where inspect.getmodule may fail even though a valid __module__ attribute is present. Mocking is the only reliable way to exercise this path in isolation, since naturally the two sys.modules lookups would always agree.

---

📱 Kick off Copilot coding agent tasks wherever you are with GitHub Mobile, available on iOS and Android.

[samwaseda]

@copilot run black

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.97%. Comparing base (62fa0e4) to head (bcffb5c).
⚠️ Report is 5 commits behind head on samwaseda-patch-1.

Additional details and impacted files

@@                  Coverage Diff                  @@
##           samwaseda-patch-1     #188      +/-   ##
=====================================================
+ Coverage              97.92%   97.97%   +0.04%     
=====================================================
  Files                     34       34              
  Lines                   2268     2268              
=====================================================
+ Hits                    2221     2222       +1     
+ Misses                    47       46       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[Copilot]

@copilot run black

Both changed files are already black-formatted — no changes needed.

[samwaseda]

@copilot Remove _version.py

[Copilot]

@copilot Remove _version.py

Done in 0c38bda — deleted flowrep/_version.py and added */_version.py to .gitignore to prevent it being committed again.