The three refutations in wrappers.bats that asserted nothing

[marcos-mendez]

tests/wrappers.bats came in with #6 after the sweep in Keel-Linux/tracker#19. Three of its seven bare negations were not the last command of their test. A bats test passes or fails on the exit status of its last command, and bash's errexit ignores a negated command, so ! cmd anywhere else is never checked:

• line 267, "refuses a target that is not a WordPress": that wp-cli is never called for a refused target;
• line 325, "USER in the environment does not decide who owns wp-content": that nothing under wp-content is handed to root;
• line 337, "WPROOT in the environment does not decide what this script rewrites": that the updater never runs chown -R root:root /.

All seven are run ! now, with bats_require_minimum_version 1.5.0. The four that were last in their test are converted too, so that no line only works because of where it sits.

Test plan

• For each of the seven, making that one command succeed turns its test red. Each fails with status 1 (grep found no match), not 2 (file missing).
• Real mutations of keel-wordpress-update for the three that were dead: calling wp-cli before the target check, chowning wp-content/uploads to root, and chowning /. Each turns this branch red and leaves master green.
• The extended gate from Negations that assert, and a gate so the class cannot come back .github#14 (6372ac2) reports nothing; on master it reports these three lines.
• bats -r tests green.
• Tests ship nothing, so no changelog entry.

Part of Keel-Linux/tracker#19. Must merge before Keel-Linux/.github#14.