feat: Add AlterField Operation to Migration Generator

[kumarUjjawal]

Add AlterField Operation to Migration Generator

Overview

This PR introduces support for the AlterField operation in the migration generator.

Related Issues

#205

Reviewer Notes

• I don't have full confidence in my implementation if you think this is not what was the intention of the Support "alter field" in automatic migration generator #205 feel free to close this for now.

[codecov]

Codecov Report

❌ Patch coverage is 71.79487% with 66 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
cot/src/db/migrations.rs	0.00%	42 Missing ⚠️
cot-cli/src/migration_generator.rs	87.50%	24 Missing ⚠️

Flag	Coverage Δ
rust	89.58% <71.79%> (-0.17%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
cot-cli/src/migration_generator.rs	89.06% <87.50%> (+0.77%)	⬆️
cot/src/db/migrations.rs	78.66% <0.00%> (-5.22%)	⬇️

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[kumarUjjawal]

@seqre any feedback for me?

[seqre]

This operation has many edge cases, and it should be thoroughly tested. Can you please add more tests first?

Think about possible cases, such as: name change (this one can be skipped for now, it's tricky), type change, attributes change, etc. All those should be either handled or visibly err when a given case is unhandled or ambiguous.

It seems that cot-cli/src/migration_generator.rs does not have a test for alter model, would you be so kind and add that too?

[kumarUjjawal]

Sure, Thank you!

[kumarUjjawal]

@seqre I added more tests can you check if they are what you are expecting and any feedbacks next steps. Thanks!

[kumarUjjawal]

@m4tx the error doesn't seem to come from my PR, can you take a look?