Issue 4954 - Implement a downloader script supporting OAuth2 authentication/authorization

tox.ini

Index: tox.ini
diff --git a/tox.ini b/tox.ini
index a4b3915f77ad471191162f16cf1de038562a87c1..5302ea80066e758c0c574124d23151d7ea44cb95 100644
--- a/tox.ini
+++ b/tox.ini
@@ -4,11 +4,11 @@ skipsdist = true
 
 [pytest]
 # The names of the test files here don't start with 'test_' so we need
-# to instruct pytest to load tests from all files.
+# to instruct pytest to load test from all files.
 python_files = *.py
 
 [run]
-# Exclude tests from coverage.
+# Exclude test from coverage.
 omit = */test/*
 
 [flake8]
@@ -77,6 +77,9 @@ per-file-ignores =
     /sitescripts/utils.py : A107,A301,D200,D202,D205,D208,D400,D401,E501,N801,N802,N806
     /sitescripts/web.py : A107,E501,N802
     /multiplexer.py : A107,E501
+    /sitescripts/oauth2dl/bin/oauth2dl.py: D413, E501, A301

[Vasily Kuznetsov, 2018/07/20 21:00:01]

It would be better not to add exceptions for new code. Is any of this really
necessary or can we adjust the code to make flake8 happy?

[Tudor Avram, 2018/07/31 09:20:18]

Done.

+    /sitescripts/oauth2dl/test/test_oauth2dl.py: A110
+    sitescripts/oauth2dl/utils/dummy_wsgi_app.py: A206
 
 [testenv]
 basepython=python2
@@ -96,6 +99,8 @@ deps =
     flake8-commas
     pep8-naming
     hg+https://hg.adblockplus.org/codingtools#egg=flake8-eyeo&subdirectory=flake8-eyeo
+    httplib2

[Vasily Kuznetsov, 2018/07/20 21:00:01]

Do we need httplib2 for the new script? Do Google libraries require it or what's
the reason?

[Tudor Avram, 2018/07/23 19:29:03]

Yeah, the google library requires an `httplib2.Http()` object to be passed to
them. From my understanding, they are using the respective object to make the
requests to their API.

[Tudor Avram, 2018/07/31 09:20:19]

Done.

+    oauth2client
 
 setenv =
     SITESCRIPTS_CONFIG = ./.sitescripts.test
@@ -115,5 +120,6 @@ commands =
         sitescripts/extensions/test \
         sitescripts/subscriptions/test \
 	sitescripts/reports/tests \
+    sitescripts/oauth2dl/test/test_oauth2dl.py \

[Vasily Kuznetsov, 2018/07/20 21:00:01]

Would be neater to keep consistent indentation.

[Tudor Avram, 2018/07/23 19:29:03]

Acknowledged.

[Tudor Avram, 2018/07/31 09:20:18]

Done.

         tests
     flake8 sitescripts multiplexer.py multiplexer.fcgi