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

📝 docs: adiciona info de testes no readme #180

Merged

Conversation

charlesribeiro
Copy link
Contributor

Descrição

Ao rodar o alias de teste npx nx test, apenas os testes localizados no diretório src/ estão sendo executados atualmente.

Screenshot 2024-09-17 at 20 56 06

Inclusive nas automações do sonar, uma vez que o sonar-project.properties aponta para o arquivo tsconfig.app.json

sonar.typescript.tsconfigPath=tsconfig.app.json

Este PR adiciona os arquivos de modules ao jest.config.app.ts e ao tsconfig.app.json

Screenshot 2024-09-17 at 20 56 55

Este PR também adiciona informações a respeito de como rodar os testes, incluido flags interessantes. Adiciona também novos aliases ao package.json para rodar testes com flag de watch e test-coverage.


Tipo da alteração

🐛 fix(environment): inclui modules nos testes via jest.config.app.ts
📝 docs(docs): atualiza readme.md e package.json com scripts de test


Outras observações

tsconfig.spec.json teve de incluir a flag "strict": false, em compilerOptions, uma vez que o erro abaixo estava ocorrendo nos testes de modules
Screenshot 2024-09-17 at 21 24 44
tsconfig.json possui a flag setada como "strict": true


@andrewarosario andrewarosario changed the title 📝 docs(docs): adiciona info de testes no readme 📝 docs: adiciona info de testes no readme Sep 24, 2024
Copy link

github-actions bot commented Sep 25, 2024

Deploy preview for portal-ascepa ready!

✅ Preview
https://portal-ascepa-lhw26s11e-andrew-rosarios-projects.vercel.app

Built with commit 6156f6a.
This pull request is being automatically deployed with vercel-action

@FelipeDuarteLuna FelipeDuarteLuna added documentation Improvements or additions to documentation setup Project setup or configuration labels Oct 1, 2024
Copy link
Contributor

@victoralvesprof victoralvesprof left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@FelipeDuarteLuna FelipeDuarteLuna self-requested a review October 8, 2024 22:15
Copy link
Contributor

@FelipeDuarteLuna FelipeDuarteLuna left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review realizado com Sucesso, OK

Copy link

sonarqubecloud bot commented Oct 8, 2024

@charlesribeiro charlesribeiro merged commit 3649c75 into main Oct 8, 2024
4 checks passed
@charlesribeiro charlesribeiro deleted the docs/170-readme-adicionar-comandos-para-testar-o-projeto branch October 8, 2024 22:17
Copy link
Contributor

@RenanBotelho RenanBotelho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review realizado com Sucesso.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation setup Project setup or configuration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants