@platformatic/service 0.27.0 → 0.29.0

package/test/autoload.test.js

@@ -47,99 +47,6 @@ test('autoload & filesystem based routing / watch disabled', async ({ teardown,
   }
 })
 
-test('autoload & filesystem based routing / watch enabled', async ({ teardown, equal }) => {
-  const config = {
-    server: {
-      hostname: '127.0.0.1',
-      port: 0
-    },
-    plugins: {
-      paths: [join(__dirname, 'fixtures', 'directories', 'routes')]
-    },
-    watch: true,
-    metrics: false
-  }
-
-  const app = await buildServer(config)
-  teardown(async () => {
-    await app.close()
-  })
-  await app.start()
-
-  {
-    const res = await request(`${app.url}/`)
-    equal(res.statusCode, 200, 'status code')
-    const body = await res.body.json()
-    equal(body.hello, 'from root', 'body')
-  }
-
-  {
-    const res = await request(`${app.url}/foo/bar`)
-    equal(res.statusCode, 200, 'status code')
-    const body = await res.body.json()
-    equal(body.hello, 'from bar', 'body')
-  }
-
-  {
-    const res = await request(`${app.url}/foo/baz`)
-    equal(res.statusCode, 200, 'status code')
-    const body = await res.body.json()
-    equal(body.hello, 'from baz', 'body')
-  }
-})
-
-test('multiple files', async ({ teardown, equal }) => {
-  const config = {
-    server: {
-      hostname: '127.0.0.1',
-      port: 0
-    },
-    plugins: {
-      paths: [{
-        path: join(__dirname, 'fixtures', 'directories', 'plugins')
-      }, {
-        path: join(__dirname, 'fixtures', 'directories', 'routes')
-      }]
-    },
-    watch: true,
-    metrics: false
-  }
-
-  const app = await buildServer(config)
-  teardown(async () => {
-    await app.close()
-  })
-  await app.start()
-
-  {
-    const res = await request(`${app.url}/`)
-    equal(res.statusCode, 200, 'status code')
-    const body = await res.body.json()
-    equal(body.hello, 'from root', 'body')
-  }
-
-  {
-    const res = await request(`${app.url}/foo/bar`)
-    equal(res.statusCode, 200, 'status code')
-    const body = await res.body.json()
-    equal(body.hello, 'from bar', 'body')
-  }
-
-  {
-    const res = await request(`${app.url}/foo/baz`)
-    equal(res.statusCode, 200, 'status code')
-    const body = await res.body.json()
-    equal(body.hello, 'from baz', 'body')
-  }
-
-  {
-    const res = await request(`${app.url}/foo/with-decorator`)
-    equal(res.statusCode, 200, 'status code')
-    const body = await res.body.json()
-    equal(body.hello, 'bar', 'body')
-  }
-})
-
 test('multiple files / watch false', async ({ teardown, equal }) => {
   const config = {
     server: {
@@ -233,95 +140,6 @@ test('autoload & filesystem based routing / watch disabled / no object', async (
   }
 })
 
-test('autoload & filesystem based routing / watch enabled / no object', async ({ teardown, equal }) => {
-  const config = {
-    server: {
-      hostname: '127.0.0.1',
-      port: 0
-    },
-    plugins: {
-      paths: [join(__dirname, 'fixtures', 'directories', 'routes')]
-    },
-    watch: true,
-    metrics: false
-  }
-
-  const app = await buildServer(config)
-  teardown(async () => {
-    await app.close()
-  })
-  await app.start()
-
-  {
-    const res = await request(`${app.url}/`)
-    equal(res.statusCode, 200, 'status code')
-    const body = await res.body.json()
-    equal(body.hello, 'from root', 'body')
-  }
-
-  {
-    const res = await request(`${app.url}/foo/bar`)
-    equal(res.statusCode, 200, 'status code')
-    const body = await res.body.json()
-    equal(body.hello, 'from bar', 'body')
-  }
-
-  {
-    const res = await request(`${app.url}/foo/baz`)
-    equal(res.statusCode, 200, 'status code')
-    const body = await res.body.json()
-    equal(body.hello, 'from baz', 'body')
-  }
-})
-
-test('multiple files / no object', async ({ teardown, equal }) => {
-  const config = {
-    server: {
-      hostname: '127.0.0.1',
-      port: 0
-    },
-    plugins: {
-      paths: [join(__dirname, 'fixtures', 'directories', 'plugins'), join(__dirname, 'fixtures', 'directories', 'routes')]
-    },
-    watch: true,
-    metrics: false
-  }
-
-  const app = await buildServer(config)
-  teardown(async () => {
-    await app.close()
-  })
-  await app.start()
-
-  {
-    const res = await request(`${app.url}/`)
-    equal(res.statusCode, 200, 'status code')
-    const body = await res.body.json()
-    equal(body.hello, 'from root', 'body')
-  }
-
-  {
-    const res = await request(`${app.url}/foo/bar`)
-    equal(res.statusCode, 200, 'status code')
-    const body = await res.body.json()
-    equal(body.hello, 'from bar', 'body')
-  }
-
-  {
-    const res = await request(`${app.url}/foo/baz`)
-    equal(res.statusCode, 200, 'status code')
-    const body = await res.body.json()
-    equal(body.hello, 'from baz', 'body')
-  }
-
-  {
-    const res = await request(`${app.url}/foo/with-decorator`)
-    equal(res.statusCode, 200, 'status code')
-    const body = await res.body.json()
-    equal(body.hello, 'bar', 'body')
-  }
-})
-
 test('multiple files / watch false / no object', async ({ teardown, equal }) => {
   const config = {
     server: {

package/test/cli/compile.test.mjs

@@ -164,7 +164,7 @@ t.test('start command should not compile typescript plugin with errors', async (
     await childProcess
     t.fail('should not compile bad typescript plugin')
   } catch (err) {
-    if (!err.stderr.includes('Found 1 error')) {
+    if (!err.stdout.includes('Found 1 error')) {
       t.comment(err.stdout)
       t.comment(err.stderr)
       t.fail('should throw one ts error')
@@ -284,3 +284,100 @@ t.test('should compile typescript plugin with start command from a folder', asyn
   }
   t.fail('should compile typescript plugin with start command')
 })
+
+t.test('should start the service if it was precompiled and typescript is `false`', async (t) => {
+  const testDir = path.join(urlDirname(import.meta.url), '..', 'fixtures', 'typescript-plugin-nocompile')
+  const cwd = path.join(urlDirname(import.meta.url), '..', 'tmp', 'typescript-plugin-clone-9')
+
+  await cp(testDir, cwd, { recursive: true })
+
+  await execa('node', [cliPath, 'compile'], { cwd })
+
+  const child = execa('node', [cliPath, 'start'], { cwd })
+
+  const splitter = split()
+  child.stdout.pipe(splitter)
+
+  for await (const data of splitter) {
+    const sanitized = stripAnsi(data)
+    if (sanitized.includes('Typescript plugin loaded')) {
+      t.pass()
+      return
+    }
+  }
+  t.fail('should load the typescript plugin without compiling it')
+})
+
+t.test('should not start the service if it was not precompiled and typescript is `false`', async (t) => {
+  const testDir = path.join(urlDirname(import.meta.url), '..', 'fixtures', 'typescript-plugin-nocompile')
+  const cwd = path.join(urlDirname(import.meta.url), '..', 'tmp', 'typescript-plugin-clone-10')
+
+  await cp(testDir, cwd, { recursive: true })
+
+  const child = execa('node', [cliPath, 'start'], { cwd })
+
+  const splitter = split()
+  child.stdout.pipe(splitter)
+  child.stderr.pipe(splitter)
+
+  for await (const data of splitter) {
+    const sanitized = stripAnsi(data)
+    if (sanitized.includes('Unknown file extension ".ts" for')) {
+      t.pass()
+      return
+    }
+  }
+  t.fail('should load the typescript plugin without compiling it')
+})
+
+t.test('should compile typescript plugin with string config', async (t) => {
+  const testDir = path.join(urlDirname(import.meta.url), '..', 'fixtures', 'typescript-plugin-string')
+  const cwd = path.join(urlDirname(import.meta.url), '..', 'tmp', 'typescript-plugin-clone-11')
+
+  await cp(testDir, cwd, { recursive: true })
+
+  const child = execa('node', [cliPath, 'compile'], { cwd })
+
+  t.teardown(exitOnTeardown(child))
+
+  const splitter = split()
+  child.stdout.pipe(splitter)
+
+  for await (const data of splitter) {
+    const sanitized = stripAnsi(data)
+    if (sanitized.includes('Typescript compilation completed successfully.')) {
+      const jsPluginPath = path.join(cwd, 'dist', 'plugin.js')
+      try {
+        await access(jsPluginPath)
+      } catch (err) {
+        t.fail(err)
+      }
+
+      t.pass()
+      return
+    }
+  }
+  t.fail('should compile typescript plugin with a compile command')
+})
+
+t.test('should not start the service if it was not precompiled and typescript is `"false"`', async (t) => {
+  const testDir = path.join(urlDirname(import.meta.url), '..', 'fixtures', 'typescript-plugin-nocompile')
+  const cwd = path.join(urlDirname(import.meta.url), '..', 'tmp', 'typescript-plugin-clone-12')
+
+  await cp(testDir, cwd, { recursive: true })
+
+  const child = execa('node', [cliPath, 'start'], { cwd })
+
+  const splitter = split()
+  child.stdout.pipe(splitter)
+  child.stderr.pipe(splitter)
+
+  for await (const data of splitter) {
+    const sanitized = stripAnsi(data)
+    if (sanitized.includes('Unknown file extension ".ts" for')) {
+      t.pass()
+      return
+    }
+  }
+  t.fail('should load the typescript plugin without compiling it')
+})

package/test/clients.test.js

@@ -60,3 +60,56 @@ test('client is loaded (ts)', async ({ teardown, equal, pass, same }) => {
   const data = await res.body.json()
   same(data, { hello: 'world' })
 })
+
+test('client is loaded dependencyless', async ({ teardown, equal, same }) => {
+  const app1 = await buildServer(join(__dirname, '..', 'fixtures', 'hello', 'warn-log.service.json'))
+
+  teardown(async () => {
+    await app1.close()
+  })
+  await app1.start()
+
+  process.env.PLT_CLIENT_URL = app1.url
+
+  const app2 = await buildServer(join(__dirname, '..', 'fixtures', 'hello-client-without-deps', 'platformatic.service.json'))
+
+  teardown(async () => {
+    await app2.close()
+  })
+  await app2.start()
+
+  const res = await request(`${app2.url}/`)
+  equal(res.statusCode, 200, 'status code')
+  const data = await res.body.json()
+  same(data, { hello: 'world' })
+})
+
+test('client is loaded (ts) depencyless', async ({ teardown, equal, pass, same }) => {
+  const app1 = await buildServer(join(__dirname, '..', 'fixtures', 'hello', 'warn-log.service.json'))
+
+  teardown(async () => {
+    await app1.close()
+  })
+  await app1.start()
+
+  process.env.PLT_CLIENT_URL = app1.url
+
+  const targetDir = join(__dirname, '..', 'fixtures', 'hello-client-ts-without-deps')
+
+  try {
+    await rmdir(join(targetDir, 'dist'))
+  } catch {}
+
+  await compile(targetDir, { server: { logger: { level: 'warn' } } })
+
+  const app2 = await buildServer(join(targetDir, 'platformatic.service.json'))
+  teardown(async () => {
+    await app2.close()
+  })
+  await app2.start()
+
+  const res = await request(`${app2.url}/`)
+  equal(res.statusCode, 200, 'status code')
+  const data = await res.body.json()
+  same(data, { hello: 'world' })
+})

package/test/fixtures/bad-typescript-plugin/platformatic.service.json

@@ -12,5 +12,5 @@
     "paths": ["plugin.ts"],
     "typescript": true
   },
-  "watch": true
+  "watch": false
 }

package/test/fixtures/typescript-plugin-nocompile-string/platformatic.service.json

@@ -0,0 +1,16 @@
+{
+  "server": {
+    "logger": {
+      "level": "info"
+    },
+    "hostname": "127.0.0.1",
+    "port": "0",
+    "pluginTimeout": 60000,
+    "keepAliveTimeout": 1
+  },
+  "plugins": {
+    "paths": ["plugin.ts"],
+    "typescript": "false"
+  },
+  "watch": false
+}

package/test/fixtures/typescript-plugin-nocompile-string/plugin.ts

@@ -0,0 +1,5 @@
+import { FastifyInstance } from 'fastify'
+
+export default async function (app: FastifyInstance) {
+  app.log.info('Typescript plugin loaded')
+}

package/test/fixtures/typescript-plugin-nocompile-string/tsconfig.json

@@ -0,0 +1,22 @@
+{
+  "compilerOptions": {
+    "module": "commonjs",
+    "esModuleInterop": true,
+    "target": "es6",
+    "moduleResolution": "node",
+    "sourceMap": true,
+    "pretty": true,
+    "noEmitOnError": true,
+    "outDir": "dist"
+  },
+  "watchOptions": {
+    "watchFile": "fixedPollingInterval",
+    "watchDirectory": "fixedPollingInterval",
+    "fallbackPolling": "dynamicPriority",
+    "synchronousWatchDirectory": true,
+    "excludeDirectories": [
+      "**/node_modules",
+      "dist"
+    ]
+  }
+}

package/test/fixtures/typescript-plugin-string/platformatic.service.json

@@ -0,0 +1,16 @@
+{
+  "server": {
+    "logger": {
+      "level": "info"
+    },
+    "hostname": "127.0.0.1",
+    "port": "0",
+    "pluginTimeout": 60000,
+    "keepAliveTimeout": 1
+  },
+  "plugins": {
+    "paths": ["plugin.ts"],
+    "typescript": "true"
+  },
+  "watch": false
+}

package/test/fixtures/typescript-plugin-string/plugin.ts

@@ -0,0 +1,5 @@
+import { FastifyInstance } from 'fastify'
+
+export default async function (app: FastifyInstance) {
+  app.log.info('Typescript plugin loaded')
+}

package/test/fixtures/typescript-plugin-string/tsconfig.json

@@ -0,0 +1,22 @@
+{
+  "compilerOptions": {
+    "module": "commonjs",
+    "esModuleInterop": true,
+    "target": "es6",
+    "moduleResolution": "node",
+    "sourceMap": true,
+    "pretty": true,
+    "noEmitOnError": true,
+    "outDir": "dist"
+  },
+  "watchOptions": {
+    "watchFile": "fixedPollingInterval",
+    "watchDirectory": "fixedPollingInterval",
+    "fallbackPolling": "dynamicPriority",
+    "synchronousWatchDirectory": true,
+    "excludeDirectories": [
+      "**/node_modules",
+      "dist"
+    ]
+  }
+}