update default values for environment variables

Author: bachtran02

src/email.py

@@ -97,13 +97,14 @@ def fmt_date(dt: datetime):
 
         return cls(
             to_email=student.get_email(),
-            from_email=Environment.get(ENV_EMAIL_FROM),
+            from_email=Environment.get_email_from(),
             cc_emails=cc_emails,
-            reply_to_email=Environment.get(ENV_EMAIL_REPLY_TO),
-            subject=Environment.get(ENV_EMAIL_SUBJECT),
+            reply_to_email=Environment.get_reply_to_email(),
+            subject=Environment.get_email_subject(),
             body=body,
         )
 
+    """
     def OLDsend(self) -> None:
         # TODO: When 162 adds HTML support, bring back HTML emails.
         # html_body = Markdown().convert(self.body)
@@ -130,7 +131,7 @@ def OLDsend(self) -> None:
 
         except Exception as e:
             raise EmailError("An error occurred while sending an email:", e)
-
+    """
 
     def send(self) -> None:
         PORT = 465  # For starttls
@@ -140,7 +141,7 @@ def send(self) -> None:
         smtp_password = os.environ['SMTP_PASSWORD']
         sender_email = os.environ['SENDER_EMAIL']
 
-        SENDERNAME = Environment.get(ENV_EMAIL_FROM)
+        SENDERNAME = Environment.get_email_from()
         receiver_email = self.to_email
         cc_emails = self.cc_emails
         reply_to_email = self.reply_to_email

src/gradescope.py

@@ -23,7 +23,7 @@ def __init__(self) -> None:
 
     @staticmethod
     def is_enabled():
-        return cast_bool(Environment.safe_get("EXTEND_GRADESCOPE_ASSIGNMENTS", "No"))
+        return Environment.get_extend_gradescope_assignments()
 
     def apply_extension(self, assignment_name: str, assignment_urls: List[str], email: str, num_days: int) -> List[str]:
         warnings = []

src/utils.py

@@ -53,6 +53,18 @@ def cast_list_int(cell: str) -> List[int]:
 
 PREFIX = "cs161extensions_"
 
+DEFAULT_COURSE_NAME = "PLEASE SET A COURSE NAME"
+DEFAULT_REPLY_TO_EMAIL = "PLEASE SET REPLY-TO EMAIL"
+
+DEFAULT_AUTO_APPROVE_THRESHOLD = 1
+DEFAULT_AUTO_APPROVE_THRESHOLD_DSP = 1
+DEFAULT_APPROVE_ASSIGNMENT_THRESDHOLD = 1
+DEFAULT_MAX_TOTAL_REQUESTED_EXTENSIONS_THRESHOLD = 3
+
+DEFAULT_EMAIL_FROM = "[{}] <{}@berkeley.edu>".format(DEFAULT_COURSE_NAME, DEFAULT_COURSE_NAME)
+DEFAULT_EMAIL_SUBJECT = "[CS 000] Extension Request Update"
+
+DEFAULT_EXTEND_GRADESCOPE_ASSIGNMENTS = "No"
 
 class Environment:
     @staticmethod
@@ -82,26 +94,46 @@ def get(key: str) -> Any:
 
     @staticmethod
     def get_auto_approve_threshold() -> int:
-        return int(Environment.get("AUTO_APPROVE_THRESHOLD"))
+        return int(Environment.safe_get(
+            "AUTO_APPROVE_THRESHOLD", DEFAULT_AUTO_APPROVE_THRESHOLD))
 
     @staticmethod
     def get_auto_approve_threshold_dsp() -> int:
-        return int(Environment.get("AUTO_APPROVE_THRESHOLD_DSP"))
+        return int(Environment.safe_get(
+            "AUTO_APPROVE_THRESHOLD_DSP", DEFAULT_AUTO_APPROVE_THRESHOLD_DSP))
 
     @staticmethod
     def get_max_total_requested_extensions_threshold() -> int:
         # If this number is -1, then assume this flag is disabled.
         # If this number is 0, then reject all extensions.
         # If this number is > 0, then reject extensions if the total number of extensions requested exceeds this number.
-        return int(Environment.safe_get("MAX_TOTAL_REQUESTED_EXTENSIONS_THRESHOLD", default=-1))
+        return int(Environment.safe_get(
+            "MAX_TOTAL_REQUESTED_EXTENSIONS_THRESHOLD", DEFAULT_MAX_TOTAL_REQUESTED_EXTENSIONS_THRESHOLD))
 
     @staticmethod
     def get_auto_approve_assignment_threshold() -> int:
-        return int(Environment.get("AUTO_APPROVE_ASSIGNMENT_THRESHOLD"))
+        return int(Environment.safe_get(
+            "AUTO_APPROVE_ASSIGNMENT_THRESHOLD", DEFAULT_APPROVE_ASSIGNMENT_THRESDHOLD))
 
     @staticmethod
     def get_course_name() -> str:
-        return str(Environment.safe_get("COURSE_NAME", ""))
+        return str(Environment.safe_get("COURSE_NAME", DEFAULT_COURSE_NAME))
+    
+    @staticmethod
+    def get_reply_to_email() -> str:
+        return str(Environment.safe_get("REPLY_TO_EMAIL", DEFAULT_REPLY_TO_EMAIL))
+    
+    @staticmethod
+    def get_email_from() -> str:
+        return str(Environment.safe_get("EMAIL_FROM", DEFAULT_EMAIL_FROM))
+    
+    @staticmethod
+    def get_email_subject() -> str:
+        return str(Environment.safe_get("EMAIL_SUBJECT", DEFAULT_EMAIL_SUBJECT))
+    
+    @staticmethod
+    def get_extend_gradescope_assignments() -> bool:
+        return cast_bool(Environment.safe_get("EXTEND_GRADESCOPE_ASSIGNMENTS", DEFAULT_EXTEND_GRADESCOPE_ASSIGNMENTS))
 
     @staticmethod
     def configure_env_vars(sheet: Sheet):