@autofleet/sadot 0.8.2-beta-71b4ad14.0 → 0.8.3

package/dist/api/v1/definition/validations.js

@@ -40,11 +40,11 @@ const DefaultValueSchema = joi_1.default.when('fieldType', {
         { is: constants_1.CustomFieldDefinitionType.BOOLEAN, then: joi_1.default.boolean().allow(null) },
         { is: constants_1.CustomFieldDefinitionType.DATE, then: joi_1.default.date().allow(null) },
         { is: constants_1.CustomFieldDefinitionType.DATETIME, then: joi_1.default.date().allow(null) },
-        { is: constants_1.CustomFieldDefinitionType.FILE, then: FileValidationSchema },
-        { is: constants_1.CustomFieldDefinitionType.IMAGE, then: joi_1.default.string().uri().allow(null) },
+        { is: constants_1.CustomFieldDefinitionType.FILE, then: joi_1.default.array().items(FileValidationSchema).allow(null) },
+        { is: constants_1.CustomFieldDefinitionType.IMAGE, then: joi_1.default.array().items(joi_1.default.string().uri()).allow(null) },
         { is: constants_1.CustomFieldDefinitionType.NUMBER, then: joi_1.default.number().allow(null) },
         { is: constants_1.CustomFieldDefinitionType.SELECT, then: joi_1.default.string().allow(null) },
-        { is: constants_1.CustomFieldDefinitionType.STATUS, then: statusValidationObject.allow(null) },
+        { is: constants_1.CustomFieldDefinitionType.STATUS, then: joi_1.default.string().allow(null) },
         { is: constants_1.CustomFieldDefinitionType.TEXT, then: joi_1.default.string().allow(null) },
     ],
 });

package/dist/errors/index.d.ts

@@ -1,4 +1,5 @@
 import { BadRequest } from '@autofleet/errors';
+import type { ValidationError } from 'joi';
 export declare class MissingRequiredCustomFieldError extends BadRequest {
     constructor(missingFields: string[]);
 }
