search

singer-io / tap-harvest

A Singer tap for extracting data from the Harvest API
GNU Affero General Public License v3.0
8 stars 13 forks source link

Tap is sending an extra field in 'time_entries' stream. #39

Closed kspeer825 closed 4 years ago

kspeer825 commented 4 years ago

The stitch docs specify a list of schema for the time_entries stream. The field "rounded_hours" is not in this schema, nor is it in the stitch docs, however it is currently being sent to the target for the time_entries stream. 

FAIL: test_run (test_harvest_pagination.PaginationTest) (stream='time_entries')
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/root/project/tests/test_harvest_pagination.py", line 371, in do_test
    msg="The fields sent to the target have an extra or missing field"
AssertionError: Items in the second set but not the first:
'rounded_hours' : The fields sent to the target have an extra or missing field
kspeer825 commented 4 years ago

This is not a correct analysis. The field is not being sent to the target.