Skip to content

fix(backend): non-OP resource test fixes #2974

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
Merged
merged 32 commits into from
Sep 16, 2024
Merged

fix(backend): non-OP resource test fixes #2974

merged 32 commits into from
Sep 16, 2024

Conversation

@njlie
Copy link
Contributor

@njlie njlie commented Sep 13, 2024

Changes proposed in this pull request

  • Fixes tests for backend services that aren't OP resources or GraphQL resolvers.

Context

Fixes #2970.

This is part of a greater effort captured in #2972.

Checklist

  • Related issues linked using fixes #number
  • Tests added/updated
  • Make sure that all checks pass
  • Bruno collection updated (if necessary)
  • Documentation issue created with user-docs label (if necessary)
  • OpenAPI specs updated (if necessary)

@github-actions github-actions bot added type: tests Testing related pkg: backend Changes in the backend package. type: source Changes business logic labels Sep 13, 2024
@njlie njlie changed the base branch from nl-fix-tenant-feature-branch-tests to 2893-multi-tenant-rafiki September 16, 2024 16:26
njlie and others added 22 commits September 16, 2024 09:31
@njlie
feat(backend): create operator kratos identity on backend startup
3a1b976
@njlie
chore: formatting
90373b3
@njlie
fix: handle new cases
9e403ac
@njlie
feat(frontend,backend): tenant management api
350a0f4
@sanducb @njlie
Add migration files
e4d0b9b
@sanducb @njlie
Update migration files
bd208cb
@golobitch @njlie
feat(tenant): basic tenant admin api schema and service
b311b5c
@sanducb @njlie
Fix file name for migrations
ee58eaf
@golobitch @njlie
feat(auth): create basic tenant service and model plus graphql schema
7090b00
@golobitch @njlie
feat(auth): add basic tenant schema and appropriate resources like mo…
78407d1 
…del and service
@golobitch @njlie
feat(backend): add apollo client do dependencies
029099d
@golobitch @njlie
feat(auth): delete tenant
ef7af00
@golobitch @njlie
chore(auth): format
607d650
@golobitch @njlie
feat(backend): create tenant service implementation
54ae837
@golobitch @njlie
feat(packages): make multi tenant work wip
41a7a15
@golobitch @njlie
feat(backend): update resolvers with tenant id and finish the tenant …
a00dbc6 
…creation
@golobitch @njlie
feat(backend): small changes to schema + mapping of tenant to graphql…
3118adf 
… + bruno
@njlie
feat(backend,frontend): create kratos identity when tenants are creat…
2655885 
…ed, permissioned admin UI pages
@njlie
fix: tenant service functions
28c2533
@njlie
feat: better kratos check conditionals
1afea35
@njlie
fix: operator secret for non-session api calls
b56ed94
@njlie
fix: wallet address tests
6c40743
@njlie njlie force-pushed the nl/2970/feature-branch-tests-non-op branch from 3bb49db to 9f6bb0d Compare September 16, 2024 17:18
@njlie njlie merged commit b7e7ed0 into 2893-multi-tenant-rafiki Sep 16, 2024
@njlie njlie deleted the nl/2970/feature-branch-tests-non-op branch September 16, 2024 17:23
@njlie njlie mentioned this pull request Sep 30, 2024
Closed
7 tasks
@njlie njlie mentioned this pull request Oct 11, 2024
Closed
@njlie njlie linked an issue Oct 11, 2024 that may be closed by this pull request
Backend Admin API middleware to verify session token; add tenantId to context #2916
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@golobitch golobitch golobitch approved these changes

@mkurapov mkurapov Awaiting requested review from mkurapov

@sanducb sanducb Awaiting requested review from sanducb

Assignees

No one assigned

Labels

pkg: auth Changes in the GNAP auth package. pkg: backend Changes in the backend package. pkg: mock-account-service-lib pkg: mock-ase type: source Changes business logic type: tests Testing related

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Backend Admin API middleware to verify session token; add tenantId to context

4 participants

@njlie @golobitch @sanducb