fix: Update test version checks to 2.9.0 and remove whitespace
- Update version checks in test_package_structure.py from 2.8.0 to 2.9.0 - Update version check in test_cli_paths.py from 2.8.0 to 2.9.0 - Remove trailing whitespace from blank lines in code_analyzer.py (lines 1436-1504) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
This commit is contained in:
@@ -1433,7 +1433,7 @@ class CodeAnalyzer:
|
|||||||
def _analyze_godot_scene(self, content: str, file_path: str) -> dict[str, Any]:
|
def _analyze_godot_scene(self, content: str, file_path: str) -> dict[str, Any]:
|
||||||
"""
|
"""
|
||||||
Analyze Godot .tscn scene file.
|
Analyze Godot .tscn scene file.
|
||||||
|
|
||||||
Extracts:
|
Extracts:
|
||||||
- Node hierarchy
|
- Node hierarchy
|
||||||
- Script attachments
|
- Script attachments
|
||||||
@@ -1443,7 +1443,7 @@ class CodeAnalyzer:
|
|||||||
nodes = []
|
nodes = []
|
||||||
resources = []
|
resources = []
|
||||||
scripts = []
|
scripts = []
|
||||||
|
|
||||||
# Extract external resources
|
# Extract external resources
|
||||||
for match in re.finditer(r'\[ext_resource.*?type="(.+?)".*?path="(.+?)".*?id="(.+?)"\]', content):
|
for match in re.finditer(r'\[ext_resource.*?type="(.+?)".*?path="(.+?)".*?id="(.+?)"\]', content):
|
||||||
res_type, path, res_id = match.groups()
|
res_type, path, res_id = match.groups()
|
||||||
@@ -1452,28 +1452,28 @@ class CodeAnalyzer:
|
|||||||
"path": path,
|
"path": path,
|
||||||
"id": res_id
|
"id": res_id
|
||||||
})
|
})
|
||||||
|
|
||||||
# Track scripts separately
|
# Track scripts separately
|
||||||
if res_type == "Script":
|
if res_type == "Script":
|
||||||
scripts.append({
|
scripts.append({
|
||||||
"path": path,
|
"path": path,
|
||||||
"id": res_id
|
"id": res_id
|
||||||
})
|
})
|
||||||
|
|
||||||
# Extract nodes
|
# Extract nodes
|
||||||
for match in re.finditer(r'\[node name="(.+?)".*?type="(.+?)".*?\]', content):
|
for match in re.finditer(r'\[node name="(.+?)".*?type="(.+?)".*?\]', content):
|
||||||
node_name, node_type = match.groups()
|
node_name, node_type = match.groups()
|
||||||
|
|
||||||
# Check if node has a script attached
|
# Check if node has a script attached
|
||||||
script_match = re.search(rf'\[node name="{re.escape(node_name)}".*?script = ExtResource\("(.+?)"\)', content, re.DOTALL)
|
script_match = re.search(rf'\[node name="{re.escape(node_name)}".*?script = ExtResource\("(.+?)"\)', content, re.DOTALL)
|
||||||
attached_script = script_match.group(1) if script_match else None
|
attached_script = script_match.group(1) if script_match else None
|
||||||
|
|
||||||
nodes.append({
|
nodes.append({
|
||||||
"name": node_name,
|
"name": node_name,
|
||||||
"type": node_type,
|
"type": node_type,
|
||||||
"script": attached_script
|
"script": attached_script
|
||||||
})
|
})
|
||||||
|
|
||||||
return {
|
return {
|
||||||
"file": file_path,
|
"file": file_path,
|
||||||
"nodes": nodes,
|
"nodes": nodes,
|
||||||
@@ -1485,11 +1485,11 @@ class CodeAnalyzer:
|
|||||||
"resource_count": len(resources)
|
"resource_count": len(resources)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
def _analyze_godot_resource(self, content: str, file_path: str) -> dict[str, Any]:
|
def _analyze_godot_resource(self, content: str, file_path: str) -> dict[str, Any]:
|
||||||
"""
|
"""
|
||||||
Analyze Godot .tres resource file.
|
Analyze Godot .tres resource file.
|
||||||
|
|
||||||
Extracts:
|
Extracts:
|
||||||
- Resource type and class
|
- Resource type and class
|
||||||
- Script reference
|
- Script reference
|
||||||
@@ -1501,7 +1501,7 @@ class CodeAnalyzer:
|
|||||||
resource_type = None
|
resource_type = None
|
||||||
script_class = None
|
script_class = None
|
||||||
script_path = None
|
script_path = None
|
||||||
|
|
||||||
# Extract resource header
|
# Extract resource header
|
||||||
header_match = re.search(r'\[gd_resource type="(.+?)"(?:\s+script_class="(.+?)")?\s+', content)
|
header_match = re.search(r'\[gd_resource type="(.+?)"(?:\s+script_class="(.+?)")?\s+', content)
|
||||||
if header_match:
|
if header_match:
|
||||||
|
|||||||
@@ -138,7 +138,7 @@ class TestUnifiedCLIEntryPoints(unittest.TestCase):
|
|||||||
|
|
||||||
# Should show version
|
# Should show version
|
||||||
output = result.stdout + result.stderr
|
output = result.stdout + result.stderr
|
||||||
self.assertIn("2.8.0", output)
|
self.assertIn("2.9.0", output)
|
||||||
|
|
||||||
except FileNotFoundError:
|
except FileNotFoundError:
|
||||||
# If skill-seekers is not installed, skip this test
|
# If skill-seekers is not installed, skip this test
|
||||||
|
|||||||
@@ -24,7 +24,7 @@ class TestCliPackage:
|
|||||||
import skill_seekers.cli
|
import skill_seekers.cli
|
||||||
|
|
||||||
assert hasattr(skill_seekers.cli, "__version__")
|
assert hasattr(skill_seekers.cli, "__version__")
|
||||||
assert skill_seekers.cli.__version__ == "2.8.0"
|
assert skill_seekers.cli.__version__ == "2.9.0"
|
||||||
|
|
||||||
def test_cli_has_all(self):
|
def test_cli_has_all(self):
|
||||||
"""Test that skill_seekers.cli package has __all__ export list."""
|
"""Test that skill_seekers.cli package has __all__ export list."""
|
||||||
@@ -88,7 +88,7 @@ class TestMcpPackage:
|
|||||||
import skill_seekers.mcp
|
import skill_seekers.mcp
|
||||||
|
|
||||||
assert hasattr(skill_seekers.mcp, "__version__")
|
assert hasattr(skill_seekers.mcp, "__version__")
|
||||||
assert skill_seekers.mcp.__version__ == "2.8.0"
|
assert skill_seekers.mcp.__version__ == "2.9.0"
|
||||||
|
|
||||||
def test_mcp_has_all(self):
|
def test_mcp_has_all(self):
|
||||||
"""Test that skill_seekers.mcp package has __all__ export list."""
|
"""Test that skill_seekers.mcp package has __all__ export list."""
|
||||||
@@ -108,7 +108,7 @@ class TestMcpPackage:
|
|||||||
import skill_seekers.mcp.tools
|
import skill_seekers.mcp.tools
|
||||||
|
|
||||||
assert hasattr(skill_seekers.mcp.tools, "__version__")
|
assert hasattr(skill_seekers.mcp.tools, "__version__")
|
||||||
assert skill_seekers.mcp.tools.__version__ == "2.8.0"
|
assert skill_seekers.mcp.tools.__version__ == "2.9.0"
|
||||||
|
|
||||||
|
|
||||||
class TestPackageStructure:
|
class TestPackageStructure:
|
||||||
@@ -212,7 +212,7 @@ class TestRootPackage:
|
|||||||
import skill_seekers
|
import skill_seekers
|
||||||
|
|
||||||
assert hasattr(skill_seekers, "__version__")
|
assert hasattr(skill_seekers, "__version__")
|
||||||
assert skill_seekers.__version__ == "2.8.0"
|
assert skill_seekers.__version__ == "2.9.0"
|
||||||
|
|
||||||
def test_root_has_metadata(self):
|
def test_root_has_metadata(self):
|
||||||
"""Test that skill_seekers root package has metadata."""
|
"""Test that skill_seekers root package has metadata."""
|
||||||
|
|||||||
Reference in New Issue
Block a user