Validaciones para los endpints

packages/sim-entrada-eventos/aplication/Sim.controller.ts

@@ -1,16 +1,8 @@
 import { Request, Response } from "express"
 import { SimUsecases } from "./Sim.usecases.js"
+import { activationValidator, iccidValidator } from "./httpValidators.js"
+import { companyFromIccid } from "#domain/companies.js"
 
-// Partiendo del caracter 3 2 de pais + 2 de compañia
-// Metiendolo a la BDD podria ser mas dinamico pero perderia
-// tiempo de query
-// Puede que esté bien crear un endpoint para administrarlo
-const COMPAÑIASICCID = new Map<string, string>(
-  [
-    ["3490", "alai"],
-    ["3510", "nos"],
-    ["3320", "objenious"]
-  ])
 
 export class SimController {
   private simUseCases: SimUsecases
@@ -25,11 +17,19 @@ export class SimController {
 
   public preactivation() {
     return async (req: Request, res: Response) => {
-      const valido = this.validateBody(req.body, res)
-      if (valido == false) return;
+      console.warn("[!] Se deberia de usar la peticion /sim/activate directamente")
+      try {
+        iccidValidator.validate(req.body)
+      } catch (e) {
+        res.status(422).json({
+          errors: {
+            msg: e
+          }
+        })
+      }
 
       const { iccid } = req.body
-      const compañia = this.compañiaFromIccid(iccid)
+      const compañia = companyFromIccid(iccid)
 
       if (compañia == undefined) {
         res.status(500).json({
@@ -62,13 +62,19 @@ export class SimController {
 
   public activation() {
     return async (req: Request, res: Response) => {
-      const valido = this.validateBody(req.body, res)
-
-      if (valido == false) return; // Si no es valido ya se ha enviado el error
+      try {
+        activationValidator.validate(req.body)
+      } catch (e) {
+        res.status(422).json({
+          errors: {
+            msg: e
+          }
+        })
+      }
 
       const { iccid, offer } = req.body
 
-      const compañia = this.compañiaFromIccid(iccid)
+      const compañia = companyFromIccid(iccid)
 
       if (compañia == undefined) {
         res.status(500).json({
@@ -101,12 +107,18 @@ export class SimController {
 
   public cancelation() {
     return async (req: Request, res: Response) => {
-      const valido = this.validateBody(req.body, res)
-
-      if (valido == false) return; // Si no es valido ya se ha enviado el error
+      try {
+        iccidValidator.validate(req.body)
+      } catch (e) {
+        res.status(422).json({
+          errors: {
+            msg: e
+          }
+        })
+      }
 
       const { iccid } = req.body
-      const compañia = this.compañiaFromIccid(iccid)
+      const compañia = companyFromIccid(iccid)
 
       try {
         await this.simUseCases.cancelation({ iccid, compañia })
@@ -127,12 +139,18 @@ export class SimController {
 
   public pause() {
     return async (req: Request, res: Response) => {
-      const valido = this.validateBody(req.body, res)
-
-      if (valido == false) return; // Si no es valido ya se ha enviado el error
+      try {
+        iccidValidator.validate(req.body)
+      } catch (e) {
+        res.status(422).json({
+          errors: {
+            msg: e
+          }
+        })
+      }
 
       const { iccid } = req.body
-      const compañia = this.compañiaFromIccid(iccid)
+      const compañia = companyFromIccid(iccid)
 
       try {
         await this.simUseCases.pause({ iccid, compañia })
@@ -153,12 +171,18 @@ export class SimController {
 
   public free() {
     return async (req: Request, res: Response) => {
-      const valido = this.validateBody(req.body, res)
-
-      if (valido == false) return; // Si no es valido ya se ha enviado el error
+      try {
+        iccidValidator.validate(req.body)
+      } catch (e) {
+        res.status(422).json({
+          errors: {
+            msg: e
+          }
+        })
+      }
 
       const { iccid } = req.body
-      const compañia = this.compañiaFromIccid(iccid)
+      const compañia = companyFromIccid(iccid)
 
       try {
         await this.simUseCases.cancelation({ iccid, compañia })
@@ -185,7 +209,7 @@ export class SimController {
       if (valido == false) return; // Si no es valido ya se ha enviado el error
 
       const { iccid } = req.body
-      const compañia = this.compañiaFromIccid(iccid)
+      const compañia = companyFromIccid(iccid)
 
       try {
         await this.simUseCases.cancelation({ iccid, compañia })
@@ -228,15 +252,4 @@ export class SimController {
     return valid;
   }
 
-  /**
-  * A partir del iccid completo devuelve la compañia a la que pertenece
-  * @throws Error si no hay una compañia definida en COMPAÑIASICCID con el codigo
-  */
-  private compañiaFromIccid(iccid: string) {
-    const caracteresCommpañia = iccid.slice(2, 6)
-    const compañia = COMPAÑIASICCID.get(caracteresCommpañia)
-
-    if (compañia == undefined) throw new Error("El la compañia es desconocida: " + caracteresCommpañia)
-    return compañia
-  }
 }

packages/sim-entrada-eventos/aplication/httpValidators.ts

@@ -0,0 +1,50 @@
+import { companyFromIccid } from "#domain/companies.js";
+import { BodyValidator, Validator } from "sim-shared/aplication/BodyValidator.js";
+
+const offers = new Map([
+  ["mensual", "SAVEFAMILY1"],
+  ["anual", "SAVEFAMILY2"]
+])
+
+const iccidLongitudValidator = <Validator<{ iccid: string }>>{
+  field: "iccid",
+  errorMsg: "La longitud del iccid es incorrecta debera ser de 19 caracteres",
+  validationFunc: (a: { iccid: string }) => a.iccid.length == 19,
+}
+
+const iccidRequired = <Validator<{ iccid: string }>>{
+  field: "iccid",
+  errorMsg: "El iccid debe estara definido",
+  validationFunc: (a: { iccid: string }) => a.iccid != undefined,
+}
+
+const iccidWithValidCompany = <Validator<{ iccid: string }>>{
+  field: "iccid",
+  errorMsg: "El iccid no corresponde a una compañia registrada",
+  validationFunc: (a: { iccid: string }) => companyFromIccid(a.iccid) != undefined,
+}
+
+
+const offerExists = <Validator<{ offer: string }>>{
+  field: "offer",
+  errorMsg: "La oferta introducida no es valida",
+  validationFunc: (a: { offer: string }) => offers.has(a.offer),
+}
+
+
+export const activationValidator = new BodyValidator<{ iccid: string, offer: string }>(
+  [
+    iccidRequired,
+    iccidLongitudValidator,
+    iccidWithValidCompany,
+    offerExists,
+  ]
+)
+
+export const iccidValidator = new BodyValidator<{ iccid: string, offer: string }>(
+  [
+    iccidRequired,
+    iccidLongitudValidator,
+    iccidWithValidCompany,
+  ]
+)