fixup! fix: fixes based on review feedback, tests etc

Author: xitij2000

openedx/core/djangoapps/agreements/data.py

@@ -28,7 +28,7 @@ class LTIPIISignatureData:
     lti_tools_hash: str
 
 
-@dataclass
+@attr.s(frozen=True, auto_attribs=True)
 class UserAgreementData:
     """
     Data for a user agreement record.
@@ -50,7 +50,7 @@ def from_model(cls, model: UserAgreement):
         )
 
 
-@dataclass
+@attr.s(frozen=True, auto_attribs=True)
 class UserAgreementRecordData:
     """
     Data for a single user agreement record.