Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .gitignore
Original file line numberDiff line numberDiff line change
Expand Up@@ -10,7 +10,7 @@ __pycache__/
.Python
build/
develop-eggs/
dist/
/dist/
downloads/
eggs/
.eggs/
Expand Down
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
# **Upcoming release**

- #730 Match on module aliases for autoimport suggestions
- #755 Remove dependency on `build` package being installed while running tests

# Release 1.12.0

Expand Down
2 changes: 1 addition & 1 deletion MANIFEST.in
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
include README.rst COPYING setup.py MANIFEST.in CHANGELOG.md
include README.rst COPYING setup.py MANIFEST.in CHANGELOG.md ropetest-package-fixtures/external_fixturepkg/dist/external_fixturepkg-1.0.0-py3-none-any.whl ropetest-package-fixtures/external_fixturepkg/dist/external_fixturepkg-1.0.0.tar.gz
recursive-include rope *.py
recursive-include docs *.rst
recursive-include ropetest *.py
1 change: 1 addition & 0 deletions pyproject.toml
Original file line numberDiff line numberDiff line change
Expand Up@@ -87,6 +87,7 @@ profile = "black"

[tool.pytest.ini_options]

testpaths = "ropetest"
python_files = [
"*test.py",
"__init__.py",
Expand Down
2 changes: 1 addition & 1 deletion rope/contrib/autoimport/utils.py
Original file line numberDiff line numberDiff line change
Expand Up@@ -54,7 +54,7 @@ def get_package_source(
if "site-packages" in package.parts:
return Source.SITE_PACKAGE
if sys.version_info < (3, 10, 0):
if str(package).startswith(sys.prefix):
if str(package).startswith(sys.base_prefix):
return Source.STANDARD
else:
if name in sys.stdlib_module_names:
Expand Down
12 changes: 12 additions & 0 deletions ropetest-package-fixtures/external_fixturepkg/README.md
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
# example_project

Just an example project for testing rope.


To build this package, run:

```bash
$ python -m build
```

This generates packages in `dist` folder.
Binary file not shown.
Binary file not shown.
18 changes: 18 additions & 0 deletions ropetest-package-fixtures/external_fixturepkg/pyproject.toml
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
[build-system]
requires = ["flit-core >= 3.8"]
build-backend = "flit_core.buildapi"

[project]
name = "external_fixturepkg"
version = "1.0.0"
description = "Just an example project built with build/flit for testing purpose"
readme = "README.md"
requires-python = ">= 3.7"

dependencies = [ ]

[project.scripts]
pyproject-build = "build.__main__:entrypoint"

[tool.flit.sdist]
exclude = ["**/__pycache__", "**/*.egg-info"]
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
def sample_function():
pass
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
def main() -> None:
pass


if __name__ == '__main__':
main()
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
foo = None
23 changes: 16 additions & 7 deletions ropetest/conftest.py
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
import pathlib
import sys
from subprocess import check_call

import pytest

Expand All@@ -13,13 +15,6 @@ def project():
testutils.remove_project(project)


@pytest.fixture
def project2():
project = testutils.sample_project("another_project")
yield project
testutils.remove_project(project)


@pytest.fixture
def project_path(project):
yield pathlib.Path(project.address)
Expand DownExpand Up@@ -52,3 +47,17 @@ def pkg1(project) -> resources.Folder:
@pytest.fixture
def mod2(project, pkg1) -> resources.Folder:
return testutils.create_module(project, "mod2", pkg1)


@pytest.fixture(scope="session")
def external_fixturepkg():
check_call([
sys.executable,
"-m",
"pip",
"install",
"--force-reinstall",
"ropetest-package-fixtures/external_fixturepkg/dist/external_fixturepkg-1.0.0-py3-none-any.whl",
])
yield
check_call([sys.executable, "-m", "pip", "uninstall", "--yes", "external-fixturepkg"])
13 changes: 6 additions & 7 deletions ropetest/contrib/autoimport/conftest.py
Original file line numberDiff line numberDiff line change
Expand Up@@ -24,19 +24,18 @@ def typing_path():


@pytest.fixture
def build_env_path():
from build import env

yield pathlib.Path(env.__file__)
def example_external_package_module_path(external_fixturepkg):
from external_fixturepkg import mod1
yield pathlib.Path(mod1.__file__)


@pytest.fixture
def build_path():
import build
def example_external_package_path(external_fixturepkg):
import external_fixturepkg

# Uses __init__.py so we need the parent

yield pathlib.Path(build.__file__).parent
yield pathlib.Path(external_fixturepkg.__file__).parent


@pytest.fixture
Expand Down
20 changes: 12 additions & 8 deletions ropetest/contrib/autoimport/utilstest.py
Original file line numberDiff line numberDiff line change
Expand Up@@ -14,31 +14,35 @@ def test_get_package_source_not_project(mod1_path):
assert utils.get_package_source(mod1_path, None, "") == Source.UNKNOWN


def test_get_package_source_pytest(build_path):
def test_get_package_source_pytest(example_external_package_path):
# pytest is not installed as part of the standard library
# but should be installed into site_packages,
# so it should return Source.SITE_PACKAGE
assert utils.get_package_source(build_path, None, "build") == Source.SITE_PACKAGE
source = utils.get_package_source(example_external_package_path, None, "mod1")
assert source == Source.SITE_PACKAGE


def test_get_package_source_typing(typing_path):

assert utils.get_package_source(typing_path, None, "typing") == Source.STANDARD


def test_get_modname_project_no_add(mod1_path, project_path):

assert utils.get_modname_from_path(mod1_path, project_path, False) == "mod1"


def test_get_modname_single_file(typing_path):

assert utils.get_modname_from_path(typing_path, typing_path) == "typing"


def test_get_modname_folder(build_path, build_env_path):

assert utils.get_modname_from_path(build_env_path, build_path) == "build.env"
def test_get_modname_folder(
example_external_package_path,
example_external_package_module_path,
):
modname = utils.get_modname_from_path(
example_external_package_module_path,
example_external_package_path,
)
assert modname == "external_fixturepkg.mod1"


def test_get_package_tuple_sample(project_path):
Expand Down
17 changes: 10 additions & 7 deletions ropetest/contrib/autoimporttest.py
Original file line numberDiff line numberDiff line change
Expand Up@@ -111,13 +111,6 @@ def test_handling_builtin_modules(self):
self.importer.update_module("sys")
self.assertIn("sys", self.importer.get_modules("exit"))

def test_search_submodule(self):
self.importer.update_module("build")
import_statement = ("from build import env", "env")
self.assertIn(import_statement, self.importer.search("env", exact_match=True))
self.assertIn(import_statement, self.importer.search("en"))
self.assertIn(import_statement, self.importer.search("env"))

def test_search_module(self):
self.importer.update_module("os")
import_statement = ("import os", "os")
Expand DownExpand Up@@ -184,6 +177,16 @@ def test_skipping_directories_not_accessible_because_of_permission_error(self):
self.assertGreater(len(self.importer._dump_all()), 0)


def test_search_submodule(external_fixturepkg):
project = testutils.sample_project(extension_modules=["sys"])
importer = autoimport.AutoImport(project, observe=False)
importer.update_module("external_fixturepkg")
import_statement = ("from external_fixturepkg import mod1", "mod1")
assert import_statement in importer.search("mod1", exact_match=True)
assert import_statement in importer.search("mo")
assert import_statement in importer.search("mod1")


class AutoImportObservingTest(unittest.TestCase):
def setUp(self):
super().setUp()
Expand Down
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .gitignore
Original file line numberDiff line numberDiff line change
Expand Up@@ -10,7 +10,7 @@ __pycache__/
.Python
build/
develop-eggs/
dist/
/dist/
downloads/
eggs/
.eggs/
Expand Down
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
# **Upcoming release**

- #730 Match on module aliases for autoimport suggestions
- #755 Remove dependency on `build` package being installed while running tests

# Release 1.12.0

Expand Down
2 changes: 1 addition & 1 deletion MANIFEST.in
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
include README.rst COPYING setup.py MANIFEST.in CHANGELOG.md
include README.rst COPYING setup.py MANIFEST.in CHANGELOG.md ropetest-package-fixtures/external_fixturepkg/dist/external_fixturepkg-1.0.0-py3-none-any.whl ropetest-package-fixtures/external_fixturepkg/dist/external_fixturepkg-1.0.0.tar.gz
recursive-include rope *.py
recursive-include docs *.rst
recursive-include ropetest *.py
1 change: 1 addition & 0 deletions pyproject.toml
Original file line numberDiff line numberDiff line change
Expand Up@@ -87,6 +87,7 @@ profile = "black"

[tool.pytest.ini_options]

testpaths = "ropetest"
python_files = [
"*test.py",
"__init__.py",
Expand Down
2 changes: 1 addition & 1 deletion rope/contrib/autoimport/utils.py
Original file line numberDiff line numberDiff line change
Expand Up@@ -54,7 +54,7 @@ def get_package_source(
if "site-packages" in package.parts:
return Source.SITE_PACKAGE
if sys.version_info < (3, 10, 0):
if str(package).startswith(sys.prefix):
if str(package).startswith(sys.base_prefix):
return Source.STANDARD
else:
if name in sys.stdlib_module_names:
Expand Down
12 changes: 12 additions & 0 deletions ropetest-package-fixtures/external_fixturepkg/README.md
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
# example_project

Just an example project for testing rope.


To build this package, run:

```bash
$ python -m build
```

This generates packages in `dist` folder.
Binary file not shown.
Binary file not shown.
18 changes: 18 additions & 0 deletions ropetest-package-fixtures/external_fixturepkg/pyproject.toml
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
[build-system]
requires = ["flit-core >= 3.8"]
build-backend = "flit_core.buildapi"

[project]
name = "external_fixturepkg"
version = "1.0.0"
description = "Just an example project built with build/flit for testing purpose"
readme = "README.md"
requires-python = ">= 3.7"

dependencies = [ ]

[project.scripts]
pyproject-build = "build.__main__:entrypoint"

[tool.flit.sdist]
exclude = ["**/__pycache__", "**/*.egg-info"]
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
def sample_function():
pass
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
def main() -> None:
pass


if __name__ == '__main__':
main()
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
foo = None
23 changes: 16 additions & 7 deletions ropetest/conftest.py
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
import pathlib
import sys
from subprocess import check_call

import pytest

Expand All@@ -13,13 +15,6 @@ def project():
testutils.remove_project(project)


@pytest.fixture
def project2():
project = testutils.sample_project("another_project")
yield project
testutils.remove_project(project)


@pytest.fixture
def project_path(project):
yield pathlib.Path(project.address)
Expand DownExpand Up@@ -52,3 +47,17 @@ def pkg1(project) -> resources.Folder:
@pytest.fixture
def mod2(project, pkg1) -> resources.Folder:
return testutils.create_module(project, "mod2", pkg1)


@pytest.fixture(scope="session")
def external_fixturepkg():
check_call([
sys.executable,
"-m",
"pip",
"install",
"--force-reinstall",
"ropetest-package-fixtures/external_fixturepkg/dist/external_fixturepkg-1.0.0-py3-none-any.whl",
])
yield
check_call([sys.executable, "-m", "pip", "uninstall", "--yes", "external-fixturepkg"])
13 changes: 6 additions & 7 deletions ropetest/contrib/autoimport/conftest.py
Original file line numberDiff line numberDiff line change
Expand Up@@ -24,19 +24,18 @@ def typing_path():


@pytest.fixture
def build_env_path():
from build import env

yield pathlib.Path(env.__file__)
def example_external_package_module_path(external_fixturepkg):
from external_fixturepkg import mod1
yield pathlib.Path(mod1.__file__)


@pytest.fixture
def build_path():
import build
def example_external_package_path(external_fixturepkg):
import external_fixturepkg

# Uses __init__.py so we need the parent

yield pathlib.Path(build.__file__).parent
yield pathlib.Path(external_fixturepkg.__file__).parent


@pytest.fixture
Expand Down
20 changes: 12 additions & 8 deletions ropetest/contrib/autoimport/utilstest.py
Original file line numberDiff line numberDiff line change
Expand Up@@ -14,31 +14,35 @@ def test_get_package_source_not_project(mod1_path):
assert utils.get_package_source(mod1_path, None, "") == Source.UNKNOWN


def test_get_package_source_pytest(build_path):
def test_get_package_source_pytest(example_external_package_path):
# pytest is not installed as part of the standard library
# but should be installed into site_packages,
# so it should return Source.SITE_PACKAGE
assert utils.get_package_source(build_path, None, "build") == Source.SITE_PACKAGE
source = utils.get_package_source(example_external_package_path, None, "mod1")
assert source == Source.SITE_PACKAGE


def test_get_package_source_typing(typing_path):

assert utils.get_package_source(typing_path, None, "typing") == Source.STANDARD


def test_get_modname_project_no_add(mod1_path, project_path):

assert utils.get_modname_from_path(mod1_path, project_path, False) == "mod1"


def test_get_modname_single_file(typing_path):

assert utils.get_modname_from_path(typing_path, typing_path) == "typing"


def test_get_modname_folder(build_path, build_env_path):

assert utils.get_modname_from_path(build_env_path, build_path) == "build.env"
def test_get_modname_folder(
example_external_package_path,
example_external_package_module_path,
):
modname = utils.get_modname_from_path(
example_external_package_module_path,
example_external_package_path,
)
assert modname == "external_fixturepkg.mod1"


def test_get_package_tuple_sample(project_path):
Expand Down
17 changes: 10 additions & 7 deletions ropetest/contrib/autoimporttest.py
Original file line numberDiff line numberDiff line change
Expand Up@@ -111,13 +111,6 @@ def test_handling_builtin_modules(self):
self.importer.update_module("sys")
self.assertIn("sys", self.importer.get_modules("exit"))

def test_search_submodule(self):
self.importer.update_module("build")
import_statement = ("from build import env", "env")
self.assertIn(import_statement, self.importer.search("env", exact_match=True))
self.assertIn(import_statement, self.importer.search("en"))
self.assertIn(import_statement, self.importer.search("env"))

def test_search_module(self):
self.importer.update_module("os")
import_statement = ("import os", "os")
Expand DownExpand Up@@ -184,6 +177,16 @@ def test_skipping_directories_not_accessible_because_of_permission_error(self):
self.assertGreater(len(self.importer._dump_all()), 0)


def test_search_submodule(external_fixturepkg):
project = testutils.sample_project(extension_modules=["sys"])
importer = autoimport.AutoImport(project, observe=False)
importer.update_module("external_fixturepkg")
import_statement = ("from external_fixturepkg import mod1", "mod1")
assert import_statement in importer.search("mod1", exact_match=True)
assert import_statement in importer.search("mo")
assert import_statement in importer.search("mod1")


class AutoImportObservingTest(unittest.TestCase):
def setUp(self):
super().setUp()
Expand Down
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix"); } } catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .gitignore
Original file line numberDiff line numberDiff line change
Expand Up@@ -10,7 +10,7 @@ __pycache__/
.Python
build/
develop-eggs/
dist/
/dist/
downloads/
eggs/
.eggs/
Expand Down
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
# **Upcoming release**

- #730 Match on module aliases for autoimport suggestions
- #755 Remove dependency on `build` package being installed while running tests

# Release 1.12.0

Expand Down
2 changes: 1 addition & 1 deletion MANIFEST.in
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
include README.rst COPYING setup.py MANIFEST.in CHANGELOG.md
include README.rst COPYING setup.py MANIFEST.in CHANGELOG.md ropetest-package-fixtures/external_fixturepkg/dist/external_fixturepkg-1.0.0-py3-none-any.whl ropetest-package-fixtures/external_fixturepkg/dist/external_fixturepkg-1.0.0.tar.gz
recursive-include rope *.py
recursive-include docs *.rst
recursive-include ropetest *.py
1 change: 1 addition & 0 deletions pyproject.toml
Original file line numberDiff line numberDiff line change
Expand Up@@ -87,6 +87,7 @@ profile = "black"

[tool.pytest.ini_options]

testpaths = "ropetest"
python_files = [
"*test.py",
"__init__.py",
Expand Down
2 changes: 1 addition & 1 deletion rope/contrib/autoimport/utils.py
Original file line numberDiff line numberDiff line change
Expand Up@@ -54,7 +54,7 @@ def get_package_source(
if "site-packages" in package.parts:
return Source.SITE_PACKAGE
if sys.version_info < (3, 10, 0):
if str(package).startswith(sys.prefix):
if str(package).startswith(sys.base_prefix):
return Source.STANDARD
else:
if name in sys.stdlib_module_names:
Expand Down
12 changes: 12 additions & 0 deletions ropetest-package-fixtures/external_fixturepkg/README.md
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
# example_project

Just an example project for testing rope.


To build this package, run:

```bash
$ python -m build
```

This generates packages in `dist` folder.
Binary file not shown.
Binary file not shown.
18 changes: 18 additions & 0 deletions ropetest-package-fixtures/external_fixturepkg/pyproject.toml
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
[build-system]
requires = ["flit-core >= 3.8"]
build-backend = "flit_core.buildapi"

[project]
name = "external_fixturepkg"
version = "1.0.0"
description = "Just an example project built with build/flit for testing purpose"
readme = "README.md"
requires-python = ">= 3.7"

dependencies = [ ]

[project.scripts]
pyproject-build = "build.__main__:entrypoint"

[tool.flit.sdist]
exclude = ["**/__pycache__", "**/*.egg-info"]
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
def sample_function():
pass
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
def main() -> None:
pass


if __name__ == '__main__':
main()
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
foo = None
23 changes: 16 additions & 7 deletions ropetest/conftest.py
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
import pathlib
import sys
from subprocess import check_call

import pytest

Expand All@@ -13,13 +15,6 @@ def project():
testutils.remove_project(project)


@pytest.fixture
def project2():
project = testutils.sample_project("another_project")
yield project
testutils.remove_project(project)


@pytest.fixture
def project_path(project):
yield pathlib.Path(project.address)
Expand DownExpand Up@@ -52,3 +47,17 @@ def pkg1(project) -> resources.Folder:
@pytest.fixture
def mod2(project, pkg1) -> resources.Folder:
return testutils.create_module(project, "mod2", pkg1)


@pytest.fixture(scope="session")
def external_fixturepkg():
check_call([
sys.executable,
"-m",
"pip",
"install",
"--force-reinstall",
"ropetest-package-fixtures/external_fixturepkg/dist/external_fixturepkg-1.0.0-py3-none-any.whl",
])
yield
check_call([sys.executable, "-m", "pip", "uninstall", "--yes", "external-fixturepkg"])
13 changes: 6 additions & 7 deletions ropetest/contrib/autoimport/conftest.py
Original file line numberDiff line numberDiff line change
Expand Up@@ -24,19 +24,18 @@ def typing_path():


@pytest.fixture
def build_env_path():
from build import env

yield pathlib.Path(env.__file__)
def example_external_package_module_path(external_fixturepkg):
from external_fixturepkg import mod1
yield pathlib.Path(mod1.__file__)


@pytest.fixture
def build_path():
import build
def example_external_package_path(external_fixturepkg):
import external_fixturepkg

# Uses __init__.py so we need the parent

yield pathlib.Path(build.__file__).parent
yield pathlib.Path(external_fixturepkg.__file__).parent


@pytest.fixture
Expand Down
20 changes: 12 additions & 8 deletions ropetest/contrib/autoimport/utilstest.py
Original file line numberDiff line numberDiff line change
Expand Up@@ -14,31 +14,35 @@ def test_get_package_source_not_project(mod1_path):
assert utils.get_package_source(mod1_path, None, "") == Source.UNKNOWN


def test_get_package_source_pytest(build_path):
def test_get_package_source_pytest(example_external_package_path):
# pytest is not installed as part of the standard library
# but should be installed into site_packages,
# so it should return Source.SITE_PACKAGE
assert utils.get_package_source(build_path, None, "build") == Source.SITE_PACKAGE
source = utils.get_package_source(example_external_package_path, None, "mod1")
assert source == Source.SITE_PACKAGE


def test_get_package_source_typing(typing_path):

assert utils.get_package_source(typing_path, None, "typing") == Source.STANDARD


def test_get_modname_project_no_add(mod1_path, project_path):

assert utils.get_modname_from_path(mod1_path, project_path, False) == "mod1"


def test_get_modname_single_file(typing_path):

assert utils.get_modname_from_path(typing_path, typing_path) == "typing"


def test_get_modname_folder(build_path, build_env_path):

assert utils.get_modname_from_path(build_env_path, build_path) == "build.env"
def test_get_modname_folder(
example_external_package_path,
example_external_package_module_path,
):
modname = utils.get_modname_from_path(
example_external_package_module_path,
example_external_package_path,
)
assert modname == "external_fixturepkg.mod1"


def test_get_package_tuple_sample(project_path):
Expand Down
17 changes: 10 additions & 7 deletions ropetest/contrib/autoimporttest.py
Original file line numberDiff line numberDiff line change
Expand Up@@ -111,13 +111,6 @@ def test_handling_builtin_modules(self):
self.importer.update_module("sys")
self.assertIn("sys", self.importer.get_modules("exit"))

def test_search_submodule(self):
self.importer.update_module("build")
import_statement = ("from build import env", "env")
self.assertIn(import_statement, self.importer.search("env", exact_match=True))
self.assertIn(import_statement, self.importer.search("en"))
self.assertIn(import_statement, self.importer.search("env"))

def test_search_module(self):
self.importer.update_module("os")
import_statement = ("import os", "os")
Expand DownExpand Up@@ -184,6 +177,16 @@ def test_skipping_directories_not_accessible_because_of_permission_error(self):
self.assertGreater(len(self.importer._dump_all()), 0)


def test_search_submodule(external_fixturepkg):
project = testutils.sample_project(extension_modules=["sys"])
importer = autoimport.AutoImport(project, observe=False)
importer.update_module("external_fixturepkg")
import_statement = ("from external_fixturepkg import mod1", "mod1")
assert import_statement in importer.search("mod1", exact_match=True)
assert import_statement in importer.search("mo")
assert import_statement in importer.search("mod1")


class AutoImportObservingTest(unittest.TestCase):
def setUp(self):
super().setUp()
Expand Down
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms"); } } catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .gitignore
Original file line numberDiff line numberDiff line change
Expand Up@@ -10,7 +10,7 @@ __pycache__/
.Python
build/
develop-eggs/
dist/
/dist/
downloads/
eggs/
.eggs/
Expand Down
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
# **Upcoming release**

- #730 Match on module aliases for autoimport suggestions
- #755 Remove dependency on `build` package being installed while running tests

# Release 1.12.0

Expand Down
2 changes: 1 addition & 1 deletion MANIFEST.in
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
include README.rst COPYING setup.py MANIFEST.in CHANGELOG.md
include README.rst COPYING setup.py MANIFEST.in CHANGELOG.md ropetest-package-fixtures/external_fixturepkg/dist/external_fixturepkg-1.0.0-py3-none-any.whl ropetest-package-fixtures/external_fixturepkg/dist/external_fixturepkg-1.0.0.tar.gz
recursive-include rope *.py
recursive-include docs *.rst
recursive-include ropetest *.py
1 change: 1 addition & 0 deletions pyproject.toml
Original file line numberDiff line numberDiff line change
Expand Up@@ -87,6 +87,7 @@ profile = "black"

[tool.pytest.ini_options]

testpaths = "ropetest"
python_files = [
"*test.py",
"__init__.py",
Expand Down
2 changes: 1 addition & 1 deletion rope/contrib/autoimport/utils.py
Original file line numberDiff line numberDiff line change
Expand Up@@ -54,7 +54,7 @@ def get_package_source(
if "site-packages" in package.parts:
return Source.SITE_PACKAGE
if sys.version_info < (3, 10, 0):
if str(package).startswith(sys.prefix):
if str(package).startswith(sys.base_prefix):
return Source.STANDARD
else:
if name in sys.stdlib_module_names:
Expand Down
12 changes: 12 additions & 0 deletions ropetest-package-fixtures/external_fixturepkg/README.md
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
# example_project

Just an example project for testing rope.


To build this package, run:

```bash
$ python -m build
```

This generates packages in `dist` folder.
Binary file not shown.
Binary file not shown.
18 changes: 18 additions & 0 deletions ropetest-package-fixtures/external_fixturepkg/pyproject.toml
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
[build-system]
requires = ["flit-core >= 3.8"]
build-backend = "flit_core.buildapi"

[project]
name = "external_fixturepkg"
version = "1.0.0"
description = "Just an example project built with build/flit for testing purpose"
readme = "README.md"
requires-python = ">= 3.7"

dependencies = [ ]

[project.scripts]
pyproject-build = "build.__main__:entrypoint"

[tool.flit.sdist]
exclude = ["**/__pycache__", "**/*.egg-info"]
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
def sample_function():
pass
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
def main() -> None:
pass


if __name__ == '__main__':
main()
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
foo = None
23 changes: 16 additions & 7 deletions ropetest/conftest.py
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
import pathlib
import sys
from subprocess import check_call

import pytest

Expand All@@ -13,13 +15,6 @@ def project():
testutils.remove_project(project)


@pytest.fixture
def project2():
project = testutils.sample_project("another_project")
yield project
testutils.remove_project(project)


@pytest.fixture
def project_path(project):
yield pathlib.Path(project.address)
Expand DownExpand Up@@ -52,3 +47,17 @@ def pkg1(project) -> resources.Folder:
@pytest.fixture
def mod2(project, pkg1) -> resources.Folder:
return testutils.create_module(project, "mod2", pkg1)


@pytest.fixture(scope="session")
def external_fixturepkg():
check_call([
sys.executable,
"-m",
"pip",
"install",
"--force-reinstall",
"ropetest-package-fixtures/external_fixturepkg/dist/external_fixturepkg-1.0.0-py3-none-any.whl",
])
yield
check_call([sys.executable, "-m", "pip", "uninstall", "--yes", "external-fixturepkg"])
13 changes: 6 additions & 7 deletions ropetest/contrib/autoimport/conftest.py
Original file line numberDiff line numberDiff line change
Expand Up@@ -24,19 +24,18 @@ def typing_path():


@pytest.fixture
def build_env_path():
from build import env

yield pathlib.Path(env.__file__)
def example_external_package_module_path(external_fixturepkg):
from external_fixturepkg import mod1
yield pathlib.Path(mod1.__file__)


@pytest.fixture
def build_path():
import build
def example_external_package_path(external_fixturepkg):
import external_fixturepkg

# Uses __init__.py so we need the parent

yield pathlib.Path(build.__file__).parent
yield pathlib.Path(external_fixturepkg.__file__).parent


@pytest.fixture
Expand Down
20 changes: 12 additions & 8 deletions ropetest/contrib/autoimport/utilstest.py
Original file line numberDiff line numberDiff line change
Expand Up@@ -14,31 +14,35 @@ def test_get_package_source_not_project(mod1_path):
assert utils.get_package_source(mod1_path, None, "") == Source.UNKNOWN


def test_get_package_source_pytest(build_path):
def test_get_package_source_pytest(example_external_package_path):
# pytest is not installed as part of the standard library
# but should be installed into site_packages,
# so it should return Source.SITE_PACKAGE
assert utils.get_package_source(build_path, None, "build") == Source.SITE_PACKAGE
source = utils.get_package_source(example_external_package_path, None, "mod1")
assert source == Source.SITE_PACKAGE


def test_get_package_source_typing(typing_path):

assert utils.get_package_source(typing_path, None, "typing") == Source.STANDARD


def test_get_modname_project_no_add(mod1_path, project_path):

assert utils.get_modname_from_path(mod1_path, project_path, False) == "mod1"


def test_get_modname_single_file(typing_path):

assert utils.get_modname_from_path(typing_path, typing_path) == "typing"


def test_get_modname_folder(build_path, build_env_path):

assert utils.get_modname_from_path(build_env_path, build_path) == "build.env"
def test_get_modname_folder(
example_external_package_path,
example_external_package_module_path,
):
modname = utils.get_modname_from_path(
example_external_package_module_path,
example_external_package_path,
)
assert modname == "external_fixturepkg.mod1"


def test_get_package_tuple_sample(project_path):
Expand Down
17 changes: 10 additions & 7 deletions ropetest/contrib/autoimporttest.py
Original file line numberDiff line numberDiff line change
Expand Up@@ -111,13 +111,6 @@ def test_handling_builtin_modules(self):
self.importer.update_module("sys")
self.assertIn("sys", self.importer.get_modules("exit"))

def test_search_submodule(self):
self.importer.update_module("build")
import_statement = ("from build import env", "env")
self.assertIn(import_statement, self.importer.search("env", exact_match=True))
self.assertIn(import_statement, self.importer.search("en"))
self.assertIn(import_statement, self.importer.search("env"))

def test_search_module(self):
self.importer.update_module("os")
import_statement = ("import os", "os")
Expand DownExpand Up@@ -184,6 +177,16 @@ def test_skipping_directories_not_accessible_because_of_permission_error(self):
self.assertGreater(len(self.importer._dump_all()), 0)


def test_search_submodule(external_fixturepkg):
project = testutils.sample_project(extension_modules=["sys"])
importer = autoimport.AutoImport(project, observe=False)
importer.update_module("external_fixturepkg")
import_statement = ("from external_fixturepkg import mod1", "mod1")
assert import_statement in importer.search("mod1", exact_match=True)
assert import_statement in importer.search("mo")
assert import_statement in importer.search("mod1")


class AutoImportObservingTest(unittest.TestCase):
def setUp(self):
super().setUp()
Expand Down
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links"); } } catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); } })(); (function(){ try { var __m = "youtube.com"; var __re = new RegExp('^' + "youtube\\.com" + '
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .gitignore
Original file line numberDiff line numberDiff line change
Expand Up@@ -10,7 +10,7 @@ __pycache__/
.Python
build/
develop-eggs/
dist/
/dist/
downloads/
eggs/
.eggs/
Expand Down
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
# **Upcoming release**

- #730 Match on module aliases for autoimport suggestions
- #755 Remove dependency on `build` package being installed while running tests

# Release 1.12.0

Expand Down
2 changes: 1 addition & 1 deletion MANIFEST.in
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
include README.rst COPYING setup.py MANIFEST.in CHANGELOG.md
include README.rst COPYING setup.py MANIFEST.in CHANGELOG.md ropetest-package-fixtures/external_fixturepkg/dist/external_fixturepkg-1.0.0-py3-none-any.whl ropetest-package-fixtures/external_fixturepkg/dist/external_fixturepkg-1.0.0.tar.gz
recursive-include rope *.py
recursive-include docs *.rst
recursive-include ropetest *.py
1 change: 1 addition & 0 deletions pyproject.toml
Original file line numberDiff line numberDiff line change
Expand Up@@ -87,6 +87,7 @@ profile = "black"

[tool.pytest.ini_options]

testpaths = "ropetest"
python_files = [
"*test.py",
"__init__.py",
Expand Down
2 changes: 1 addition & 1 deletion rope/contrib/autoimport/utils.py
Original file line numberDiff line numberDiff line change
Expand Up@@ -54,7 +54,7 @@ def get_package_source(
if "site-packages" in package.parts:
return Source.SITE_PACKAGE
if sys.version_info < (3, 10, 0):
if str(package).startswith(sys.prefix):
if str(package).startswith(sys.base_prefix):
return Source.STANDARD
else:
if name in sys.stdlib_module_names:
Expand Down
12 changes: 12 additions & 0 deletions ropetest-package-fixtures/external_fixturepkg/README.md
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
# example_project

Just an example project for testing rope.


To build this package, run:

```bash
$ python -m build
```

This generates packages in `dist` folder.
Binary file not shown.
Binary file not shown.
18 changes: 18 additions & 0 deletions ropetest-package-fixtures/external_fixturepkg/pyproject.toml
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
[build-system]
requires = ["flit-core >= 3.8"]
build-backend = "flit_core.buildapi"

[project]
name = "external_fixturepkg"
version = "1.0.0"
description = "Just an example project built with build/flit for testing purpose"
readme = "README.md"
requires-python = ">= 3.7"

dependencies = [ ]

[project.scripts]
pyproject-build = "build.__main__:entrypoint"

[tool.flit.sdist]
exclude = ["**/__pycache__", "**/*.egg-info"]
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
def sample_function():
pass
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
def main() -> None:
pass


if __name__ == '__main__':
main()
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
foo = None
23 changes: 16 additions & 7 deletions ropetest/conftest.py
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
import pathlib
import sys
from subprocess import check_call

import pytest

Expand All@@ -13,13 +15,6 @@ def project():
testutils.remove_project(project)


@pytest.fixture
def project2():
project = testutils.sample_project("another_project")
yield project
testutils.remove_project(project)


@pytest.fixture
def project_path(project):
yield pathlib.Path(project.address)
Expand DownExpand Up@@ -52,3 +47,17 @@ def pkg1(project) -> resources.Folder:
@pytest.fixture
def mod2(project, pkg1) -> resources.Folder:
return testutils.create_module(project, "mod2", pkg1)


@pytest.fixture(scope="session")
def external_fixturepkg():
check_call([
sys.executable,
"-m",
"pip",
"install",
"--force-reinstall",
"ropetest-package-fixtures/external_fixturepkg/dist/external_fixturepkg-1.0.0-py3-none-any.whl",
])
yield
check_call([sys.executable, "-m", "pip", "uninstall", "--yes", "external-fixturepkg"])
13 changes: 6 additions & 7 deletions ropetest/contrib/autoimport/conftest.py
Original file line numberDiff line numberDiff line change
Expand Up@@ -24,19 +24,18 @@ def typing_path():


@pytest.fixture
def build_env_path():
from build import env

yield pathlib.Path(env.__file__)
def example_external_package_module_path(external_fixturepkg):
from external_fixturepkg import mod1
yield pathlib.Path(mod1.__file__)


@pytest.fixture
def build_path():
import build
def example_external_package_path(external_fixturepkg):
import external_fixturepkg

# Uses __init__.py so we need the parent

yield pathlib.Path(build.__file__).parent
yield pathlib.Path(external_fixturepkg.__file__).parent


@pytest.fixture
Expand Down
20 changes: 12 additions & 8 deletions ropetest/contrib/autoimport/utilstest.py
Original file line numberDiff line numberDiff line change
Expand Up@@ -14,31 +14,35 @@ def test_get_package_source_not_project(mod1_path):
assert utils.get_package_source(mod1_path, None, "") == Source.UNKNOWN


def test_get_package_source_pytest(build_path):
def test_get_package_source_pytest(example_external_package_path):
# pytest is not installed as part of the standard library
# but should be installed into site_packages,
# so it should return Source.SITE_PACKAGE
assert utils.get_package_source(build_path, None, "build") == Source.SITE_PACKAGE
source = utils.get_package_source(example_external_package_path, None, "mod1")
assert source == Source.SITE_PACKAGE


def test_get_package_source_typing(typing_path):

assert utils.get_package_source(typing_path, None, "typing") == Source.STANDARD


def test_get_modname_project_no_add(mod1_path, project_path):

assert utils.get_modname_from_path(mod1_path, project_path, False) == "mod1"


def test_get_modname_single_file(typing_path):

assert utils.get_modname_from_path(typing_path, typing_path) == "typing"


def test_get_modname_folder(build_path, build_env_path):

assert utils.get_modname_from_path(build_env_path, build_path) == "build.env"
def test_get_modname_folder(
example_external_package_path,
example_external_package_module_path,
):
modname = utils.get_modname_from_path(
example_external_package_module_path,
example_external_package_path,
)
assert modname == "external_fixturepkg.mod1"


def test_get_package_tuple_sample(project_path):
Expand Down
17 changes: 10 additions & 7 deletions ropetest/contrib/autoimporttest.py
Original file line numberDiff line numberDiff line change
Expand Up@@ -111,13 +111,6 @@ def test_handling_builtin_modules(self):
self.importer.update_module("sys")
self.assertIn("sys", self.importer.get_modules("exit"))

def test_search_submodule(self):
self.importer.update_module("build")
import_statement = ("from build import env", "env")
self.assertIn(import_statement, self.importer.search("env", exact_match=True))
self.assertIn(import_statement, self.importer.search("en"))
self.assertIn(import_statement, self.importer.search("env"))

def test_search_module(self):
self.importer.update_module("os")
import_statement = ("import os", "os")
Expand DownExpand Up@@ -184,6 +177,16 @@ def test_skipping_directories_not_accessible_because_of_permission_error(self):
self.assertGreater(len(self.importer._dump_all()), 0)


def test_search_submodule(external_fixturepkg):
project = testutils.sample_project(extension_modules=["sys"])
importer = autoimport.AutoImport(project, observe=False)
importer.update_module("external_fixturepkg")
import_statement = ("from external_fixturepkg import mod1", "mod1")
assert import_statement in importer.search("mod1", exact_match=True)
assert import_statement in importer.search("mo")
assert import_statement in importer.search("mod1")


class AutoImportObservingTest(unittest.TestCase):
def setUp(self):
super().setUp()
Expand Down
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default"); } } catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .gitignore
Original file line numberDiff line numberDiff line change
Expand Up@@ -10,7 +10,7 @@ __pycache__/
.Python
build/
develop-eggs/
dist/
/dist/
downloads/
eggs/
.eggs/
Expand Down
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
# **Upcoming release**

- #730 Match on module aliases for autoimport suggestions
- #755 Remove dependency on `build` package being installed while running tests

# Release 1.12.0

Expand Down
2 changes: 1 addition & 1 deletion MANIFEST.in
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
include README.rst COPYING setup.py MANIFEST.in CHANGELOG.md
include README.rst COPYING setup.py MANIFEST.in CHANGELOG.md ropetest-package-fixtures/external_fixturepkg/dist/external_fixturepkg-1.0.0-py3-none-any.whl ropetest-package-fixtures/external_fixturepkg/dist/external_fixturepkg-1.0.0.tar.gz
recursive-include rope *.py
recursive-include docs *.rst
recursive-include ropetest *.py
1 change: 1 addition & 0 deletions pyproject.toml
Original file line numberDiff line numberDiff line change
Expand Up@@ -87,6 +87,7 @@ profile = "black"

[tool.pytest.ini_options]

testpaths = "ropetest"
python_files = [
"*test.py",
"__init__.py",
Expand Down
2 changes: 1 addition & 1 deletion rope/contrib/autoimport/utils.py
Original file line numberDiff line numberDiff line change
Expand Up@@ -54,7 +54,7 @@ def get_package_source(
if "site-packages" in package.parts:
return Source.SITE_PACKAGE
if sys.version_info < (3, 10, 0):
if str(package).startswith(sys.prefix):
if str(package).startswith(sys.base_prefix):
return Source.STANDARD
else:
if name in sys.stdlib_module_names:
Expand Down
12 changes: 12 additions & 0 deletions ropetest-package-fixtures/external_fixturepkg/README.md
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
# example_project

Just an example project for testing rope.


To build this package, run:

```bash
$ python -m build
```

This generates packages in `dist` folder.
Binary file not shown.
Binary file not shown.
18 changes: 18 additions & 0 deletions ropetest-package-fixtures/external_fixturepkg/pyproject.toml
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
[build-system]
requires = ["flit-core >= 3.8"]
build-backend = "flit_core.buildapi"

[project]
name = "external_fixturepkg"
version = "1.0.0"
description = "Just an example project built with build/flit for testing purpose"
readme = "README.md"
requires-python = ">= 3.7"

dependencies = [ ]

[project.scripts]
pyproject-build = "build.__main__:entrypoint"

[tool.flit.sdist]
exclude = ["**/__pycache__", "**/*.egg-info"]
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
def sample_function():
pass
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
def main() -> None:
pass


if __name__ == '__main__':
main()
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
foo = None
23 changes: 16 additions & 7 deletions ropetest/conftest.py
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
import pathlib
import sys
from subprocess import check_call

import pytest

Expand All@@ -13,13 +15,6 @@ def project():
testutils.remove_project(project)


@pytest.fixture
def project2():
project = testutils.sample_project("another_project")
yield project
testutils.remove_project(project)


@pytest.fixture
def project_path(project):
yield pathlib.Path(project.address)
Expand DownExpand Up@@ -52,3 +47,17 @@ def pkg1(project) -> resources.Folder:
@pytest.fixture
def mod2(project, pkg1) -> resources.Folder:
return testutils.create_module(project, "mod2", pkg1)


@pytest.fixture(scope="session")
def external_fixturepkg():
check_call([
sys.executable,
"-m",
"pip",
"install",
"--force-reinstall",
"ropetest-package-fixtures/external_fixturepkg/dist/external_fixturepkg-1.0.0-py3-none-any.whl",
])
yield
check_call([sys.executable, "-m", "pip", "uninstall", "--yes", "external-fixturepkg"])
13 changes: 6 additions & 7 deletions ropetest/contrib/autoimport/conftest.py
Original file line numberDiff line numberDiff line change
Expand Up@@ -24,19 +24,18 @@ def typing_path():


@pytest.fixture
def build_env_path():
from build import env

yield pathlib.Path(env.__file__)
def example_external_package_module_path(external_fixturepkg):
from external_fixturepkg import mod1
yield pathlib.Path(mod1.__file__)


@pytest.fixture
def build_path():
import build
def example_external_package_path(external_fixturepkg):
import external_fixturepkg

# Uses __init__.py so we need the parent

yield pathlib.Path(build.__file__).parent
yield pathlib.Path(external_fixturepkg.__file__).parent


@pytest.fixture
Expand Down
20 changes: 12 additions & 8 deletions ropetest/contrib/autoimport/utilstest.py
Original file line numberDiff line numberDiff line change
Expand Up@@ -14,31 +14,35 @@ def test_get_package_source_not_project(mod1_path):
assert utils.get_package_source(mod1_path, None, "") == Source.UNKNOWN


def test_get_package_source_pytest(build_path):
def test_get_package_source_pytest(example_external_package_path):
# pytest is not installed as part of the standard library
# but should be installed into site_packages,
# so it should return Source.SITE_PACKAGE
assert utils.get_package_source(build_path, None, "build") == Source.SITE_PACKAGE
source = utils.get_package_source(example_external_package_path, None, "mod1")
assert source == Source.SITE_PACKAGE


def test_get_package_source_typing(typing_path):

assert utils.get_package_source(typing_path, None, "typing") == Source.STANDARD


def test_get_modname_project_no_add(mod1_path, project_path):

assert utils.get_modname_from_path(mod1_path, project_path, False) == "mod1"


def test_get_modname_single_file(typing_path):

assert utils.get_modname_from_path(typing_path, typing_path) == "typing"


def test_get_modname_folder(build_path, build_env_path):

assert utils.get_modname_from_path(build_env_path, build_path) == "build.env"
def test_get_modname_folder(
example_external_package_path,
example_external_package_module_path,
):
modname = utils.get_modname_from_path(
example_external_package_module_path,
example_external_package_path,
)
assert modname == "external_fixturepkg.mod1"


def test_get_package_tuple_sample(project_path):
Expand Down
17 changes: 10 additions & 7 deletions ropetest/contrib/autoimporttest.py
Original file line numberDiff line numberDiff line change
Expand Up@@ -111,13 +111,6 @@ def test_handling_builtin_modules(self):
self.importer.update_module("sys")
self.assertIn("sys", self.importer.get_modules("exit"))

def test_search_submodule(self):
self.importer.update_module("build")
import_statement = ("from build import env", "env")
self.assertIn(import_statement, self.importer.search("env", exact_match=True))
self.assertIn(import_statement, self.importer.search("en"))
self.assertIn(import_statement, self.importer.search("env"))

def test_search_module(self):
self.importer.update_module("os")
import_statement = ("import os", "os")
Expand DownExpand Up@@ -184,6 +177,16 @@ def test_skipping_directories_not_accessible_because_of_permission_error(self):
self.assertGreater(len(self.importer._dump_all()), 0)


def test_search_submodule(external_fixturepkg):
project = testutils.sample_project(extension_modules=["sys"])
importer = autoimport.AutoImport(project, observe=False)
importer.update_module("external_fixturepkg")
import_statement = ("from external_fixturepkg import mod1", "mod1")
assert import_statement in importer.search("mod1", exact_match=True)
assert import_statement in importer.search("mo")
assert import_statement in importer.search("mod1")


class AutoImportObservingTest(unittest.TestCase):
def setUp(self):
super().setUp()
Expand Down
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers"); } } catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .gitignore
Original file line numberDiff line numberDiff line change
Expand Up@@ -10,7 +10,7 @@ __pycache__/
.Python
build/
develop-eggs/
dist/
/dist/
downloads/
eggs/
.eggs/
Expand Down
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
# **Upcoming release**

- #730 Match on module aliases for autoimport suggestions
- #755 Remove dependency on `build` package being installed while running tests

# Release 1.12.0

Expand Down
2 changes: 1 addition & 1 deletion MANIFEST.in
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
include README.rst COPYING setup.py MANIFEST.in CHANGELOG.md
include README.rst COPYING setup.py MANIFEST.in CHANGELOG.md ropetest-package-fixtures/external_fixturepkg/dist/external_fixturepkg-1.0.0-py3-none-any.whl ropetest-package-fixtures/external_fixturepkg/dist/external_fixturepkg-1.0.0.tar.gz
recursive-include rope *.py
recursive-include docs *.rst
recursive-include ropetest *.py
1 change: 1 addition & 0 deletions pyproject.toml
Original file line numberDiff line numberDiff line change
Expand Up@@ -87,6 +87,7 @@ profile = "black"

[tool.pytest.ini_options]

testpaths = "ropetest"
python_files = [
"*test.py",
"__init__.py",
Expand Down
2 changes: 1 addition & 1 deletion rope/contrib/autoimport/utils.py
Original file line numberDiff line numberDiff line change
Expand Up@@ -54,7 +54,7 @@ def get_package_source(
if "site-packages" in package.parts:
return Source.SITE_PACKAGE
if sys.version_info < (3, 10, 0):
if str(package).startswith(sys.prefix):
if str(package).startswith(sys.base_prefix):
return Source.STANDARD
else:
if name in sys.stdlib_module_names:
Expand Down
12 changes: 12 additions & 0 deletions ropetest-package-fixtures/external_fixturepkg/README.md
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
# example_project

Just an example project for testing rope.


To build this package, run:

```bash
$ python -m build
```

This generates packages in `dist` folder.
Binary file not shown.
Binary file not shown.
18 changes: 18 additions & 0 deletions ropetest-package-fixtures/external_fixturepkg/pyproject.toml
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
[build-system]
requires = ["flit-core >= 3.8"]
build-backend = "flit_core.buildapi"

[project]
name = "external_fixturepkg"
version = "1.0.0"
description = "Just an example project built with build/flit for testing purpose"
readme = "README.md"
requires-python = ">= 3.7"

dependencies = [ ]

[project.scripts]
pyproject-build = "build.__main__:entrypoint"

[tool.flit.sdist]
exclude = ["**/__pycache__", "**/*.egg-info"]
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
def sample_function():
pass
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
def main() -> None:
pass


if __name__ == '__main__':
main()
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
foo = None
23 changes: 16 additions & 7 deletions ropetest/conftest.py
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
import pathlib
import sys
from subprocess import check_call

import pytest

Expand All@@ -13,13 +15,6 @@ def project():
testutils.remove_project(project)


@pytest.fixture
def project2():
project = testutils.sample_project("another_project")
yield project
testutils.remove_project(project)


@pytest.fixture
def project_path(project):
yield pathlib.Path(project.address)
Expand DownExpand Up@@ -52,3 +47,17 @@ def pkg1(project) -> resources.Folder:
@pytest.fixture
def mod2(project, pkg1) -> resources.Folder:
return testutils.create_module(project, "mod2", pkg1)


@pytest.fixture(scope="session")
def external_fixturepkg():
check_call([
sys.executable,
"-m",
"pip",
"install",
"--force-reinstall",
"ropetest-package-fixtures/external_fixturepkg/dist/external_fixturepkg-1.0.0-py3-none-any.whl",
])
yield
check_call([sys.executable, "-m", "pip", "uninstall", "--yes", "external-fixturepkg"])
13 changes: 6 additions & 7 deletions ropetest/contrib/autoimport/conftest.py
Original file line numberDiff line numberDiff line change
Expand Up@@ -24,19 +24,18 @@ def typing_path():


@pytest.fixture
def build_env_path():
from build import env

yield pathlib.Path(env.__file__)
def example_external_package_module_path(external_fixturepkg):
from external_fixturepkg import mod1
yield pathlib.Path(mod1.__file__)


@pytest.fixture
def build_path():
import build
def example_external_package_path(external_fixturepkg):
import external_fixturepkg

# Uses __init__.py so we need the parent

yield pathlib.Path(build.__file__).parent
yield pathlib.Path(external_fixturepkg.__file__).parent


@pytest.fixture
Expand Down
20 changes: 12 additions & 8 deletions ropetest/contrib/autoimport/utilstest.py
Original file line numberDiff line numberDiff line change
Expand Up@@ -14,31 +14,35 @@ def test_get_package_source_not_project(mod1_path):
assert utils.get_package_source(mod1_path, None, "") == Source.UNKNOWN


def test_get_package_source_pytest(build_path):
def test_get_package_source_pytest(example_external_package_path):
# pytest is not installed as part of the standard library
# but should be installed into site_packages,
# so it should return Source.SITE_PACKAGE
assert utils.get_package_source(build_path, None, "build") == Source.SITE_PACKAGE
source = utils.get_package_source(example_external_package_path, None, "mod1")
assert source == Source.SITE_PACKAGE


def test_get_package_source_typing(typing_path):

assert utils.get_package_source(typing_path, None, "typing") == Source.STANDARD


def test_get_modname_project_no_add(mod1_path, project_path):

assert utils.get_modname_from_path(mod1_path, project_path, False) == "mod1"


def test_get_modname_single_file(typing_path):

assert utils.get_modname_from_path(typing_path, typing_path) == "typing"


def test_get_modname_folder(build_path, build_env_path):

assert utils.get_modname_from_path(build_env_path, build_path) == "build.env"
def test_get_modname_folder(
example_external_package_path,
example_external_package_module_path,
):
modname = utils.get_modname_from_path(
example_external_package_module_path,
example_external_package_path,
)
assert modname == "external_fixturepkg.mod1"


def test_get_package_tuple_sample(project_path):
Expand Down
17 changes: 10 additions & 7 deletions ropetest/contrib/autoimporttest.py
Original file line numberDiff line numberDiff line change
Expand Up@@ -111,13 +111,6 @@ def test_handling_builtin_modules(self):
self.importer.update_module("sys")
self.assertIn("sys", self.importer.get_modules("exit"))

def test_search_submodule(self):
self.importer.update_module("build")
import_statement = ("from build import env", "env")
self.assertIn(import_statement, self.importer.search("env", exact_match=True))
self.assertIn(import_statement, self.importer.search("en"))
self.assertIn(import_statement, self.importer.search("env"))

def test_search_module(self):
self.importer.update_module("os")
import_statement = ("import os", "os")
Expand DownExpand Up@@ -184,6 +177,16 @@ def test_skipping_directories_not_accessible_because_of_permission_error(self):
self.assertGreater(len(self.importer._dump_all()), 0)


def test_search_submodule(external_fixturepkg):
project = testutils.sample_project(extension_modules=["sys"])
importer = autoimport.AutoImport(project, observe=False)
importer.update_module("external_fixturepkg")
import_statement = ("from external_fixturepkg import mod1", "mod1")
assert import_statement in importer.search("mod1", exact_match=True)
assert import_statement in importer.search("mo")
assert import_statement in importer.search("mod1")


class AutoImportObservingTest(unittest.TestCase):
def setUp(self):
super().setUp()
Expand Down
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode"); } } catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); } })(); })();
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .gitignore
Original file line numberDiff line numberDiff line change
Expand Up@@ -10,7 +10,7 @@ __pycache__/
.Python
build/
develop-eggs/
dist/
/dist/
downloads/
eggs/
.eggs/
Expand Down
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
# **Upcoming release**

- #730 Match on module aliases for autoimport suggestions
- #755 Remove dependency on `build` package being installed while running tests

# Release 1.12.0

Expand Down
2 changes: 1 addition & 1 deletion MANIFEST.in
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
include README.rst COPYING setup.py MANIFEST.in CHANGELOG.md
include README.rst COPYING setup.py MANIFEST.in CHANGELOG.md ropetest-package-fixtures/external_fixturepkg/dist/external_fixturepkg-1.0.0-py3-none-any.whl ropetest-package-fixtures/external_fixturepkg/dist/external_fixturepkg-1.0.0.tar.gz
recursive-include rope *.py
recursive-include docs *.rst
recursive-include ropetest *.py
1 change: 1 addition & 0 deletions pyproject.toml
Original file line numberDiff line numberDiff line change
Expand Up@@ -87,6 +87,7 @@ profile = "black"

[tool.pytest.ini_options]

testpaths = "ropetest"
python_files = [
"*test.py",
"__init__.py",
Expand Down
2 changes: 1 addition & 1 deletion rope/contrib/autoimport/utils.py
Original file line numberDiff line numberDiff line change
Expand Up@@ -54,7 +54,7 @@ def get_package_source(
if "site-packages" in package.parts:
return Source.SITE_PACKAGE
if sys.version_info < (3, 10, 0):
if str(package).startswith(sys.prefix):
if str(package).startswith(sys.base_prefix):
return Source.STANDARD
else:
if name in sys.stdlib_module_names:
Expand Down
12 changes: 12 additions & 0 deletions ropetest-package-fixtures/external_fixturepkg/README.md
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
# example_project

Just an example project for testing rope.


To build this package, run:

```bash
$ python -m build
```

This generates packages in `dist` folder.
Binary file not shown.
Binary file not shown.
18 changes: 18 additions & 0 deletions ropetest-package-fixtures/external_fixturepkg/pyproject.toml
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
[build-system]
requires = ["flit-core >= 3.8"]
build-backend = "flit_core.buildapi"

[project]
name = "external_fixturepkg"
version = "1.0.0"
description = "Just an example project built with build/flit for testing purpose"
readme = "README.md"
requires-python = ">= 3.7"

dependencies = [ ]

[project.scripts]
pyproject-build = "build.__main__:entrypoint"

[tool.flit.sdist]
exclude = ["**/__pycache__", "**/*.egg-info"]
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
def sample_function():
pass
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
def main() -> None:
pass


if __name__ == '__main__':
main()
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
foo = None
23 changes: 16 additions & 7 deletions ropetest/conftest.py
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
import pathlib
import sys
from subprocess import check_call

import pytest

Expand All@@ -13,13 +15,6 @@ def project():
testutils.remove_project(project)


@pytest.fixture
def project2():
project = testutils.sample_project("another_project")
yield project
testutils.remove_project(project)


@pytest.fixture
def project_path(project):
yield pathlib.Path(project.address)
Expand DownExpand Up@@ -52,3 +47,17 @@ def pkg1(project) -> resources.Folder:
@pytest.fixture
def mod2(project, pkg1) -> resources.Folder:
return testutils.create_module(project, "mod2", pkg1)


@pytest.fixture(scope="session")
def external_fixturepkg():
check_call([
sys.executable,
"-m",
"pip",
"install",
"--force-reinstall",
"ropetest-package-fixtures/external_fixturepkg/dist/external_fixturepkg-1.0.0-py3-none-any.whl",
])
yield
check_call([sys.executable, "-m", "pip", "uninstall", "--yes", "external-fixturepkg"])
13 changes: 6 additions & 7 deletions ropetest/contrib/autoimport/conftest.py
Original file line numberDiff line numberDiff line change
Expand Up@@ -24,19 +24,18 @@ def typing_path():


@pytest.fixture
def build_env_path():
from build import env

yield pathlib.Path(env.__file__)
def example_external_package_module_path(external_fixturepkg):
from external_fixturepkg import mod1
yield pathlib.Path(mod1.__file__)


@pytest.fixture
def build_path():
import build
def example_external_package_path(external_fixturepkg):
import external_fixturepkg

# Uses __init__.py so we need the parent

yield pathlib.Path(build.__file__).parent
yield pathlib.Path(external_fixturepkg.__file__).parent


@pytest.fixture
Expand Down
20 changes: 12 additions & 8 deletions ropetest/contrib/autoimport/utilstest.py
Original file line numberDiff line numberDiff line change
Expand Up@@ -14,31 +14,35 @@ def test_get_package_source_not_project(mod1_path):
assert utils.get_package_source(mod1_path, None, "") == Source.UNKNOWN


def test_get_package_source_pytest(build_path):
def test_get_package_source_pytest(example_external_package_path):
# pytest is not installed as part of the standard library
# but should be installed into site_packages,
# so it should return Source.SITE_PACKAGE
assert utils.get_package_source(build_path, None, "build") == Source.SITE_PACKAGE
source = utils.get_package_source(example_external_package_path, None, "mod1")
assert source == Source.SITE_PACKAGE


def test_get_package_source_typing(typing_path):

assert utils.get_package_source(typing_path, None, "typing") == Source.STANDARD


def test_get_modname_project_no_add(mod1_path, project_path):

assert utils.get_modname_from_path(mod1_path, project_path, False) == "mod1"


def test_get_modname_single_file(typing_path):

assert utils.get_modname_from_path(typing_path, typing_path) == "typing"


def test_get_modname_folder(build_path, build_env_path):

assert utils.get_modname_from_path(build_env_path, build_path) == "build.env"
def test_get_modname_folder(
example_external_package_path,
example_external_package_module_path,
):
modname = utils.get_modname_from_path(
example_external_package_module_path,
example_external_package_path,
)
assert modname == "external_fixturepkg.mod1"


def test_get_package_tuple_sample(project_path):
Expand Down
17 changes: 10 additions & 7 deletions ropetest/contrib/autoimporttest.py
Original file line numberDiff line numberDiff line change
Expand Up@@ -111,13 +111,6 @@ def test_handling_builtin_modules(self):
self.importer.update_module("sys")
self.assertIn("sys", self.importer.get_modules("exit"))

def test_search_submodule(self):
self.importer.update_module("build")
import_statement = ("from build import env", "env")
self.assertIn(import_statement, self.importer.search("env", exact_match=True))
self.assertIn(import_statement, self.importer.search("en"))
self.assertIn(import_statement, self.importer.search("env"))

def test_search_module(self):
self.importer.update_module("os")
import_statement = ("import os", "os")
Expand DownExpand Up@@ -184,6 +177,16 @@ def test_skipping_directories_not_accessible_because_of_permission_error(self):
self.assertGreater(len(self.importer._dump_all()), 0)


def test_search_submodule(external_fixturepkg):
project = testutils.sample_project(extension_modules=["sys"])
importer = autoimport.AutoImport(project, observe=False)
importer.update_module("external_fixturepkg")
import_statement = ("from external_fixturepkg import mod1", "mod1")
assert import_statement in importer.search("mod1", exact_match=True)
assert import_statement in importer.search("mo")
assert import_statement in importer.search("mod1")


class AutoImportObservingTest(unittest.TestCase):
def setUp(self):
super().setUp()
Expand Down