synchronization fix

[mjuskiewicz66]

📜 Checklist

• Commits are cleanly separated with meaningful messages (simple features and bug fixes should be squashed to one commit)
• A changelog entry or entries has been added for any significant changes
• Follows the Pulp policy on AI Usage
• (For new features) - User documentation and test coverage has been added

See: Pull Request Walkthrough

Additional information: This PR attempts to fix #331

[jobselko]

@mjuskiewicz66 Could you please add a changelog entry as described in the checklist in the PR description, and include some tests based on what was mentioned in the issue? Thanks!

[mjuskiewicz66]

Changelog entry added.

[mjuskiewicz66]

Regarding tests, I don’t feel familiar enough with the entire project to write them. Testing this specific function requires extensive setup and mocking, and I’m unsure how to approach that.

[jobselko]

Regarding tests, I don’t feel familiar enough with the entire project to write them. Testing this specific function requires extensive setup and mocking, and I’m unsure how to approach that.

Thanks for the update! Would it be okay if I add tests for this in a separate PR sometime next week? I can include you as a co-author so your work is acknowledged. Let me know if that works for you.

[mjuskiewicz66]

Yeah, that works for me. Thanks!

[jobselko]

Hi @mjuskiewicz66, I opened #386. Could you please check if my commit where I added you as co-author has the correct name and email? Thanks!

[mjuskiewicz66]

Hey @jobselko, there is a small typo in my username, it's mjuskiewicz66. And the email is michal.juskiewicz@blik.com.