Skip to content

Commit

Permalink
Extends tests for ModuleStream.depends_on_stream()
Browse files Browse the repository at this point in the history
[updated]The tests for depends_on_stream() and build_depends_on_stream() is modified to handle empty lists

Fixes:fedora-modularity#192
  • Loading branch information
athira-selvam committed Mar 15, 2019
1 parent 5e42e86 commit ee0ee95
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 1 deletion.
12 changes: 11 additions & 1 deletion modulemd/v2/tests/ModulemdTests/modulestream.py
Original file line number Diff line number Diff line change
Expand Up @@ -1085,7 +1085,17 @@ def test_depends_on_stream(self):
'platform', 'f28'), False)

self.assertEqual(stream.depends_on_stream('base', 'f30'), False)
self.assertEqual(stream.depends_on_stream('base', 'f30'), False)
self.assertEqual(
stream.build_depends_on_stream(
'base', 'f30'), False)

if version >= Modulemd.ModuleStreamVersionEnum.TWO:
self.assertEqual(
stream.depends_on_stream(
'streamname', 'f30'), True)
self.assertEqual(
stream.build_depends_on_stream(
'streamname', 'f30'), True)


if __name__ == '__main__':
Expand Down
2 changes: 2 additions & 0 deletions modulemd/v2/tests/test_data/dependson_v2.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,10 @@ data:
dependencies:
- buildrequires:
platform: [ f30 ]
streamname: [ ]
requires:
platform: [ -f28 ]
streamname: [ ]
references:
community: http://nodejs.org
documentation: http://nodejs.org/en/docs
Expand Down

0 comments on commit ee0ee95

Please sign in to comment.