BHBC-2144: Fix Project Team Member interface + Amend system user GUID implementation

[curtisupshall]

Overview

Links to Jira tickets

• https://quartech.atlassian.net/browse/BHBC-2144

Description of relevant changes

• Fixes a bug with adding participants to a project;
• Removes the User GUID text field from the Add System User form;
• Makes the user_guid property in the system_users table nullable;
• Adds a service method for finding users by user identifier + identity source;
• Adds a function which patches the user's GUID entry whenever their user context is set during a database connection.

[github-actions]

Openshift URLs for the PR Deployment:

• App Route: https://biohubbc-app-926-af2668-dev.apps.silver.devops.gov.bc.ca/
• Api Route: https://biohubbc-api-926-af2668-dev.apps.silver.devops.gov.bc.ca/
• Pods: https://console.apps.silver.devops.gov.bc.ca/k8s/ns/af2668-dev/pods?name=926

[codecov]

Codecov Report

Merging #926 (1fa86fd) into dev (b7dd6a4) will increase coverage by 0.02%.
The diff coverage is 90.90%.

@@            Coverage Diff             @@
##              dev     #926      +/-   ##
==========================================
+ Coverage   65.73%   65.75%   +0.02%     
==========================================
  Files         351      349       -2     
  Lines       11319    11317       -2     
  Branches     1963     1961       -2     
==========================================
+ Hits         7440     7441       +1     
+ Misses       3475     3471       -4     
- Partials      404      405       +1     

Impacted Files	Coverage Δ
api/src/models/user.ts	69.23% <ø> (ø)
.../src/paths/project/{projectId}/participants/get.ts	96.00% <ø> (ø)
api/src/paths/user/list.ts	85.71% <ø> (ø)
api/src/queries/queries.ts	100.00% <ø> (ø)
app/src/features/admin/users/ActiveUsersList.tsx	32.50% <ø> (ø)
...pp/src/features/admin/users/AddSystemUsersForm.tsx	20.00% <ø> (+2.60%)	⬆️
api/src/repositories/user-repository.ts	97.87% <75.00%> (-2.13%)	⬇️
api/src/services/user-service.ts	62.90% <85.71%> (+2.18%)	⬆️
api/src/database/db.ts	85.98% <92.30%> (+2.17%)	⬆️
...c/paths/project/{projectId}/participants/create.ts	86.11% <100.00%> (-0.38%)	⬇️
... and 3 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[NickPhura]

Just curious regarding the set guid sql + set context sql: was there no way to do it in 1 call in the end? Because of the views, or the way functions work?

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

[KjartanE]

Looks great! ☄️