Fix location of a targeted entity

CHANGELOG.md

@@ -2,6 +2,8 @@
 
 ### vNEXT
 
+- **github**: Fix location of a targeted entity[PR #104](https://github.com/kamilkisiela/graphql-inspector/pull/104)
+
 ### v1.14.0
 
 - **core**: Fix how GraphQL Schema is fetched [PR #101](https://github.com/kamilkisiela/graphql-inspector/pull/101)

packages/github/__tests__/location.ts

@@ -0,0 +1,55 @@
+import {buildSchema, printSchema} from 'graphql';
+import {getLocation} from '../src/location';
+import {GraphQLSchema} from 'graphql';
+
+const schema = buildSchema(/* GraphQL */ `
+  type Query {
+    user(id: ID!): User
+    users: [User!]
+  }
+
+  type User {
+    id: ID!
+    name: String!
+  }
+`);
+
+function printedLine(schema: GraphQLSchema, line: number): string {
+  const printed = printSchema(schema);
+  return printed.split('\n')[line - 1];
+}
+
+test('location of a Type', async () => {
+  const {line} = getLocation({
+    schema,
+    path: 'User',
+  });
+
+  expect(printedLine(schema, line)).toMatch('type User {');
+});
+test('location of a Type.Field', async () => {
+  const {line} = getLocation({
+    schema,
+    path: 'User.id',
+  });
+
+  expect(printedLine(schema, line)).toMatch('id: ID!');
+});
+
+test('location of a Type.Field.Arg', async () => {
+  const {line} = getLocation({
+    schema,
+    path: 'Query.user.id',
+  });
+
+  expect(printedLine(schema, line)).toMatch('user(id: ID!): User');
+});
+
+test('location of a RootType.Field', async () => {
+  const {line} = getLocation({
+    schema,
+    path: 'Query.user',
+  });
+
+  expect(printedLine(schema, line)).toMatch('user(id: ID!): User');
+});

packages/github/jest.config.js

@@ -0,0 +1,3 @@
+const config = require('../../jest.config');
+
+module.exports = config;

packages/github/package.json

@@ -25,6 +25,7 @@
   "scripts": {
     "start": "probot run ./dist/index.js",
     "dev": "yarn build && yarn start",
+    "test": "jest",
     "build": "tsc",
     "clean": "rm -rf dist/",
     "deploy": "now --docker",

packages/github/src/location.ts

@@ -1,6 +1,7 @@
 import {
   GraphQLSchema,
   printSchema,
+  buildSchema,
   isObjectType,
   isDirective,
   isEnumType,
@@ -10,15 +11,20 @@ import {
   getLocation as graphqlGetLocation,
 } from 'graphql';
 
+function normalizeSchema(schema: GraphQLSchema): GraphQLSchema {
+  return buildSchema(printSchema(schema));
+}
+
 export function getLocation({
   path,
   schema,
 }: {
   path: string;
   schema: GraphQLSchema;
 }) {
-  const printed = printSchema(schema);
-  const loc = getNodeLocation({path, schema});
+  const normalizedSchema = normalizeSchema(schema);
+  const printed = printSchema(normalizedSchema);
+  const loc = getNodeLocation({path, schema: normalizedSchema});
   const source = new Source(printed);
 
   if (!loc) {

packages/github/tsconfig.test.json

@@ -0,0 +1,7 @@
+{
+  "extends": "./tsconfig.json",
+  "compilerOptions": {
+    "types": ["jest"]
+  },
+  "include": ["__tests__/**/*.ts"]
+}