-
Notifications
You must be signed in to change notification settings - Fork 0
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
32 GitHub actions backend and analyzer test automation #80
32 GitHub actions backend and analyzer test automation #80
Conversation
Signed-off-by: Deniz Deli <[email protected]>
Signed-off-by: Deniz Deli <[email protected]>
Signed-off-by: Deniz Deli <[email protected]>
Signed-off-by: Deniz Deli <[email protected]>
Signed-off-by: Deniz Deli <[email protected]>
Signed-off-by: Deniz Deli <[email protected]>
Signed-off-by: Deniz Deli <[email protected]>
Signed-off-by: Deniz Deli <[email protected]>
Signed-off-by: Deniz Deli <[email protected]>
Signed-off-by: Deniz Deli <[email protected]>
Signed-off-by: Deniz Deli <[email protected]>
Signed-off-by: Deniz Deli <[email protected]>
I just saw that i need to adjust the target branch. I will do that as soon as i get confirmation that all backend and analyzer tests under development are running local and are merged to dev |
Signed-off-by: Deniz Deli <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are these commented out sections necessary? If not then I think it would be better practice to remove them.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The tests are definitely necessary but are currently failing. In order to build my test runner, I have commented them out for now. They must be fixed or removed in the next sprint
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work!
Added Github actions pipline for automated tests. had to comment out two backend tests in backupData.service.spec.ts to make it run