fix-1909: handle default_pool updates with slots/include_deferred mask

[bmanan7]

Description

• Update Airflow client to respect default_pool PATCH rules: send only slots and include_deferred with an update_mask.
• Added a regression test so future changes keep the special-case behavior intact.

🎟 Issue(s)

Related #1909

🧪 Functional Testing

Tried copying pool with different slot values and number of pools and each request was successful. Attached screenshots from UI.

./astro deployment pool copy --source-id cme0XXXXXX --target-id cme0XXXXXX

Using an Astro API Token
Copying Pool default_pool
Copying Pool test-pool
Copying Pool second_pool
==============
./astro deployment pool copy --source-id cme0XXXXXX --target-id cme0XXXXXX

Using an Astro API Token
Copying Pool default_pool
Copying Pool test-pool
Copying Pool second_pool
Copying Pool Fourth-pool

go test -count=1 ./airflow-client
ok  	github.com/astronomer/astro-cli/airflow-client	0.667s

📸 Screenshots

Confirmed from UI that the default pool slot value is being updated and reflected without any errors.

📋 Checklist

• Rebased from the main (or release if patching) branch (before testing)
• Ran make test before taking out of draft
• Ran make lint before taking out of draft
• Added/updated applicable tests
• Tested against Astro-API (if necessary).
• Tested against Houston-API and [Astronomer] (https://github.com/astronomer/astronomer/) (if necessary). - NA
• Communicated to/tagged owners of respective clients potentially impacted by these changes.
• Updated any related documentation - NA

[neel-astro]

Correct me if I am wrong, but based on the airflow code, I think you just need to pass the update_mask in the path param for the default pool. So no need to exclusively send these two fields, airflow will take care of selecting these two fields from the request body based on the update_mask path param

[bmanan7]

Thanks for the suggestion. You’re right. Airflow handler only enforces that update_mask is supplied and constrained, it doesn’t object to extra fields in the body. I have updated code to keep it straight forward i.e just appending update_mask=slots&update_mask=include_deferred when the pool name is default_pool.

[coveralls-official]

Pull Request Test Coverage Report for Build ac408698-168f-4800-b0a2-f20ccdda36df

Details

• 6 of 6 (100.0%) changed or added relevant lines in 1 file are covered.
• 2 unchanged lines in 1 file lost coverage.
• Overall coverage increased (+0.01%) to 38.161%

---

Files with Coverage Reduction	New Missed Lines	%
docker/docker.go	2	46.81%

Totals
Change from base Build 04c39e19-4d56-4706-a6ef-480413713603:	0.01%
Covered Lines:	23779
Relevant Lines:	62312

---

💛 - Coveralls