Got a lot of unused variables (I assume build failed). Also the do nothing setup and teardownModule aren't needed.

Other questions

• Why the URI changes?
• Do you plan on implementing more in this PR?

(Sorry for hacky review. On phone.)

Coverage remained the same at 100.0% when pulling b906f33 on tseaver:pubsub-begin_regression_suite into b45a909 on GoogleCloudPlatform:master.

I rebased to remove #764 and adjust the tests to accommodate the changed URIs.

Per the API docs, the URIs are all based at https://pubsub.googleapis.com/v1beta2, rather than https://www.googleapis.com/pubsub/v1beta1. I know they are right because the regression tests were all 404ing with non-JSON errors before. :)

I plan to add more tests in later PRs (e.g., once we decide about returning Topic instances from pubsub.api.list_topics()).

Coverage remained the same at 100.0% when pulling c1ebe23 on tseaver:pubsub-begin_regression_suite into b45a909 on GoogleCloudPlatform:master.

Still have unused vars HTTP and SHARED_BUCKETS and an unused import httplib2.

Just call me the human linter :)

Other than LGTM (if you don't don't can you squash these fixed into the original commit, 4 commits for tint change is sadness)

Coverage remained the same at 100.0% when pulling 5981067 on tseaver:pubsub-begin_regression_suite into b45a909 on GoogleCloudPlatform:master.

@dhermes I removed those variables and the import, and squashed it down to two commits.

Looks great thanks