core: Remove GetVdsHooksById2Query

GetVdsHooksByIdQuery and GetVdsHooksById2Query are completely
identical except for the usage of different APIs of VdsHooksParser.

Since GetVdsHooksById is not used anywhere, it is removed, and
GetVdsHooksById2 loses it "2" and get a name the better suits
conventions (not to mention common sense).

This patch includes:
1. Removing the old GetVdsHooksByIdQuery class and renaming
   GetVdsHooksById2Query to take its place.
2. Removing the old GetVdsHooksById constant in the VdsQueryTye
   enum, and renaming GetVdsHooksById2 to take its place.
3. Clearing up the VdsHooksParser class from methods with "2" in their
   name - either by removing unused methods, or by renaming them to a
   more sensible name.
4. Removing the TestVdsHooks class, as it only tested the removed
   methods.

Change-Id: I1890f429a12cbf7e42727b7360a85ebb1b07b5b0
Signed-off-by: Allon Mureinik <amureini@redhat.com>