chore(tests): move to compact, human readable, commentable toml layout for test records

[blaggacao]

cc @ruthra-kumar — inspired by your feedback about the inconvenience to work with json records

[ruthra-kumar]

How are the toml files read? Is there any upstream change in Frappe?

[blaggacao]

How are the toml files read? Is there any upstream change in Frappe?

Yes, indeed: https://github.com/frappe/frappe/blob/c67c1b05cb401b702ff430dbbe18d57cd6ea2a33/frappe/tests/utils/generators.py#L121-L143

def load_test_records_for(index_doctype) -> dict[str, Any] | list:
	module_path = get_module_path(get_doctype_module(index_doctype), "doctype", frappe.scrub(index_doctype))

	json_path = os.path.join(module_path, "test_records.json")
	if os.path.exists(json_path):
		if not frappe.flags.deprecation_dumpster_invoked:
			from frappe.deprecation_dumpster import deprecation_warning

			deprecation_warning(
				"2024-10-09",
				"v18",
				"Use TOML files for test records; migration script: https://github.com/frappe/frappe/pull/28065",
			)
		with open(json_path) as f:
			return json.load(f)

	toml_path = os.path.join(module_path, "test_records.toml")
	if os.path.exists(toml_path):
		with open(toml_path, "rb") as f:
			return tomli.load(f)

	else:
		return {}

It supports both, still.