update snapshots and fix linting errors

HoudiniGraphql · Mar 6, 2024 · 90ca6ad · 90ca6ad
1 parent 571b765
commit 90ca6ad
Show file tree

Hide file tree

Showing 10 changed files with 57 additions and 18 deletions.
diff --git a/packages/houdini-svelte/src/plugin/artifactData.test.ts b/packages/houdini-svelte/src/plugin/artifactData.test.ts
@@ -132,7 +132,8 @@ describe('load', () => {
 			        },
 
 			        "types": {},
-			        "defaults": {}
+			        "defaults": {},
+			        "runtimeScalars": {}
 			    },
 
 			    "policy": "CacheOrNetwork",

diff --git a/packages/houdini/src/codegen/generators/artifacts/tests/pagination.test.ts b/packages/houdini/src/codegen/generators/artifacts/tests/pagination.test.ts
@@ -203,7 +203,9 @@ test('pagination arguments stripped from key', async function () {
 
 		        "defaults": {
 		            "first": 10
-		        }
+		        },
+
+		        "runtimeScalars": {}
 		    }
 		};
 
@@ -414,7 +416,9 @@ test('pagination arguments stays in key as it s a SinglePage Mode', async functi
 
 		        "defaults": {
 		            "first": 10
-		        }
+		        },
+
+		        "runtimeScalars": {}
 		    }
 		};
 
@@ -518,7 +522,9 @@ test('offset based pagination marks appropriate field', async function () {
 
 		        "defaults": {
 		            "limit": 10
-		        }
+		        },
+
+		        "runtimeScalars": {}
 		    }
 		};
 
@@ -784,7 +790,9 @@ test('cursor as scalar gets the right pagination query argument types', async fu
 
 		        "defaults": {
 		            "first": 10
-		        }
+		        },
+
+		        "runtimeScalars": {}
 		    },
 
 		    "policy": "CacheOrNetwork",
@@ -1152,7 +1160,9 @@ test("sibling aliases don't get marked", async function () {
 
 		        "defaults": {
 		            "first": 10
-		        }
+		        },
+
+		        "runtimeScalars": {}
 		    }
 		};
 

diff --git a/packages/houdini/src/codegen/generators/artifacts/tests/selection.test.ts b/packages/houdini/src/codegen/generators/artifacts/tests/selection.test.ts
@@ -1312,7 +1312,8 @@ test('fragment argument passed to directive', async function () {
 		        },
 
 		        "types": {},
-		        "defaults": {}
+		        "defaults": {},
+		        "runtimeScalars": {}
 		    }
 		};
 

diff --git a/packages/houdini/src/codegen/generators/definitions/schema.test.ts b/packages/houdini/src/codegen/generators/definitions/schema.test.ts
@@ -91,6 +91,8 @@ test('adds internal documents to schema', async function () {
 			  Infinite
 			  SinglePage
 			}
+
+			scalar ViewerIDFromSession
 		`)
 })
 
@@ -179,6 +181,8 @@ test('list operations are included', async function () {
 			  SinglePage
 			}
 
+			scalar ViewerIDFromSession
+
 			directive @User_delete repeatable on FIELD
 		`)
 
@@ -285,6 +289,8 @@ test('list operations are included but delete directive should not be in when we
 			  SinglePage
 			}
 
+			scalar ViewerIDFromSession
+
 			directive @User_delete repeatable on FIELD
 		`)
 
@@ -403,5 +409,7 @@ test("writing twice doesn't duplicate definitions", async function () {
 			  Infinite
 			  SinglePage
 			}
+
+			scalar ViewerIDFromSession
 		`)
 })
diff --git a/packages/houdini/src/codegen/generators/typescript/typescript.test.ts b/packages/houdini/src/codegen/generators/typescript/typescript.test.ts
@@ -263,6 +263,7 @@ describe('typescript', function () {
 			        };
 			        "types": {};
 			        "defaults": {};
+			        "runtimeScalars": {};
 			    };
 			};
 		`)
@@ -353,6 +354,7 @@ describe('typescript', function () {
 			        };
 			        "types": {};
 			        "defaults": {};
+			        "runtimeScalars": {};
 			    };
 			};
 		`)
@@ -865,6 +867,7 @@ describe('typescript', function () {
 			        };
 			        "types": {};
 			        "defaults": {};
+			        "runtimeScalars": {};
 			    };
 			    "policy": "CacheOrNetwork";
 			    "partial": false;
@@ -1128,6 +1131,7 @@ describe('typescript', function () {
 			            };
 			        };
 			        "defaults": {};
