Skip to content

Add tests for gRPC ProjectService#113

Merged
das-Abroxas merged 14 commits intofeat/version2.0reworkfrom
test/grpc-tests
Aug 25, 2023
Merged

Add tests for gRPC ProjectService#113
das-Abroxas merged 14 commits intofeat/version2.0reworkfrom
test/grpc-tests

Conversation

@das-Abroxas
Copy link
Copy Markdown
Contributor

@das-Abroxas das-Abroxas commented Aug 25, 2023

This creates a basic set of tests for the ProjectService.

For this purpose, the corresponding init functions of all modules were created in the test utils, which are also used in the normal start-up process of the ArunaServer. This means that every ArunaAPI service implemented in the ArunaServer can now be emulated in the tests.

Added Tests:

  • ProjectService

ToDo:

  • InfoService
  • UserService
  • AuthorizationService
  • NotificationService
  • EndpointService
  • CollectionService
  • DatasetService
  • ObjectService
  • RelationService
  • SearchService

For the future, a concept should also be developed on how to additionally detect side effects in the tests. So far, the functionalities are only considered in isolation, but could potentially influence/modify other data in case of implementation errors.

@das-Abroxas das-Abroxas added the enhancement New feature or request label Aug 25, 2023
@das-Abroxas das-Abroxas requested review from St4NNi and lfbrehm August 25, 2023 07:13
@das-Abroxas das-Abroxas self-assigned this Aug 25, 2023
@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 25, 2023

Codecov Report

Patch coverage: 28.57% and project coverage change: +6.29% 🎉

Comparison is base (b38418d) 40.53% compared to head (4d9ede6) 46.82%.
Report is 1 commits behind head on feat/version2.0rework.

Additional details and impacted files
@@                    Coverage Diff                    @@
##           feat/version2.0rework     #113      +/-   ##
=========================================================
+ Coverage                  40.53%   46.82%   +6.29%     
=========================================================
  Files                         60       60              
  Lines                       7636     7643       +7     
=========================================================
+ Hits                        3095     3579     +484     
+ Misses                      4541     4064     -477     
Files Changed Coverage Δ
src/grpc/projects.rs 99.08% <ø> (+99.08%) ⬆️
src/search/meilisearch_client.rs 39.62% <0.00%> (+5.29%) ⬆️
src/middlelayer/update_db_handler.rs 81.86% <66.66%> (+1.20%) ⬆️
src/database/dsls/object_dsl.rs 87.01% <100.00%> (+0.41%) ⬆️

... and 12 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Copy Markdown
Member

@lfbrehm lfbrehm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is looks good to me so far.

Copy link
Copy Markdown
Member

@St4NNi St4NNi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+6.53% 🎉 Nice !

@das-Abroxas das-Abroxas merged commit 762a9eb into feat/version2.0rework Aug 25, 2023
@St4NNi St4NNi deleted the test/grpc-tests branch September 20, 2023 08:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants