django-mercadopago icon indicating copy to clipboard operation
django-mercadopago copied to clipboard

Testing multiple items per preference (and typo errors fixed)

Open pwqw opened this issue 6 years ago • 5 comments

What do you think about it?

I cant run the test because I believe that the Account app_id and secret_key are not valid.

======================================================================
ERROR: test_existing_notification (tests.test_views.CreateNotificationTestCase)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/user/django-mercadopago/tests/test_views.py", line 17, in setUp
    self.preference.submit()
  File "/home/user/django-mercadopago/django_mercadopago/models.py", line 201, in submit
    pref_result = mercadopago_service.create_preference(request)
  File "/home/user/django-mercadopago/.tox/py36/lib/python3.6/site-packages/mercadopago/mercadopago.py", line 181, in create_preference
    access_token = self.get_access_token()
  File "/home/user/django-mercadopago/.tox/py36/lib/python3.6/site-packages/mercadopago/mercadopago.py", line 79, in get_access_token
    raise MPInvalidCredentials(str(access_data))
mercadopago.mercadopago.MPInvalidCredentials: {'status': 400, 'response': {'message': 'Invalid client_id', 'error': 'bad_request', 'status': 400, 'cause': {'code': 11, 'description': 'Invalid client_id', 'data': None}}}

pwqw avatar Mar 25 '19 16:03 pwqw

The file could be simply named test/urls.py instead of test/test_urls.py really?

pwqw avatar Mar 25 '19 19:03 pwqw

Yup, test/urls.py is better.

WhyNotHugo avatar Mar 25 '19 20:03 WhyNotHugo

I miss a little detail in urls.py

pwqw avatar Mar 25 '19 20:03 pwqw

Hi @WhyNotHugo! Is this PR still considered to be accepted? I'm about using this app in a site and I can contribute code to make this work. Gracias y saludos desde Uruguay!

tadeo avatar Nov 11 '19 16:11 tadeo

Wow, it’s been a year, my deepest apologies!

I’ve been working on porting most of this library into django-payments, which has a better design in general, plus, it integrates multiple other payment methods though.

I doubt I’ll work on new features here myself. However, I won’t reject an updated version of this PR if the comments here are addressed.

WhyNotHugo avatar Dec 04 '20 21:12 WhyNotHugo