Panels aufgeraeumt (alle vier Addons) + Test-Suite instandgesetzt
Panel-Umbau
-----------
Jedes Addon hatte seine Einstellungen als Kette dauerhaft offener Boxen - beim
Rock Generator 15 Stueck, man musste am ganzen Panel vorbeiscrollen, um an den
Erzeugen-Knopf zu kommen. Jetzt steht oben nur, was man bei jedem Asset anfasst
(Vorlage, Name, grosser Erzeugen-Knopf, Zufall), der Rest liegt in nummerierten,
eingeklappten Unter-Panels in der Reihenfolge des Arbeitsablaufs:
Rock Gen 9 Unter-Panels (Benennung ... Textur-Bake)
Tree Gen 4 (Wachstum, Blattwerk, Zusatz-Ebenen, Export)
Grass Gen 4 (Form, Haltung, Batch, Kanal-Belegung)
Asset Utils 5 (war schon so, dient als Vorlage)
Schalter, die ein ganzes Unter-Panel steuern (Vertex-Farben backen,
Wachstums-Stufen), sitzen jetzt als Haekchen im Panel-Kopf - Zustand sichtbar
ohne aufzuklappen.
Gemessen statt geglaubt: test_panels.py registriert alle vier Addons, laesst
jedes Panel zeichnen, prueft Tab-Zugehoerigkeit und Auf-/Zugeklappt-Absicht,
dass jeder angebotene Operator existiert - und vor allem, dass JEDE definierte
Property irgendwo erreichbar bleibt (Zweige wie base_shape='BOX' werden dabei
durchgeschaltet). Ergebnis: 0 von 122 Properties beim Umbau verloren.
Test-Suite
----------
Beim Gegenpruefen kam heraus, dass fuenf Tests still veraltet waren und eine
echte Regression nicht gemeldet haetten:
test_rock_gen suchte Objekte mit Praefix "Rock_" - die heissen seit dem
Benennungssystem "Fels_A_M_01". Fand 0 von 3.
test_base_shapes loeschte am Schleifenanfang ALLE Objekte, auch die eigenen
Kopien aus vorigen Durchlaeufen -> StructRNA removed.
test_ecogame_tree erwartete eine feste Namensliste ohne die spaeter
dazugekommenen _Leaf-Objekte.
test_tree_addon dito, ausserdem Zip-Version 1.22.0 fest verdrahtet.
test_leaves Zip-Version 1.23.0 fest verdrahtet.
Drei Muster vermieden statt nur die Symptome geflickt: neue Objekte ueber die
Differenz vorher/nachher finden statt ueber ein Namens-Praefix, Zips per glob
statt mit fester Version, Zusatz-Ebenen beim Zaehlen herausfiltern.
Neu: run_tests.ps1 faehrt die komplette Suite mit einem Befehl (-Filter fuer
einzelne). Genau das fehlte - deshalb war das Veralten nicht aufgefallen.
13 von 13 Tests bestehen.
Nebenbei
--------
build.ps1 ruft server-generate jetzt mit --html auf (browsbare index.html fuer
statisches Hosting). Nachgemessen und im README festgehalten: zwei Zips
derselben Extension-ID sind KEIN Versions-Archiv - server-generate warnt
("archive found with duplicates ... conflicting blender versions"), mehrere
Versionen einer ID gehen nur bei verschiedenen Blender-Bereichen.
Versionen: Rock 2.13.0, Tree 1.24.0, Grass 1.1.0, Asset Utils 1.1.0
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This commit is contained in:
Claude Opus 4.8
@@ -26,21 +26,31 @@ VARS=[("Wuerfel",'CUBE',None),
|
||||
print("%-9s %-6s %s" % ("Variante","Form","Abmessungen X/Y/Z (m)"))
|
||||
keep=[]; x=0.0
|
||||
for name,shape,prop in VARS:
|
||||
for o in list(bpy.data.objects): bpy.data.objects.remove(o,do_unlink=True)
|
||||
# FALLE: das hier loeschte frueher ALLE Objekte - auch die Kopien aus den
|
||||
# vorigen Durchlaeufen. Die waren beim Rendern tot ("StructRNA ... removed").
|
||||
kept = {k[0] for k in keep}
|
||||
for o in list(bpy.data.objects):
|
||||
if o not in kept: bpy.data.objects.remove(o,do_unlink=True)
|
||||
s.base_shape=shape; s.box_variation=0.0
|
||||
if prop: s.box_proportions=prop
|
||||
s.seed=7
|
||||
# Die behaltenen Kopien stehen weiter in der Szene - [0] wuerde eine davon
|
||||
# greifen statt des frisch erzeugten Steins. Deshalb ueber die Differenz.
|
||||
before=set(bpy.context.scene.objects)
|
||||
bpy.ops.object.generate_rocks_batch()
|
||||
ob=[o for o in bpy.context.scene.objects if o.type=='MESH'][0]
|
||||
ob=[o for o in bpy.context.scene.objects
|
||||
if o.type=='MESH' and o not in before][0]
|
||||
bb=ob.bound_box
|
||||
d=[max(v[i] for v in bb)-min(v[i] for v in bb) for i in range(3)]
|
||||
print("%-9s %-6s %.2f / %.2f / %.2f" % (name,shape,d[0],d[1],d[2]))
|
||||
if name=="Platte" and not (d[2] < d[0]*0.45): fails.append("platte_nicht_flach")
|
||||
if name=="Riegel" and not (d[0] > d[1]*1.5): fails.append("riegel_nicht_laenglich")
|
||||
cp=ob.copy(); cp.data=ob.data.copy(); keep.append((cp,x))
|
||||
cp=ob.copy(); cp.data=ob.data.copy()
|
||||
bpy.context.collection.objects.link(cp) # sofort verlinken, nicht spaeter
|
||||
keep.append((cp,x))
|
||||
x+=2.4
|
||||
for cp,px in keep:
|
||||
bpy.context.collection.objects.link(cp); cp.location=(px,0,0)
|
||||
cp.location=(px,0,0)
|
||||
cp.data.materials.clear(); cp.data.materials.append(mat)
|
||||
for o in list(bpy.context.scene.objects):
|
||||
if o.type=='MESH' and o not in [k[0] for k in keep]: bpy.data.objects.remove(o,do_unlink=True)
|
||||
|
||||
@@ -16,17 +16,21 @@ if m.TreeGenSettings.bl_rna.properties['generate_lightmap_uv'].default: fails.ap
|
||||
s.preset='birke'; s.art_name="Birke"; s.variant=1
|
||||
s.use_growth=True; s.stages=4; s.apply_modifier=False
|
||||
bpy.ops.object.treegen_create()
|
||||
# Seit der Blattwerk-Ebene entsteht zu jeder Stufe ein zweites Objekt "_Leaf".
|
||||
# Deshalb Stamm und Ebene getrennt pruefen statt eine feste Gesamtliste.
|
||||
names=sorted(o.name for o in bpy.context.scene.objects if o.type=='MESH')
|
||||
print("STUFEN:", names)
|
||||
if names != ["Birke_01_S1","Birke_01_S2","Birke_01_S3","Birke_01_S4"]: fails.append("stufen_namen")
|
||||
stems=[n for n in names if not n.endswith(("_Leaf","_Frucht"))]
|
||||
if stems != ["Birke_01_S1","Birke_01_S2","Birke_01_S3","Birke_01_S4"]: fails.append("stufen_namen")
|
||||
if [n for n in names if n.endswith("_Leaf")] != [s_+"_Leaf" for s_ in stems]: fails.append("stufen_leaf")
|
||||
|
||||
# Einzelbaum, Art aus Preset abgeleitet
|
||||
for o in list(bpy.data.objects): bpy.data.objects.remove(o,do_unlink=True)
|
||||
s.use_growth=False; s.art_name=""; s.preset='eiche'; s.variant=2
|
||||
bpy.ops.object.treegen_create()
|
||||
single=[o.name for o in bpy.context.scene.objects if o.type=='MESH']
|
||||
single=sorted(o.name for o in bpy.context.scene.objects if o.type=='MESH')
|
||||
print("EINZELN:", single)
|
||||
if single != ["Eiche_02"]: fails.append("einzel_name")
|
||||
if [n for n in single if not n.endswith(("_Leaf","_Frucht"))] != ["Eiche_02"]: fails.append("einzel_name")
|
||||
|
||||
# Ursprung unten-mittig + echte Meter
|
||||
ob=bpy.context.scene.objects[single[0]]
|
||||
|
||||
@@ -1,5 +1,12 @@
|
||||
import bpy, sys, zipfile, os, tempfile, importlib.util, math, bmesh
|
||||
ZIP=r"A:\eco\stylized_rock_generator\dist\stylized_tree_generator-1.23.0.zip"
|
||||
import bpy, glob, sys, zipfile, os, tempfile, importlib.util, math, bmesh
|
||||
# Version NICHT fest verdrahten - sonst schlaegt der Test nach jedem
|
||||
# Versions-Bump fehl, obwohl am Addon nichts kaputt ist.
|
||||
_d = os.path.join(os.path.dirname(os.path.dirname(os.path.abspath(__file__))), "dist")
|
||||
_z = sorted(glob.glob(os.path.join(_d, "stylized_tree_generator-*.zip")))
|
||||
if not _z:
|
||||
print("ERGEBNIS: FEHLER: kein gebautes Tree-Zip in dist/ (erst build.ps1)")
|
||||
sys.exit(1)
|
||||
ZIP = _z[-1]
|
||||
tmp=tempfile.mkdtemp()
|
||||
with zipfile.ZipFile(ZIP) as z: z.extractall(tmp)
|
||||
init=[os.path.join(r,"__init__.py") for r,_,f in os.walk(tmp) if "__init__.py" in f][0]
|
||||
|
||||
@@ -0,0 +1,234 @@
|
||||
"""Registriert alle vier Addons und prueft die Panel-Struktur.
|
||||
|
||||
Warum das eigene Test-Datei ist: py_compile sagt nur, dass die Datei syntaktisch
|
||||
gueltig ist. Ob Blender die Panels ANNIMMT, ist eine andere Frage - bl_parent_id
|
||||
muss auf ein bereits registriertes Panel zeigen, bl_category muss zwischen
|
||||
Eltern- und Unter-Panel uebereinstimmen (sonst landet das Unter-Panel in einem
|
||||
anderen Tab und ist praktisch unsichtbar), und jedes Panel muss zeichnen koennen,
|
||||
ohne zu werfen.
|
||||
|
||||
Aufruf:
|
||||
blender --background --factory-startup --python tests/test_panels.py
|
||||
"""
|
||||
import os
|
||||
import sys
|
||||
import importlib.util
|
||||
|
||||
import bpy
|
||||
|
||||
ROOT = os.path.dirname(os.path.dirname(os.path.abspath(__file__)))
|
||||
|
||||
# Addon-Datei -> (Haupt-Panel-idname, Anzahl Unter-Panels, die offen sein DUERFEN)
|
||||
#
|
||||
# In den Generatoren sind Unter-Panels Einstellungen, die man selten anfasst -
|
||||
# die gehoeren alle zugeklappt, damit der Erzeugen-Knopf oben sichtbar bleibt.
|
||||
# In Asset Utils ist jedes Unter-Panel ein Arbeitsschritt: 1/2/4 macht man bei
|
||||
# JEDEM Asset, die bleiben offen. Deshalb steht die Erwartung hier pro Addon
|
||||
# statt als pauschale Regel.
|
||||
ADDONS = {
|
||||
"stylized_rock_generator.py": ("VIEW3D_PT_rock_generator", 9,
|
||||
"rock_gen_settings", set()),
|
||||
"stylized_tree_generator.py": ("VIEW3D_PT_tree_generator", 4,
|
||||
"tree_gen_settings", set()),
|
||||
"stylized_grass_generator.py": ("VIEW3D_PT_grass_generator", 4,
|
||||
"grass_gen_settings", set()),
|
||||
"stylized_asset_utils.py": ("VIEW3D_PT_asset_utils", 5,
|
||||
"asset_utils_settings",
|
||||
{"1 - Vorbereiten", "2 - Vertex-Farben",
|
||||
"4 - Pruefen"}),
|
||||
}
|
||||
|
||||
failures = []
|
||||
|
||||
|
||||
def load(path, name):
|
||||
spec = importlib.util.spec_from_file_location(name, path)
|
||||
mod = importlib.util.module_from_spec(spec)
|
||||
sys.modules[name] = mod
|
||||
spec.loader.exec_module(mod)
|
||||
return mod
|
||||
|
||||
|
||||
class FakeSelf:
|
||||
"""Ersatz fuer die Panel-Instanz: draw() braucht nur self.layout."""
|
||||
|
||||
def __init__(self, layout):
|
||||
self.layout = layout
|
||||
|
||||
|
||||
class FakeLayout:
|
||||
"""Minimal-Layout, das jeden UI-Aufruf schluckt und mitzaehlt.
|
||||
|
||||
Damit laesst sich draw() ohne echten UI-Kontext ausfuehren - der ist im
|
||||
--background-Modus nicht verfuegbar.
|
||||
"""
|
||||
|
||||
def __init__(self, counter):
|
||||
self._c = counter
|
||||
|
||||
def _child(self, *a, **k):
|
||||
return FakeLayout(self._c)
|
||||
|
||||
row = column = box = split = _child
|
||||
|
||||
def prop(self, data, name, **k):
|
||||
# Zugriff erzwingen: schlaegt fehl, wenn die Property nicht existiert.
|
||||
getattr(data, name)
|
||||
self._c["prop"] += 1
|
||||
|
||||
def operator(self, idname, **k):
|
||||
self._c["op"] += 1
|
||||
self._c["ops"].add(idname)
|
||||
return self
|
||||
|
||||
def label(self, **k):
|
||||
self._c["label"] += 1
|
||||
|
||||
def separator(self, **k):
|
||||
pass
|
||||
|
||||
def menu(self, *a, **k):
|
||||
pass
|
||||
|
||||
def template_ID(self, *a, **k):
|
||||
pass
|
||||
|
||||
def __setattr__(self, k, v):
|
||||
# scale_y / active / enabled einfach annehmen
|
||||
object.__setattr__(self, k, v)
|
||||
|
||||
|
||||
class CollectLayout(FakeLayout):
|
||||
"""Wie FakeLayout, sammelt aber die Namen der gezeichneten Properties."""
|
||||
|
||||
def __init__(self, names):
|
||||
FakeLayout.__init__(self, {"prop": 0, "op": 0, "label": 0, "ops": set()})
|
||||
self._names = names
|
||||
|
||||
def _child(self, *a, **k):
|
||||
return CollectLayout(self._names)
|
||||
|
||||
row = column = box = split = _child
|
||||
|
||||
def prop(self, data, name, **k):
|
||||
self._names.add(name)
|
||||
|
||||
|
||||
for filename, (parent_id, want_subs, settings_attr, open_ok) in ADDONS.items():
|
||||
path = os.path.join(ROOT, filename)
|
||||
mod = load(path, filename[:-3])
|
||||
try:
|
||||
mod.register()
|
||||
except Exception as exc:
|
||||
failures.append("%s: register() wirft %s" % (filename, exc))
|
||||
continue
|
||||
|
||||
panels = [c for c in mod.classes if issubclass(c, bpy.types.Panel)]
|
||||
parent = [p for p in panels if getattr(p, "bl_idname", "") == parent_id]
|
||||
if not parent:
|
||||
failures.append("%s: Haupt-Panel %s fehlt" % (filename, parent_id))
|
||||
mod.unregister()
|
||||
continue
|
||||
parent = parent[0]
|
||||
|
||||
subs = [p for p in panels if getattr(p, "bl_parent_id", "") == parent_id]
|
||||
if len(subs) != want_subs:
|
||||
failures.append("%s: %d Unter-Panels, erwartet %d"
|
||||
% (filename, len(subs), want_subs))
|
||||
|
||||
for sub in subs:
|
||||
# Gleicher Tab wie das Eltern-Panel?
|
||||
if sub.bl_category != parent.bl_category:
|
||||
failures.append("%s/%s: bl_category '%s' != Eltern-Tab '%s'"
|
||||
% (filename, sub.__name__, sub.bl_category,
|
||||
parent.bl_category))
|
||||
# Eingeklappt - ausser bei den ausdruecklich als offen gewollten.
|
||||
is_closed = 'DEFAULT_CLOSED' in getattr(sub, "bl_options", set())
|
||||
if not is_closed and sub.bl_label not in open_ok:
|
||||
failures.append("%s/%s (%s): offen, sollte eingeklappt sein"
|
||||
% (filename, sub.__name__, sub.bl_label))
|
||||
if is_closed and sub.bl_label in open_ok:
|
||||
failures.append("%s/%s (%s): eingeklappt, sollte offen sein"
|
||||
% (filename, sub.__name__, sub.bl_label))
|
||||
# Wirklich registriert? Blender haengt bl_rna an registrierte Klassen.
|
||||
if not hasattr(sub, "bl_rna"):
|
||||
failures.append("%s/%s: nicht registriert"
|
||||
% (filename, sub.__name__))
|
||||
|
||||
# Jedes Panel einmal zeichnen lassen.
|
||||
total = {"prop": 0, "op": 0, "label": 0, "ops": set()}
|
||||
for p in [parent] + subs:
|
||||
counter = {"prop": 0, "op": 0, "label": 0, "ops": set()}
|
||||
# FALLE: registrierte Panel-Klassen lassen sich nicht instanziieren
|
||||
# ("bpy_struct.__new__(struct): expected a single argument"). draw() ist
|
||||
# aber eine normale Python-Funktion - self darf alles sein, was .layout hat.
|
||||
inst = FakeSelf(FakeLayout(counter))
|
||||
try:
|
||||
p.draw(inst, bpy.context)
|
||||
if "draw_header" in p.__dict__:
|
||||
p.draw_header(inst, bpy.context)
|
||||
except Exception as exc:
|
||||
failures.append("%s/%s: draw() wirft %s: %s"
|
||||
% (filename, p.__name__, type(exc).__name__, exc))
|
||||
continue
|
||||
if counter["prop"] + counter["op"] + counter["label"] == 0:
|
||||
failures.append("%s/%s: zeichnet gar nichts"
|
||||
% (filename, p.__name__))
|
||||
for k in ("prop", "op", "label"):
|
||||
total[k] += counter[k]
|
||||
total["ops"] |= counter["ops"]
|
||||
|
||||
# Jede definierte Property muss irgendwo im Panel erreichbar sein.
|
||||
#
|
||||
# Das ist der eigentliche Regressionstest beim Umbauen der Panels: beim
|
||||
# Verschieben von Reglern zwischen Panels faellt leicht einer heraus, und
|
||||
# dann gibt es eine Einstellung, an die man in Blender nicht mehr rankommt.
|
||||
# Ein paar Regler haengen an einem Zweig (BOX-Proportionen nur bei
|
||||
# base_shape='BOX'), deshalb werden diese Zweige vorher durchgeschaltet.
|
||||
st = getattr(bpy.context.scene, settings_attr, None)
|
||||
if st is not None:
|
||||
drawn = set()
|
||||
branches = [{}]
|
||||
for attr, values in (("base_shape", ('CUBE', 'BOX', 'ICOSPHERE')),
|
||||
("g_mode", ('SHELLS', 'MOSS')),
|
||||
("shading_mode", ('AUTO', 'FLAT', 'SMOOTH'))):
|
||||
if hasattr(st, attr):
|
||||
branches += [{attr: v} for v in values]
|
||||
for combo in branches:
|
||||
for attr, val in combo.items():
|
||||
try:
|
||||
setattr(st, attr, val)
|
||||
except TypeError:
|
||||
continue
|
||||
for p in [parent] + subs:
|
||||
try:
|
||||
p.draw(FakeSelf(CollectLayout(drawn)), bpy.context)
|
||||
if "draw_header" in p.__dict__:
|
||||
p.draw_header(FakeSelf(CollectLayout(drawn)), bpy.context)
|
||||
except Exception:
|
||||
pass
|
||||
defined = set(type(st).__annotations__)
|
||||
unreachable = sorted(defined - drawn)
|
||||
if unreachable:
|
||||
failures.append("%s: Property(s) in keinem Panel erreichbar: %s"
|
||||
% (filename, ", ".join(unreachable)))
|
||||
|
||||
# Jeder Operator, den ein Panel anbietet, muss es auch geben.
|
||||
for idname in sorted(total["ops"]):
|
||||
grp, _, op = idname.partition(".")
|
||||
if not hasattr(getattr(bpy.ops, grp, None), op):
|
||||
failures.append("%s: Panel ruft unbekannten Operator %s"
|
||||
% (filename, idname))
|
||||
|
||||
print("%-32s %d Unter-Panels %3d Props %2d Buttons %2d Labels"
|
||||
% (filename, len(subs), total["prop"], total["op"], total["label"]))
|
||||
|
||||
mod.unregister()
|
||||
|
||||
print("")
|
||||
if failures:
|
||||
print("ERGEBNIS: %d FEHLER" % len(failures))
|
||||
for f in failures:
|
||||
print(" - " + f)
|
||||
sys.exit(1)
|
||||
print("ERGEBNIS: ALLE PANELS OK")
|
||||
@@ -81,12 +81,16 @@ def main():
|
||||
settings.assign_material = False
|
||||
|
||||
# 3. Batch erzeugen
|
||||
# Vorher merken, was schon da ist: das Benennungsschema kommt aus den
|
||||
# Einstellungen (Fels_A_M_01 ...), ein fest verdrahtetes Praefix wie "Rock_"
|
||||
# geht schief, sobald jemand die Benennung aendert. Differenz statt Ratename.
|
||||
before = set(bpy.context.scene.objects)
|
||||
result = bpy.ops.object.generate_rocks_batch()
|
||||
if 'FINISHED' not in result:
|
||||
failures.append("Operator lieferte nicht {'FINISHED'}: %s" % (result,))
|
||||
|
||||
rocks = [o for o in bpy.context.scene.objects
|
||||
if o.type == 'MESH' and o.name.startswith("Rock_")]
|
||||
if o.type == 'MESH' and o not in before]
|
||||
|
||||
if len(rocks) != BATCH:
|
||||
failures.append("Erwartet %d Rocks, gefunden %d" % (BATCH, len(rocks)))
|
||||
|
||||
@@ -1,7 +1,15 @@
|
||||
"""Prueft das gebaute Tree-Addon: registrieren, Operator ausfuehren, Ergebnis messen."""
|
||||
import bpy, sys, zipfile, os, tempfile, importlib.util, traceback
|
||||
|
||||
ZIP = r"A:\eco\stylized_rock_generator\dist\stylized_tree_generator-1.22.0.zip"
|
||||
# Version NICHT fest verdrahten - sonst schlaegt der Test bei jedem Bump fehl,
|
||||
# obwohl am Addon nichts kaputt ist.
|
||||
import glob
|
||||
_d = os.path.join(os.path.dirname(os.path.dirname(os.path.abspath(__file__))), "dist")
|
||||
_z = sorted(glob.glob(os.path.join(_d, "stylized_tree_generator-*.zip")))
|
||||
if not _z:
|
||||
print("ERGEBNIS: FEHLER: kein gebautes Tree-Zip in dist/ (erst build.ps1 laufen lassen)")
|
||||
sys.exit(1)
|
||||
ZIP = _z[-1]
|
||||
fails = []
|
||||
|
||||
tmp = tempfile.mkdtemp()
|
||||
@@ -46,7 +54,11 @@ for o in list(bpy.data.objects):
|
||||
bpy.data.objects.remove(o, do_unlink=True)
|
||||
s.preset = "baum"; s.use_growth = True; s.stages = 4
|
||||
r = bpy.ops.object.treegen_create()
|
||||
obs = sorted([o for o in bpy.data.objects if o.type == 'MESH'], key=lambda o: o.name)
|
||||
# Nur die Staemme: seit der Blattwerk-Ebene gibt es zu jeder Stufe noch ein
|
||||
# "_Leaf"-Objekt. Mit denen zusammen ist die Vertex-Folge nicht mehr aufsteigend.
|
||||
obs = sorted([o for o in bpy.data.objects
|
||||
if o.type == 'MESH' and not o.name.endswith(("_Leaf", "_Frucht"))],
|
||||
key=lambda o: o.name)
|
||||
bpy.context.view_layer.update()
|
||||
dg = bpy.context.evaluated_depsgraph_get()
|
||||
counts = [len(o.evaluated_get(dg).data.vertices) for o in obs]
|
||||
|
||||
Reference in New Issue
Block a user