Test files:scan

[skshetry]

This commit, although a bit hacky, adds the test for the files:scan
in the Occ. The config didn't seem to have come into effect when set.
But, i noticed that the config was coming into effect when it was
force-scanned. Needs further investigation if files_external:option
is the problem or not.

Related Issue

• Part of Test files:scan occ command #33617

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

Checklist:

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

Open tasks:

• Backport (if applicable set "backport-request" label and remove when the backport was done)

[skshetry]

@phil-davis It seemed to me that, if you used any APIs to check for that file, it would just add the entry to it's filecache, though I cannot fully confirm this. So, I had to use the api from testing app.

EDIT: files_external:config works the same way. As per the docs, it manages backend configuration for mounts and option manages configuration for mounts. It seems confusing, but maybe it 's like backend/frontend for mounts. But, both seems to require force-scan to come into effect.

[phil-davis]

Test scenario looks good.
See the comments.

[phil-davis]

Looking good. We will see what happens when drone is available some day.

[codecov]

Codecov Report

Merging #33647 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master   #33647   +/-   ##
=========================================
  Coverage     64.34%   64.34%           
  Complexity    18305    18305           
=========================================
  Files          1195     1195           
  Lines         69228    69228           
  Branches       1276     1276           
=========================================
  Hits          44548    44548           
  Misses        24308    24308           
  Partials        372      372

Flag	Coverage Δ	Complexity Δ
#javascript	52.96% <ø> (ø)	0 <ø> (ø)	⬇️
#phpunit	65.66% <ø> (ø)	18305 <ø> (ø)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 171a41d...3105164. Read the comment docs.

[skshetry]

@phil-davis @individual-it please review

[phil-davis]

@ownclouders rebase

[ownclouders]

Hey! I'm GitMate.io! This pull request is being rebased automatically. Please DO NOT push while rebase is in progress or your changes would be lost permanently ⚠️

[ownclouders]

Automated rebase with GitMate.io was successful! 🎉

[phil-davis]

drone and CI is all functioning now, so I asked ownclouders to rebase this.
It can be merged when CI passes.
@skshetry please make a backport

[phil-davis]

Backport stable10 #33685