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.

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

tests: Include tests for pkg/api #138

Merged
merged 3 commits into from
Dec 22, 2023
Merged

Commits on Dec 20, 2023

  1. tests: Include UT tests to pkg/api/store

    Add unit tests for `pkg/api/store` coverage as 84%
    
    Signed-off-by: Kairo de Araujo <kairo.araujo@testifysec.com>
    kairoaraujo committed Dec 20, 2023
    Configuration menu
    Copy the full SHA
    b7314d2 View commit details
    Browse the repository at this point in the history

Commits on Dec 21, 2023

  1. tests: Include UT tests to pkg/api/download

    Add unit tests for `pkg/api/download` coverage as 85%
    
    The UT found a bug in the `DownloadWithWriter`. This bug returns nil
    instead of error and affects users while downloading envelopes. For
    example, the `archivistactl retrieve envelope` command, if giving an
    invalid server or if the server is unavailable, will return no error.
    
    In the `archivistactl`, the user will behave the same when the
    server is available, and there is no envelope with the expected
    `gitoid`.
    
    Signed-off-by: Kairo de Araujo <kairo.araujo@testifysec.com>
    kairoaraujo committed Dec 21, 2023
    Configuration menu
    Copy the full SHA
    477fe51 View commit details
    Browse the repository at this point in the history
  2. tests: Include UT tests to pkg/api/graphql

    Add unit tests for `pkg/api/graphql` coverage as 86%
    
    Signed-off-by: Kairo de Araujo <kairo.araujo@testifysec.com>
    kairoaraujo committed Dec 21, 2023
    Configuration menu
    Copy the full SHA
    8d28176 View commit details
    Browse the repository at this point in the history