+			        "runtimeScalars": {};
 			    };
 			};
 		`)
@@ -1356,6 +1360,7 @@ describe('typescript', function () {
 			            };
 			        };
 			        "defaults": {};
+			        "runtimeScalars": {};
 			    };
 			    "policy": "CacheOrNetwork";
 			    "partial": false;
@@ -2289,6 +2294,7 @@ describe('typescript', function () {
 			        };
 			        "types": {};
 			        "defaults": {};
+			        "runtimeScalars": {};
 			    };
 			    "policy": "CacheOrNetwork";
 			    "partial": false;
@@ -2693,6 +2699,7 @@ describe('typescript', function () {
 			            };
 			        };
 			        "defaults": {};
+			        "runtimeScalars": {};
 			    };
 			};
 		`)
@@ -2809,7 +2816,8 @@ describe('typescript', function () {
 			        },
 
 			        "types": {},
-			        "defaults": {}
+			        "defaults": {},
+			        "runtimeScalars": {}
 			    },
 
 			    "policy": "CacheOrNetwork",

diff --git a/packages/houdini/src/codegen/transforms/fragmentVariables.test.ts b/packages/houdini/src/codegen/transforms/fragmentVariables.test.ts
@@ -519,7 +519,8 @@ test('thread query variables to inner fragments', async function () {
 		        },
 
 		        "types": {},
-		        "defaults": {}
+		        "defaults": {},
+		        "runtimeScalars": {}
 		    },
 
 		    "policy": "CacheOrNetwork",
@@ -1028,7 +1029,9 @@ test('persists fragment variables in artifact', async function () {
 		        "defaults": {
 		            "name": "Hello",
 		            "cool": true
-		        }
+		        },
+
+		        "runtimeScalars": {}
 		    }
 		};
 
@@ -1203,7 +1206,8 @@ test('variables referenced deeply in objects', async function () {
 		        },
 
 		        "types": {},
-		        "defaults": {}
+		        "defaults": {},
+		        "runtimeScalars": {}
 		    }
 		};
 

diff --git a/packages/houdini/src/codegen/transforms/paginate.test.ts b/packages/houdini/src/codegen/transforms/paginate.test.ts
@@ -746,7 +746,9 @@ test('embeds node pagination query as a separate document', async function () {
 
 		        "defaults": {
 		            "first": 10
-		        }
+		        },
+
+		        "runtimeScalars": {}
 		    },
 
 		    "policy": "CacheOrNetwork",
@@ -1021,7 +1023,9 @@ test('embeds custom pagination query as a separate document', async function ()
 
 		        "defaults": {
 		            "first": 10
-		        }
+		        },
+
+		        "runtimeScalars": {}
 		    },
 
 		    "policy": "CacheOrNetwork",
@@ -1810,7 +1814,9 @@ test('generated query has same refetch spec', async function () {
 		        "defaults": {
 		            "first": 10,
 		            "after": "1234"
-		        }
+		        },
+
+		        "runtimeScalars": {}
 		    },
 
 		    "policy": "CacheOrNetwork",
@@ -2073,7 +2079,9 @@ test('default defaultPaginateMode to SinglePage', async function () {
 
 		        "defaults": {
 		            "first": 10
-		        }
+		        },
+
+		        "runtimeScalars": {}
 		    }
 		};
 

diff --git a/packages/houdini/src/codegen/transforms/runtimeScalars.ts b/packages/houdini/src/codegen/transforms/runtimeScalars.ts
@@ -1,4 +1,3 @@
-import { directive } from '@babel/types'
 import * as graphql from 'graphql'
 
 import { unwrapType, wrapType, type Config, type Document } from '../../lib'

diff --git a/packages/houdini/src/runtime/client/plugins/cache.test.ts b/packages/houdini/src/runtime/client/plugins/cache.test.ts
@@ -3,7 +3,7 @@ import { beforeEach, expect, test, vi } from 'vitest'
 import { createPluginHooks, HoudiniClient, type HoudiniClientConstructorArgs } from '..'
 import { testConfigFile } from '../../../test'
 import { Cache } from '../../cache/cache'
-import { CachePolicy, PendingValue, QueryArtifact } from '../../lib'
+import { CachePolicy, PendingValue, type QueryArtifact } from '../../lib'
 import { setMockConfig } from '../../lib/config'
 import { ArtifactKind, DataSource } from '../../lib/types'
 import type { ClientPlugin } from '../documentStore'

diff --git a/packages/houdini/src/runtime/client/plugins/query.ts b/packages/houdini/src/runtime/client/plugins/query.ts
@@ -1,5 +1,5 @@
 import cache from '../../cache'
-import { type RuntimeScalarPayload } from '../../lib'
+import type { RuntimeScalarPayload } from '../../lib'
 import { type SubscriptionSpec, ArtifactKind, DataSource } from '../../lib/types'
 import type { ClientPlugin } from '../documentStore'
 import { documentPlugin } from '../utils'