test: fix test case to modified behaviour

We throw instead of showing warning now

frappe/core/doctype/docshare/test_docshare.py

@@ -151,7 +151,7 @@ def test_share_disabled_add_with_ignore_permissions(self):
 		self.assertFalse(self.event.has_permission("share"))
 
 		# Test if behaviour is consistent for developer overrides
-		frappe.share.add_docshare(
+		frappe.share.add(
 			"Event", self.event.name, "test1@example.com", flags={"ignore_share_permission": True}
 		)
 
@@ -177,18 +177,13 @@ def test_share_disabled_assign_to(self):
 		Assigning a document to a user without access must not share the document,
 		if sharing disabled.
 		"""
-		from frappe.desk.form.assign_to import add, get
+		from frappe.desk.form.assign_to import add
 
 		frappe.share.add("Event", self.event.name, self.user, share=1)
 		frappe.set_user(self.user)
 
-		# Assign to 'test1@example.com'
-		add({"doctype": "Event", "name": self.event.name, "assign_to": ["test1@example.com"]})
-
-		# Check if assigned to 'test1@example.com'
-		assignments = get(dict(doctype="Event", name=self.event.name))
-		self.assertEqual(len(assignments), 1)
-
-		# Check if not shared with 'test1@example.com'
-		shared_users = [x.user for x in frappe.share.get_users("Event", self.event.name)]
-		self.assertNotIn("test1@example.com", shared_users)
+		self.assertRaises(
+			frappe.ValidationError,
+			add,
+			{"doctype": "Event", "name": self.event.name, "assign_to": ["test1@example.com"]},
+		)

frappe/tests/utils.py

@@ -44,7 +44,7 @@ def test_case(self):
 		# change setting
 		for key, value in settings_dict.items():
 			setattr(settings, key, value)
-		settings.save()
+		settings.save(ignore_permissions=True)
 		# singles are cached by default, clear to avoid flake
 		frappe.db.value_cache[settings] = {}
 		yield  # yield control to calling function
@@ -54,7 +54,7 @@ def test_case(self):
 		settings = frappe.get_doc(doctype)
 		for key, value in previous_settings.items():
 			setattr(settings, key, value)
-		settings.save()
+		settings.save(ignore_permissions=True)
 		frappe.db.value_cache[settings] = {}