Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update objetivo-7.md #220

Closed
wants to merge 14 commits into from
Closed

Update objetivo-7.md #220

wants to merge 14 commits into from

Conversation

MarioGuisado
Copy link
Contributor

Entrega del objetivo del proyecto de Infraestructura virtual

Primero y mas importante

  • NOCERRAR: No voy a cerrar ningún PR pase lo que pase. Ni voy a abrir otro
    para el mismo objetivo.

Lista de comprobación para hitos

  • GUION: He leído el guión del hito
  • PRE: Cumplo los prerrequisitos (he aprobado el hito anterior, por ejemplo)

@JJ
Copy link
Owner

JJ commented Jan 5, 2024

¿No vas a intentar arreglarlo?

@MarioGuisado
Copy link
Contributor Author

@JJ No sé cómo solucionarlo. El error (Error: EACCES: permission denied, open './logs.log') dice que hay un problema con los permisos del archivo logs.log, que supongo que salta en el test que comprueba que se escribe correctamente en este archivo. Es un error que a mí no me salta y no sé como replicarlo.

@JJ
Copy link
Owner

JJ commented Jan 5, 2024

No, no es sólo ese error, y de hecho que dé ese error puede ser culpa de otra cosa. Míralo bien. Y no te da en local por razones obvias.

@MarioGuisado
Copy link
Contributor Author

Lo único que se me ocurre es que esté relacionado con el "[[email protected]][DEBUG] Failed to load /app/test/.env ENOENT: no such file or directory, open '/app/test/.env'". Entiendo que si falta el fichero .env dará problemas, pero se supone que este fichero no podemos subirlo, así que no sé qué hacer.

@JJ
Copy link
Owner

JJ commented Jan 5, 2024

Lo único que se me ocurre es que esté relacionado con el "[[email protected]][DEBUG] Failed to load /app/test/.env ENOENT: no such file or directory, open '/app/test/.env'". Entiendo que si falta el fichero .env dará problemas, pero se supone que este fichero no podemos subirlo, así que no sé qué hacer.

Si no puedes subirlo, ¿no será conveniente que uses otro fichero?

@MarioGuisado
Copy link
Contributor Author

He eliminado la necesidad de cargar .env y ahora los tests usan la configuración por defecto de la clase Config. Si elimino el archivo .env me sigue pasando los tests, pero el error sigue siendo el mismo: Error: EACCES: permission denied, open './logs.log'

@JJ
Copy link
Owner

JJ commented Jan 6, 2024

Tendrás que poner otro fichero para que guarde los logs, o ninguno en el caso de hacer los tests.

@JJ
Copy link
Owner

JJ commented Jan 6, 2024

Precisamente la idea de usar los logs en un objeto es que se puedan configurar para que no escriban en fichero, sino a memoria, cambiando la construcción o el tipo del log...

@JJ
Copy link
Owner

JJ commented Jan 6, 2024

Piensa que es difícil, si no imposible, testear con un log que escriba en fichero. Tendrías que leer el fichero para ver si ha escrito lo que deseas. Por eso os explico el concepto de inyección de dependencias, para que inyectéis el objeto del log, en cada fase (test, producción) con características diferentes (posiblemente configuradas mediante el objeto de configuración.

@JJ JJ closed this Jan 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants