Add missing properties to packets

[Stoops-ML]

• Adds epoch to PositionList
• Adds outlineColor and outline to Polygon

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.25%. Comparing base (21653db) to head (f80d168).
Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #147   +/-   ##
=======================================
  Coverage   99.25%   99.25%           
=======================================
  Files          12       12           
  Lines         805      808    +3     
=======================================
+ Hits          799      802    +3     
  Misses          6        6           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Stoops-ML]

I didn't add a test for these new properties, yet the Codecov report is says that the added lines are being hit.

It's because all classes initialise all properties (usually to None). For example, from test_polygon_with_hole() the created polygon is:

p = Polygon(
    positions=PositionList(cartographicDegrees=[30.0, 40.0, 1.0]),
    holes=PositionListOfLists(
        cartographicDegrees=[[20.0, 20.0, 0.0], [10.0, 10.0, 0.0]]
    ),
)

Which creates an object with the following properties:

p = Polygon(
    positions=PositionList(
        interpolationAlgorithm=None,
        interpolationDegree=None,
        delete=None,
        referenceFrame=None,
        cartesian=None,
        cartographicRadians=None,
        cartographicDegrees=[30.0, 40.0, 1.0],
        references=None,
        interval=None,
        epoch=None,
    ),
    show=None,
    arcType=None,
    granularity=None,
    material=None,
    shadows=None,
    distanceDisplayCondition=None,
    classificationType=None,
    zIndex=None,
    holes=PositionListOfLists(
        delete=None,
        referenceFrame=None,
        cartesian=None,
        cartographicRadians=None,
        cartographicDegrees=[[20.0, 20.0, 0.0], [10.0, 10.0, 0.0]],
        references=None,
    ),
    outlineColor=None,
    outline=None,
)

All the properties with None values are removed when calling __str__.

It's somewhat deceiving to say that these lines are actually covered despite having no actual checks for them.

[astrojuanlu]

Makes sense, thanks for the investigation!