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

adding integration tests #34

Merged
merged 30 commits into from
Jun 12, 2024
Merged

adding integration tests #34

merged 30 commits into from
Jun 12, 2024

Conversation

unglaublicherdude
Copy link
Member

No description provided.

@unglaublicherdude unglaublicherdude marked this pull request as ready for review June 6, 2024 14:17
@unglaublicherdude
Copy link
Member Author

This PR adds 2 simple test that only tests the upload function for now.
But that already tests if the vaas app does function within the nextcloud version.

We can and should add more like running the cron and checking in the logs if files get tagged as unscanned and such stuff.

@unglaublicherdude
Copy link
Member Author

added the following tests for each user (admin and testuser)

  • upload eicar
  • upload clean
  • upload pup

added a test for eicar upload when vaas does not work (which should not reject the upload)

@unglaublicherdude unglaublicherdude changed the title adding the install script adding integration tests Jun 11, 2024
@unglaublicherdude
Copy link
Member Author

added 2 more tests

  1. background scan for admin user
  2. background scan for testuser

@unglaublicherdude unglaublicherdude force-pushed the add-tests branch 3 times, most recently from 1428eea to 93c70a6 Compare June 11, 2024 15:52
this commit adds commands for tagging and scanning so the test don't have to rely on a cron run.
@unglaublicherdude unglaublicherdude merged commit 8965e55 into main Jun 12, 2024
2 checks passed
@pstadermann pstadermann deleted the add-tests branch June 12, 2024 11:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants