Skip to content

/ cpython

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

[3.7] Add test cases for dataclasses. (GH-17909) #17920

Open
wants to merge 1 commit into
base: 3.7
from
Open

[3.7] Add test cases for dataclasses. (GH-17909) #17920

wants to merge 1 commit into from
+19 −0

Conversation

@miss-islington
Copy link

miss-islington commented Jan 9, 2020 •
edited

  • Add test cases for dataclasses.

  • Add test for repr output of field.

  • Add test for ValueError to be raised when both default and default_factory are passed.
    (cherry picked from commit eef1b02)

Co-authored-by: Karthikeyan Singaravelan tir.karthi@gmail.com

Automerge-Triggered-By: @ericvsmith
@tirkarthi @miss-islington
Add test cases for dataclasses. (GH-17909)
Loading status checks…
1cf491b 
* Add test cases for dataclasses.

* Add test for repr output of field.
* Add test for ValueError to be raised when both default and default_factory are passed.
(cherry picked from commit eef1b02)

Co-authored-by: Karthikeyan Singaravelan <tir.karthi@gmail.com>
@miss-islington miss-islington requested a review from ericvsmith as a code owner Jan 9, 2020
@bedevere-bot bedevere-bot mentioned this pull request Jan 9, 2020
Merged
@miss-islington

This comment has been minimized.

Sign in to view
Copy link
Author

miss-islington commented Jan 9, 2020

@tirkarthi and @ericvsmith: Status check is done, and it's a success ✅ .
1 similar comment
@miss-islington

This comment has been minimized.

Sign in to view
Copy link
Author

miss-islington commented Jan 9, 2020

@tirkarthi and @ericvsmith: Status check is done, and it's a success ✅ .
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ericvsmith ericvsmith

Assignees
No one assigned
Labels
🤖 automerge CLA signed awaiting review skip issue skip news type-tests
Projects
None yet
Milestone
No milestone
5 participants
@miss-islington @ericvsmith @the-knights-who-say-ni @bedevere-bot @tirkarthi
You can’t perform that action at this time.