Add App Engine system tests by theacodes · Pull Request #59 · googleapis/google-auth-library-python

theacodes · 2016-10-26T22:33:00Z

~~Has #58 as a diffbase.~~

Resolves #41, progress towards #55.

system_tests/app_engine/app/.gitignore

@@ -0,0 +1 @@
+lib


system_tests/app_engine/app/appengine_config.py

+
+
+# Patch os.path.expanduser. This should be fixed in GAE
+# versions released after Nov 2016.


system_tests/app_engine/app/main.py

+import contextlib
+import json
+import sys
+from StringIO import StringIO


system_tests/app_engine/app/main.py

+    response = HTTP_REQUEST(url=url, method='GET')
+    token_info = json.loads(response.data.decode('utf-8'))
+
+    assert token_info['scope'] == EMAIL_SCOPE


system_tests/app_engine/app/main.py

+
+    # Get token info and verify scope
+    url = _helpers.update_query(TOKEN_INFO_URL, {
+        'access_token': scoped_credentials.token


system_tests/app_engine/app/requirements.txt

@@ -0,0 +1,3 @@
+urllib3
+# Relative path to google-auth-python's source.
+../../..


system_tests/app_engine/test_app_engine.py

+def vendor_app_dependencies():
+    """Vendors in the test application's third-party dependencies."""
+    subprocess.check_call(
+        ['pip', 'install', '-t', 'lib', '-r', 'requirements.txt'])


system_tests/app_engine/test_app_engine.py

+def deploy_app():
+    """Deploys the test application using gcloud."""
+    subprocess.check_call(
+        ['gcloud', 'app', 'deploy', '-q', 'app.yaml'])


system_tests/app_engine/test_app_engine.py

+
+
+@pytest.mark.skipif(
+    'SKIP_APP_ENGINE_SYSTEM_TEST' in os.environ,


system_tests/app_engine/test_app_engine.py

+    reason='Explicitly skipping App Engine system tests.')
+def test_live_application(app, http_request):
+    response = http_request(method='GET', url=app)
+    assert response.status == 200, response.data.decode('utf-8')


system_tests/app_engine/app/main.py

+            output = (
+                'Test {} failed: {}\n\n'
+                'Stacktrace:\n{}\n\n'
+                'Captured output:\n{}').format(


system_tests/app_engine/app/main.py

-    output += 'Captured output:\n{}'.format(capsys.getvalue())
+    tests = (test_credentials, test_default)
+
+    for test in tests:


theacodes added this to the 1.0.0 milestone Oct 26, 2016

theacodes assigned dhermes Oct 26, 2016

Jon Wayne Parrott added 2 commits October 26, 2016 21:08

Add App Engine system tests

42e5d7e

Add token info check

1f4e69b

theacodes force-pushed the gae-system-tests branch from 98ed4bf to 1f4e69b Compare October 27, 2016 04:08

dhermes reviewed Oct 27, 2016

View reviewed changes

Address review comments

1aaf384

dhermes approved these changes Oct 27, 2016

View reviewed changes

theacodes merged commit 7a27f47 into master Oct 27, 2016

theacodes unassigned dhermes Oct 11, 2018



		# Patch os.path.expanduser. This should be fixed in GAE
		# versions released after Nov 2016.



		@pytest.mark.skipif(
		'SKIP_APP_ENGINE_SYSTEM_TEST' in os.environ,

		@@ -0,0 +1 @@
		lib

Conversation

theacodes commented Oct 26, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

theacodes commented Oct 26, 2016 •

edited

Loading