🧪 Create Cypress test for Roles of variables

app.py

@@ -610,6 +610,7 @@ def parse():
                 response['has_sleep'] = True
 
             response['variables'] = transpile_result.roles_of_variables
+            response['show_roles'] = show_roles_of_variables()
         except Exception:
             pass
 
@@ -2918,6 +2919,7 @@ def on_offline_mode():
     env_defaults = dict(
         BASE_URL=f"http://localhost:{config['port']}/",
         ADMIN_USER="admin",
+        SHOW_ROLES=False,
     )
     for key, value in env_defaults.items():
         if key not in os.environ:

static/js/appbundle.js

@@ -103004,7 +103004,6 @@ def note_with_error(value, err):
   var theLanguage;
   var TRADUCTION2;
   var variable_view = true;
-  var showRoles = false;
   var step_debugger = false;
   var fullLineCommands = [
     "print",
@@ -103056,6 +103055,7 @@ def note_with_error(value, err):
         if (variables[i][1]) {
           const variableName = variables[i][0].replace(/^_/, "");
           const role = programData == null ? void 0 : programData.variables[variableName];
+          const showRoles = programData == null ? void 0 : programData.show_roles;
           if (showRoles) {
             variableList.append(`<li style=color:${variables[i][2]}>${variableName}: ${variables[i][1]} (${role})</li>`);
           } else {

static/js/debugging.ts

@@ -8,7 +8,7 @@ let TRADUCTION: Map<string,string>;
 
 //Feature flag for variable and values view
 let variable_view = true;
-let showRoles = false;
+//let showRoles = true;
 let step_debugger = false;
 const fullLineCommands = [
   'print',
@@ -79,6 +79,7 @@ export function load_variables(variables: any) {
       if (variables[i][1]) {
         const variableName = variables[i][0].replace(/^_/, '');
         const role = programData?.variables[variableName];
+        const showRoles = programData?.show_roles;
         if (showRoles) {
           variableList.append(`<li style=color:${variables[i][2]}>${variableName}: ${variables[i][1]} (${role})</li>`);
         } else {

utils.py

@@ -98,6 +98,13 @@ def is_debug_mode():
     """
     return DEBUG_MODE
 
+def show_roles_of_variables():
+    """Whether the roles of variables should be shown in the variable view.
+    (If not, only variable names and values will be shown in the list).
+    Checks the configuration in the environment variable $SHOW_ROLES    
+    """
+    can_show_roles = os.getenv("SHOW_ROLES", False)
+    return can_show_roles
 
 def is_offline_mode():
     """Return whether or not we're in offline mode.

website/auth.py

@@ -159,7 +159,6 @@ def forget_current_user():
     session.pop("keyword_lang", None)  # Delete session keyword language if existing
     session.pop("profile_image", None)  # Delete profile image id if existing
 
-
 def is_admin(user):
     """Whether the given user (object) is an admin.
 
@@ -173,7 +172,6 @@ def is_admin(user):
 
     return user.get("username") in admin_users or user.get("email") in admin_users
 
-
 def is_teacher(user, cls=None):
     # the `is_teacher` field is either `0`, `1` or not present.
     return bool(user.get("is_teacher", False))