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

ci: initial test pipeline UI #109

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

ci: initial test pipeline UI #109

wants to merge 7 commits into from

Conversation

frank-bee
Copy link
Member

@frank-bee frank-bee commented Dec 12, 2024

fixes #107

Copy link

Code Coverage Report

Project Statements Rate Branch Rate Indicator
packages/autopilot-utils 96.71% 90.08% 💚
dist/coverage/python 96.71% 90.08% 💚
apps/splunk-fetcher 92.51% 82.00% 💚
apps/sharepoint-fetcher 84.83% 76.62% 💚
apps/sharepoint-evaluator 94.09% 90.48% 💚
apps/sharepoint 84.06% 67.24% 💚
apps/security-scanner 0.00% 0.00% 💔
apps/pex-tool 93.07% 78.05% 💚
apps/pdf-signature-evaluator 67.48% 48.80% 💛
apps/papsr 98.28% 94.74% 💚
apps/filecheck 98.53% 96.30% 💚
apps/excel-tools 78.37% 62.89% 💛
apps/artifactory-fetcher 98.51% 92.86% 💚
Total 87.16% 75.59% 💚

@frank-bee
Copy link
Member Author

@fbufler @FlorianZieglerBosch I think this is rather a general issue with the test execution than actual errors

@fbufler
Copy link
Member

fbufler commented Dec 12, 2024

indeed, so i ran the tests out from the yaku-ui folder could that be the difference?
yaku-test

@frank-bee
Copy link
Member Author

@fbufler what do you mean , you run it inside the folder in your screen capture, not?

@fbufler
Copy link
Member

fbufler commented Dec 13, 2024

@fbufler what do you mean , you run it inside the folder in your screen capture, not?

Yes I do.

cd yaku-ui
npm install -ws --include-workspace-root
npm run build
npm run test

(Unsure if it works without the build as I cant really test it well)

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.

Unit test UI
3 participants