Handle user attributes of type list about django-mama-cas HOT 7 CLOSED

I have created a pull request
#75

Please let me know if this looks ok.

from django-mama-cas.

@tarunpaul thanks! I've taken a quick look and the PR is missing a test. I'd say that the for loop you created overwrites the attribute.text with the last value of the list. A test would prove me wrong ;)

Could you please add the test?

from django-mama-cas.

@manelclos I have added a test case. Please take a look.
And thank you for keeping up the code standards. Really appreciate it. :)

from django-mama-cas.

Hi @manelclos, Please let me know if this looks ok or if I have missed anything.
Few test environments seem to fail on travis-ci. Two of which are unable to connect to internet to download the specific python version.
But locally it seems to pass all test cases.

$ python3 runtest.py
Creating test database for alias 'default'...
Ran 157 tests in 1.767s
OK
Destroying test database for alias 'default'...

from django-mama-cas.

@tarunpaul just checked the code locally, all good! Let's see if @jbittel has some time to approve it.

from django-mama-cas.

Great!
Thank you @manelclos :)

from django-mama-cas.

Merged. Much appreciated!

from django-mama-cas.