Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft: Fix ci #1096

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
5 changes: 5 additions & 0 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,11 +16,15 @@ matrix:
python: 3.7
before_install:
- choco install python3 --version=3.7.4
- choco install openssl --version=1.1.1.900
- pip install ntc_templates==1.4.1
- pip install textfsm==0.4.1
env:
- PATH=/c/Python37:/c/Python37/Scripts:$PATH
- TRAVIS_PYTHON_VERSION=3.7
- CRYPTOGRAPHY_DONT_BUILD_RUST=1
- INCLUDE='C:\OpenSSL-Win32\include'
- LIB='C:\OpenSSL-Win32\lib'
addons:
apt:
packages:
Expand All @@ -29,6 +33,7 @@ addons:
- libxslt1-dev

install:
- "set"
- "pip install -r development.txt"
- "pip install -r requirements.txt"
- "pip install ."
Expand Down
4 changes: 4 additions & 0 deletions lib/jnpr/junos/utils/config.py
Original file line number Diff line number Diff line change
Expand Up @@ -260,6 +260,10 @@ def diff(self, rb_id=0, ignore_warning=False, use_fast_diff=False):
else:
raise

# Check return type is an etree Element, if not return None for no changes
if not type(rsp) == etree._Element:
return None

diff_txt = rsp.find("configuration-output").text
return None if diff_txt == "\n" else diff_txt

Expand Down