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

Feature/SK-1288 | Test all get (/) api endpoints #786

Merged
merged 71 commits into from
Jan 23, 2025
Merged

Feature/SK-1288 | Test all get (/) api endpoints #786

merged 71 commits into from
Jan 23, 2025

Conversation

niklastheman
Copy link
Contributor

Added unit tests for all routes, that is api/v1/clients, api/v1/combiners etc. More tests will be added in a later stage.

niklastheman and others added 30 commits December 20, 2024 15:01
@niklastheman
added graphql server
2710e6e
@niklastheman
sessions added
bab877e
@niklastheman
get single session
9c4ad30
@niklastheman
added models
0fe3cfd
@niklastheman
added validations
a29f0a1
@niklastheman
statuses added
750cf3a
@niklastheman
added plura/singular naming to follow convention
6dcb6af
@niklastheman
added abstract parent class, Store along with MongoDBStore
0e596dc
@niklastheman
removed use_typings option
9271df6
@niklastheman
clean up
e55ad72
@niklastheman
merged SK-1109
b069844
@niklastheman
removed use_typings, changes to complie with feature/SK-1109
0ec6d6c
@niklastheman
started on replacing mongostatestore
4ebb1eb
@niklastheman
removed get_combiners from mongostatestore and started using combiner…
eaf75c0 
…_store in Network class
@niklastheman
removed add/delete combiner from network and mongostatestore
08f8583
@niklastheman
moved get and list clients from mongostatestore
fab412a
@niklastheman
update sessions with session_store
328217b
@niklastheman
removed set_session_status from mongostatestore
5a5303f
@niklastheman
added round_store to combiner
1ac6880
@niklastheman
clean up
9d9bfa8
@niklastheman
upload model moved to api/v1/models
010528f
@niklastheman
removed get model
d0df714
@niklastheman
moved add client to api/v1/clients/add
70fc83d
@niklastheman
moved download package to api/v1/packages/download
d542140
@niklastheman
moved get checksum to api/v1/packages/checksum
a1ac5b1
@niklastheman
removed old client and old package runtime class
839bf39
@niklastheman
mongostatestore dependecy to control removed and repository dependecy…
db14426 
… injected into control
@niklastheman
get_client_config moved to api/v1/clients/config
57ba8d1
@niklastheman
removed API (interface) class
5fdc75c
@niklastheman
added indexes to mogno stores
64fdd83
@niklastheman
added status store
420f347
@niklastheman
added validations sql store
099ff2e
@niklastheman
added client store
dbf3f5a
@niklastheman
started on prediction store
fd1c34b
@niklastheman
prediction store v1
633d7cf
@niklastheman
create sql tables only if sql stores
51c463c
@niklastheman
linting fix
6920404
@niklastheman
sqllite postgres or mongo based on env:s. moved table creation to cot…
34a5818 
…roller only
@niklastheman
implemented connected_client_count for client sql store
a7d7e08
@niklastheman
clean up
361d967
@niklastheman
client_store update should be like every other update
b8f89b7
@niklastheman
run table creation script from combiner to fix timing issue
669c154
@niklastheman
merge of master
fdc77d9
@niklastheman
removed debug logging
ab7eafc
@niklastheman
This SQL query depends on a user-provided value fix.
6816327
@niklastheman
ruff linting fix#
d734c55
@niklastheman
api client should use api/v1
72bff6a
@niklastheman
should be able to upload package without name and description
1de8855
@niklastheman
added tests for all /get endpoints
df5f412
@niklastheman
refactor to test all endpoints in one functions
013ea88
@github-actions github-actions bot added feature New feature or request minor labels Jan 13, 2025
@niklastheman niklastheman requested a review from Wrede January 23, 2025 10:13
@niklastheman niklastheman merged commit a464c7a into master Jan 23, 2025
17 checks passed
@niklastheman niklastheman deleted the feature/SK-1288 branch January 23, 2025 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ahellander ahellander Awaiting requested review from ahellander

@Wrede Wrede Awaiting requested review from Wrede

Assignees
No one assigned
Labels
feature New feature or request minor
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@niklastheman @Wrede