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

Quarterly security updates #119

Merged
merged 10 commits into from
Jul 29, 2024
Merged

Quarterly security updates #119

merged 10 commits into from
Jul 29, 2024

Conversation

wilwong89
Copy link
Contributor

Description

Update versions of Node, Github actions, and various NPM packages.

Types of changes

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)

Checklist

  • I have read the CONTRIBUTING doc
  • I have checked that unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

Further comments

Copy link

codeclimate bot commented Jul 17, 2024

Code Climate has analyzed commit df0dc88 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 43.1% (0.0% change).

View more on Code Climate.

Copy link

github-actions bot commented Jul 17, 2024

Coverage Report (Frontend)

Totals Coverage
Statements: 32.47% ( 1421 / 4376 )
Methods: 27.87% ( 245 / 879 )
Lines: 37.17% ( 878 / 2362 )
Branches: 26.26% ( 298 / 1135 )

Copy link

Coverage Report (Application)

Totals Coverage
Statements: 45.45% ( 918 / 2020 )
Methods: 35.5% ( 131 / 369 )
Lines: 56.37% ( 597 / 1059 )
Branches: 32.09% ( 190 / 592 )

frontend/package.json Outdated Show resolved Hide resolved
@qhanson55
Copy link
Collaborator

qhanson55 commented Jul 18, 2024

I'm getting these warning/errors when I run npm run test on my local


You may find that it works just fine, or you may not.

SUPPORTED TYPESCRIPT VERSIONS: >=4.7.4 <5.5.0

YOUR TYPESCRIPT VERSION: 5.5.3

Please only submit bug reports when using the officially supported version.

=============

/Users/qhanson/dev/nr-permitconnect-navigator-service/frontend/src/views/DeveloperView.vue
  20:16  warning  'ssoRequestBasicAccess' is defined but never used  @typescript-eslint/no-unused-vars

✖ 1 problem (0 errors, 1 warning)


> [email protected] test
> vitest run --coverage

The CJS build of Vite's Node API is deprecated. See https://vitejs.dev/guide/troubleshooting.html#vite-cjs-node-api-deprecated for more details.

 RUN  v1.6.0 /Users/qhanson/dev/nr-permitconnect-navigator-service/frontend
      Coverage enabled with istanbul

stderr | tests/unit/service/configService.spec.ts > Config Store > init > getConfig reaquires missing config
Configuration missing. Attempting to reacquire...

stderr | tests/unit/service/configService.spec.ts > Config Store > init > getConfig fails to reaquire missing config
Configuration missing. Attempting to reacquire...
Failed to reacquire configuration: () => Promise.reject({
          data: testData
        })

stdout | tests/unit/components/submission/SubmissionBringForwardCalendar.spec.ts > SubmissionBringForwardCalendar > renders component
[UserManager] getUser: user not found in storage

stderr | tests/unit/utils/formatters.spec.ts > formatters.ts > formatDate > returns empty string if given invalid format
_dateFnsFormat: Error parsing asd to RangeError: Invalid time value```

@wilwong89 wilwong89 force-pushed the feature/security+updates branch from dcb5b6d to ffa7295 Compare July 25, 2024 19:19
@wilwong89 wilwong89 force-pushed the feature/security+updates branch from ffa7295 to df0dc88 Compare July 25, 2024 19:23
@wilwong89 wilwong89 closed this Jul 25, 2024
@wilwong89 wilwong89 reopened this Jul 25, 2024
@kyle1morel kyle1morel merged commit 4ff1602 into master Jul 29, 2024
30 checks passed
@kyle1morel kyle1morel deleted the feature/security+updates branch July 29, 2024 17:32
@kyle1morel kyle1morel changed the title Feature/security+updates Quarterly security updates Jul 29, 2024
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.

4 participants