I've implemented the mock --shell 'echo foo' test in
ticket-78-test-fix branch.
Would be nice if we could verify that is what we want to do. But it
seems to
work pretty fine with my testing.
This approach won't work. If mock root is clean then mock shell will fail.
$ mock clean
$ ./try-fedora-review -vb 1
INFO: mock.py version 1.1.28 starting...
Start: init plugins
INFO: selinux enabled
Finish: init plugins
Start: run
CRITICAL: No package specified to rebuild command.
DEBUG: Exception down the road...
Traceback (most recent call last):
File "/home/kojan/git/FedoraReview/src/FedoraReview/review_helper.py", line
194, in run
self._do_run()
File "/home/kojan/git/FedoraReview/src/FedoraReview/review_helper.py", line
164, in _do_run
Settings.init()
File "/home/kojan/git/FedoraReview/src/FedoraReview/settings.py", line 283, in
init
_check_mock_grp(self.mock_config)
File "/home/kojan/git/FedoraReview/src/FedoraReview/settings.py", line 53, in
_check_mock_grp
output, = p.communicate()
ValueError: too many values to unpack
ERROR: Exception down the road...(logs in /home/kojan/.cache/fedora-review.log)