Skip to content

Commit 262f55a

Browse files
dhermeslandrito
authored andcommitted
Skipping system tests when credentials env. var is unset. (googleapis#3475)
1 parent 66d313a commit 262f55a

File tree

15 files changed

+17
-17
lines changed

15 files changed

+17
-17
lines changed

.circleci/config.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ version: 2
33
jobs:
44
build:
55
docker:
6-
- image: googleapis/nox:0.11.2
6+
- image: googleapis/nox:0.17.0
77
steps:
88
- checkout
99
- run:

appveyor/requirements.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,4 +3,4 @@
33
# pip will build them from source using the MSVC compiler matching the
44
# target Python version and architecture
55
wheel
6-
nox-automation==0.11.2
6+
nox-automation>=0.17.0

bigquery/nox.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ def system_tests(session, python_version):
4949

5050
# Sanity check: Only run system tests if the environment variable is set.
5151
if not os.environ.get('GOOGLE_APPLICATION_CREDENTIALS', ''):
52-
return
52+
session.skip('Credentials must be set via environment variable.')
5353

5454
# Run the system tests against latest Python 2 and Python 3 only.
5555
session.interpreter = 'python{}'.format(python_version)

bigtable/nox.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ def system_tests(session, python_version):
4848

4949
# Sanity check: Only run system tests if the environment variable is set.
5050
if not os.environ.get('GOOGLE_APPLICATION_CREDENTIALS', ''):
51-
return
51+
session.skip('Credentials must be set via environment variable.')
5252

5353
# Run the system tests against latest Python 2 and Python 3 only.
5454
session.interpreter = 'python{}'.format(python_version)

datastore/nox.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ def system_tests(session, python_version):
4949

5050
# Sanity check: Only run system tests if the environment variable is set.
5151
if not os.environ.get('GOOGLE_APPLICATION_CREDENTIALS', ''):
52-
return
52+
session.skip('Credentials must be set via environment variable.')
5353

5454
# Run the system tests against latest Python 2 and Python 3 only.
5555
session.interpreter = 'python{}'.format(python_version)
@@ -70,7 +70,7 @@ def doctests(session):
7070

7171
# Sanity check: Only run system tests if the environment variable is set.
7272
if not os.environ.get('GOOGLE_APPLICATION_CREDENTIALS', ''):
73-
return
73+
session.skip('Credentials must be set via environment variable.')
7474

7575
# Doctests run against Python 3.6 only.
7676
# It is difficult to make doctests run against both Python 2 and Python 3

error_reporting/nox.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ def system_tests(session, python_version):
7878

7979
# Sanity check: Only run system tests if the environment variable is set.
8080
if not os.environ.get('GOOGLE_APPLICATION_CREDENTIALS', ''):
81-
return
81+
session.skip('Credentials must be set via environment variable.')
8282

8383
# Run the system tests against latest Python 2 and Python 3 only.
8484
session.interpreter = 'python{}'.format(python_version)

language/nox.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ def system_tests(session, python_version):
4949

5050
# Sanity check: Only run system tests if the environment variable is set.
5151
if not os.environ.get('GOOGLE_APPLICATION_CREDENTIALS', ''):
52-
return
52+
session.skip('Credentials must be set via environment variable.')
5353

5454
# Run the system tests against latest Python 2 and Python 3 only.
5555
session.interpreter = 'python{}'.format(python_version)

logging/nox.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ def system_tests(session, python_version):
5252

5353
# Sanity check: Only run system tests if the environment variable is set.
5454
if not os.environ.get('GOOGLE_APPLICATION_CREDENTIALS', ''):
55-
return
55+
session.skip('Credentials must be set via environment variable.')
5656

5757
# Run the system tests against latest Python 2 and Python 3 only.
5858
session.interpreter = 'python{}'.format(python_version)

monitoring/nox.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ def system_tests(session, python_version):
4949

5050
# Sanity check: Only run system tests if the environment variable is set.
5151
if not os.environ.get('GOOGLE_APPLICATION_CREDENTIALS', ''):
52-
return
52+
session.skip('Credentials must be set via environment variable.')
5353

5454
# Run the system tests against latest Python 2 and Python 3 only.
5555
session.interpreter = 'python{}'.format(python_version)

pubsub/nox.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ def system_tests(session, python_version):
4949

5050
# Sanity check: Only run system tests if the environment variable is set.
5151
if not os.environ.get('GOOGLE_APPLICATION_CREDENTIALS', ''):
52-
return
52+
session.skip('Credentials must be set via environment variable.')
5353

5454
# Run the system tests against latest Python 2 and Python 3 only.
5555
session.interpreter = 'python{}'.format(python_version)

0 commit comments

Comments
 (0)