POC definitions

ngarbezza · Nov 4, 2024 · 6558d73 · 6558d73
1 parent 78a7cd0
commit 6558d73
Show file tree

Hide file tree

Showing 4 changed files with 18 additions and 2 deletions.
diff --git a/lib/core/test_runner.js b/lib/core/test_runner.js
@@ -15,6 +15,7 @@ export class TestRunner {
   #failFastMode;
   #randomOrder;
   #testExecutionTimeoutMs;
+  #archivoActual;
 
   constructor(callbacks) {
     this.#suites = [];
@@ -25,7 +26,7 @@ export class TestRunner {
   // Configuration
 
   registerSuite(name, suiteBody, callbacks) {
-    const suiteToAdd = new TestSuite(name, suiteBody, callbacks);
+    const suiteToAdd = new TestSuite(name, suiteBody, callbacks, this.#archivoActual);
     return this.addSuite(suiteToAdd);
   }
 
@@ -139,4 +140,9 @@ export class TestRunner {
       return this.#callbacks.onSuccess(this);
     }
   }
+
+  cargandoArchivo(file) {
+    console.log(`Cargando archivo: ${file}`);
+    this.#archivoActual = file;
+  }
 }
diff --git a/lib/core/test_suite.js b/lib/core/test_suite.js
@@ -38,14 +38,18 @@ export class TestSuite {
 
   // Instance creation
 
-  constructor(name, body, callbacks) {
+  // TODO: ajustar la regla de eslint para que max-params permita 6 parámetros
+  // eslint-disable-next-line max-params
+  constructor(name, body, callbacks, definition) {
     this.#initializeName(name);
     this.#initializeBody(body);
     this.#tests = [];
     this.#callbacks = callbacks;
     this.#before = undefined;
     this.#after = undefined;
     this.#isSkipped = false;
+    // TODO: hacer privado
+    this.definition = definition;
   }
 
   // Initializing / Configuring

diff --git a/lib/testy.js b/lib/testy.js
@@ -178,6 +178,9 @@ class Testy {
 
   async #loadFileHandlingErrors(file) {
     try {
+      // OJO: este approach solo funciona porque estamos cargando los archivos en secuencia.
+      // TODO: crear una ADR para documentar esta decisión.
+      testRunner.cargandoArchivo(file);
       await import(file);
     } catch (err) {
       ui.exitWithError(

diff --git a/lib/ui/formatter.js b/lib/ui/formatter.js
@@ -55,6 +55,9 @@ export class Formatter {
   displaySuiteEnd(suite) {
     this.#displaySeparator('-');
     this.#console.log(`${this.#translated('summary_of')} ${suite.name()}:`);
+    // TODO: internacionalizar mensaje
+    // TODO: testear que se muestre la definición
+    this.#console.log(`Defined in: ${suite.definition}`);
     this.#displayCountFor(suite);
     this.#displaySeparator();
   }