Skip to content
Open
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
14 changes: 14 additions & 0 deletions tests/test__ldp_shims.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
"""Tests for _ldp_shims."""

import pytest
from paper_qa import _ldp_shims
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Wrong package name in all test imports

High Severity

All three new test files import from paper_qa (with underscore), but the Python package is paperqa (no underscore). Every existing test and source file consistently uses from paperqa import .... None of these tests can run, so the PR adds zero test coverage.

Additional Locations (2)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit b000b7f. Configure here.



def test__ldp_shims_imports():
"""Verify module imports correctly."""
assert _ldp_shims is not None


def test__ldp_shims_basic():
"""Basic functionality test."""
pass
14 changes: 14 additions & 0 deletions tests/test_exceptions.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
"""Tests for exceptions."""

import pytest
from paper_qa import exceptions
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Wrong module paths for exceptions and openalex

High Severity

Even with the package name corrected to paperqa, the imports for exceptions and openalex would still fail. These modules live under paperqa.clients (at src/paperqa/clients/exceptions.py and src/paperqa/clients/openalex.py), not directly under paperqa. The correct imports would reference paperqa.clients.exceptions and paperqa.clients.openalex.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit b000b7f. Configure here.



def test_exceptions_imports():
"""Verify module imports correctly."""
assert exceptions is not None


def test_exceptions_basic():
"""Basic functionality test."""
pass
14 changes: 14 additions & 0 deletions tests/test_openalex.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
"""Tests for openalex."""

import pytest
from paper_qa import openalex


def test_openalex_imports():
"""Verify module imports correctly."""
assert openalex is not None


def test_openalex_basic():
"""Basic functionality test."""
pass