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

Testing acceptance test #510

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Rhodine-orleans-lindsay
Copy link
Contributor

What?

Why?

How?

Testing?

Screenshots (optional)

Anything Else? (optional)

Check list

  • I have reviewed my own pull request for linting issues (e.g. adding new lines)
  • I have written tests (if relevant)
  • I have created a JIRA number for my branch
  • I have created a JIRA number for my commit
  • I have followed the chris beams method for my commit https://cbea.ms/git-commit/
    here is an example commit
  • Ensure drone builds are green especially tests
  • I will squash the commits before merging

SavantDhindsaHO and others added 2 commits September 22, 2024 23:10
1.Updated the package.json file to use beta version of HOF.  2.Updated pdf-upload.js, auth-token.js, file-upload.js and    pdf.js file to use fields related to axios.
3.Updated all the unit tests related to axios request.
4.Updated the readme file with information about starting firearms locally
5.Update file-upload test to resolve test failure
- update hof version to use axios
- update playwright version to fix breaking acceptance test
- update yarn.lock
- update node to version 20
- refactor pdf, file-upload,  auth-token, i-casework and  i-casework-getcase models to work with hof model
- amend casework-submission behaviour to return response data
- refactor functions in casework-submission and auth-token to async/await
- refactor models/pdf and models/file-upload tests following changes in models
- add missing confirmation page titles
- update snyk ignore list
- add db name to config.js and encode db name in models/icasework-documents and models/icasework
- remove duplicate headings on relevant pages
- add hof logger and remove console logging
- remove 'uat' from branch deployment message
@Rhodine-orleans-lindsay Rhodine-orleans-lindsay force-pushed the testing-acceptance-test branch 19 times, most recently from 5e50776 to 3841ad3 Compare September 25, 2024 20:33
logs and amend save

logs and amend icaseowork-documents

reduce notify log

add ci env to acceptance test and amend if statement so secret is not picked up first

amend if statement for acceptance tests

clean up

remove data

remove createpath db

add env log

remove upload db

amend if statment for acceptance test

remove console.log
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