Stop uisng a designated tests project ID.

This is because the project ID can be determined directly from
the file pointed to by GOOGLE_APPLICATION_CREDENTIALS.

Fixes #2263.

CONTRIBUTING.rst

@@ -153,8 +153,6 @@ Running System Tests
   so you'll need to provide some environment variables to facilitate
   authentication to your project:
 
-  - ``GOOGLE_CLOUD_TESTS_PROJECT_ID``: Developers Console project ID (e.g.
-    bamboo-shift-455).
   - ``GOOGLE_APPLICATION_CREDENTIALS``: The path to a JSON key file;
     see ``system_tests/app_credentials.json.sample`` as an example. Such a file
     can be downloaded directly from the developer's console by clicking
@@ -182,12 +180,12 @@ Running System Tests
    $ export CLOUDSDK_PYTHON_SITEPACKAGES=1
 
    # Authenticate the gcloud tool with your account.
-   $ JSON_CREDENTIALS_FILE="path/to/app_credentials.json"
-   $ gcloud auth activate-service-account --key-file=$JSON_CREDENTIALS_FILE
+   $ GOOGLE_APPLICATION_CREDENTIALS="path/to/app_credentials.json"
+   $ gcloud auth activate-service-account \
+   > --key-file=${GOOGLE_APPLICATION_CREDENTIALS}
 
    # Create the indexes
-   $ gcloud preview datastore create-indexes system_tests/data/index.yaml \
-   > --project=$GOOGLE_CLOUD_TESTS_PROJECT_ID
+   $ gcloud preview datastore create-indexes system_tests/data/index.yaml
 
    # Restore your environment to its previous state.
    $ unset CLOUDSDK_PYTHON_SITEPACKAGES

google/cloud/environment_vars.py

@@ -21,9 +21,6 @@
 PROJECT = 'GOOGLE_CLOUD_PROJECT'
 """Environment variable defining default project."""
 
-TESTS_PROJECT = 'GOOGLE_CLOUD_TESTS_PROJECT_ID'
-"""Environment variable defining project for tests."""
-
 GCD_DATASET = 'DATASTORE_DATASET'
 """Environment variable defining default dataset ID under GCD."""
 

system_tests/bigquery.py

@@ -17,7 +17,6 @@
 import unittest
 
 from google.cloud import _helpers
-from google.cloud.environment_vars import TESTS_PROJECT
 from google.cloud import bigquery
 from google.cloud.exceptions import Forbidden
 
@@ -52,7 +51,6 @@ class Config(object):
 
 
 def setUpModule():
-    _helpers.PROJECT = TESTS_PROJECT
     Config.CLIENT = bigquery.Client()
 
 

system_tests/bigtable.py

@@ -29,7 +29,6 @@
 from google.cloud.bigtable.row_filters import RowFilterUnion
 from google.cloud.bigtable.row_data import Cell
 from google.cloud.bigtable.row_data import PartialRowData
-from google.cloud.environment_vars import TESTS_PROJECT
 
 from retry import RetryErrors
 from retry import RetryResult
@@ -92,7 +91,6 @@ def _retry_on_unavailable(exc):
 
 def setUpModule():
     from grpc._channel import _Rendezvous
-    _helpers.PROJECT = TESTS_PROJECT
     Config.CLIENT = Client(admin=True)
     Config.INSTANCE = Config.CLIENT.instance(INSTANCE_ID, LOCATION_ID)
     retry = RetryErrors(_Rendezvous, error_predicate=_retry_on_unavailable)

system_tests/clear_datastore.py

@@ -21,7 +21,6 @@
 import six
 
 from google.cloud import datastore
-from google.cloud.environment_vars import TESTS_PROJECT
 
 
 FETCH_MAX = 20
@@ -90,7 +89,7 @@ def remove_kind(kind, client):
 def remove_all_entities(client=None):
     if client is None:
         # Get a client that uses the test dataset.
-        client = datastore.Client(project=os.getenv(TESTS_PROJECT))
+        client = datastore.Client()
     for kind in ALL_KINDS:
         remove_kind(kind, client)
 

system_tests/datastore.py

@@ -23,7 +23,6 @@
 from google.cloud import datastore
 from google.cloud.datastore.helpers import GeoPoint
 from google.cloud.environment_vars import GCD_DATASET
-from google.cloud.environment_vars import TESTS_PROJECT
 from google.cloud.exceptions import Conflict
 
 import clear_datastore
@@ -55,7 +54,6 @@ def setUpModule():
     # Isolated namespace so concurrent test runs don't collide.
     test_namespace = 'ns' + unique_resource_id()
     if emulator_dataset is None:
-        _helpers.PROJECT = TESTS_PROJECT
         Config.CLIENT = datastore.Client(namespace=test_namespace)
     else:
         credentials = EmulatorCreds()

system_tests/language.py

@@ -15,7 +15,6 @@
 import unittest
 
 from google.cloud import _helpers
-from google.cloud.environment_vars import TESTS_PROJECT
 from google.cloud import exceptions
 from google.cloud import language
 from google.cloud import storage
@@ -39,7 +38,6 @@ class Config(object):
 
 
 def setUpModule():
