chore: improve repo integration test (#57)

Author: danadajian

.github/workflows/ci.yml

@@ -29,7 +29,7 @@ jobs:
         run: bun tsc
 
       - name: Unit Tests
-        run: bun test
+        run: bun unit
 
       - uses: actions/setup-java@v4
         with:

.prettierignore

@@ -1 +1,2 @@
 *.hbs
+test/integration/expected.graphql

build.gradle.kts

@@ -1,21 +1,32 @@
+import com.expediagroup.graphql.plugin.gradle.graphql
+
 repositories {
     mavenCentral()
 }
 
 plugins {
     kotlin("jvm") version "1.9.23"
+    id("com.expediagroup.graphql") version "7.1.1"
 }
 
 dependencies {
     implementation("com.expediagroup", "graphql-kotlin-schema-generator", "7.1.1")
+    implementation("com.expediagroup", "graphql-kotlin-server", "7.1.1")
     implementation("com.expediagroup", "graphql-kotlin-federation", "7.1.1")
 }
 
 sourceSets {
-    test {
+    main {
         kotlin {
-            srcDirs("test/unit", "test/integration")
+            srcDirs("test/unit")
             exclude("**/actual.kt")
+            srcDirs("test/integration")
         }
     }
 }
+
+graphql {
+    schema {
+        packages = listOf("test.integration")
+    }
+}

codegen.ts

@@ -0,0 +1,25 @@
+import { GraphQLKotlinCodegenConfig } from "./src/plugin";
+import { CodegenConfig } from "@graphql-codegen/cli";
+
+export default {
+  overwrite: true,
+  schema: "test/**/*.graphql",
+  config: {
+    namingConvention: "keep",
+  },
+  generates: {
+    "test/integration/Types.kt": {
+      plugins: [
+        {
+          "dist/plugin.cjs": {
+            resolverClasses: [
+              {
+                typeName: "Query",
+              },
+            ],
+          } satisfies GraphQLKotlinCodegenConfig,
+        },
+      ],
+    },
+  },
+} satisfies CodegenConfig;

codegen.yml

@@ -51,7 +51,7 @@ import com.types.generated.MyType
 import com.types.generated.Query as QueryInterface
 
 class MyQuery : Query, QueryInterface() {
-  override suspend fun resolveMyType(input: String): MyType =
+  override fun resolveMyType(input: String): MyType =
     MyType(
       field1 = myExpensiveCall1(),
       field2 = myExpensiveCall2()
@@ -103,7 +103,7 @@ import com.types.generated.MyType as MyTypeInterface
 import com.expediagroup.graphql.generator.annotations.GraphQLIgnore
 
 class MyQuery : Query, QueryInterface() {
-    override suspend fun resolveMyType(input: String): MyType = MyType()
+    override fun resolveMyType(input: String): MyType = MyType()
 }
 
 @GraphQLIgnore

docs/docs/recommended-usage.md

@@ -39,11 +39,11 @@
     "build": "tsup src/plugin.ts --clean --dts --external graphql",
     "format": "prettier --write .",
     "format-check": "prettier --check .",
-    "integration": "bun run build && graphql-codegen && ./gradlew build",
+    "integration": "bun run build && graphql-codegen && ./gradlew graphqlGenerateSDL && bun test ./test/integration.test.ts",
     "lint": "eslint .",
     "prepack": "bun run build",
     "prepare": "husky",
-    "test": "bun test"
+    "unit": "bun test ./test/plugin.test.ts"
   },
   "type": "module"
 }

package.json

@@ -0,0 +1,13 @@
+import { describe, expect, it } from "bun:test";
+
+describe("Integration", () => {
+  it("schema generated by GraphQL Kotlin should match expected", async () => {
+    const generatedSchema = await Bun.file(
+      `${process.cwd()}/build/schema.graphql`,
+    ).text();
+    const expectedSchema = await Bun.file(
+      `${process.cwd()}/test/integration/expected.graphql`,
+    ).text();
+    expect(generatedSchema).toBe(expectedSchema);
+  });
+});

test/integration.test.ts

@@ -0,0 +1,9 @@
+package test.integration
+
+import com.expediagroup.graphql.server.operations.Query
+import graphql.schema.DataFetchingEnvironment
+import test.integration.Query as QueryInterface
+
+class IntegrationTestQuery() : Query, QueryInterface() {
+    override fun testQuery(dataFetchingEnvironment: DataFetchingEnvironment): SomeType = SomeType()
+}

test/integration/Query.kt

@@ -0,0 +1,38 @@
+schema {
+  query: Query
+}
+
+"Marks the field, argument, input field or enum value as deprecated"
+directive @deprecated(
+    "The reason for the deprecation"
+    reason: String = "No longer supported"
+  ) on FIELD_DEFINITION | ARGUMENT_DEFINITION | ENUM_VALUE | INPUT_FIELD_DEFINITION
+
+"Directs the executor to include this field or fragment only when the `if` argument is true"
+directive @include(
+    "Included when true."
+    if: Boolean!
+  ) on FIELD | FRAGMENT_SPREAD | INLINE_FRAGMENT
+
+"Indicates an Input Object is a OneOf Input Object."
+directive @oneOf on INPUT_OBJECT
+
+"Directs the executor to skip this field or fragment when the `if` argument is true."
+directive @skip(
+    "Skipped when true."
+    if: Boolean!
+  ) on FIELD | FRAGMENT_SPREAD | INLINE_FRAGMENT
+
+"Exposes a URL that specifies the behaviour of this scalar."
+directive @specifiedBy(
+    "The URL that specifies the behaviour of this scalar."
+    url: String!
+  ) on SCALAR
+
+type Query {
+  testQuery: SomeType!
+}
+
+type SomeType {
+  someField: String
+}