added duplicate clause in move.py

[arnchlmcodes]

Added an ON DUPLICATE KEY UPDATE clause to move.py which updates m_new_namespace and m_new_article

as discussed in #101

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.78%. Comparing base (df62769) to head (cdfd76d).
⚠️ Report is 11 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1098   +/-   ##
=======================================
  Coverage   92.78%   92.78%           
=======================================
  Files          74       74           
  Lines        4311     4311           
=======================================
  Hits         4000     4000           
  Misses        311      311           

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[audiodude]

This looks good, and existing tests pass. Honestly though, this should come with a "failing test", aka a test that demonstrates the issue and is failing at HEAD, but works in the PR. Up to you.

[arnchlmcodes]

Hi @audiodude, added a failing test as requested. The test inserts an initial move record, verifies it was inserted correctly, then inserts a second move with the same primary key but different destination fields. It then checks whether the record was updated with the new values from move2 rather than raising an IntegrityError. Let me know if any changes are required :)

[audiodude]

THanks for the test, definitely the right idea!

Unfortunately you have accidentally made more work for yourself, but the end result wil be much stronger.

[audiodude]

Failing Black format, please fix.

[arnchlmcodes]

My bad, added black formatting now