Repository navigation
Un script para probar a mano ya no genera un PR: cuenta como un test — 0.8.6 - #41
Merged
Merged
Conversation
…— 0.8.6 probar-arca.js, prueba.php, test-afip.js y ejemplo-factura.js se tratan como tests: el hallazgo sigue en el scan y, si no hay otro cambio, pr deja el texto como issue. Viene del PR #16 de Elitali1/proyectoPaginaWeb, cerrado por el mantenedor porque el servicio ya mandaba el campo.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Qué cambia
Un script para probar a mano (
probar-arca.js,prueba.php,test-afip.js,ejemplo-factura.js) cuenta como un test. Si es lo único que cambiaría,prno abre un PR: el hallazgo sigue en el scan y el texto queda como issue en.parche/.Por qué
El PR Elitali1/proyectoPaginaWeb#16 tocaba solo
probar-arca.js. El servicio (src/services/arca.service.js) ya mandaba el campo, y el cuerpo del PR lo decía. El mantenedor lo cerró igual: "El sistema ya informa CondicionIVAReceptorId según el receptor". Un PR que no cambia el código que corre se lee como ruido, y eso le saca confianza al resto.Cómo se reconoce
Va por el nombre del archivo, sin la extensión (
isProbeScript, ensrc/paths.js):-,_o.;test_data.js,testimonios.js,contest.js,demo.jsy una carpetaprobar/siguen siendo código.Verificación
npm test: 318 de 318 (4 tests nuevos entest/scripts-de-prueba.test.js).2d96e02):pr --dry-runya no abre PR, deja el reporte y mantiene el aviso decuitReceptor ? 1 : 5.Los dos tests de punta a punta que usaban
probar.jscomo archivo a arreglar ahora usansrc/caja.js, que es código.Versión 0.8.6.