fix resetting lifecycle rules in update-bucket

CHANGELOG.md

@@ -6,6 +6,9 @@
 
 ## [Unreleased]
 
+### Fixed
+* Fix lifecycle rules being cleared after using `update-bucket` command if not explictly set again.
+
 ## [3.10.0] - 2023-09-10
 
 ### Added

b2/console_tool.py

@@ -549,7 +549,7 @@ def _setup_parser(cls, parser):
         lifecycle_group.add_argument(
             '--lifecycleRule',
             action='append',
-            default=[],
+            default=None,
             type=functools.partial(validated_loads, expected_type=LifecycleRule),
             dest='lifecycleRules',
             help="Lifecycle rule in JSON format. Can be supplied multiple times.",

test/unit/test_console_tool.py

@@ -523,6 +523,42 @@ def test_create_bucket_key_and_authorize_with_it(self):
             0,
         )
 
+    def test_update_bucket_without_lifecycle(self):
+        # Start with authorizing with the master key
+        self._authorize_account()
+
+        bucket_name = 'my-bucket-liferules'
+        # Create a bucket with lifecycleRule
+        self._run_command([
+            'create-bucket',
+            '--lifecycleRule',
+            '{"daysFromHidingToDeleting": 2, "fileNamePrefix": "foo"}',
+            bucket_name,
+            'allPrivate'],
+            'bucket_0\n', '', 0)
+
+        expected_stdout_dict = {
+            "accountId": self.account_id,
+            "bucketId": "bucket_0",
+            "bucketInfo": {
+                "xxx": "123"
+            },
+            "bucketName": "my-bucket-liferules",
+            "bucketType": "allPrivate",
+            "lifecycleRules": [
+                {
+                    "daysFromHidingToDeleting": 2,
+                    "fileNamePrefix": "foo"
+                }
+            ],
+        }
+
+        # Update some other attribute than lifecycleRule, which should remain intact
+        self._run_command(
+            ['update-bucket', bucket_name, '--bucketInfo', '{"xxx": "123"}'],
+            expected_json_in_stdout=expected_stdout_dict,
+        )
+
     def test_clear_account(self):
         # Initial condition
         self._authorize_account()