@@ -8,8 +9,11 @@ export declare class UnsupportedCustomFieldTypeError extends BadRequest {
 export declare class UnsupportedCustomValidationError extends BadRequest {
     constructor(fieldType: string);
 }
+export declare class InvalidFieldTypeError extends BadRequest {
+    constructor(fieldType: string);
+}
 export declare class InvalidValueError extends BadRequest {
-    constructor(value: any, fieldType: string);
+    constructor(value: any, fieldDefinitionName: string, joiValidationError: ValidationError);
 }
 export declare class MissingDefinitionError extends BadRequest {
     constructor(fieldNames: string[]);

package/dist/errors/index.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.MissingDefinitionError = exports.InvalidValueError = exports.UnsupportedCustomValidationError = exports.UnsupportedCustomFieldTypeError = exports.MissingRequiredCustomFieldError = void 0;
+exports.MissingDefinitionError = exports.InvalidValueError = exports.InvalidFieldTypeError = exports.UnsupportedCustomValidationError = exports.UnsupportedCustomFieldTypeError = exports.MissingRequiredCustomFieldError = void 0;
 /* eslint-disable max-classes-per-file */
 const errors_1 = require("@autofleet/errors");
 class MissingRequiredCustomFieldError extends errors_1.BadRequest {
@@ -27,11 +27,24 @@ class UnsupportedCustomValidationError extends errors_1.BadRequest {
     }
 }
 exports.UnsupportedCustomValidationError = UnsupportedCustomValidationError;
+class InvalidFieldTypeError extends errors_1.BadRequest {
+    constructor(fieldType) {
+        const err = new Error(`Invalid field type ${fieldType}`);
+        super([err], null, null);
+        this.message = 'INVALID_FIELD_TYPE';
+    }
+}
+exports.InvalidFieldTypeError = InvalidFieldTypeError;
 class InvalidValueError extends errors_1.BadRequest {
-    constructor(value, fieldType) {
-        const err = new Error(`Invalid "${fieldType}" value ${JSON.stringify(value)}`);
+    constructor(value, fieldDefinitionName, joiValidationError) {
+        const formattedErrorMessage = joiValidationError.message
+            .replace(/"/g, '')
+            .replace('value', `'${fieldDefinitionName}'`);
+        const formattedValue = typeof value === 'object' ? JSON.stringify(value) : value;
+        const invalidValueMessage = `Invalid Value on field '${fieldDefinitionName}'. ${formattedErrorMessage}. received: '${formattedValue}'`;
+        const err = new Error(invalidValueMessage);
         super([err], null, null);
-        this.message = 'INVALID_VALUE';
+        this.message = invalidValueMessage;
     }
 }
 exports.InvalidValueError = InvalidValueError;

package/dist/events/index.js

@@ -7,18 +7,16 @@ exports.sendDimEvent = void 0;
 const events_1 = __importDefault(require("@autofleet/events"));
 const logger_1 = __importDefault(require("../utils/logger"));
 const events = new events_1.default({ logger: logger_1.default });
-const KEYS_TO_CONVERT = ['value', 'defaultValue'];
+const KEYS_TO_CONVERT = ['value', 'defaultValue', 'blockEditingFromUI'];
 const stringifyBooleans = (savedObject, keysToConvert) => {
-    if (!Object.keys(savedObject).some((key) => keysToConvert.includes(key))) {
+    const savedObjectKeySet = new Set(Object.keys(savedObject));
+    if (keysToConvert.every((key) => !savedObjectKeySet.has(key))) {
         return savedObject;
     }
-    const objectToReturn = { ...savedObject };
-    keysToConvert.forEach((key) => {
-        if (typeof savedObject[key] === 'boolean') {
-            objectToReturn[key] = savedObject[key].toString();
-        }
-    });
-    return objectToReturn;
+    return {
+        ...savedObject,
+        ...Object.fromEntries(keysToConvert.map((key) => [key, typeof savedObject[key] === 'boolean' ? savedObject[key].toString() : savedObject[key]])),
+    };
 };
 const modelTableMapping = {
     CustomFieldDefinition: {

package/dist/models/CustomFieldDefinition.js

@@ -28,8 +28,8 @@ let CustomFieldDefinition = class CustomFieldDefinition extends sequelize_typesc
         }
         if (![null, undefined].includes(instance.defaultValue)) {
             const isValid = (0, validations_1.validateValue)(instance.defaultValue, instance.fieldType, instance.validation);
-            if (!isValid) {
-                throw new errors_1.InvalidValueError(instance.defaultValue, instance.fieldType);
+            if (isValid.error) {
+                throw new errors_1.InvalidValueError(instance.defaultValue, instance.name, isValid.error);
             }
         }
     }

package/dist/models/CustomFieldValue.d.ts

@@ -8,8 +8,9 @@ declare class CustomFieldValue extends Model {
     updatedAt?: Date;
     deletedAt?: Date;
     customFieldDefinition: CustomFieldDefinition;
-    static validateValues(instances: CustomFieldValue[]): Promise<void>;
-    static validateValue(instance: CustomFieldValue): Promise<void>;
+    private static validateValueAgainstDefinition;
+    static validateCustomFieldValues(instances: CustomFieldValue[]): Promise<void>;
+    static validateCustomFieldValue(instance: CustomFieldValue): Promise<void>;
     static afterSaveHandler(instance: CustomFieldValue, options: any): void;
 }
 export default CustomFieldValue;

package/dist/models/CustomFieldValue.js

@@ -39,7 +39,21 @@ const validations_1 = require("../utils/validations");
 const CustomFieldDefinitionRepo = __importStar(require("../repository/definition"));
 const errors_1 = require("../errors");
 let CustomFieldValue = class CustomFieldValue extends sequelize_typescript_1.Model {
-    static async validateValues(instances) {
+    static validateValueAgainstDefinition(instance, definition) {
+        const { validation, fieldType, name } = definition;
+        const isValidFieldType = (0, validations_1.validateFieldType)(fieldType);
+        if (!isValidFieldType) {
+            throw new errors_1.InvalidFieldTypeError(fieldType);
+        }
+        // Always allow null values
+        if (instance.value === null)
+            return;
+        const validateValueResponse = (0, validations_1.validateValue)(instance.value, fieldType, validation);
+        if (validateValueResponse.error) {
+            throw new errors_1.InvalidValueError(instance.value, name, validateValueResponse.error);
+        }
+    }
+    static async validateCustomFieldValues(instances) {
         const ids = instances.map((instance) => instance.customFieldDefinitionId);
         const uniqueIds = [...new Set(ids)];
         const definitions = await CustomFieldDefinitionRepo.findByIds(uniqueIds, { withDisabled: true });
@@ -47,23 +61,17 @@ let CustomFieldValue = class CustomFieldValue extends sequelize_typescript_1.Mod
             throw new Error('Definitions not found');
         }
         instances.forEach((instance) => {
-            const { validation, fieldType, } = definitions
-                .find((definition) => definition.id === instance.customFieldDefinitionId);
-            const isValid = (0, validations_1.validateValue)(instance.value, fieldType, validation);
-            if (!isValid) {
-                throw new errors_1.InvalidValueError(instance.value, fieldType);
+            const definition = definitions.find((d) => d.id === instance.customFieldDefinitionId);
+            if (definition) {
+                this.validateValueAgainstDefinition(instance, definition);
             }
         });
     }
-    static async validateValue(instance) {
+    static async validateCustomFieldValue(instance) {
         const { customFieldDefinitionId } = instance;
         // eslint-disable-next-line max-len
         const cfd = await CustomFieldDefinitionRepo.findById(customFieldDefinitionId, { withDisabled: true });
-        const { validation, fieldType } = cfd;
-        const isValid = (0, validations_1.validateValue)(instance.value, fieldType, validation);
-        if (!isValid) {
-            throw new errors_1.InvalidValueError(instance.value, fieldType);
-        }
+        this.validateValueAgainstDefinition(instance, cfd);
     }
     static afterSaveHandler(instance, options) {
         if (options.transaction) {
@@ -120,7 +128,7 @@ __decorate([
     __metadata("design:type", Function),
     __metadata("design:paramtypes", [Array]),
     __metadata("design:returntype", Promise)
-], CustomFieldValue, "validateValues", null);
+], CustomFieldValue, "validateCustomFieldValues", null);
 __decorate([
     sequelize_typescript_1.BeforeUpdate,
     sequelize_typescript_1.BeforeCreate,
@@ -128,7 +136,7 @@ __decorate([
     __metadata("design:type", Function),
     __metadata("design:paramtypes", [CustomFieldValue]),
     __metadata("design:returntype", Promise)
-], CustomFieldValue, "validateValue", null);
+], CustomFieldValue, "validateCustomFieldValue", null);
 __decorate([
     sequelize_typescript_1.AfterUpsert,
     __metadata("design:type", Function),

package/dist/tests/mocks/definition.mock.d.ts

@@ -15,13 +15,13 @@ export declare const coolFieldDefinition2: {
     updatedAt?: Date;
     deletedAt?: Date;
     defaultValue?: any;
+    blockEditingFromUI?: boolean;
     displayName?: string;
     validation?: any;
     fieldType: string;
     entityId: string;
     entityType: string;
     modelType: string;
-    blockEditingFromUI?: boolean;
 };
 export declare const coolFieldDefinition3: {
     name: string;
@@ -32,16 +32,17 @@ export declare const coolFieldDefinition3: {
     updatedAt?: Date;
     deletedAt?: Date;
     defaultValue?: any;
+    blockEditingFromUI?: boolean;
     displayName?: string;
     validation?: any;
     fieldType: string;
     entityId: string;
     entityType: string;
     modelType: string;
-    blockEditingFromUI?: boolean;
 };
 export declare const booleanField: (modelType: string) => CreateCustomFieldDefinition;
 export declare const selectField: (modelType: string, options: any) => CreateCustomFieldDefinition;
 export declare const statusField: (modelType: string, options: any) => CreateCustomFieldDefinition;
+export declare const fileField: (modelType: string) => CreateCustomFieldDefinition;
 export declare const createDefinition: (defaults: Partial<CustomFieldDefinitionDTO>) => CreateCustomFieldDefinition;
 export declare const createDefinitions: (defaults: Partial<CustomFieldDefinitionDTO>, length?: number) => CreateCustomFieldDefinition[];

package/dist/tests/mocks/definition.mock.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.createDefinitions = exports.createDefinition = exports.statusField = exports.selectField = exports.booleanField = exports.coolFieldDefinition3 = exports.coolFieldDefinition2 = exports.coolFieldDefinition = exports.contextAwareFieldDefinition = void 0;
+exports.createDefinitions = exports.createDefinition = exports.fileField = exports.statusField = exports.selectField = exports.booleanField = exports.coolFieldDefinition3 = exports.coolFieldDefinition2 = exports.coolFieldDefinition = exports.contextAwareFieldDefinition = void 0;
 const node_crypto_1 = require("node:crypto");
 exports.contextAwareFieldDefinition = {
     name: 'cool field',
@@ -49,6 +49,14 @@ const statusField = (modelType, options) => ({
     entityType: 'fleetId',
 });
 exports.statusField = statusField;
+const fileField = (modelType) => ({
+    name: 'file',
+    modelType,
+    fieldType: 'file',
+    entityId: (0, node_crypto_1.randomUUID)(),
+    entityType: 'fleetId',
+});
+exports.fileField = fileField;
 // eslint-disable-next-line max-len
 const createDefinition = (defaults) => ({
     name: defaults?.name || `def_${(0, node_crypto_1.randomUUID)()}`,

package/dist/utils/validations/index.d.ts

@@ -1,2 +1,3 @@
 import type { CustomFieldDefinitionType } from '../constants';
-export declare const validateValue: (value: unknown, valueType: CustomFieldDefinitionType, validation?: unknown) => boolean;
+export declare const validateFieldType: (type: CustomFieldDefinitionType) => boolean;
+export declare const validateValue: (value: unknown, valueType: CustomFieldDefinitionType, validation?: unknown) => import("joi").ValidationResult;

package/dist/utils/validations/index.js

@@ -1,15 +1,12 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.validateValue = void 0;
+exports.validateValue = exports.validateFieldType = void 0;
 const validators_1 = require("./validators");
+const validateFieldType = (type) => Object.keys(validators_1.validators).includes(type);
+exports.validateFieldType = validateFieldType;
 const validateValue = (value, valueType, validation) => {
     const validator = validators_1.validators[valueType];
-    if (!validator) {
-        // Unsupported field type
-        return false;
-    }
-    // Always allow null values
-    return value === null || validator(value, validation);
+    return validator(value, validation);
     /** TODO: Add validation for required fields
      * @example
      * if (validations.required && !value) {

package/dist/utils/validations/type.d.ts

@@ -1,10 +1,11 @@
+import type { ValidationResult } from 'joi';
 import type { CustomFieldDefinitionType } from '../constants';
 /**
  * Validator is a function that validates a custom-field `value`,
  * against a custom-field definition `validation object`.
  * @returns `true` if the value is valid, `false` otherwise.
  */
-export type Validator<Value, DefinitionValidationObject> = (value: Value, validation?: DefinitionValidationObject) => boolean;
+export type Validator<Value, DefinitionValidationObject> = (value: Value, validation?: DefinitionValidationObject) => ValidationResult;
 /**
  * Validators is a map of custom-field types to their respective validators.
  * The key is the custom-field type, and the value is the validator function.

package/dist/utils/validations/validators/index.js

@@ -22,19 +22,19 @@ exports.CustomValidationTypes = {
 exports.validators = {
     [constants_1.CustomFieldDefinitionType.SELECT]: select_validator_1.validateSelect,
     [constants_1.CustomFieldDefinitionType.STATUS]: status_validator_1.validateStatus,
-    [constants_1.CustomFieldDefinitionType.TEXT]: (value) => (typeof value === 'string'),
-    [constants_1.CustomFieldDefinitionType.NUMBER]: (value) => (typeof value === 'number'),
-    [constants_1.CustomFieldDefinitionType.BOOLEAN]: (value) => (typeof value === 'boolean'),
-    [constants_1.CustomFieldDefinitionType.DATE]: (value) => (!joi_1.default.date().validate(value).error),
-    [constants_1.CustomFieldDefinitionType.DATETIME]: (value) => (!joi_1.default.date().validate(value).error),
-    [constants_1.CustomFieldDefinitionType.IMAGE]: (value) => (!joi_1.default.array().min(1).unique()
+    [constants_1.CustomFieldDefinitionType.TEXT]: (value) => joi_1.default.string().min(0).validate(value),
+    [constants_1.CustomFieldDefinitionType.NUMBER]: (value) => joi_1.default.number().strict(true).validate(value),
+    [constants_1.CustomFieldDefinitionType.BOOLEAN]: (value) => joi_1.default.boolean().strict().validate(value),
+    [constants_1.CustomFieldDefinitionType.DATE]: (value) => joi_1.default.date().validate(value),
+    [constants_1.CustomFieldDefinitionType.DATETIME]: (value) => joi_1.default.date().validate(value),
+    [constants_1.CustomFieldDefinitionType.IMAGE]: (value) => joi_1.default.array().min(1).unique()
         .items(joi_1.default.string().uri())
-        .validate(value).error),
-    [constants_1.CustomFieldDefinitionType.FILE]: (value) => (!joi_1.default.array().min(1).unique().items(joi_1.default.object({
+        .validate(value),
+    [constants_1.CustomFieldDefinitionType.FILE]: (value) => joi_1.default.array().min(1).unique().items(joi_1.default.object({
         name: joi_1.default.string().required(),
         type: joi_1.default.string(),
         size: joi_1.default.string(),
         addedBy: joi_1.default.string().uuid(),
     }))
-        .validate(value).error),
+        .validate(value),
 };

package/dist/utils/validations/validators/select.validator.js

@@ -1,9 +1,12 @@
 "use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.validateSelect = void 0;
+const joi_1 = __importDefault(require("joi"));
 /**
  * Validate that the value is one of the select values
  */
-const validateSelect = (value, selectValues) => (Array.isArray(selectValues)
-    && selectValues.includes(value));
+const validateSelect = (value, selectValues) => (joi_1.default.string().allow(null).valid(...selectValues).validate(value));
 exports.validateSelect = validateSelect;

package/dist/utils/validations/validators/status.validator.js

@@ -1,9 +1,15 @@
 "use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.validateStatus = void 0;
+const joi_1 = __importDefault(require("joi"));
 /**
 * Validate that the value is one of the status values
 */
-const validateStatus = (value, statusValues) => (Array.isArray(statusValues)
-    && statusValues.some((status) => status.value === value));
+const validateStatus = (value, statusValues) => (joi_1.default.string()
+    .allow(null)
+    .valid(...statusValues.map((statusValue) => statusValue.value))
+    .validate(value));
 exports.validateStatus = validateStatus;

package/package.json

@@ -1,11 +1,11 @@
 {
   "name": "@autofleet/sadot",
-  "version": "0.8.2-beta-71b4ad14.0",
+  "version": "0.8.3",
   "description": "",
   "main": "dist/index.js",
   "scripts": {
     "start": "ts-node src/index.ts",
-    "build": "rm -rf dist && tsc -p tsconfig.build.json",
+    "build": "rm -rf dist && tsc",
     "linter": "eslint .",
     "test": "jest --forceExit --runInBand",
     "coverage": "jest --coverage --forceExit --runInBand && rm -rf ./coverage",
@@ -41,7 +41,6 @@
   "devDependencies": {
     "@types/express": "^4.17.17",
     "@types/jest": "^29.5.13",
-    "@types/supertest": "^6.0.2",
     "@typescript-eslint/eslint-plugin": "^7.18.0",
     "@typescript-eslint/parser": "^7.18.0",
     "eslint": "^8.57.0",
@@ -49,7 +48,6 @@
     "eslint-plugin-import": "^2.22.1",
     "jest": "^29.7.0",
     "npm-watch": "^0.11.0",
-    "supertest": "^7.0.0",
     "ts-jest": "^29.1.2",
     "ts-node": "^8.6.2",
     "typescript": "^5.3.3"

package/src/api/v1/definition/validations.ts

@@ -36,11 +36,11 @@ const DefaultValueSchema = Joi.when('fieldType', {
     { is: CustomFieldDefinitionType.BOOLEAN, then: Joi.boolean().allow(null) },
     { is: CustomFieldDefinitionType.DATE, then: Joi.date().allow(null) },
     { is: CustomFieldDefinitionType.DATETIME, then: Joi.date().allow(null) },
-    { is: CustomFieldDefinitionType.FILE, then: FileValidationSchema },
-    { is: CustomFieldDefinitionType.IMAGE, then: Joi.string().uri().allow(null) },
+    { is: CustomFieldDefinitionType.FILE, then: Joi.array().items(FileValidationSchema).allow(null) },
+    { is: CustomFieldDefinitionType.IMAGE, then: Joi.array().items(Joi.string().uri()).allow(null) },
     { is: CustomFieldDefinitionType.NUMBER, then: Joi.number().allow(null) },
     { is: CustomFieldDefinitionType.SELECT, then: Joi.string().allow(null) },
-    { is: CustomFieldDefinitionType.STATUS, then: statusValidationObject.allow(null) },
+    { is: CustomFieldDefinitionType.STATUS, then: Joi.string().allow(null) },
     { is: CustomFieldDefinitionType.TEXT, then: Joi.string().allow(null) },
   ],
 });

package/src/errors/index.ts

@@ -1,5 +1,6 @@
 /* eslint-disable max-classes-per-file */
 import { BadRequest } from '@autofleet/errors';
+import type { ValidationError } from 'joi';
 
 export class MissingRequiredCustomFieldError extends BadRequest {
   constructor(missingFields: string[]) {
@@ -25,11 +26,27 @@ export class UnsupportedCustomValidationError extends BadRequest {
   }
 }
 
+export class InvalidFieldTypeError extends BadRequest {
+  constructor(fieldType: string) {
+    const err = new Error(`Invalid field type ${fieldType}`);
+    super([err], null, null);
+    this.message = 'INVALID_FIELD_TYPE';
+  }
+}
+
 export class InvalidValueError extends BadRequest {
-  constructor(value: any, fieldType: string) {
-    const err = new Error(`Invalid "${fieldType}" value ${JSON.stringify(value)}`);
+  constructor(value: any, fieldDefinitionName: string, joiValidationError: ValidationError) {
+    const formattedErrorMessage = joiValidationError.message
+      .replace(/"/g, '')
+      .replace('value', `'${fieldDefinitionName}'`);
+
+    const formattedValue = typeof value === 'object' ? JSON.stringify(value) : value;
+
+    const invalidValueMessage = `Invalid Value on field '${fieldDefinitionName}'. ${formattedErrorMessage}. received: '${formattedValue}'`;
+
+    const err = new Error(invalidValueMessage);
     super([err], null, null);
-    this.message = 'INVALID_VALUE';
+    this.message = invalidValueMessage;
   }
 }
 

package/src/events/index.ts

@@ -3,19 +3,17 @@ import logger from '../utils/logger';
 
 const events = new Events({ logger });
 
-const KEYS_TO_CONVERT = ['value', 'defaultValue'];
+const KEYS_TO_CONVERT = ['value', 'defaultValue', 'blockEditingFromUI'];
 
-const stringifyBooleans = (savedObject: any, keysToConvert: Array<string>) => {
-  if (!Object.keys(savedObject).some((key) => keysToConvert.includes(key))) {
+const stringifyBooleans = (savedObject: any, keysToConvert: string[]) => {
+  const savedObjectKeySet = new Set(Object.keys(savedObject));
+  if (keysToConvert.every((key) => !savedObjectKeySet.has(key))) {
     return savedObject;
   }
-  const objectToReturn = { ...savedObject };
-  keysToConvert.forEach((key) => {
-    if (typeof savedObject[key] === 'boolean') {
-      objectToReturn[key] = savedObject[key].toString();
-    }
-  });
-  return objectToReturn;
+  return {
+    ...savedObject,
+    ...Object.fromEntries(keysToConvert.map((key) => [key, typeof savedObject[key] === 'boolean' ? savedObject[key].toString() : savedObject[key]])),
+  };
 };
 
 const modelTableMapping = {

package/src/models/CustomFieldDefinition.ts

@@ -143,8 +143,8 @@ class CustomFieldDefinition extends Model {
     }
     if (![null, undefined].includes(instance.defaultValue)) {
       const isValid = validateValue(instance.defaultValue, instance.fieldType, instance.validation);
-      if (!isValid) {
-        throw new InvalidValueError(instance.defaultValue, instance.fieldType);
+      if (isValid.error) {
+        throw new InvalidValueError(instance.defaultValue, instance.name, isValid.error);
       }
     }
   }

package/src/models/CustomFieldValue.ts

@@ -15,9 +15,9 @@ import {
 } from 'sequelize-typescript';
 import { sendDimEvent } from '../events';
 import { CustomFieldDefinition } from '.';
-import { validateValue } from '../utils/validations';
+import { validateFieldType, validateValue } from '../utils/validations';
 import * as CustomFieldDefinitionRepo from '../repository/definition';
-import { InvalidValueError } from '../errors';
+import { InvalidFieldTypeError, InvalidValueError } from '../errors';
 
 @Table({
   timestamps: true,
@@ -56,9 +56,26 @@ class CustomFieldValue extends Model {
   @BelongsTo(() => CustomFieldDefinition, { scope: { disabled: false } })
     customFieldDefinition: CustomFieldDefinition;
 
+  private static validateValueAgainstDefinition(
+    instance: CustomFieldValue,
+    definition: CustomFieldDefinition,
+  ): void {
+    const { validation, fieldType, name } = definition;
+    const isValidFieldType = validateFieldType(fieldType);
+    if (!isValidFieldType) {
+      throw new InvalidFieldTypeError(fieldType);
+    }
+    // Always allow null values
+    if (instance.value === null) return;
+    const validateValueResponse = validateValue(instance.value, fieldType, validation);
+    if (validateValueResponse.error) {
+      throw new InvalidValueError(instance.value, name, validateValueResponse.error);
+    }
+  }
+
   @BeforeBulkCreate
   @BeforeBulkUpdate
-  static async validateValues(instances: CustomFieldValue[]): Promise<void> {
+  static async validateCustomFieldValues(instances: CustomFieldValue[]): Promise<void> {
     const ids = instances.map((instance) => instance.customFieldDefinitionId);
     const uniqueIds = [...new Set(ids)];
     const definitions = await CustomFieldDefinitionRepo.findByIds(
@@ -71,14 +88,9 @@ class CustomFieldValue extends Model {
     }
 
     instances.forEach((instance) => {
-      const {
-        validation,
-        fieldType,
-      } = definitions
-        .find((definition) => definition.id === instance.customFieldDefinitionId);
-      const isValid = validateValue(instance.value, fieldType, validation);
-      if (!isValid) {
-        throw new InvalidValueError(instance.value, fieldType);
+      const definition = definitions.find((d) => d.id === instance.customFieldDefinitionId);
+      if (definition) {
+        this.validateValueAgainstDefinition(instance, definition);
       }
     });
   }
@@ -86,15 +98,11 @@ class CustomFieldValue extends Model {
   @BeforeUpdate
   @BeforeCreate
   @BeforeUpsert
-  static async validateValue(instance: CustomFieldValue): Promise<void> {
+  static async validateCustomFieldValue(instance: CustomFieldValue): Promise<void> {
     const { customFieldDefinitionId } = instance;
     // eslint-disable-next-line max-len
     const cfd = await CustomFieldDefinitionRepo.findById(customFieldDefinitionId, { withDisabled: true });
-    const { validation, fieldType } = cfd;
-    const isValid = validateValue(instance.value, fieldType, validation);
-    if (!isValid) {
-      throw new InvalidValueError(instance.value, fieldType);
-    }
+    this.validateValueAgainstDefinition(instance, cfd);
   }
 
   @AfterUpsert

package/src/tests/mocks/definition.mock.ts

@@ -52,6 +52,14 @@ export const statusField = (modelType: string, options): CreateCustomFieldDefini
   entityType: 'fleetId',
 });
 
+export const fileField = (modelType: string): CreateCustomFieldDefinition => ({
+  name: 'file',
+  modelType,
+  fieldType: 'file',
+  entityId: uuidv4(),
+  entityType: 'fleetId',
+});
+
 // eslint-disable-next-line max-len
 export const createDefinition = (defaults: Partial<CustomFieldDefinitionDTO>): CreateCustomFieldDefinition => ({
   name: defaults?.name || `def_${uuidv4()}`,

package/src/utils/validations/index.ts

@@ -1,18 +1,15 @@
 import type { CustomFieldDefinitionType } from '../constants';
 import { validators } from './validators';
 
+export const validateFieldType = (type: CustomFieldDefinitionType): boolean => Object.keys(validators).includes(type);
+
 export const validateValue = (
   value: unknown,
   valueType: CustomFieldDefinitionType,
   validation?: unknown,
 ) => {
   const validator = validators[valueType];
-  if (!validator) {
-    // Unsupported field type
-    return false;
-  }
-  // Always allow null values
-  return value === null || validator(value, validation);
+  return validator(value, validation);
   /** TODO: Add validation for required fields
    * @example
    * if (validations.required && !value) {

package/src/utils/validations/type.ts

@@ -1,3 +1,4 @@
+import type { ValidationResult } from 'joi';
 import type { CustomFieldDefinitionType } from '../constants';
 
 /**
@@ -8,7 +9,7 @@ import type { CustomFieldDefinitionType } from '../constants';
 export type Validator<Value, DefinitionValidationObject> = (
    value: Value,
    validation?: DefinitionValidationObject
-  ) => boolean;
+  ) => ValidationResult;
 
 /**
  * Validators is a map of custom-field types to their respective validators.

package/src/utils/validations/validators/index.ts

@@ -20,19 +20,19 @@ export const CustomValidationTypes = {
 export const validators: Validators = {
   [CustomFieldDefinitionType.SELECT]: validateSelect,
   [CustomFieldDefinitionType.STATUS]: validateStatus,
-  [CustomFieldDefinitionType.TEXT]: (value) => (typeof value === 'string'),
-  [CustomFieldDefinitionType.NUMBER]: (value) => (typeof value === 'number'),
-  [CustomFieldDefinitionType.BOOLEAN]: (value) => (typeof value === 'boolean'),
-  [CustomFieldDefinitionType.DATE]: (value) => (!Joi.date().validate(value).error),
-  [CustomFieldDefinitionType.DATETIME]: (value) => (!Joi.date().validate(value).error),
-  [CustomFieldDefinitionType.IMAGE]: (value) => (!Joi.array().min(1).unique()
+  [CustomFieldDefinitionType.TEXT]: (value) => Joi.string().min(0).validate(value),
+  [CustomFieldDefinitionType.NUMBER]: (value) => Joi.number().strict(true).validate(value),
+  [CustomFieldDefinitionType.BOOLEAN]: (value) => Joi.boolean().strict().validate(value),
+  [CustomFieldDefinitionType.DATE]: (value) => Joi.date().validate(value),
+  [CustomFieldDefinitionType.DATETIME]: (value) => Joi.date().validate(value),
+  [CustomFieldDefinitionType.IMAGE]: (value) => Joi.array().min(1).unique()
     .items(Joi.string().uri())
-    .validate(value).error),
-  [CustomFieldDefinitionType.FILE]: (value) => (!Joi.array().min(1).unique().items(Joi.object({
+    .validate(value),
+  [CustomFieldDefinitionType.FILE]: (value) => Joi.array().min(1).unique().items(Joi.object({
     name: Joi.string().required(),
     type: Joi.string(),
     size: Joi.string(),
     addedBy: Joi.string().uuid(),
   }))
-    .validate(value).error),
+    .validate(value),
 };

package/src/utils/validations/validators/select.validator.ts

@@ -1,3 +1,4 @@
+import Joi from 'joi';
 import type { Validator } from '../type';
 
 /**
@@ -6,6 +7,6 @@ import type { Validator } from '../type';
 export const validateSelect: Validator<string, string[]> = (
   value,
   selectValues,
-) => (Array.isArray(selectValues)
-      && selectValues.includes(value)
+) => (
+  Joi.string().allow(null).valid(...selectValues).validate(value)
 );

package/src/utils/validations/validators/status.validator.ts

@@ -1,3 +1,4 @@
+import Joi from 'joi';
 import type { Validator } from '../type';
 
 type StatusColor = string | null; // TODO: Takes from @autofleet/colors ?
@@ -13,6 +14,9 @@ type StatusOption = {
 export const validateStatus: Validator<StatusValue, StatusOption[]> = (
   value,
   statusValues,
-) => (Array.isArray(statusValues)
-   && statusValues.some((status) => status.value === value)
+) => (
+  Joi.string()
+    .allow(null)
+    .valid(...statusValues.map((statusValue) => statusValue.value))
+    .validate(value)
 );

package/tsconfig.build.json

@@ -3,5 +3,5 @@
   "include": [
     "src/**/*",
   ],
-  "exclude": ["node_modules", "**/*.test.ts"]
+  "exclude": ["node_modules", "src/tests/**", "**/*.test.ts"]
 }