remove tests that assert on internals

Also remove comment
Leave only run method exposed publically in Vcloud::Query

Also deleted some unused methods.
remove command line options that are not useful
Split out query functionality into two classes

The Vcloud::Query class was combining two separate concerns -
handling the command line inputs/formatting the output and
interacting with Fog. This commit splits out those two pieces of
functionality. The Query class handles the command line input and
formats the output and the QueryRunner class does the interactions
with Fog.

Also removed the code for debug output, which wouldn't have worked
anyway due to no require for 'pp' and the fog mock code.
Refactor vcloud_query code to be easier to follow
expose deprecated original public method to ensure consistent public interface
Bump the version and add a changelog entry
Add unit tests for Vcloud::QueryRunner
rename variable for clarity
additional tests to complete coverage of QueryRunner
Independent unit tests for Vcloud::Query

Remove references to fog and reunitise by mocking out QueryRunner
Clean up Vcloud::QueryRunner.get_results_page

* Remove argument nil check that will never occur.
* Remove haphazard exception handling.
Ensure files end with a blank line
use Vcloud::QueryRunner instead of depricated call to Vcloud::Query
remove unnecessary blank lines
Change test data for clarity in query_runner_spec

The test just requires two pieces of data and it was found to be confusing
using the same piece of data twice. Made to use two different pieces of
data to avoid confusion.
Merge pull request #14 from alphagov/restructure_vcloud_query

Separate query runner api from query tool