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

Issue/3315 New Integration test facility & add integration tests for auth #3344

Merged
merged 76 commits into from
Apr 21, 2022
Merged

Issue/3315 New Integration test facility & add integration tests for auth #3344

merged 76 commits into from
Apr 21, 2022

Conversation

t83714
Copy link
Contributor

@t83714 t83714 commented Apr 14, 2022 •
edited
Loading

What this PR does

Fixes

Checklist

  • There are unit tests to verify my changes are correct
  • I've updated CHANGES.md with what I changed.

t83714 added 30 commits April 5, 2022 23:44
@t83714
launch service using Dockerode
7419393
@t83714
- move integration tests to a seperate module
f40e6e9 
- upgrade pg to 7.18.2
@t83714
add es to authServiceRunner
21853c7
@t83714
only create /data when it doesn't exist
0308be1
@t83714
- add opa support
57e9524 
- use latest dockerode-compose forked commit
@t83714
add db migrator
62b3b04
@t83714
turn off elasticsearch by default
a8c8b13
@t83714
auto remove containers for migrator
e7299f3
@t83714
add auth api to authServiceRunner
9894094
@t83714
renamed to service runner
94cd37f
@t83714
move test cases to tests folder
830b2e8
@t83714
Add run-service CLI tool
d9888fc
@t83714
add registry api & aspect migrator
252aee5
@t83714
- refactor auth api
4827cb0 
- add public create user API
@t83714
- refactor user related type definition
24c9a30 
- make create / update user return current user data
- make `displayName` & `email` compulsory field when create user
@t83714
add more registry test case
a9a5d70
@t83714
- make user related api also available at /public/user as well (to be…
d57d9e1 
… backwards compatible )

- better error message
- make AuthApiClient use api endpoint at `/public/users` (/public/user will be removed in future)
@t83714
- not support draft datasets without "publishing.state" set to "draft…
009d85e 
…" anymore.

i.e. A draft dataset must has "pulishing" aspect `state` field set to "draft".
Our frontend will always create draft dataset with "publishing" aspect. Thus, this change won't impact our frontend.
@t83714
fix test cases
36c2144
@t83714
adjust CI pipeline
26cd0e6
@t83714
- remove legacy int-tests
06dd9f7 
- make new int-tests a compulsory step for all release jobs
@t83714
update cache key
d7566db
@t83714
increase default wait service live time to 5 mins
476b76b
@t83714
auto pull migrator image
4d1da71
@t83714
auto setup port-forwarding
8c0488a
@t83714
fix: should run setDockerServiceForwardHost in ServiceRunner constr…
079059c 
…uctor
@t83714
fix incorrect quote in port-forward command
897b991
@t83714
port-forward should not wait for process exit
c19abc2
@t83714
adjust how sbt task is run
a132361
@t83714
should build magda-migrator-registry-aspects before run integration t…
606bcff 
…ests
t83714 added 22 commits April 18, 2022 01:53
@t83714
add storage api test case
9d1163f
@t83714
add more test cases
552d666
@t83714
complete storage api test cases
6487481
@t83714
use original docker compose file to launach elasticsearch
6a63cf1
@t83714
- add allowAutoCrawlOnStartingUp option to indexer
1c7520b 
- add `/v0/live` & `/v0/ready` endpoint to indexer
- ServiceRuner: auth service is not enabled by default
- ServiceRuner: index setup will be performed after elastic search is online
@t83714
fix indexer test case
8f50561
@t83714
move status endpoint of indexer to /v0/status/x
172b89f
@t83714
add search api service to service runner
ca448ca
@t83714
#3345 Add dataset index / delete API endpoints to indexer
a4b375f
@t83714
reformat code
527ae03
@t83714
make indexer auto-exit unless user explicitly requests the service
e8d9aa8
@t83714
indexer api client
3fd9315
@t83714
Indexer index & delete endpoint will auto refresh index
12240a9
@t83714
fix: search api should be able to locate dataset by dataset id
f77fcdf
@t83714
add debug mode to search api
1cdd015
@t83714
bug fixes
721e92d
@t83714
add more search api auth test cases
2c790a7
@t83714
reformat code
a046897
@t83714
fix MockIndexer
1390e27
@t83714
make registry db migrator run first
7d460d1
@t83714
increase max timeout
b84433c
@t83714
update docs
7c14058
@t83714 t83714 mentioned this pull request Apr 21, 2022
Closed
@t83714 t83714 marked this pull request as ready for review April 21, 2022 00:29
@t83714
Copy link
Contributor Author

t83714 commented Apr 21, 2022

tested in pipelines & locally

@t83714 t83714 merged commit 1867f2d into next Apr 21, 2022
@t83714 t83714 deleted the issue/3315 branch April 21, 2022 04:19
@t83714 t83714 mentioned this pull request May 3, 2022
Closed
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.
1 participant
@t83714