@hazeljs/typeorm 0.2.0-rc.8 → 0.2.1

package/README.md

@@ -153,9 +153,9 @@ export class TransferService {
 ## Links
 
 - [TypeORM docs](https://typeorm.io)
-- [HazelJS](https://hazeljs.com)
+- [HazelJS](https://hazeljs.ai)
 - [GitHub](https://github.com/hazel-js/hazeljs)
 
 ## License
 
-Apache 2.0 © [HazelJS](https://hazeljs.com)
+Apache 2.0 © [HazelJS](https://hazeljs.ai)

package/dist/base.repository.test.js

@@ -42,6 +42,43 @@ describe('BaseRepository', () => {
         expect(typeof result).toBe('number');
         expect(result).toBe(0);
     });
+    it('should delegate findOne to TypeORM repository', async () => {
+        const entity = { id: 1, name: 'a' };
+        repo['repository'].findOne.mockResolvedValueOnce(entity);
+        const result = await repo.findOne({ where: { id: 1 } });
+        expect(result).toEqual(entity);
+    });
+    it('should delegate create to TypeORM repository', async () => {
+        const created = { id: 1, name: 'new' };
+        repo['repository'].create.mockReturnValueOnce(created);
+        repo['repository'].save.mockResolvedValueOnce(created);
+        const result = await repo.create({ name: 'new' });
+        expect(repo['repository'].create).toHaveBeenCalledWith({ name: 'new' });
+        expect(repo['repository'].save).toHaveBeenCalledWith(created);
+        expect(result).toEqual(created);
+    });
+    it('should delegate save to TypeORM repository', async () => {
+        const entity = { id: 1, name: 'updated' };
+        repo['repository'].save.mockResolvedValueOnce(entity);
+        const result = await repo.save(entity);
+        expect(result).toEqual(entity);
+    });
+    it('should delegate update to TypeORM repository', async () => {
+        repo['repository'].update.mockResolvedValueOnce(undefined);
+        await repo.update({ id: 1 }, { name: 'updated' });
+        expect(repo['repository'].update).toHaveBeenCalledWith({ id: 1 }, { name: 'updated' });
+    });
+    it('should delegate delete to TypeORM repository', async () => {
+        repo['repository'].delete.mockResolvedValueOnce(undefined);
+        await repo.delete({ id: 1 });
+        expect(repo['repository'].delete).toHaveBeenCalledWith({ id: 1 });
+    });
+    it('should call find with empty options when options undefined', async () => {
+        repo['repository'].find.mockClear();
+        repo['repository'].find.mockResolvedValueOnce([]);
+        await repo.find();
+        expect(repo['repository'].find).toHaveBeenCalledWith({});
+    });
     describe('handleError', () => {
         it('should throw on generic error', () => {
             expect(() => repo['handleError'](new Error('db error'))).toThrow('Database error: db error');
@@ -51,10 +88,48 @@ describe('BaseRepository', () => {
             err.code = '23505';
             expect(() => repo['handleError'](err)).toThrow(/Unique constraint/);
         });
+        it('should map unique constraint 23505 with constraint name', () => {
+            const err = new Error('duplicate');
+            err.code = '23505';
+            err.constraint = 'users_email_key';
+            expect(() => repo['handleError'](err)).toThrow('users_email_key');
+        });
         it('should map foreign key code 23503', () => {
             const err = new Error('fk');
             err.code = '23503';
             expect(() => repo['handleError'](err)).toThrow('Foreign key constraint violation');
         });
+        it('should map EntityNotFoundError by name', () => {
+            const err = new Error('not found');
+            err.name = 'EntityNotFoundError';
+            expect(() => repo['handleError'](err)).toThrow('Record not found');
+        });
+    });
+    describe('error propagation', () => {
+        it('should call handleError when findOne throws', async () => {
+            repo['repository'].findOne.mockRejectedValueOnce(new Error('db fail'));
+            await expect(repo.findOne({ where: { id: 1 } })).rejects.toThrow('Database error: db fail');
+        });
+        it('should call handleError when create/save throws', async () => {
+            repo['repository'].create.mockReturnValue({});
+            repo['repository'].save.mockRejectedValueOnce(new Error('insert fail'));
+            await expect(repo.create({ name: 'x' })).rejects.toThrow('Database error: insert fail');
+        });
+        it('should call handleError when save (entity) throws', async () => {
+            repo['repository'].save.mockRejectedValueOnce(new Error('save fail'));
+            await expect(repo.save({ id: 1, name: 'x' })).rejects.toThrow('Database error: save fail');
+        });
+        it('should call handleError when update throws', async () => {
+            repo['repository'].update.mockRejectedValueOnce(new Error('update fail'));
+            await expect(repo.update({ id: 1 }, { name: 'y' })).rejects.toThrow('Database error: update fail');
+        });
+        it('should call handleError when delete throws', async () => {
+            repo['repository'].delete.mockRejectedValueOnce(new Error('delete fail'));
+            await expect(repo.delete({ id: 1 })).rejects.toThrow('Database error: delete fail');
+        });
+        it('should call handleError when count throws', async () => {
+            repo['repository'].count.mockRejectedValueOnce(new Error('count fail'));
+            await expect(repo.count({})).rejects.toThrow('Database error: count fail');
+        });
     });
 });

package/dist/typeorm.module.test.js

@@ -1,7 +1,41 @@
 "use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
 Object.defineProperty(exports, "__esModule", { value: true });
 const typeorm_module_1 = require("./typeorm.module");
 const typeorm_service_1 = require("./typeorm.service");
+const typeorm = __importStar(require("typeorm"));
 describe('TypeOrmModule', () => {
     it('should export TypeOrmModule class', () => {
         expect(typeorm_module_1.TypeOrmModule).toBeDefined();
@@ -35,8 +69,30 @@ describe('TypeOrmModule', () => {
             });
             const provider = result.providers[0];
             expect(provider.useFactory).toBeDefined();
-            // useFactory creates DataSource(options) and TypeOrmService; skip calling it to avoid requiring pg driver
             expect(typeof provider.useFactory).toBe('function');
         });
+        it('useFactory creates DataSource and TypeOrmService (forRoot body coverage)', async () => {
+            const mockDs = {
+                isInitialized: false,
+                initialize: jest.fn().mockResolvedValue(undefined),
+                destroy: jest.fn().mockResolvedValue(undefined),
+                getRepository: jest.fn(() => ({})),
+            };
+            const spy = jest.spyOn(typeorm, 'DataSource').mockImplementation(() => mockDs);
+            const result = typeorm_module_1.TypeOrmModule.forRoot({
+                type: 'sqlite',
+                database: ':memory:',
+                synchronize: true,
+            });
+            const provider = result.providers[0];
+            const service = provider.useFactory();
+            expect(service).toBeDefined();
+            expect(service.ready).toBeDefined();
+            await service.ready();
+            expect(service.dataSource).toBe(mockDs);
+            expect(mockDs.initialize).toHaveBeenCalledTimes(1);
+            await service.onModuleDestroy();
+            spy.mockRestore();
+        });
     });
 });

package/dist/typeorm.service.test.js

@@ -1,5 +1,39 @@
 "use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
 Object.defineProperty(exports, "__esModule", { value: true });
+const typeorm = __importStar(require("typeorm"));
 const typeorm_service_1 = require("./typeorm.service");
 describe('TypeOrmService', () => {
     describe('with provided DataSource', () => {
@@ -65,6 +99,57 @@ describe('TypeOrmService', () => {
             expect(repo).toBe(mockRepo);
         });
     });
+    describe('with options.options (DataSourceOptions)', () => {
+        it('creates DataSource from options and initializes', async () => {
+            const mockDs = {
+                isInitialized: false,
+                initialize: jest.fn().mockResolvedValue(undefined),
+                destroy: jest.fn().mockResolvedValue(undefined),
+                getRepository: jest.fn(() => ({})),
+            };
+            const spy = jest
+                .spyOn(typeorm, 'DataSource')
+                .mockImplementation(() => mockDs);
+            const service = new typeorm_service_1.TypeOrmService({
+                options: {
+                    type: 'sqlite',
+                    database: ':memory:',
+                    synchronize: true,
+                },
+            });
+            await service.ready();
+            expect(service.dataSource).toBe(mockDs);
+            expect(mockDs.initialize).toHaveBeenCalledTimes(1);
+            spy.mockRestore();
+        });
+    });
+    describe('when initialize() rejects', () => {
+        it('surfaces error from ready() and leaves DataSource uninitialized', async () => {
+            const mockDs = {
+                isInitialized: false,
+                initialize: jest.fn().mockRejectedValue(new Error('connection refused')),
+                destroy: jest.fn(),
+                getRepository: jest.fn(() => ({})),
+            };
+            const dataSource = mockDs;
+            const service = new typeorm_service_1.TypeOrmService({ dataSource });
+            await expect(service.ready()).rejects.toThrow('connection refused');
+        });
+    });
+    describe('when destroy() throws in onModuleDestroy', () => {
+        it('logs and rethrows', async () => {
+            const mockDs = {
+                isInitialized: true,
+                initialize: jest.fn(),
+                destroy: jest.fn().mockRejectedValue(new Error('destroy failed')),
+                getRepository: jest.fn(() => ({})),
+            };
+            const dataSource = mockDs;
+            const service = new typeorm_service_1.TypeOrmService({ dataSource });
+            await service.ready();
+            await expect(service.onModuleDestroy()).rejects.toThrow('destroy failed');
+        });
+    });
     describe('without options or dataSource', () => {
         const origEnv = process.env.DATABASE_URL;
         afterEach(() => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hazeljs/typeorm",
-  "version": "0.2.0-rc.8",
+  "version": "0.2.1",
   "description": "TypeORM integration for HazelJS framework",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -45,9 +45,9 @@
   "bugs": {
     "url": "https://github.com/hazeljs/hazel-js/issues"
   },
-  "homepage": "https://hazeljs.com",
+  "homepage": "https://hazeljs.ai",
   "peerDependencies": {
     "@hazeljs/core": ">=0.2.0-beta.0"
   },
-  "gitHead": "c0cfcca112ad5a11c6d5545acc98bb5d39355896"
+  "gitHead": "736a56e3d70b51050608cffae2394de3a3487f20"
 }