Skip to content

Commit 23febe2

Browse files
tentatively add user input mock
1 parent 2b6fd28 commit 23febe2

File tree

1 file changed

+18
-13
lines changed

1 file changed

+18
-13
lines changed

test/test_modules/test_cli_clone.py

Lines changed: 18 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -115,7 +115,8 @@ def test_create_metadata_item_rule(self, mock_add_metadata_rule):
115115

116116
@patch.object(clone_metadata_utils, 'create_metadata_items')
117117
@patch.object(clone_metadata_utils, 'list_metadata_items')
118-
def test_compare_create_metadata_items_new_fields(self, mock_list, mock_create):
118+
@patch('cloudinary_cli.utils.utils.confirm_action')
119+
def test_compare_create_metadata_items_new_fields(self, mock_list, mock_create, mock_confirm):
119120
"""Test comparing and creating new metadata fields"""
120121
metadata_fields = {
121122
'metadata_fields': [
@@ -131,7 +132,7 @@ def test_compare_create_metadata_items_new_fields(self, mock_list, mock_create):
131132
}
132133
]
133134
}
134-
135+
mock_confirm.return_value = True
135136
mock_source_fields = metadata_fields
136137
mock_list.return_value = metadata_fields
137138
mock_destination_fields = {
@@ -151,7 +152,8 @@ def test_compare_create_metadata_items_new_fields(self, mock_list, mock_create):
151152

152153
@patch.object(clone_metadata_utils, 'create_metadata_items')
153154
@patch.object(clone_metadata_utils, 'list_metadata_items')
154-
def test_compare_create_metadata_items_new_rules(self, mock_list, mock_create):
155+
@patch('cloudinary_cli.utils.utils.confirm_action')
156+
def test_compare_create_metadata_items_new_rules(self, mock_list, mock_create, mock_confirm):
155157
"""Test comparing and creating new metadata rules"""
156158
metadata_rules = {
157159
'metadata_rules': [
@@ -175,7 +177,7 @@ def test_compare_create_metadata_items_new_rules(self, mock_list, mock_create):
175177
}
176178
]
177179
}
178-
180+
mock_confirm.return_value = True
179181
mock_source_metadata_rules = metadata_rules
180182
mock_list.return_value = metadata_rules
181183

@@ -195,7 +197,8 @@ def test_compare_create_metadata_items_new_rules(self, mock_list, mock_create):
195197
self.assertEqual(result, mock_list.return_value)
196198

197199
@patch.object(clone_metadata_utils, 'create_metadata_items')
198-
def test_compare_create_metadata_items_existing_fields(self, mock_create):
200+
@patch('cloudinary_cli.utils.utils.confirm_action')
201+
def test_compare_create_metadata_items_existing_fields(self, mock_create, mock_confirm):
199202
"""Test comparing when fields already exist"""
200203
mock_source_fields = {
201204
'metadata_fields': [
@@ -217,15 +220,15 @@ def test_compare_create_metadata_items_existing_fields(self, mock_create):
217220
}
218221
]
219222
}
220-
221-
mock_source_fields
223+
mock_confirm.return_value = True
222224
clone_metadata_utils.compare_create_metadata_items(mock_source_fields, mock_destination_fields, key="metadata_fields", **self.mock_target_config)
223225

224226
# No fields should be created
225227
mock_create.assert_not_called()
226228

227229
@patch.object(clone_metadata_utils, 'create_metadata_items')
228-
def test_compare_create_metadata_items_existing_rules(self, mock_create):
230+
@patch('cloudinary_cli.utils.utils.confirm_action')
231+
def test_compare_create_metadata_items_existing_rules(self, mock_create, mock_confirm):
229232
"""Test comparing when rules already exist"""
230233

231234
mock_source_metadata_rules = {
@@ -256,15 +259,16 @@ def test_compare_create_metadata_items_existing_rules(self, mock_create):
256259
}
257260
]
258261
}
259-
262+
mock_confirm.return_value = True
260263
clone_metadata_utils.compare_create_metadata_items(mock_source_metadata_rules, mock_destination_metadata_rules, key="metadata_rules", **self.mock_target_config)
261264

262265
# No rules should be created
263266
mock_create.assert_not_called()
264267

265268
@patch.object(clone_metadata_utils, 'create_metadata_items')
266269
@patch.object(clone_metadata_utils, 'list_metadata_items')
267-
def test_compare_create_metadata_items_mixed_scenario(self, mock_list, mock_create):
270+
@patch('cloudinary_cli.utils.utils.confirm_action')
271+
def test_compare_create_metadata_items_mixed_scenario(self, mock_list, mock_create, mock_confirm):
268272
"""Test comparing with mix of new and existing fields"""
269273
metadata_fields = {
270274
'metadata_fields': [
@@ -291,7 +295,7 @@ def test_compare_create_metadata_items_mixed_scenario(self, mock_list, mock_crea
291295
}
292296
]
293297
}
294-
298+
mock_confirm.return_value = True
295299
mock_source_fields= metadata_fields
296300
mock_list.return_value = metadata_fields
297301

@@ -306,7 +310,8 @@ def test_compare_create_metadata_items_mixed_scenario(self, mock_list, mock_crea
306310

307311
@patch.object(clone_metadata_utils, 'create_metadata_items')
308312
@patch.object(clone_metadata_utils, 'list_metadata_items')
309-
def test_compare_create_metadata_items_mixed_rules_scenario(self, mock_list, mock_create):
313+
@patch('cloudinary_cli.utils.utils.confirm_action')
314+
def test_compare_create_metadata_items_mixed_rules_scenario(self, mock_list, mock_create, mock_confirm):
310315
"""Test comparing with mix of new and existing rules"""
311316
metadata_rules = {
312317
'metadata_rules': [
@@ -345,7 +350,7 @@ def test_compare_create_metadata_items_mixed_rules_scenario(self, mock_list, moc
345350
}
346351
]
347352
}
348-
353+
mock_confirm.return_value = True
349354
mock_source_metadata_rules = metadata_rules
350355
mock_list.return_value = metadata_rules
351356

0 commit comments

Comments
 (0)