Skip to content

test: check if deprecated package name raises DeprecationWarning#11

Merged
ketozhang merged 2 commits into
mainfrom
test-check-old-name-raises-warning
Apr 30, 2026
Merged

test: check if deprecated package name raises DeprecationWarning#11
ketozhang merged 2 commits into
mainfrom
test-check-old-name-raises-warning

Conversation

@ketozhang

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds CI and a regression test to verify that the legacy package name (pytest-artifact / pytest_artifact) triggers a DeprecationWarning, ensuring users migrating to pytest-artifacts get a clear warning.

Changes:

  • Add a new pytester-based test that expects a DeprecationWarning when the deprecated package/module path is used.
  • Introduce a deprecated dependency group to install the legacy package in CI.
  • Update GitHub Actions workflow to run tests with the deprecated dependency group.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 6 comments.

File Description
tests/test_deprecated_package_name.py New test intended to validate deprecated package-name behavior via pytester.
pyproject.toml Adds a deprecated dependency group for installing pytest-artifact.
.github/workflows/main.yml Runs the test suite again with the deprecated dependency group installed.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread pyproject.toml
Comment thread .github/workflows/main.yml Outdated
Comment thread tests/test_deprecated_package_name.py
Comment thread tests/test_deprecated_package_name.py
Comment thread tests/test_deprecated_package_name.py
Comment thread tests/test_deprecated_package_name.py Outdated
@ketozhang ketozhang force-pushed the test-check-old-name-raises-warning branch from 5c18fc4 to 8498d5e Compare April 30, 2026 05:45
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@ketozhang ketozhang merged commit 27b200f into main Apr 30, 2026
7 checks passed
@ketozhang ketozhang deleted the test-check-old-name-raises-warning branch April 30, 2026 05:49
ketozhang added a commit that referenced this pull request Apr 30, 2026
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.

2 participants