remove unused var
cleanup test-packages

- Errors in merging the cordova branch against 273b70bea48a189bd73 left
  in an unnecessary getLocalPackages function

- LocalCatalog.getLocalPackageNames is unnecessary (it's the same as
  getAllPackageNames!)  CompleteCatalog.getLocalPackageNames is OK I
  guess.

- getPackagesForTest returned some extra data that was only used to call
  the nonexistent catalog.removeLocalPackage method.  Meh, we don't
  clean up the catalog when the command ends now (which doesn't happens
  unless you're using --once anyway). We'll probably fix this by making
  catalog.complete less global soon anyway (ie, it should be build-specific).

- Fix a duplicate test runner app cp_r