* Allow verisure locks to be configured with a default code * linting fix * PR feedback * PR feedback - try harder to prevent future typos A python mock is a magical thing, and will respond to basicaly any method you call on it. It's somewhat better to assert against an explicit variable named 'mock', rather than to assert on the method name you wanted to mock... could prevent a typo from messing up tests. * PR feedback: convert tests to integration-style tests Set up a fake verisure hub, stub out a _lot_ of calls, then test after platform discovery and service calls. It should be noted that we're overriding the `update()` calls in these tests. This was done to prevent even further mocking of the verisure hub's responses. Hopefully, this'll be a foundation for people to write more tests. * more pr feedback
364 lines
9.2 KiB
Python
Executable file
364 lines
9.2 KiB
Python
Executable file
#!/usr/bin/env python3
|
|
"""Generate an updated requirements_all.txt."""
|
|
import importlib
|
|
import os
|
|
import pkgutil
|
|
import re
|
|
import sys
|
|
import fnmatch
|
|
|
|
COMMENT_REQUIREMENTS = (
|
|
'RPi.GPIO',
|
|
'raspihats',
|
|
'rpi-rf',
|
|
'Adafruit-DHT',
|
|
'Adafruit_BBIO',
|
|
'fritzconnection',
|
|
'pybluez',
|
|
'beacontools',
|
|
'bluepy',
|
|
'opencv-python',
|
|
'python-lirc',
|
|
'pyuserinput',
|
|
'evdev',
|
|
'pycups',
|
|
'python-eq3bt',
|
|
'avion',
|
|
'decora',
|
|
'face_recognition',
|
|
'blinkt',
|
|
'smbus-cffi',
|
|
'envirophat',
|
|
'i2csense',
|
|
'credstash',
|
|
'bme680',
|
|
'homekit',
|
|
'py_noaa',
|
|
)
|
|
|
|
TEST_REQUIREMENTS = (
|
|
'aioautomatic',
|
|
'aiohttp_cors',
|
|
'aiohue',
|
|
'aiounifi',
|
|
'apns2',
|
|
'caldav',
|
|
'coinmarketcap',
|
|
'defusedxml',
|
|
'dsmr_parser',
|
|
'enturclient',
|
|
'ephem',
|
|
'evohomeclient',
|
|
'feedparser',
|
|
'foobot_async',
|
|
'geojson_client',
|
|
'georss_client',
|
|
'gTTS-token',
|
|
'ha-ffmpeg',
|
|
'hangups',
|
|
'HAP-python',
|
|
'haversine',
|
|
'hbmqtt',
|
|
'hdate',
|
|
'holidays',
|
|
'home-assistant-frontend',
|
|
'homematicip',
|
|
'influxdb',
|
|
'jsonpath',
|
|
'libpurecoollink',
|
|
'libsoundtouch',
|
|
'luftdaten',
|
|
'mficlient',
|
|
'numpy',
|
|
'paho-mqtt',
|
|
'pexpect',
|
|
'pilight',
|
|
'pmsensor',
|
|
'prometheus_client',
|
|
'pushbullet.py',
|
|
'py-canary',
|
|
'pyblackbird',
|
|
'pydeconz',
|
|
'pydispatcher',
|
|
'pyhomematic',
|
|
'pylitejet',
|
|
'pymonoprice',
|
|
'pynx584',
|
|
'pyopenuv',
|
|
'pyotp',
|
|
'pysonos',
|
|
'pyqwikswitch',
|
|
'PyRMVtransport',
|
|
'PyTransportNSW',
|
|
'pyspcwebgw',
|
|
'python-forecastio',
|
|
'python-nest',
|
|
'python_awair',
|
|
'pytradfri\\[async\\]',
|
|
'pyunifi',
|
|
'pyupnp-async',
|
|
'pywebpush',
|
|
'regenmaschine',
|
|
'restrictedpython',
|
|
'rflink',
|
|
'ring_doorbell',
|
|
'rxv',
|
|
'simplisafe-python',
|
|
'sleepyq',
|
|
'smhi-pkg',
|
|
'somecomfort',
|
|
'sqlalchemy',
|
|
'srpenergy',
|
|
'statsd',
|
|
'uvcclient',
|
|
'vsure',
|
|
'warrant',
|
|
'pythonwhois',
|
|
'wakeonlan',
|
|
'vultr',
|
|
'YesssSMS',
|
|
'ruamel.yaml',
|
|
)
|
|
|
|
IGNORE_PACKAGES = (
|
|
'homeassistant.components.recorder.models',
|
|
'homeassistant.components.homekit.*',
|
|
'homeassistant.components.hangouts.hangups_utils'
|
|
)
|
|
|
|
IGNORE_PIN = ('colorlog>2.1,<3', 'keyring>=9.3,<10.0', 'urllib3')
|
|
|
|
IGNORE_REQ = (
|
|
'colorama<=1', # Windows only requirement in check_config
|
|
)
|
|
|
|
URL_PIN = ('https://developers.home-assistant.io/docs/'
|
|
'creating_platform_code_review.html#1-requirements')
|
|
|
|
|
|
CONSTRAINT_PATH = os.path.join(os.path.dirname(__file__),
|
|
'../homeassistant/package_constraints.txt')
|
|
CONSTRAINT_BASE = """
|
|
pycryptodome>=3.6.6
|
|
|
|
# Breaks Python 3.6 and is not needed for our supported Python versions
|
|
enum34==1000000000.0.0
|
|
|
|
# This is a old unmaintained library and is replaced with pycryptodome
|
|
pycrypto==1000000000.0.0
|
|
|
|
# Contains code to modify Home Assistant to work around our rules
|
|
python-systemair-savecair==1000000000.0.0
|
|
"""
|
|
|
|
|
|
def explore_module(package, explore_children):
|
|
"""Explore the modules."""
|
|
module = importlib.import_module(package)
|
|
|
|
found = []
|
|
|
|
if not hasattr(module, '__path__'):
|
|
return found
|
|
|
|
for _, name, _ in pkgutil.iter_modules(module.__path__, package + '.'):
|
|
found.append(name)
|
|
|
|
if explore_children:
|
|
found.extend(explore_module(name, False))
|
|
|
|
return found
|
|
|
|
|
|
def core_requirements():
|
|
"""Gather core requirements out of setup.py."""
|
|
with open('setup.py') as inp:
|
|
reqs_raw = re.search(
|
|
r'REQUIRES = \[(.*?)\]', inp.read(), re.S).group(1)
|
|
return re.findall(r"'(.*?)'", reqs_raw)
|
|
|
|
|
|
def comment_requirement(req):
|
|
"""Comment out requirement. Some don't install on all systems."""
|
|
return any(ign in req for ign in COMMENT_REQUIREMENTS)
|
|
|
|
|
|
def gather_modules():
|
|
"""Collect the information."""
|
|
reqs = {}
|
|
|
|
errors = []
|
|
|
|
for package in sorted(
|
|
explore_module('homeassistant.components', True) +
|
|
explore_module('homeassistant.scripts', True) +
|
|
explore_module('homeassistant.auth', True)):
|
|
try:
|
|
module = importlib.import_module(package)
|
|
except ImportError:
|
|
for pattern in IGNORE_PACKAGES:
|
|
if fnmatch.fnmatch(package, pattern):
|
|
break
|
|
else:
|
|
errors.append(package)
|
|
continue
|
|
|
|
if not getattr(module, 'REQUIREMENTS', None):
|
|
continue
|
|
|
|
for req in module.REQUIREMENTS:
|
|
if req in IGNORE_REQ:
|
|
continue
|
|
if '://' in req:
|
|
errors.append(
|
|
"{}[Only pypi dependencies are allowed: {}]".format(
|
|
package, req))
|
|
if req.partition('==')[1] == '' and req not in IGNORE_PIN:
|
|
errors.append(
|
|
"{}[Please pin requirement {}, see {}]".format(
|
|
package, req, URL_PIN))
|
|
reqs.setdefault(req, []).append(package)
|
|
|
|
for key in reqs:
|
|
reqs[key] = sorted(reqs[key],
|
|
key=lambda name: (len(name.split('.')), name))
|
|
|
|
if errors:
|
|
print("******* ERROR")
|
|
print("Errors while importing: ", ', '.join(errors))
|
|
print("Make sure you import 3rd party libraries inside methods.")
|
|
return None
|
|
|
|
return reqs
|
|
|
|
|
|
def generate_requirements_list(reqs):
|
|
"""Generate a pip file based on requirements."""
|
|
output = []
|
|
for pkg, requirements in sorted(reqs.items(), key=lambda item: item[0]):
|
|
for req in sorted(requirements,
|
|
key=lambda name: (len(name.split('.')), name)):
|
|
output.append('\n# {}'.format(req))
|
|
|
|
if comment_requirement(pkg):
|
|
output.append('\n# {}\n'.format(pkg))
|
|
else:
|
|
output.append('\n{}\n'.format(pkg))
|
|
return ''.join(output)
|
|
|
|
|
|
def requirements_all_output(reqs):
|
|
"""Generate output for requirements_all."""
|
|
output = []
|
|
output.append('# Home Assistant core')
|
|
output.append('\n')
|
|
output.append('\n'.join(core_requirements()))
|
|
output.append('\n')
|
|
output.append(generate_requirements_list(reqs))
|
|
|
|
return ''.join(output)
|
|
|
|
|
|
def requirements_test_output(reqs):
|
|
"""Generate output for test_requirements."""
|
|
output = []
|
|
output.append('# Home Assistant test')
|
|
output.append('\n')
|
|
with open('requirements_test.txt') as test_file:
|
|
output.append(test_file.read())
|
|
output.append('\n')
|
|
filtered = {key: value for key, value in reqs.items()
|
|
if any(
|
|
re.search(r'(^|#){}($|[=><])'.format(ign),
|
|
key) is not None for ign in TEST_REQUIREMENTS)}
|
|
output.append(generate_requirements_list(filtered))
|
|
|
|
return ''.join(output)
|
|
|
|
|
|
def gather_constraints():
|
|
"""Construct output for constraint file."""
|
|
return '\n'.join(core_requirements() + [''])
|
|
|
|
|
|
def write_requirements_file(data):
|
|
"""Write the modules to the requirements_all.txt."""
|
|
with open('requirements_all.txt', 'w+', newline="\n") as req_file:
|
|
req_file.write(data)
|
|
|
|
|
|
def write_test_requirements_file(data):
|
|
"""Write the modules to the requirements_test_all.txt."""
|
|
with open('requirements_test_all.txt', 'w+', newline="\n") as req_file:
|
|
req_file.write(data)
|
|
|
|
|
|
def write_constraints_file(data):
|
|
"""Write constraints to a file."""
|
|
with open(CONSTRAINT_PATH, 'w+', newline="\n") as req_file:
|
|
req_file.write(data + CONSTRAINT_BASE)
|
|
|
|
|
|
def validate_requirements_file(data):
|
|
"""Validate if requirements_all.txt is up to date."""
|
|
with open('requirements_all.txt', 'r') as req_file:
|
|
return data == req_file.read()
|
|
|
|
|
|
def validate_requirements_test_file(data):
|
|
"""Validate if requirements_test_all.txt is up to date."""
|
|
with open('requirements_test_all.txt', 'r') as req_file:
|
|
return data == req_file.read()
|
|
|
|
|
|
def validate_constraints_file(data):
|
|
"""Validate if constraints is up to date."""
|
|
with open(CONSTRAINT_PATH, 'r') as req_file:
|
|
return data + CONSTRAINT_BASE == req_file.read()
|
|
|
|
|
|
def main(validate):
|
|
"""Run the script."""
|
|
if not os.path.isfile('requirements_all.txt'):
|
|
print('Run this from HA root dir')
|
|
return 1
|
|
|
|
data = gather_modules()
|
|
|
|
if data is None:
|
|
return 1
|
|
|
|
constraints = gather_constraints()
|
|
|
|
reqs_file = requirements_all_output(data)
|
|
reqs_test_file = requirements_test_output(data)
|
|
|
|
if validate:
|
|
errors = []
|
|
if not validate_requirements_file(reqs_file):
|
|
errors.append("requirements_all.txt is not up to date")
|
|
|
|
if not validate_requirements_test_file(reqs_test_file):
|
|
errors.append("requirements_test_all.txt is not up to date")
|
|
|
|
if not validate_constraints_file(constraints):
|
|
errors.append(
|
|
"home-assistant/package_constraints.txt is not up to date")
|
|
|
|
if errors:
|
|
print("******* ERROR")
|
|
print('\n'.join(errors))
|
|
print("Please run script/gen_requirements_all.py")
|
|
return 1
|
|
|
|
return 0
|
|
|
|
write_requirements_file(reqs_file)
|
|
write_test_requirements_file(reqs_test_file)
|
|
write_constraints_file(constraints)
|
|
return 0
|
|
|
|
|
|
if __name__ == '__main__':
|
|
_VAL = sys.argv[-1] == 'validate'
|
|
sys.exit(main(_VAL))
|