Consistently handle due time for III day loans to avoid duedate earlier outdate error

[branchedelac]

[Moved from https://github.com/FOLIO-FSE/service_tools/issues/15)

Sierra and Millennium don't export due times.

item_barcode	out_date	due_date	renewal_count	status	patron_barcode	next_item_status
789	2019-02-22 10:53:00	2019-02-22	0	n	1234

Parsing the dates as datetime typically translates that to 00:00:00, which for 1-day-loans results in a duedate earlier than outdate error.

item_barcode	out_date	due_date	renewal_count	status	patron_barcode	next_item_status
789	2019-02-22 10:53:00	2019-02-22 00:00:00	0	n	1234

Should we be setting all due times to 23:59:59PM (for III and other libraries that don't provide one) to avoid this?

[fontanka16]

@branchedelac could you check fcd8fad#diff-6276fd63df8b1563f60b13aa05727af12188396505613c9a8a953311dcd34728 to see if the test covers your use case?

[fontanka16]

Actually it does not. I am adding a new test case.

[fontanka16]

Now this case is covered as well