Skip to content

resolve_type: Tests for Interface#868

Closed
tony wants to merge 1 commit into
graphql-python:masterfrom
tony:test/resolve-type
Closed

resolve_type: Tests for Interface#868
tony wants to merge 1 commit into
graphql-python:masterfrom
tony:test/resolve-type

Conversation

@tony
Copy link
Copy Markdown

@tony tony commented Nov 26, 2018

Hi,

I am trying to get def resolve_type working with Interface. My goal is to get this functioning (if it's the case it's not working). Forgive me if I'm misunderstanding how resolve_type works! Hopefully this test helps assure the the example we document in the interfaces.rst documentation will work.

As a way to create a minimal example, I've created a PR, but not sure if I'm correct on it yet. I think we should probably put it in test_query and also test for the resolved data + field. I also tried putting it in test_interface.py, but that may be too low level for resolve_type to run.

Here are some of the places I've researched prior to coming here:

@tony
Copy link
Copy Markdown
Author

tony commented Dec 30, 2018

This issue was due to me using a field named type. I'm abandoning this PR since my case is probably solved by #775

@tony tony closed this Dec 30, 2018
@tony tony deleted the test/resolve-type branch December 30, 2018 19:03
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