Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: editable install cover package navigation #535

Closed
wants to merge 1 commit into from

Conversation

LecrisUT
Copy link
Collaborator

TODO:

  • Add test-cases for relative navigation
    • python-python navigation (py1_pkg.py1_module -> py2_pkg.py2_module)
    • namespace-python-python navigations ((ns1)py3_pkg.pya_module -> (ns2)py3_pkg.pyb_module)
    • c-c nagivation ((_c_lib.so)c1_pkg.c1_module -> (_c_lib.so)c2_pkg.c2_module; (_c3_lib.so)c3_pkg.c3_module -> (_c4_lib.so)c4_pkg.c4_mdoule)
    • Mixed packaging (shared_pkg.c1_pkg.c1_module with c1_pkg and c1_module in the same .so file; shared_pkg.c1_pkg.py2_module with py2_module from python)
  • Fix python-python navigation
  • Fix c-python navigation

Naming could be a bit cumbersome. Is there a convention or some reusable structure that we can import

Comment on lines +72 to +76
# Navigate from py_package.py_module to py_package
value = isolated.execute(
"import py_pkg.py1_pkg.py1_module; py_pkg.py1_pkg.py1_module.py2_method_b()"
)
assert value == "py2_method_b"
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dialecticDolt, I have reproduced your error here

@LecrisUT
Copy link
Collaborator Author

LecrisUT commented Nov 8, 2023

Hmm these are still failing, I guess because they cover the .. navigation case. The error log is not quite descriptive though. Oh, nvm found the log and the typo

@LecrisUT LecrisUT changed the title fix: Editable package navigation test: editable install cover package navigation Nov 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant