@platformatic/sql-graphql 0.13.0 → 0.14.0

package/index.js

@@ -4,6 +4,7 @@ const fp = require('fastify-plugin')
 const constructGraph = require('./lib/entity-to-type')
 const mercurius = require('mercurius')
 const graphql = require('graphql')
+const { mercuriusFederationPlugin } = require('@mercuriusjs/federation')
 const establishRelations = require('./lib/relationship')
 const setupSubscriptions = require('./lib/subscriptions')
 const scalars = require('graphql-scalars')
@@ -156,7 +157,12 @@ async function mapperToGraphql (app, opts) {
 
   opts.graphiql = opts.graphiql !== false
 
-  await app.register(mercurius, {
+  let plugin = mercurius
+  if (opts.federationMetadata) {
+    plugin = mercuriusFederationPlugin
+  }
+
+  await app.register(plugin, {
     ...opts,
     schema: sdl,
     loaders,

package/lib/relationship.js

@@ -23,7 +23,7 @@ module.exports = function establishRelations (app, relations, resolvers, loaders
 
     const current = entitiesTypeMap[entityName]
     const foreign = entitiesTypeMap[foreignEntityName]
-    assert(foreign !== undefined, `No foreign table named "${foreign_table_name}" was found ${enhanceAssertLogMsg}`)
+    assert(foreign !== undefined, `No foreign table named "${foreign_table_name}" was found ${enhanceAssertLogMsg}. Relation: ${JSON.stringify(relation, null, 2)}`)
 
     // current to foreign, we skip this if the foreign table has a composite primary key
     // TODO implement support for this case

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@platformatic/sql-graphql",
-  "version": "0.13.0",
+  "version": "0.14.0",
   "description": "Map SQL dbs to GraphQL",
   "main": "index.js",
   "repository": {
@@ -14,23 +14,25 @@
   },
   "homepage": "https://github.com/platformatic/platformatic#readme",
   "devDependencies": {
+    "@mercuriusjs/gateway": "^1.0.0",
     "fastify": "^4.10.2",
     "snazzy": "^9.0.0",
     "standard": "^17.0.0",
     "tap": "^16.3.2",
     "tsd": "^0.25.0",
     "ws": "^8.11.0",
-    "@platformatic/sql-mapper": "0.13.0",
-    "@platformatic/sql-events": "0.13.0"
+    "@platformatic/sql-events": "0.14.0",
+    "@platformatic/sql-mapper": "0.14.0"
   },
   "dependencies": {
+    "@mercuriusjs/federation": "^1.0.0",
     "camelcase": "^6.3.0",
     "fastify-plugin": "^4.4.0",
     "graphql": "^16.6.0",
     "graphql-scalars": "^1.20.1",
     "graphql-type-json": "^0.3.2",
     "inflected": "^2.1.0",
-    "mercurius": "^11.4.0"
+    "mercurius": "^12.0.0"
   },
   "tsd": {
     "directory": "test/types"

package/test/datatypes.test.js

@@ -723,7 +723,7 @@ test('[SQLite] - UUID', { skip: !isSQLite }, async ({ pass, teardown, same, equa
   }
 })
 
-test('BIGINT!', { skip: isSQLite, only: true }, async ({ pass, teardown, same, equal }) => {
+test('BIGINT!', { skip: isSQLite }, async ({ pass, teardown, same, equal }) => {
   const app = fastify()
   app.register(sqlMapper, {
     ...connInfo,

package/test/federation.test.js

@@ -2,17 +2,17 @@
 
 const { test } = require('tap')
 const Fastify = require('fastify')
-const mercurius = require('mercurius')
+const Gateway = require('@mercuriusjs/gateway')
+const { mercuriusFederationPlugin } = require('@mercuriusjs/federation')
 const sqlGraphQL = require('..')
 const sqlMapper = require('@platformatic/sql-mapper')
 const { clear, connInfo, isSQLite } = require('./helper')
 
 async function createTestService (t, schema, resolvers = {}) {
   const service = Fastify({ logger: { level: 'error' } })
-  service.register(mercurius, {
+  service.register(mercuriusFederationPlugin, {
     schema,
-    resolvers,
-    federationMetadata: true
+    resolvers
   })
   await service.listen({ port: 0 })
   return [service, service.server.address().port]
@@ -155,7 +155,7 @@ async function createTestGatewayServer (t, cacheOpts) {
     await categoryService.close()
     await postService.close()
   })
-  gateway.register(mercurius, {
+  gateway.register(Gateway, {
     gateway: {
       services: [{
         name: 'category',

package/test/helper.js

@@ -127,6 +127,16 @@ module.exports.clear = async function (db, sql) {
   } catch (err) {
   }
 
+  try {
+    await db.query(sql`DROP TABLE test4.books`)
+  } catch (err) {
+  }
+
+  try {
+    await db.query(sql`DROP TABLE test3.authors`)
+  } catch (err) {
+  }
+
   try {
     await db.query(sql`DROP TABLE test2.books`)
   } catch (err) {
@@ -137,6 +147,16 @@ module.exports.clear = async function (db, sql) {
   } catch (err) {
   }
 
+  try {
+    await db.query(sql`DROP SCHEMA test4`)
+  } catch (err) {
+  }
+
+  try {
+    await db.query(sql`DROP SCHEMA test3`)
+  } catch (err) {
+  }
+
   try {
     await db.query(sql`DROP SCHEMA test2`)
   } catch (err) {

package/test/schema.test.js

@@ -4,7 +4,7 @@ const { test } = require('tap')
 const sqlGraphQL = require('..')
 const sqlMapper = require('@platformatic/sql-mapper')
 const fastify = require('fastify')
-const { isSQLite, connInfo, isMysql, clear } = require('./helper')
+const { isSQLite, isPg, connInfo, isMysql, clear } = require('./helper')
 
 test('should handle relationships with different schemas', { skip: isSQLite }, async ({ pass, teardown, same, equal }) => {
   const app = fastify()
@@ -165,3 +165,113 @@ test('should handle relationships with different schemas', { skip: isSQLite }, a
     }, 'query authors response')
   }
 })
+
+test('should not throw if all of the schema with contraint references are loaded on the config', { skip: !isPg }, async ({ pass, teardown, same }) => {
+  const app = fastify()
+  app.register(sqlMapper, {
+    ...connInfo,
+    schema: ['test1', 'test2', 'test3', 'test4'],
+    async onDatabaseLoad (db, sql) {
+      pass('onDatabaseLoad called')
+
+      await clear(db, sql)
+
+      await db.query(sql`CREATE SCHEMA IF NOT EXISTS test1;`)
+      await db.query(sql`CREATE SCHEMA IF NOT EXISTS test2;`)
+      await db.query(sql`CREATE SCHEMA IF NOT EXISTS test3;`)
+      await db.query(sql`CREATE SCHEMA IF NOT EXISTS test4;`)
+      await db.query(sql`
+        CREATE TABLE test1.authors (
+          id INTEGER PRIMARY KEY,
+          name VARCHAR(42)
+        );
+        
+        CREATE TABLE test2.books (
+          id INTEGER PRIMARY KEY,
+          title VARCHAR(42),
+          author_id INTEGER
+        );
+        
+        CREATE TABLE test3.authors (
+          id INTEGER PRIMARY KEY,
+          name VARCHAR(42)
+        );
+        
+        CREATE TABLE test4.books (
+          id INTEGER PRIMARY KEY,
+          title VARCHAR(42),
+          author_id INTEGER
+        );
+        
+        ALTER TABLE ONLY test2.books
+          ADD CONSTRAINT authors_fkey FOREIGN KEY (author_id) REFERENCES test1.authors(id);
+        
+        ALTER TABLE ONLY test4.books
+          ADD CONSTRAINT authors_fkey FOREIGN KEY (author_id) REFERENCES test3.authors(id);
+      `)
+    }
+  })
+  app.register(sqlGraphQL)
+  teardown(app.close.bind(app))
+
+  try {
+    await app.ready()
+  } catch (error) {
+    same(true, false, 'app should not throw')
+  }
+})
+
+test('should not throw if some of the schema with contraint references are not passed to the config', { skip: !isPg }, async ({ pass, teardown, same }) => {
+  const app = fastify()
+  app.register(sqlMapper, {
+    ...connInfo,
+    schema: ['test1', 'test2'],
+    async onDatabaseLoad (db, sql) {
+      pass('onDatabaseLoad called')
+
+      await clear(db, sql)
+
+      await db.query(sql`CREATE SCHEMA IF NOT EXISTS test1;`)
+      await db.query(sql`CREATE SCHEMA IF NOT EXISTS test2;`)
+      await db.query(sql`CREATE SCHEMA IF NOT EXISTS test3;`)
+      await db.query(sql`CREATE SCHEMA IF NOT EXISTS test4;`)
+      await db.query(sql`
+        CREATE TABLE test1.authors (
+          id INTEGER PRIMARY KEY,
+          name VARCHAR(42)
+        );
+        
+        CREATE TABLE test2.books (
+          id INTEGER PRIMARY KEY,
+          title VARCHAR(42),
+          author_id INTEGER
+        );
+        
+        CREATE TABLE test3.authors (
+          id INTEGER PRIMARY KEY,
+          name VARCHAR(42)
+        );
+        
+        CREATE TABLE test4.books (
+          id INTEGER PRIMARY KEY,
+          title VARCHAR(42),
+          author_id INTEGER
+        );
+        
+        ALTER TABLE ONLY test2.books
+          ADD CONSTRAINT authors_fkey FOREIGN KEY (author_id) REFERENCES test1.authors(id);
+        
+        ALTER TABLE ONLY test4.books
+          ADD CONSTRAINT authors_fkey FOREIGN KEY (author_id) REFERENCES test3.authors(id);
+      `)
+    }
+  })
+  app.register(sqlGraphQL)
+  teardown(app.close.bind(app))
+
+  try {
+    await app.ready()
+  } catch (error) {
+    same(true, false, 'we expect the app to not throw')
+  }
+})