Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test_recurrence fails #113

Closed
dvzrv opened this issue Apr 19, 2018 · 3 comments · Fixed by #115
Closed

test_recurrence fails #113

dvzrv opened this issue Apr 19, 2018 · 3 comments · Fixed by #115

Comments

@dvzrv
Copy link

dvzrv commented Apr 19, 2018

During packaging for Arch Linux, test_recurrence fails, after successful build (for python2/3). I have shipped the package before, so I'm wondering, why it breaks now:

======================================================================
ERROR: test_recurrence (tests.TestIcalendar)          
----------------------------------------------------------------------
Traceback (most recent call last):           
  File "/build/python-vobject/src/python-vobject-0.9.5/tests.py", line 754, in test_recurrence
    dates = list(cal.vevent.getrruleset())                       
  File "/build/python-vobject/src/python-vobject-0.9.5/vobject/icalendar.py", line 453, in getrruleset
    ignoretz=isinstance(dtstart, datetime.date))
  File "/usr/lib/python3.6/site-packages/dateutil/rrule.py", line 1664, in __call__
    return self._parse_rfc(s, **kwargs)                
  File "/usr/lib/python3.6/site-packages/dateutil/rrule.py", line 1547, in _parse_rfc
    tzinfos=tzinfos)                     
  File "/usr/lib/python3.6/site-packages/dateutil/rrule.py", line 1506, in _parse_rfc_rrule
    return rrule(dtstart=dtstart, cache=cache, **rrkwargs)
  File "/usr/lib/python3.6/site-packages/dateutil/rrule.py", line 461, in __init__
    'RRULE UNTIL values must be specified in UTC when DTSTART '
ValueError: RRULE UNTIL values must be specified in UTC when DTSTART is timezone-aware
                                   
----------------------------------------------------------------------
Ran 39 tests in 0.182s                        

I'm running the tests with python setup.py test. Is there anything I'm missing?

@dvzrv
Copy link
Author

dvzrv commented May 16, 2018

Could you please take your time to actually look into this, instead of closing this bug? The test is still failing.

I'm also not sure, what your dismissive comment is supposed to tell me, but that you probably don't know what's going on in your setup.py?
It clearly states test_suite="tests".
Therefore it doesn't matter, if I call python setup.py test or python tests.py, as it will lead to the same result.

Apart from that, I guess currently having dateutil in version 2.7.3 on Arch Linux causes this behavior (as mentioned in #115), so this is a valid bug.

@wpercy wpercy reopened this May 17, 2018
@wpercy
Copy link
Contributor

wpercy commented May 17, 2018

Yes, this is caused by us not currently supporting dateutil 2.6+. I'll test against the PR you referenced above when I get a chance and put out a new release.

@dvzrv
Copy link
Author

dvzrv commented May 17, 2018

Thanks! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants