S3: delete_objects(): BypassGovernanceMode should handle both True and true

[bblommers]

Looks like the casing changed (True -> true in a recent botocore update (potentially this one), and Moto didn't understand the provided parameter anymore. This should fix the tests on master.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.80%. Comparing base (448c745) to head (9230a00).
Report is 7 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #8752   +/-   ##
=======================================
  Coverage   92.80%   92.80%           
=======================================
  Files        1260     1260           
  Lines      109606   109606           
=======================================
  Hits       101723   101723           
  Misses       7883     7883           

Flag	Coverage Δ
servertests	28.21% <ø> (ø)
unittests	92.78% <ø> (ø)

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

☔ 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.

[bpandola]

Just out of curiosity, I went digging for the change. It's here.

[bblommers]

Just out of curiosity, I went digging for the change. It's here.

They changed the entire serialization format so that all querystring booleans are now lowercased.. and our implementation only failed once? I'm impressed 🙂