[Bf-blender-cvs] [24b47c00eaa] master: Fix related to T65963: Ctest: Failing test script_load_keymap passes.

Bastien Montagne noreply at git.blender.org
Tue Jun 25 16:09:25 CEST 2019


Commit: 24b47c00eaabe12b5262b59a185da154a477bd98
Author: Bastien Montagne
Date:   Tue Jun 25 16:07:28 2019 +0200
Branches: master
https://developer.blender.org/rB24b47c00eaabe12b5262b59a185da154a477bd98

Fix related to T65963: Ctest: Failing test script_load_keymap passes.

Fix things to make test actually fail as expected (one cannot compare
functions to strings, so no more sorting for now).

Not sure how to actually fix the test though, not even sure test make
any sense anymore actually, with all those weirdo gizmos and tools
keymaps thingy...

===================================================================

M	tests/python/bl_keymap_completeness.py

===================================================================

diff --git a/tests/python/bl_keymap_completeness.py b/tests/python/bl_keymap_completeness.py
index 19541b355e2..30dac64f88b 100644
--- a/tests/python/bl_keymap_completeness.py
+++ b/tests/python/bl_keymap_completeness.py
@@ -49,14 +49,17 @@ def check_maps():
     test = maps_py - maps_bl
     if test:
         print("Keymaps that are in 'bl_keymap_utils.keymap_hierarchy' but not blender")
-        for km_id in sorted(test):
+        for km_id in test:
+            if callable(km_id):
+                # Keymap functions of tools are not in blender anyway...
+                continue
             print("\t%s" % km_id)
-        err = True
+            err = True
 
     test = maps_bl - maps_py
     if test:
         print("Keymaps that are in blender but not in 'bl_keymap_utils.keymap_hierarchy'")
-        for km_id in sorted(test):
+        for km_id in test:
             km = keyconf.keymaps[km_id]
             print("    ('%s', '%s', '%s', [])," % (km_id, km.space_type, km.region_type))
         err = True



More information about the Bf-blender-cvs mailing list