Fix __geo_interface__

[gadomski]

Related Issue(s):

• Closes Item __geo_interface__ returns the wrong represenation #1048

Description:

__geo_interface__ should be the feature, not just the geometry. I misread the protocol on my first implementation. I made the choice to just call to_dict with all the dials set to no to make it as quick and light as possible. An alternative implementation would be to build a new dict with only the __geo_interface__ required fields (type, bbox, properties, geometry, coordinates).

PR Checklist:

• pre-commit hooks pass locally
• Tests pass (run scripts/test)
• Documentation has been updated to reflect changes, if applicable
• This PR maintains or improves overall codebase code coverage.
• Changes are added to the CHANGELOG. See the docs for information about adding to the changelog.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.34%. Comparing base (0f9ff83) to head (1cd25e7).
Report is 258 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1049   +/-   ##
=======================================
  Coverage   90.34%   90.34%           
=======================================
  Files          47       47           
  Lines        6213     6213           
  Branches      929      929           
=======================================
  Hits         5613     5613           
  Misses        422      422           
  Partials      178      178           

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

[philvarner]

I think returning every field (instead of just the required GeoJSON Feature fields) is the correct behavior.