builder/utils/execute_cmd: remove useless return value and add a test
    
    Since we never pass in subprocess.PIPE for stdin/stdout/stderr,
    subprocess.communicate() does nothing, and the return out/err tuple
    was always empty.