add handle field to shipping lines

[serezha93]

this field described here https://shopify.dev/docs/api/admin-rest/2023-07/resources/draftorder

[sx328]

this field described here https://shopify.dev/docs/api/admin-rest/2023-07/resources/draftorder

lgtm!

[oliver006]

🥳

[oliver006]

Change looks ok but the tests are unhappy - can you have a look plz?

[sx328]

=== RUN   TestShippingLines_UnmarshallJSON
    order_test.go:1165: ShippingLines.Handle should be (test), was (third_party_handle)
--- FAIL: TestShippingLines_UnmarshallJSON (0.00s)
=== RUN   TestShippingLines_UnmarshallJSON_RequestFulfillmentServiceIDNumber
    order_test.go:1165: ShippingLines.Handle should be (test), was ()
--- FAIL: TestShippingLines_UnmarshallJSON_RequestFulfillmentServiceIDNumber (0.00s)
=== RUN   TestShippingLines_UnmarshallJSON_RequestFulfillmentServiceIDNull
    order_test.go:1165: ShippingLines.Handle should be (test), was ()
--- FAIL: TestShippingLines_UnmarshallJSON_RequestFulfillmentServiceIDNull (0.00s)

these specific tests seem to be failing. should be an easy fix.

[serezha93]

yep, I fixed tests

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (030224d) 100.00% compared to head (c83ae20) 100.00%.

Additional details and impacted files

@@            Coverage Diff            @@
##            master      #249   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           49        49           
  Lines         2133      2133           
=========================================
  Hits          2133      2133           

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

[oliver006]

Thanks for the PR!