coverage: 100.0%. remained the same
when pulling bb8d6b6 on roharvey:issue-1852-related-primary-key-import
into 5f66a65 on django-import-export:main.

Thanks so much for raising this and crafting a fix, much appreciated 👍

I have refactored to use classes from the example application, I hope that's ok with you.

Feel free to add your name to AUTHORS if you wish.

Thanks! Yes much better test, I figured I was barking up the wrong tree. I added myself to AUTHORS and appreciate your responsiveness!

One little thing... it looks like a test for #1846 (test_declared_field_export_order()) made its way into this PR. Not that it bothers me, but I wondered if that was intentional?

I will merge soon once tests complete. Thanks for all your input on this PR.

it looks like a test for #1846 (test_declared_field_export_order()) made its way into this PR.

I think that is because I merged the new test in a separate PR. If you pull and merge from upstream it should fix it. The test isn't showing in this PR's changes so it should be ok.

@matthewhegarty I think there is typo in the code. The commit description states [test for Resource export order with declared fields (1868), but the comment in the commit points to #1846.