Skip to content

Commit

Permalink
minor fixes in changelog and style tweaking in tests
Browse files Browse the repository at this point in the history
(I'm fighting agains the automated style, it does not always give the prettiest results)
  • Loading branch information
tobixen committed Dec 27, 2023
1 parent d9dee78 commit cd860b3
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 7 deletions.
7 changes: 2 additions & 5 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -15,14 +15,11 @@ This project should more or less adhere to [Semantic Versioning](https://semver.
* Georges Toth (@sim0nx) did a lot of efforts lifting up the project to more modern standards.
* A hook for collecting debug information has been in the pull request queue for ages. I've decided to include it in 1.4.0.

### Fixes
### Fixed

* Code formatting / style fixes.
* Search method did some logic handling non-conformant servers (loading data from the server if the search response didn't include the icalendar data, ignoring trash from the Google server when it returns data without a VTODO/VEVENT/VJOURNAL component.
* Tests - a breakage was introduced for servers not supporting MKCALENDAR - details in https://github.com/python-caldav/caldav/pull/368

### Fixed

* Revisited a problem that Google sometimes delivers junk when doing searches - credits to github user @zhwei in https://github.com/python-caldav/caldav/pull/366
* There were some compatibility-logic loading objects if the server does not deliver icalendar data (as it's suppsoed to do according to the RFC), but only if passing the `expand`-flag to the `search`-method. Fixed that it loads regardless of weather `expand` is set or not. Also in https://github.com/python-caldav/caldav/pull/366

Expand Down Expand Up @@ -51,7 +48,7 @@ Thinking it through three times, I'm not too concerned - if someone has access t

Some bugfixes.

### Fixes
### Fixed

* Some parts of the library would throw OverflowError on very weird dates/timestamps. Now those are converted to the minimum or maximum accepted date/timestamp. Credits to github user @tamarinvs19 in https://github.com/python-caldav/caldav/pull/327
* `DAVResponse.davclient` was always set to None, now it may be set to the `DAVClient` instance. Credits to github user @sobolevn in https://github.com/python-caldav/caldav/pull/323
Expand Down
10 changes: 8 additions & 2 deletions tests/test_caldav.py
Original file line number Diff line number Diff line change
Expand Up @@ -1359,9 +1359,15 @@ def testSearchSortTodo(self):
dtstart=datetime(2022, 12, 11, 10, 9, 8),
uid="tsst3",
)
t4 = c.save_todo(summary="4 task priority low", priority=9, uid="tsst4")
t4 = c.save_todo(
summary="4 task priority is set to nine which is the lowest",
priority=9,
uid="tsst4",
)
t5 = c.save_todo(
summary="5 task status completed", status="COMPLETED", uid="tsst5"
summary="5 task status is set to COMPLETED and this will disappear from the ordinary todo search",
status="COMPLETED",
uid="tsst5",
)
t6 = c.save_todo(
summary="6 task has categories",
Expand Down

0 comments on commit cd860b3

Please sign in to comment.