-    _helpers.PROJECT = TESTS_PROJECT
     Config.CLIENT = language.Client()
     # Now create a bucket for GCS stored content.
     storage_client = storage.Client()

system_tests/local_test_setup.sample

@@ -1,4 +1,3 @@
 export GOOGLE_APPLICATION_CREDENTIALS="app_credentials.json.sample"
-export GOOGLE_CLOUD_TESTS_PROJECT_ID="my-project"
 export GOOGLE_CLOUD_REMOTE_FOR_LINT="upstream"
 export GOOGLE_CLOUD_BRANCH_FOR_LINT="master"

system_tests/logging_.py

@@ -21,7 +21,6 @@
 from google.cloud.logging.handlers.transports import SyncTransport
 from google.cloud.logging import client
 from google.cloud import _helpers
-from google.cloud.environment_vars import TESTS_PROJECT
 
 from retry import RetryErrors
 from retry import RetryResult
@@ -52,7 +51,6 @@ class Config(object):
 
 
 def setUpModule():
-    _helpers.PROJECT = TESTS_PROJECT
     Config.CLIENT = client.Client()
 
 

system_tests/monitoring.py

@@ -15,7 +15,6 @@
 import unittest
 
 from google.cloud import _helpers
-from google.cloud.environment_vars import TESTS_PROJECT
 from google.cloud.exceptions import InternalServerError
 from google.cloud.exceptions import NotFound
 from google.cloud.exceptions import ServiceUnavailable
@@ -30,10 +29,6 @@
 retry_503 = RetryErrors(ServiceUnavailable)
 
 
-def setUpModule():
-    _helpers.PROJECT = TESTS_PROJECT
-
-
 class TestMonitoring(unittest.TestCase):
 
     def test_fetch_metric_descriptor(self):

system_tests/populate_datastore.py

@@ -22,7 +22,6 @@
 import six
 
 from google.cloud import datastore
-from google.cloud.environment_vars import TESTS_PROJECT
 
 
 ANCESTOR = ('Book', 'GoT')
@@ -91,7 +90,7 @@ def print_func(message):
 def add_characters(client=None):
     if client is None:
         # Get a client that uses the test dataset.
-        client = datastore.Client(project=os.getenv(TESTS_PROJECT))
+        client = datastore.Client()
     with client.transaction() as xact:
         for key_path, character in six.moves.zip(KEY_PATHS, CHARACTERS):
             if key_path[-1] != character['name']:

system_tests/pubsub.py

@@ -23,7 +23,6 @@
 # pylint: disable=ungrouped-imports
 from google.cloud import _helpers
 from google.cloud.environment_vars import PUBSUB_EMULATOR
-from google.cloud.environment_vars import TESTS_PROJECT
 from google.cloud.pubsub import client
 # pylint: enable=ungrouped-imports
 
@@ -51,7 +50,6 @@ class Config(object):
 
 
 def setUpModule():
-    _helpers.PROJECT = TESTS_PROJECT
     if os.getenv(PUBSUB_EMULATOR) is None:
         Config.CLIENT = client.Client()
     else:

system_tests/storage.py

@@ -21,7 +21,6 @@
 import six
 
 from google.cloud import _helpers
-from google.cloud.environment_vars import TESTS_PROJECT
 from google.cloud import exceptions
 from google.cloud import storage
 from google.cloud.storage._helpers import _base64_md5hash
@@ -33,7 +32,6 @@
 
 retry_429 = RetryErrors(exceptions.TooManyRequests)
 HTTP = httplib2.Http()
-_helpers.PROJECT = TESTS_PROJECT
 
 
 class Config(object):

system_tests/system_test_utils.py

@@ -18,11 +18,9 @@
 import time
 
 from google.cloud.environment_vars import CREDENTIALS as TEST_CREDENTIALS
-from google.cloud.environment_vars import TESTS_PROJECT
 
 
 # From shell environ. May be None.
-PROJECT_ID = os.getenv(TESTS_PROJECT)
 CREDENTIALS = os.getenv(TEST_CREDENTIALS)
 
 ENVIRON_ERROR_MSG = """\
@@ -44,24 +42,15 @@ def create_scoped_required():
 
 
 def check_environ():
-    missing = []
-    extra = ''
-
-    if PROJECT_ID is None:
-        missing.append(TESTS_PROJECT)
-
+    err_msg = None
     if CREDENTIALS is None:
-        missing.append(TEST_CREDENTIALS)
+        err_msg = '\nMissing variables: ' + TEST_CREDENTIALS
     elif not os.path.isfile(CREDENTIALS):
-        extra = '\nThe %s path %r is not a file.' % (TEST_CREDENTIALS,
-                                                     CREDENTIALS)
+        err_msg = '\nThe %s path %r is not a file.' % (TEST_CREDENTIALS,
+                                                       CREDENTIALS)
 
-    if missing or extra:
-        msg = ENVIRON_ERROR_MSG
-        if missing:
-            msg += '\nMissing variables: ' + ', '.join(missing)
-        if extra:
-            msg += extra
+    if err_msg is not None:
+        msg = ENVIRON_ERROR_MSG + err_msg
         print(msg, file=sys.stderr)
         sys.exit(1)