From 31a3db0c60b38a6504faa7984a3229ff8cad5c88 Mon Sep 17 00:00:00 2001
From: neatguycoding <15627489+NeatGuyCoding@users.noreply.github.com>
Date: Thu, 17 Jul 2025 12:16:40 +0800
Subject: [PATCH] minor code fix: remove duplicate hash calculation and fix
 typo

---
 api/core/workflow/entities/variable_pool.py         | 1 -
 api/tests/unit_tests/core/variables/test_segment.py | 2 +-
 2 files changed, 1 insertion(+), 2 deletions(-)

diff --git a/api/core/workflow/entities/variable_pool.py b/api/core/workflow/entities/variable_pool.py
index 646a9d3402..fbb8df6b01 100644
--- a/api/core/workflow/entities/variable_pool.py
+++ b/api/core/workflow/entities/variable_pool.py
@@ -152,7 +152,6 @@ class VariablePool(BaseModel):
             self.variable_dictionary[selector[0]] = {}
             return
         key, hash_key = self._selector_to_keys(selector)
-        hash_key = hash(tuple(selector[1:]))
         self.variable_dictionary[key].pop(hash_key, None)
 
     def convert_template(self, template: str, /):
diff --git a/api/tests/unit_tests/core/variables/test_segment.py b/api/tests/unit_tests/core/variables/test_segment.py
index cdc261fd42..4c8d983d20 100644
--- a/api/tests/unit_tests/core/variables/test_segment.py
+++ b/api/tests/unit_tests/core/variables/test_segment.py
@@ -376,7 +376,7 @@ class TestSegmentDumpAndLoad:
                 f"get_segment_discriminator failed for serialized form of type {type(variable)}"
             )
 
-    def test_invlaid_value_for_discriminator(self):
+    def test_invalid_value_for_discriminator(self):
         # Test invalid cases
         assert get_segment_discriminator({"value_type": "invalid"}) is None
         assert get_segment_discriminator({}) is None