Skip to content

Add tests for DjangoObjectType only/exclude_fields #132

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

Merged
merged 1 commit into from
Mar 16, 2017

Conversation

samdroid-apps
Copy link
Contributor

I was planning to write a new feature to rename fields on a DjangoObjectType, because I wanted my foreign key relationships to have nice reverse names. So I started working on that by writing these tests.

However, I eventually opened the "tests/models.py" field and saw how the "related_name" can be used to rename the relationship. So I didn't need to write the feature any more, but tests are still cool :)

@coveralls
Copy link

coveralls commented Mar 10, 2017

Coverage Status

Coverage remained the same at 93.75% when pulling 46048cd on LearntEmail:rename_fields into 1139507 on graphql-python:master.

@syrusakbary syrusakbary merged commit f6034ab into graphql-python:master Mar 16, 2017
@syrusakbary
Copy link
Member

Thanks for the contribution!

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.

3 participants