routes file organization (#46)

Author: msimerson

lib/group/store/mysql.js

@@ -1,6 +1,6 @@
 import Mysql from '../../mysql.js'
 import GroupBase from './base.js'
-import Permission from '../../permission.js'
+import Permission from '../../permission/index.js'
 import { mapToDbColumn } from '../../util.js'
 
 const groupDbMap = { id: 'nt_group_id', parent_gid: 'parent_group_id' }

lib/test/group.json lib/group/test/group.jsonlib/test/group.json renamed to lib/group/test/group.json

@@ -1,7 +1,7 @@
 import assert from 'node:assert/strict'
 import { describe, it, after, before } from 'node:test'
 
-import Group from './index.js'
+import Group from '../index.js'
 
 import testCase from '../test/group.json' with { type: 'json' }
 

lib/group/test.js lib/group/test/index.jslib/group/test.js renamed to lib/group/test/index.js

@@ -0,0 +1,18 @@
+const storeType = process.env.NICTOOL_DATA_STORE ?? 'mysql'
+
+let RepoClass
+switch (storeType) {
+  case 'toml':
+    RepoClass = (await import('./store/toml.js')).default
+    break
+  case 'mongodb':
+    RepoClass = (await import('./store/mongodb.js')).default
+    break
+  case 'elasticsearch':
+    RepoClass = (await import('./store/elasticsearch.js')).default
+    break
+  default:
+    RepoClass = (await import('./store/mysql.js')).default
+}
+
+export default new RepoClass()

lib/nameserver/index.js

@@ -0,0 +1,44 @@
+/**
+ * Nameserver store base class – pure attributes and business logic.
+ *
+ * Has zero knowledge of how nameservers are persisted. All nameserver repository classes
+ * must extend this class and implement the repo contract methods.
+ *
+ * Repo contract:
+ *   get(args)              → object[]
+ *   create(args)           → number  (zoneId)
+ *   put(args)              → boolean
+ *   delete(args)           → boolean
+ *   destroy(args)          → boolean
+ */
+class NameserverBase {
+  constructor(args = {}) {
+    this.debug = args?.debug ?? false
+  }
+
+  // -------------------------------------------------------------------------
+  // Repo contract – subclasses must implement these
+  // -------------------------------------------------------------------------
+
+  async get(_args) {
+    throw new Error('get() not implemented by this repo')
+  }
+
+  async create(_args) {
+    throw new Error('create() not implemented by this repo')
+  }
+
+  async put(_args) {
+    throw new Error('put() not implemented by this repo')
+  }
+
+  async delete(_args) {
+    throw new Error('delete() not implemented by this repo')
+  }
+
+  async destroy(_args) {
+    throw new Error('destroy() not implemented by this repo')
+  }
+}
+
+export default NameserverBase

lib/nameserver/store/base.js

@@ -1,11 +1,13 @@
-import Mysql from './mysql.js'
-import { mapToDbColumn } from './util.js'
+import Mysql from '../../mysql.js'
+import NameserverBase from './base.js'
+import { mapToDbColumn } from '../../util.js'
 
 const nsDbMap = { id: 'nt_nameserver_id', gid: 'nt_group_id' }
 const boolFields = ['deleted', 'export_serials']
 
-class Nameserver {
+class Nameserver extends NameserverBase {
   constructor() {
+    super()
     this.mysql = Mysql
   }
 
@@ -94,7 +96,7 @@ class Nameserver {
   }
 }
 
-export default new Nameserver()
+export default Nameserver
 
 function dbToObject(rows) {
   for (const row of rows) {

lib/nameserver.js lib/nameserver/store/mysql.jslib/nameserver.js renamed to lib/nameserver/store/mysql.js

@@ -1,9 +1,9 @@
 import assert from 'node:assert/strict'
 import { describe, it, after, before } from 'node:test'
 
-import Nameserver from './nameserver.js'
+import Nameserver from '../index.js'
 
-import testCase from './test/nameserver.json' with { type: 'json' }
+import testCase from './nameserver.json' with { type: 'json' }
 
 before(async () => {
   await Nameserver.destroy({ id: testCase.id })

lib/nameserver.test.js lib/nameserver/test/index.jslib/nameserver.test.js renamed to lib/nameserver/test/index.js

@@ -1,5 +1,5 @@
-import Mysql from './mysql.js'
-import { mapToDbColumn } from './util.js'
+import Mysql from '../mysql.js'
+import { mapToDbColumn } from '../util.js'
 
 const permDbMap = {
   id: 'nt_perm_id',

lib/test/nameserver.json lib/nameserver/test/nameserver.jsonlib/test/nameserver.json renamed to lib/nameserver/test/nameserver.json

@@ -1,13 +1,13 @@
 import assert from 'node:assert/strict'
 import { describe, it, after, before } from 'node:test'
 
-import Group from './group/index.js'
-import User from './user/index.js'
-import Permission from './permission.js'
+import Group from '../../group/index.js'
+import User from '../../user/index.js'
+import Permission from '../index.js'
 
-import groupTestCase from './test/group.json' with { type: 'json' }
-import userTestCase from './test/user.json' with { type: 'json' }
-import permTestCase from './test/permission.json' with { type: 'json' }
+import groupTestCase from '../../group/test/group.json' with { type: 'json' }
+import userTestCase from '../../user/test/user.json' with { type: 'json' }
+import permTestCase from './permission.json' with { type: 'json' }
 
 before(async () => {
   await Group.create(groupTestCase)