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

fix: Fix pytest plugin declaration so it can be used without requiring pytest_plugins #1943

Merged
merged 1 commit into from
Sep 11, 2023

Conversation

edgarrmondragon
Copy link
Collaborator

@edgarrmondragon edgarrmondragon commented Sep 7, 2023

No description provided.

@edgarrmondragon edgarrmondragon changed the base branch from main to v1 September 7, 2023 20:20
@edgarrmondragon edgarrmondragon changed the title fix: Fix pytest plugin declaration so it can be used without requiring using pytest_plugins fix: Fix pytest plugin declaration so it can be used without requiring pytest_plugins Sep 7, 2023
@edgarrmondragon edgarrmondragon changed the title fix: Fix pytest plugin declaration so it can be used without requiring pytest_plugins fix: Fix pytest plugin declaration so it can be used without requiring pytest_plugins Sep 7, 2023
@codecov
Copy link

codecov bot commented Sep 7, 2023

Codecov Report

Merging #1943 (620fc8e) into v1 (9a30927) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##               v1    #1943   +/-   ##
=======================================
  Coverage   86.94%   86.94%           
=======================================
  Files          59       59           
  Lines        5079     5079           
  Branches      820      820           
=======================================
  Hits         4416     4416           
  Misses        467      467           
  Partials      196      196           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

📢 Have feedback on the report? Share it here.

@edgarrmondragon
Copy link
Collaborator Author

@kgpayne This removes the need for users to do

# tests/conftest.py

pytest_plugins = ("singer_sdk.testing.pytest_plugin",)

@edgarrmondragon edgarrmondragon marked this pull request as ready for review September 7, 2023 20:34
@edgarrmondragon edgarrmondragon requested review from a team and kgpayne as code owners September 7, 2023 20:34
@edgarrmondragon edgarrmondragon merged commit 4ffac97 into v1 Sep 11, 2023
22 checks passed
@edgarrmondragon edgarrmondragon deleted the fix/pytest-plugin-v1 branch September 11, 2023 14:33
edgarrmondragon added a commit that referenced this pull request Sep 11, 2023
edgarrmondragon added a commit that referenced this pull request Sep 13, 2023
edgarrmondragon added a commit that referenced this pull request Sep 13, 2023
edgarrmondragon added a commit that referenced this pull request Sep 13, 2023
edgarrmondragon added a commit that referenced this pull request Sep 22, 2023
edgarrmondragon added a commit that referenced this pull request Sep 22, 2023
edgarrmondragon added a commit that referenced this pull request Sep 25, 2023
edgarrmondragon added a commit that referenced this pull request Sep 25, 2023
edgarrmondragon added a commit that referenced this pull request Oct 6, 2023
edgarrmondragon added a commit that referenced this pull request Oct 9, 2023
edgarrmondragon added a commit that referenced this pull request Oct 9, 2023
edgarrmondragon added a commit that referenced this pull request Oct 9, 2023
edgarrmondragon added a commit that referenced this pull request Oct 11, 2023
edgarrmondragon added a commit that referenced this pull request Oct 11, 2023
edgarrmondragon added a commit that referenced this pull request Oct 11, 2023
edgarrmondragon added a commit that referenced this pull request Oct 11, 2023
edgarrmondragon added a commit that referenced this pull request Nov 6, 2023
edgarrmondragon added a commit that referenced this pull request Nov 6, 2023
edgarrmondragon added a commit that referenced this pull request Nov 8, 2023
edgarrmondragon added a commit that referenced this pull request Nov 9, 2023
edgarrmondragon added a commit that referenced this pull request Nov 17, 2023
edgarrmondragon added a commit that referenced this pull request Nov 22, 2023
edgarrmondragon added a commit that referenced this pull request Nov 28, 2023
edgarrmondragon added a commit that referenced this pull request Nov 30, 2023
edgarrmondragon added a commit that referenced this pull request Nov 30, 2023
edgarrmondragon added a commit that referenced this pull request Dec 2, 2023
edgarrmondragon added a commit that referenced this pull request Dec 4, 2023
edgarrmondragon added a commit that referenced this pull request Dec 5, 2023
edgarrmondragon added a commit that referenced this pull request Dec 5, 2023
edgarrmondragon added a commit that referenced this pull request Dec 11, 2023
edgarrmondragon added a commit that referenced this pull request Dec 20, 2023
edgarrmondragon added a commit that referenced this pull request Dec 21, 2023
edgarrmondragon added a commit that referenced this pull request Dec 21, 2023
edgarrmondragon added a commit that referenced this pull request Dec 21, 2023
edgarrmondragon added a commit that referenced this pull request Jan 2, 2024
edgarrmondragon added a commit that referenced this pull request Jan 9, 2024
edgarrmondragon added a commit that referenced this pull request Jan 9, 2024
edgarrmondragon added a commit that referenced this pull request Jan 9, 2024
edgarrmondragon added a commit that referenced this pull request Jan 9, 2024
edgarrmondragon added a commit that referenced this pull request Jan 10, 2024
edgarrmondragon added a commit that referenced this pull request Jan 11, 2024
edgarrmondragon added a commit that referenced this pull request Jan 11, 2024
edgarrmondragon added a commit that referenced this pull request Jan 11, 2024
edgarrmondragon added a commit that referenced this pull request Jan 16, 2024
edgarrmondragon added a commit that referenced this pull request Jan 16, 2024
edgarrmondragon added a commit that referenced this pull request Jan 19, 2024
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