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.

Sign up for GitHub

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

Jump to bottom

Run backend tests in pipeline #6350

Draft
wants to merge 17 commits into
base: main
Choose a base branch
from
Draft

Run backend tests in pipeline #6350

wants to merge 17 commits into from

Conversation

duncanc19
Copy link
Collaborator

@duncanc19 duncanc19 commented Sep 5, 2024 •
edited
Loading

Work done

  • Fixed issue with incorrect stubbing of an S3 call which was causing backend tests to fail in pipeline
  • Updated the test-backend buildspec to run the backend tests and not the placeholder lambda unit test coverage command
  • Added install of bulk upload dependencies to stop missing module error in test-backend
  • Updated puppeteer to latest version in frontend and removed from backend package as not needed
  • Moved build of staging frontend into separate buildspec and renamed build.yml to build-backend.yml

duncanc19 and others added 17 commits September 4, 2024 09:04
@duncanc19
Update test-backend buildspec to install bulk upload dependencies and…
dfa8730 
… run backend unit tests
@duncanc19
Fix pre_build phase name
b034f4f
@duncanc19
cd to backend directory from root
584df25
@duncanc19
Use relative path to cd into backend directory
6545df8
@duncanc19
Skip tests with potential incorrect stubbing
80a5750
@duncanc19
Ensure putObject is stubbed for all tests in uploadFiles spec as it w…
89eac23 
…as not stubbed in uploadedStarGet tests where it is called
@kapppa-joe
Merge branch 'main' into run-backend-tests-in-pipeline
51190a1
@kapppa-joe
remove puppeteer in backend, bump up version of puppeteer in frontend…
e35aa95 
…, add command to test-frontend.yml to install chrome before running test
@kapppa-joe
remove npm ci from test-backend.yml
281da6f
@kapppa-joe
add back npm ci to test-backend.yml
bf62f00
@kapppa-joe
configure puppeteer to install cache in local folder
ab12370
@kapppa-joe
remove npx command to install puppeteer chrome from test-frontend.yml
0b296ae
@kapppa-joe
empty commit to test pipeline
08c3972
@kapppa-joe
add npm install to test-frontend build project
1ef8ff7
@kapppa-joe
add npm install to test-performance build project
ca9269e
@kapppa-joe
add a new build spec for build-frontend-staging, remove frontend inst…
9f25458 
…all & build step from build.yml
@kapppa-joe
rename file build.yml --> build-backend.yml
8fea114
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@duncanc19 @kapppa-joe