Revert "fix(usage-record): Add Unique constraints"

press/patches.txt

@@ -110,7 +110,6 @@ press.press.doctype.virtual_machine_volume.patches.rename_aws_fields
 press.patches.v0_7_0.convert_marketplace_description_to_html
 press.press.doctype.team.patches.remove_invalid_email_addresses
 press.saas.doctype.product_trial.patches.rename_saas_product_doctypes_to_product_trial
-press.press.doctype.usage_record.patches.add_unique_constraint
 
 [post_model_sync]
 press.patches.v0_7_0.rename_plan_to_site_plan

press/press/doctype/subscription/test_subscription.py

@@ -1,3 +1,4 @@
+# -*- coding: utf-8 -*-
 # Copyright (c) 2020, Frappe and Contributors
 # See license.txt
 
@@ -92,44 +93,6 @@
 		invoice = frappe.get_doc("Invoice", {"team": self.team.name, "status": "Draft"})
 		self.assertEqual(invoice.total, desired_value)
 
-	def test_subscription_for_duplicate_usage_record(self):
-		todo = frappe.get_doc(doctype="ToDo", description="Test todo").insert()
-		plan = frappe.get_doc(
-			doctype="Site Plan",
-			name="Plan-10",
-			document_type="ToDo",
-			interval="Daily",
-			price_usd=30,
-			price_inr=30,
-		).insert()
-
-		subscription = frappe.get_doc(
-			doctype="Subscription",
-			team=self.team.name,
-			document_type="ToDo",
-			document_name=todo.name,
-			plan_type="Site Plan",
-			plan=plan.name,
-		).insert()
-
-		# create a usage record
-		subscription.create_usage_record()
-
-		# since create_usage_record silently skips if a record already exists
-		# check by manually creating a duplicate record
-		duplicate_usage_record = frappe.new_doc(
-			doctype="Usage Record",
-			team=self.team.name,
-			document_type="ToDo",
-			document_name=todo.name,
-			plan_type="Site Plan",
-			plan=plan.name,
-			subscription=subscription.name,
-			interval="Daily",
-		)
-		with self.assertRaises(frappe.UniqueValidationError):
-			duplicate_usage_record.insert()
-
 	def test_subscription_for_non_chargeable_document(self):
 		todo = frappe.get_doc(doctype="ToDo", description="Test todo").insert()
 		plan = frappe.get_doc(

press/press/doctype/usage_record/usage_record.py

@@ -124,4 +124,4 @@ def link_unlinked_usage_records():
 
 
 def on_doctype_update():
-	frappe.db.add_unique("Usage Record", ["subscription", "date"], constraint_name="unique_usage_record")
+	frappe.db.add_index("Usage Record", ["subscription", "date"])