• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

opsdroid / opsdroid / 696
97%

Build:
DEFAULT BRANCH: master
Ran 23 Oct 2017 08:22AM UTC
Jobs 3
Files 19
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

pending completion
696

push

travis-ci

jacobtomlinson
Handle missing section details in load_modules_from_config (#271)

* Handle missing section details in load_modules_from_config

If the user has a 'databases:' section in their configuration, but nothing
listed as part of it, the opsdroid.loader.load_modules_from_config() would
call _load_modules with the modules parameter set to None, which would cause
a problem when trying to iterate it on line 223.

* Add some failing tests for PR#271

I found that I was getting two different errors with with the
load_modules_from_config() call when the minimal.yaml file was in
different states. Hence I've added a minimal_2.yaml to cover both error
conditions.

* Fix bug where configuration node has no children elements.

If the configuration has elements with no children (such as the "hello"
skill in the tests/configs/minimal.yaml file) then a yaml import gives you
that element as a string, not a dict. This change makes
Loader._load_modules() deal with that sensibly.

* Fix up tests for #271

As part of it's work the load_modules_from_config() file imports modules.
However those modules rely on opsdroid being fully instantiated, and that is
not the case in these unit tests.  So some mocking was necessary to isolate
the test, and only test the successful parsing of the configuration file.

* Minor change to satisfy pylint.

800 of 827 relevant lines covered (96.74%)

1.93 hits per line

Jobs
ID Job ID Ran Files Coverage
1 696.1 (TOXENV=lint) 23 Oct 2017 08:23AM UTC 0
Travis Job 696.1
2 696.2 (TOXENV=py35) 23 Oct 2017 08:22AM UTC 0
96.74
Travis Job 696.2
3 696.3 (TOXENV=py36) 23 Oct 2017 08:22AM UTC 0
96.74
Travis Job 696.3
Source Files on build 696
Detailed source file information is not available for this build.
  • Back to Repo
  • Travis Build #696
  • e8b89cc6 on github
  • Prev Build on master (#688)
  • Next Build on master (#708)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE TRIAL · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc