diff options
| author | Benjamin Schubert <ben.c.schubert@gmail.com> | 2019-07-16 18:31:19 +0100 |
|---|---|---|
| committer | Benjamin Schubert <ben.c.schubert@gmail.com> | 2019-07-16 18:32:48 +0100 |
| commit | f2ece705b7bf36d7d923264af2b1f51b66a9f9a7 (patch) | |
| tree | 7df4a79b12013f6094371e86bcf52bb6aa8719e7 /tests/internals/yaml.py | |
| parent | fbb8eea8334023e30c8719cb52f9b0e226b4eb8b (diff) | |
| download | buildstream-bschubert/api-improvements.tar.gz | |
node: Add 'get_str_list' on 'MappingNode'bschubert/api-improvements
`mapping.get_sequence(...).as_str_list()` is a very common
pattern seen both in plugins and the core. Adding a helper to reduce
the number of operations will make usage smoother
Diffstat (limited to 'tests/internals/yaml.py')
| -rw-r--r-- | tests/internals/yaml.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/tests/internals/yaml.py b/tests/internals/yaml.py index d94d006e3..93619dc4c 100644 --- a/tests/internals/yaml.py +++ b/tests/internals/yaml.py @@ -146,7 +146,7 @@ def test_node_set_overwrite(datafiles): assert base.get_str('kind') == 'cow' # Overwrite a list as a string - assert base.get_sequence('moods').as_str_list() == ['happy', 'sad'] + assert base.get_str_list('moods') == ['happy', 'sad'] base['moods'] = 'unemotional' assert base.get_str('moods') == 'unemotional' @@ -160,10 +160,10 @@ def test_node_set_list_element(datafiles): base = _yaml.load(filename) - assert base.get_sequence('moods').as_str_list() == ['happy', 'sad'] + assert base.get_str_list('moods') == ['happy', 'sad'] base.get_sequence('moods')[0] = 'confused' - assert base.get_sequence('moods').as_str_list() == ['confused', 'sad'] + assert base.get_str_list('moods') == ['confused', 'sad'] # Really this is testing _yaml.node_copy(), we want to |