-
Notifications
You must be signed in to change notification settings - Fork 20
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: Add test for mimetype blocking on share #321
Conversation
juliusknorr
commented
Mar 13, 2023
•
edited
Loading
edited
- test case for fix: Avoid failing with duplicate checks server#36830
- add cases for existing code around this change Also check the path based mimetype for flow rule checks server#23096 Only execute plain mimetype check for directories and do the fallback… server#24459
- Remove last TMP commit once fix: Avoid failing with duplicate checks server#36830 is merged
a0c7d45
to
17a1a83
Compare
9081722
to
80969c5
Compare
5e9d36a
to
0468063
Compare
02423f2
to
c9fba5e
Compare
c9fba5e
to
b3245ae
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice!
* @param array $arrayOfArrays | ||
* @return array | ||
*/ | ||
public function simplifyArray($arrayOfArrays) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is it actually used?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think I added for completeness as there were usages from Joas initial commit:
tests/Integration/features/bootstrap/WebDav.php
579: $elementsSimplified = $this->simplifyArray($elementRows);
930: $elementsSimplified = $this->simplifyArray($elementRows);
Signed-off-by: Julius Härtl <jus@bitgrid.net> test: Add pdf data for integration tests Signed-off-by: Julius Härtl <jus@bitgrid.net>