fix: add hide_status to the env class and fix tests

Signed-off-by: Zack Koppert <[email protected]>

Author: zkoppert

config.py

@@ -75,7 +75,7 @@ def __init__(
         hide_time_to_close: bool,
         hide_time_to_first_response: bool,
         hide_created_at: bool,
-        hide_status: bool,  # New attribute
+        hide_status: bool,
         ignore_user: List[str],
         labels_to_measure: List[str],
         enable_mentor_count: bool,
@@ -105,7 +105,7 @@ def __init__(
         self.hide_time_to_close = hide_time_to_close
         self.hide_time_to_first_response = hide_time_to_first_response
         self.hide_created_at = hide_created_at
-        self.hide_status = hide_status  # Initialize the new attribute
+        self.hide_status = hide_status
         self.enable_mentor_count = enable_mentor_count
         self.min_mentor_comments = min_mentor_comments
         self.max_comments_eval = max_comments_eval
@@ -134,6 +134,7 @@ def __repr__(self):
             f"{self.hide_time_to_close},"
             f"{self.hide_time_to_first_response},"
             f"{self.hide_created_at},"
+            f"{self.hide_status},"
             f"{self.ignore_users},"
             f"{self.labels_to_measure},"
             f"{self.enable_mentor_count},"
@@ -242,7 +243,7 @@ def get_env_vars(test: bool = False) -> EnvVars:
     hide_time_to_close = get_bool_env_var("HIDE_TIME_TO_CLOSE", False)
     hide_time_to_first_response = get_bool_env_var("HIDE_TIME_TO_FIRST_RESPONSE", False)
     hide_created_at = get_bool_env_var("HIDE_CREATED_AT", True)
-    hide_status = get_bool_env_var("HIDE_STATUS", True)  # New attribute
+    hide_status = get_bool_env_var("HIDE_STATUS", True)
     enable_mentor_count = get_bool_env_var("ENABLE_MENTOR_COUNT", False)
     min_mentor_comments = os.getenv("MIN_MENTOR_COMMENTS", "10")
     max_comments_eval = os.getenv("MAX_COMMENTS_EVAL", "20")

test_config.py

@@ -131,6 +131,7 @@ def test_get_env_vars_with_github_app(self):
             hide_time_to_close=False,
             hide_time_to_first_response=False,
             hide_created_at=True,
+            hide_status=True,
             ignore_user=[],
             labels_to_measure=[],
             enable_mentor_count=False,
@@ -186,6 +187,7 @@ def test_get_env_vars_with_token(self):
             hide_time_to_close=False,
             hide_time_to_first_response=False,
             hide_created_at=True,
+            hide_status=True,
             ignore_user=[],
             labels_to_measure=[],
             enable_mentor_count=False,
@@ -276,6 +278,7 @@ def test_get_env_vars_optional_values(self):
             hide_time_to_close=True,
             hide_time_to_first_response=True,
             hide_created_at=True,
+            hide_status=True,
             ignore_user=[],
             labels_to_measure=["waiting-for-review", "waiting-for-manager"],
             enable_mentor_count=False,
@@ -320,6 +323,7 @@ def test_get_env_vars_optionals_are_defaulted(self):
             hide_time_to_close=False,
             hide_time_to_first_response=False,
             hide_created_at=True,
+            hide_status=True,
             ignore_user=[],
             labels_to_measure=[],
             enable_mentor_count=False,