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

refactor: add test on sponsor #3

Closed
wants to merge 1 commit into from
Closed

Conversation

baby230211
Copy link
Owner

Types of changes

Thanks for sending a pull request! Please fill in the following content to let us know better about this change.
Please put an x in the box that applies

  • Bugfix
  • New feature
  • Refactoring
  • Breaking change (any change that would cause existing functionality to not work as expected)
  • Documentation Update
  • Other (please describe)

Description

Describe what the change is
You don't need to explain HOW you change the code. Your code, including comments in the code should explicitly
self-describe enough about how you change the code.

Steps to Test This Pull Request

Steps to reproduce the behavior:

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

Expected behavior

A clear and concise description of what you expected to happen.

Related Issue

If applicable, refernce to the issue related to this pull request.

More Information

Screenshots
If applicable, add screenshots to help explain your problem.

Additional context
Add any other context about the problem here. You may also want to refer
to how to wirte the perfect pull request

@codecov-commenter
Copy link

codecov-commenter commented May 6, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.90 🎉

Comparison is base (411b8b5) 69.49% compared to head (77c8443) 70.39%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master       #3      +/-   ##
==========================================
+ Coverage   69.49%   70.39%   +0.90%     
==========================================
  Files          82       85       +3     
  Lines        3357     3378      +21     
==========================================
+ Hits         2333     2378      +45     
+ Misses       1024     1000      -24     
Impacted Files Coverage Δ
src/proposals/forms/mixins.py 100.00% <ø> (ø)
src/proposals/forms/proposals.py 100.00% <ø> (ø)
src/pycontw2016/urls.py 85.71% <ø> (ø)
src/core/choices.py 100.00% <100.00%> (ø)
src/core/models.py 95.71% <100.00%> (-0.44%) ⬇️
src/proposals/api/urls.py 100.00% <100.00%> (ø)
src/proposals/api/views.py 100.00% <100.00%> (ø)

... and 4 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@baby230211 baby230211 force-pushed the refactor/test-sponsor branch 15 times, most recently from fdc3a68 to bd90e34 Compare May 6, 2023 15:27
@baby230211 baby230211 closed this May 6, 2023
baby230211 pushed a commit that referenced this pull request Jun 8, 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.

2 participants