Allow to disable preinstalled apps after installation

[staticdev]

Follow-up from #150

• Add example on README
• Add example on coverge for molecule test

[staticdev]

@aalaesar @wiktor2200 @jspaans91 I tried to resurrect this feature, but not sure how is the intended use do disable an app. Were any of you successful testing it? Maybe can you help me adding an example of useful/common app to disable?

[jspaans91]

Hi @staticdev,

Sorry for not respond in the other PR. I'll respond with an example this weekend. I made multiple successful deployments with the version I had committed.

[aalaesar]

As I said in this comment (but I switched disabled with enabled in my example 😨 )
This can be achieved in one task with a loop and I would prefer this simplier implementation:

• I don't think it is required to fetch the list of app prior to disabling any
• it is better of readability and idempotence display to loop over all apps declared as "I want this disabled" in the var by having the changed state defined in function of the occ app:disable command's stdout/err.

Regards,
Aal

PS: I'll seriously consider doing a module for managing nextcloud apps once the repo is in the nextcloud organization.

[staticdev]

@jspaans91 friendly ping on the example ;D I want to merge this one.

i comited some code

[aalaesar]

Should be all that was missing.
can you review @staticdev ?
I've commited to much to properly review

[aalaesar]

ho @staticdev I just realised that tests playbook under /tests/ are not used anymore. I am right ?
but that mean that code coverage has reduced. 😞

[staticdev]

Lgtm @aalaesar ?