Conversation
| id: String | ||
| } | ||
|
|
||
| type A implements I { |
There was a problem hiding this comment.
Alla implementing objects måste vara entities med samma keys som entity interfacet. https://www.apollographql.com/docs/graphos/schema-design/federated-schemas/entities/interfaces
Borde vi laga en sak på att lägga till det i validering?
There was a problem hiding this comment.
Det er et godt poeng, samtidig mistenker jeg at det er federation selv som må validere dette. Det kan være det skjer til vanlig, men det bare blir ikke fanget opp her siden vi ikke gjør transformen
Erkelinux
force-pushed
the
GG-375-key-resolvability
branch
from
e3a6ab4 to
b2f13f4
Compare
|
La på litt integrasjonstesting og fant en liten bug som gjorde at filteret egentlig ikke fikk skrevet endringene skikkelig. Nå burde det virke som tiltenkt. |
Erkelinux
force-pushed
the
GG-375-key-resolvability
branch
from
b2f13f4 to
0085c73
Compare
| { | ||
| "representations": [ | ||
| { | ||
| "__typename": "FederatedCustomerOneNonResolvable", | ||
| "id": "RmVkZXJhdGVkQ3VzdG9tZXJPbmVOb25SZXNvbHZhYmxlOjg=" | ||
| } | ||
| ] | ||
| }, | ||
| { | ||
| "representations": [ | ||
| { | ||
| "__typename": "FederatedCustomerOneNonResolvable", | ||
| "firstName": "SUSAN", | ||
| "lastName": "WILSON" | ||
| } | ||
| ] | ||
| }, |
There was a problem hiding this comment.
Er så mange innslag i den fila her at jeg mister litt oversikt over de ulike variantene.
Kanskje vi skal legge inn kommentarer som gjør det lettere å holde kontroll. Noe slikt:
{
"_comment": "#5 - <eventuell beskrivelse av hva som er forventet>",
"representations": [
Med et tall i kommentaren ser man raskere hvilken _X.result.approved.json fil som svarer til inputen
| "representations": [ | ||
| { | ||
| "__typename": "FederatedFilm", | ||
| "id": "RmVkZXJhdGVkRmlsbTo0" | ||
| } | ||
| ] | ||
| }, |
There was a problem hiding this comment.
hører muligens hjemme i query_federation-entities-invalid.variables.json?
Legg i tifelle gjerne på _comment i den fila også som forklarer forskjellen mellom test-casene
3 participants
Ganske enkel løsning. For generator-skjema forsvinner keys som ikke er resolvable, så vi ikke genererer kode for dem.