Skip to content

Commit 2d17437

Browse files
dottspinamark-inderhees
authored andcommitted
Revert "edtlib: test "last modified" semantic for ... specs"
This unit test was added to cover the change introduced by [1]. Further work on related issues [2] showed that the chosen approach is dead end. We're reverting all changes made in [1]. [1] edtlib: fix last modified semantic in included property specs [2] edtlib: Preserve paths of properties from included child bindings See also: zephyrproject-rtos#65221, zephyrproject-rtos#78095 This reverts commit 70eaa61. Signed-off-by: Christophe Dufaza <[email protected]>
1 parent e5941ff commit 2d17437

File tree

4 files changed

+0
-47
lines changed

4 files changed

+0
-47
lines changed

scripts/dts/python-devicetree/tests/test-bindings-include/base.yaml

Lines changed: 0 additions & 7 deletions
This file was deleted.

scripts/dts/python-devicetree/tests/test-bindings-include/modified.yaml

Lines changed: 0 additions & 7 deletions
This file was deleted.

scripts/dts/python-devicetree/tests/test-bindings-include/top.yaml

Lines changed: 0 additions & 21 deletions
This file was deleted.

scripts/dts/python-devicetree/tests/test_edtlib.py

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -365,18 +365,6 @@ def test_include_filters():
365365
assert set(child.prop2specs.keys()) == {'child-prop-1', 'child-prop-2',
366366
'x', 'z'} # root level 'y' is blocked
367367

368-
def test_include_paths():
369-
'''Test "last modified" semantic for included bindings paths.'''
370-
371-
fname2path = {'base.yaml': 'test-bindings-include/base.yaml',
372-
'modified.yaml': 'test-bindings-include/modified.yaml'}
373-
374-
with from_here():
375-
top = edtlib.Binding('test-bindings-include/top.yaml', fname2path)
376-
377-
assert 'modified.yaml' == os.path.basename(top.prop2specs["x"].path)
378-
assert 'base.yaml' == os.path.basename(top.prop2specs["y"].path)
379-
assert 'top.yaml' == os.path.basename(top.prop2specs["p"].path)
380368

381369
def test_bus():
382370
'''Test 'bus:' and 'on-bus:' in bindings'''

0 commit comments

Comments
 (0)