search

google-code-export / gdata-python-client

Automatically exported from code.google.com/p/gdata-python-client
1 stars 0 forks source link

Failed client_test.py 'https' != 'http' #371

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Hi guys,

I received an error report this morning on my project, with the following 
debugging information posted:

---
running gdata test, i receive this error, the only:
Running all tests in module gdata_tests.client_test
............F..
======================================================================
FAIL: test_simple_request (gdata_tests.client_test.RequestTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/backup/store/gdata-2.0.10/tests/gdata_tests/client_test.py", line 
218, in test_simple_request
    self.assertEqual(response.getheader('Echo-Scheme'), 'http')
AssertionError: 'https' != 'http'

----------------------------------------------------------------------
Ran 15 tests in 0.009s

FAILED (failures=1)

---

The link to the issue report is here: http://code.google.com/p/google-docs-
fs/issues/detail?id=27 with the reporters system information.

Any information on the causes of this issue on how to fix it would be 
greatly appreciated.

Cheers,
Scott W

Original issue reported on code.google.com by d38dm8nw81k1ng@gmail.com on 27 May 2010 at 9:34

GoogleCodeExporter commented 9 years ago 
Apologies, the link is:

http://code.google.com/p/google-docs-fs/issues/detail?id=27

Original comment by d38dm8nw81k1ng@gmail.com on 27 May 2010 at 9:39

GoogleCodeExporter commented 9 years ago 
I see this as well in 2.0.10. It looks like gdata.GDClient.request() calls into 
its parent class' atom.client.AtomPubClient.request() method. 

AtomPubClient.request() defaults to overriding the scheme to https and the test 
doesn't change this default. (see src/atom/client.py lines 39 and 101-102). It 
only does this for requests specified using a full http_request; requests 
passed as a uri alone don't get munged - this is why the following test (which 
also uses http, but passed as a uri) works.

Original comment by d...@google.com on 16 Jun 2010 at 11:39

GoogleCodeExporter commented 9 years ago 
oops, actually the following test suffers the same fate - it just doesn't get 
run because the AssertionError takes down the function.

Original comment by d...@google.com on 16 Jun 2010 at 11:44

GoogleCodeExporter commented 9 years ago 
Will fix tomorrow.

Original comment by vicfry...@google.com on 29 Jun 2010 at 8:35

GoogleCodeExporter commented 9 years ago 
Did this framework ever work? I went back several versions and still got the 
same errors.

Original comment by aobje...@gmail.com on 30 Jun 2010 at 4:39

GoogleCodeExporter commented 9 years ago 
I've started working on a patch for this:

  http://codereview.appspot.com/1677050

Not complete, as I believe the Calendar API will only accept SSL connections 
and so its default should change also.

Original comment by joe.gregorio@gmail.com on 30 Jun 2010 at 1:54

GoogleCodeExporter commented 9 years ago 
This patch did not resolve the error when running tests/run_data_tests.py

Running all tests in module gdata_tests.client_test
............F..
======================================================================
FAIL: test_simple_request (gdata_tests.client_test.RequestTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/var/www/html/pygdata_trunk/tests/gdata_tests/client_test.py", line 218, in test_simple_request
    self.assertEqual(response.getheader('Echo-Scheme'), 'http')
AssertionError: 'https' != 'http'

----------------------------------------------------------------------
Ran 15 tests in 0.009s

FAILED (failures=1)

Original comment by aobje...@gmail.com on 30 Jun 2010 at 5:01

GoogleCodeExporter commented 9 years ago 
I spoke too soon. Seems like I got it to go. I though I could omit the email 
change. But of course we are testing all systems. So naturally it would fail. 
All compiled fine, now.

Original comment by aobje...@gmail.com on 30 Jun 2010 at 5:08

GoogleCodeExporter commented 9 years ago 
http://codereview.appspot.com/1677050 has now been committed to trunk. I will 
mark this issue as Fixed, please reopen if you are still seeing issues.

Original comment by joe.gregorio@gmail.com on 2 Jul 2010 at 7:35

GoogleCodeExporter commented 9 years ago 
i just downloaded & installed the latest version of the zip file and got this 
same error message.

Running all tests in module gdata_tests.client_test
............F..
======================================================================
FAIL: test_simple_request (gdata_tests.client_test.RequestTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "C:\1dnlds\gdata-python-client\gdata-2.0.10\tests\gdata_tests\client_test.py", line 218, in test_simple_request
    self.assertEqual(response.getheader('Echo-Scheme'), 'http')
AssertionError: 'https' != 'http'

It was the only error in the bunch of tests.

Original comment by JoeCodes...@gmail.com on 7 Jul 2010 at 7:14

GoogleCodeExporter commented 9 years ago 
Yes, this is fixed in trunk, but a new release that incorporates the fix hasn't 
been done yet.

Original comment by joe.gregorio@gmail.com on 9 Jul 2010 at 3:39