mmabey / fhirdatetime

57%

main: 63%

LAST BUILD ON BRANCH 0.1.0b5

branch: 0.1.0b5

CHANGE BRANCH

x

Reset

• 0.1.0b5
• 0.1.0-alpha.1
• 0.1.0-alpha.2
• 0.1.0b1
• 0.1.0b2
• 0.1.0b3
• 0.1.0b4
• 0.1.0b6
• 0.1.0b7
• 0.1.0b8
• deleteme
• dev
• main

Build # 46

Build Type

push

travis-ci-com

Committed by mmabey

Commit Message

Too many problems with private properties

When I attempted to copy over portions of the datetime.datetime class
piecemeal, I ultimately had the problem that there were too many methods
that accessed the private attributes of the original datetime class. For
reasons touched on in the commit message for d7036c1, those private
attributes are critical and not accessible to inheriting types.

So, instead of having FhirDateTime inherit directly from datetime, I'm
now using multiple inheritance (hey, it was the best solution that still
allowed for `isinstance(fhir_date_time, datetime.datetime)` to still
return `True`), where a customized version of datetime is the primary
parent class. The only customizations done are to include all methods
from the std lib's datetime and date classes and put them in one place,
plus added an `__init__()` function that blindly stores the private
attributes' values (meaning without any validation). Plus some
formatting from black, haha.

Run Details

475 of 838 relevant lines covered (56.68%)

1.7 hits per line