Spaces filters

[micbar]

Description

This adds two possible filter terms (driveType, id) to the /me/drives endpoint on the graph api. These can be used with the odata query parameter "$filter".

We only support the "eq" operator for now.

This is leveraging the spaces-registry branch and needs to be rebased after that one has been merged.

Examples

Filter by Id

curl -L -X GET "https://localhost:9200/graph/v1.0/me/drives?$filter=id eq 'f98b7cc7-15ba-4715-9d16-23a746dfe648'" \
-H "Authorization: Basic YWRtaW46YWRtaW4="

Filter by type

curl -L -X GET "https://localhost:9200/graph/v1.0/me/drives?$filter=driveType eq 'project'" \
-H "Authorization: Basic YWRtaW46YWRtaW4="

Motivation and Context

How Has This Been Tested?

• curl
• needs API tests

Screenshots (if appropriate):

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• Technical debt
• Tests only (no source changes)

Checklist:

• Code changes
• Unit tests added
• Acceptance tests added
• Documentation ticket raised:

[micbar]

@butonic @kobergj I think I found a bug.

Regardless of the filters, the shared jail (driveType=="virtual") is always present.

[ownclouders]

💥 Acceptance test localApiTests-apiSpaces-ocis failed. Further test are cancelled...

[micbar]

@kobergj let me add some API tests

[dragotin]

Maybe not so nitpicky nitpicks.

[dragotin]

Nice from my POV

[butonic]

The test could be less redundant.

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

0.0% Coverage
0.0% Duplication