@voilab/vmol-objection - npm Package Compare versions

Comparing version 0.8.1 to 0.9.0

package.json

 {
   "name": "@voilab/vmol-objection",
-  "version": "0.8.1",
+  "version": "0.9.0",
   "description": "Moleculer mixin for Objection.js and knex",
@@ -29,6 +29,6 @@ "main": "index.js",
   "peerDependencies": {
-    "knex": "^2.0.0",
-    "objection": "^3.1.0",
     "ajv": "^8.11.0",
-    "ajv-keywords": "^5.1.0"
+    "ajv-keywords": "^5.1.0",
+    "knex": "^3.0.1",
+    "objection": "^3.1.0"
   },
@@ -39,3 +39,3 @@ "devDependencies": {
     "jest-cli": "^29.6.1",
-    "knex": "^2.5.0",
+    "knex": "^3.0.1",
     "moleculer": "^0.14.29",
@@ -42,0 +42,0 @@ "objection": "^3.0.4",

test/unit/service.spec.js

@@ -6,5 +6,5 @@ 'use strict';
 const Knex = require('knex');
-const ObjectionService = require('../../src');
+const { BaseModel, DBMiddleware } = require('../../index');
 
-const { Model, QueryBuilder } = require('objection');
+const { QueryBuilder } = require('objection');
 
@@ -17,3 +17,3 @@ class MockQueryBuilder extends QueryBuilder {
 
-class MockModel extends Model {
+class MockModel extends BaseModel {
     static get tableName() {
@@ -28,3 +28,3 @@ return 'base_model';
 
-describe('Test ObjectionService creation', () => {
+describe('Test DBMiddleware creation', () => {
     const broker = new ServiceBroker({ logger: false, validation: false });
@@ -47,6 +47,6 @@ const knexFile = {
         function createService() {
-            broker.createService(ObjectionService, lodash.omit(baseService, ['models']));
+            broker.createService(DBMiddleware, lodash.omit(baseService, ['models']));
         }
 
-        expect(createService).toThrowError('must be an array of models instances!');
+        expect(createService).toThrowError('`models` must be either an array or an object of model instances!');
     });
@@ -56,3 +56,3 @@
         function createService() {
-            broker.createService(ObjectionService, lodash.omit(baseService, ['knex']));
+            broker.createService(DBMiddleware, lodash.omit(baseService, ['knex']));
         }
@@ -65,3 +65,3 @@
         function createService() {
-            broker.createService(ObjectionService, {
+            broker.createService(DBMiddleware, {
                 name: 'objection',
@@ -79,3 +79,3 @@ models: [{}],
 
-        const service = broker.createService(ObjectionService, {
+        const service = broker.createService(DBMiddleware, {
             name: 'objection',
@@ -93,5 +93,5 @@ models: [MockModel],
     it('should NOT run migrations by default', async () => {
-        const spie = jest.spyOn(ObjectionService.methods, 'migrateDatabase')
+        const spie = jest.spyOn(DBMiddleware.methods, 'migrateDatabase')
             .mockImplementation(jest.fn(() => {}));
-        const service = broker.createService(ObjectionService, baseService);
+        const service = broker.createService(DBMiddleware, baseService);
 
@@ -106,3 +106,3 @@ await broker.start();
     it('should use the default migration table name when none is provided', async () => {
-        const service = broker.createService(ObjectionService, baseService);
+        const service = broker.createService(DBMiddleware, baseService);
 
@@ -120,3 +120,3 @@ await broker.start();
 
-        const service = broker.createService(ObjectionService, {
+        const service = broker.createService(DBMiddleware, {
             name: 'objection',
@@ -134,3 +134,3 @@ models: [MockModel],
     it('should run migrations only once', async () => {
-        const spie = jest.spyOn(ObjectionService.methods, 'runMigration')
+        const spie = jest.spyOn(DBMiddleware.methods, 'runMigration')
             .mockImplementation(jest.fn(() => {}));
@@ -142,3 +142,3 @@
 
-        const service = broker.createService(ObjectionService, {
+        const service = broker.createService(DBMiddleware, {
             name: 'objection',
@@ -189,3 +189,3 @@ models: [MockModel],
 
-    const service = broker.createService(ObjectionService, baseService);
+    const service = broker.createService(DBMiddleware, baseService);
 
@@ -241,3 +241,3 @@ beforeAll(() => broker.start());
 
-    const service = broker.createService(ObjectionService, baseService);
+    const service = broker.createService(DBMiddleware, baseService);
 
@@ -244,0 +244,0 @@ beforeAll(async () => {

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Worsened metrics

Total package byte prevSize

17305

decreased by-0.08%

No dependency changes