summaryrefslogtreecommitdiff
path: root/testxml
diff options
context:
space:
mode:
Diffstat (limited to 'testxml')
-rwxr-xr-xtestxml/testxml.py119
1 files changed, 112 insertions, 7 deletions
diff --git a/testxml/testxml.py b/testxml/testxml.py
index 9312bd5..7c3bdd4 100755
--- a/testxml/testxml.py
+++ b/testxml/testxml.py
@@ -7,7 +7,7 @@ import xml
from xml.dom import minidom
from PIL import Image
-filt = re.compile(".+[.]xml", re.IGNORECASE)
+filt = re.compile(".+[.](xml|tmx)", re.IGNORECASE)
dyesplit1 = re.compile(";")
dyesplit2 = re.compile(",")
parentDir = "../../clientdata"
@@ -113,6 +113,7 @@ def enumDirs(parentDir):
errors = errors + 1
def loadPaths():
+ global warnings
try:
dom = minidom.parse(parentDir + "/paths.xml")
for node in dom.getElementsByTagName("option"):
@@ -341,14 +342,118 @@ def testSpriteFile(id, fullPath, file, fileLoc, dnum, iserr):
showMsgSprite(fileLoc, "image file not exist: " + image, iserr)
return
sizes = testImageFile(image, fullPath, 0, iserr)
- s = int(sizes[0] / int(width)) * int(width)
- if sizes[0] != s:
+ s1 = int(sizes[0] / int(width)) * int(width)
+ if sizes[0] != s1:
showMsgSprite(fileLoc, "image width " + str(sizes[0]) + \
- " (need " + str(s) + ") is not multiply to frame size " + width + ", image:" + image, False)
- s = int(sizes[1] / int(height)) * int(height)
- if sizes[1] != s:
+ " (need " + str(s1) + ") is not multiply to frame size " + width + ", image:" + image, False)
+ s2 = int(sizes[1] / int(height)) * int(height)
+ if sizes[1] != s2:
showMsgSprite(fileLoc, "image height " + str(sizes[1]) + \
- " (need " + str(s) + ") is not multiply to frame size " + height + ", image:" + image, False)
+ " (need " + str(s2) + ") is not multiply to frame size " + height + ", image:" + image, False)
+
+ num = (s1 / int(width)) * (s2 / int(height))
+ if num < 1:
+ showMsgSprite(fileLoc, "image have zero frames: " + iamge, iserr)
+
+ try:
+ includes = dom.getElementsByTagName("include")
+ except:
+ includes = None
+
+ #todo need parse included files
+
+ try:
+ actions = dom.getElementsByTagName("action")
+ except:
+ actions = None
+
+ if (actions == None or len(actions) == 0) and (includes == None or len(includes) == 0):
+ showMsgSprite(fileLoc, "no actions in sprite file", iserr)
+ else:
+ actset = set()
+ for action in actions:
+ try:
+ name = action.attributes["name"].value
+ except:
+ showMsgSprite("no action name", iserr)
+ continue
+
+ testSpriteAction(fileLoc, name, action, num, iserr)
+
+ if name in actset:
+ showMsgSprite(fileLoc, "duplicate action: " + name, iserr)
+ continue
+ actset.add(name)
+
+
+
+def testSpriteAction(file, name, action, numframes, iserr):
+ try:
+ animations = action.getElementsByTagName("animation")
+ except:
+ animations = None
+
+ if animations == None or len(animations) == 0:
+ showMsgSprite(file, "no animation tags in action: " + name, False)
+
+ aniset = set()
+ for animation in animations:
+ try:
+ direction = animation.attributes["direction"].value
+ except:
+ direction = "default"
+
+ if direction is aniset:
+ showMsgSprite(file, "duplicate direction in action: " + name, iserr)
+ continue
+ aniset.add(direction)
+ try:
+ frames = animation.getElementsByTagName("frame")
+ for frame in frames:
+ try:
+ idx = int(frame.attributes["index"].value)
+ if idx >= numframes or idx < 0:
+ showMsgSprite(file, "incorrect frame index " + str(idx) + \
+ "in action: " + name, iserr)
+
+ except:
+ showMsgSprite(file, "no frame index in action: " + name, iserr)
+
+ cnt = len(frames)
+ except:
+ cnt = 0
+
+ try:
+ sequences = animation.getElementsByTagName("sequence")
+ cnt = cnt + len(sequences)
+
+ for sequence in sequences:
+ try:
+ i1 = int(sequence.attributes["start"].value)
+ i2 = int(sequence.attributes["end"].value)
+ if i1 >= numframes or i1 < 0:
+ showMsgSprite(file, "incorrect start sequence index " + str(i1) + \
+ "in action: " + name, iserr)
+ if i2 >= numframes or i2 < 0:
+ showMsgSprite(file, "incorrect end sequence index " + str(i2) + \
+ "in action: " + name, iserr)
+ except:
+ showMsgSprite(file, "no sequence start or end index in action: " + name, iserr)
+ except:
+ None
+
+ if cnt == 0:
+ showMsgSprite(file, "no frames or sequences in action: " + name, iserr)
+
+ if "default" not in aniset:
+ if "down" not in aniset:
+ showMsgSprite(file, "no down direction in animation: " + name, iserr)
+ if "up" not in aniset:
+ showMsgSprite(file, "no up direction in animation: " + name, iserr)
+ if "left" not in aniset:
+ showMsgSprite(file, "no left direction in animation: " + name, iserr)
+ if "right" not in aniset:
+ showMsgSprite(file, "no right direction in animation: " + name, iserr)