Bugfix: removed conf_platform

[ghost]

Description: Bugfix, serial_pm doesn't load with the current code in 0.26

Related issue (if applicable): fixes ERROR:homeassistant.bootstrap:Invalid config for [sensor.serial_pm]: required key not provided @ data['platform']. Got None

Checklist:

If user exposed functionality or configuration variables are added/changed:

• Documentation added/updated in home-assistant.io

If code communicates with devices, web services, or a:

• Local tests with tox run successfully. Your PR cannot be merged unless tests pass
• New dependencies have been added to the REQUIREMENTS variable ([example][ex-requir]).
• New dependencies are only imported inside functions that use them ([example][ex-import]).
• New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
• New files were added to .coveragerc.

If the code does not interact with devices:

• Local tests with tox run successfully. Your PR cannot be merged unless tests pass
• Tests have been added to verify that the new code works.

[balloob]

I'm confused, why doesn't this work ? It should work. platform should always be set to serial_pm, or else this platform would have never been loaded.

[ghost]

I'm also confused. It worked here before I installed 0.26
With the 0.26 release, I get the following error message:
ERROR:homeassistant.bootstrap:Invalid config for [sensor.serial_pm]: required key not provided @ data['platform']. Got None

When I removed the CONF_PLATFORM as a required parameter, it loads the sensor (platform is still given in the configuration)
I still don't understand the internal not good enough to say what might be the problem with this.

[balloob]

You can try adding a print(p_validated) before this line; https://github.com/home-assistant/home-assistant/blob/dev/homeassistant/bootstrap.py#L135

[ghost]

Looks good:
{'name': 'SDS21', 'serial_device': '/dev/ttyUSB0', 'brand': 'novafitness,sds021', 'platform': 'serial_pm'}

[balloob]

Managed a repro:

› python3
Python 3.4.3 (default, Aug 11 2015, 08:57:25)
[GCC 4.2.1 Compatible Apple LLVM 6.1.0 (clang-602.0.53)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> from homeassistant.components.sensor.serial_pm import PLATFORM_SCHEMA
>>> PLATFORM_SCHEMA({'name': 'SDS21', 'serial_device': '/dev/ttyUSB0', 'brand': 'novafitness,sds021', 'platform': 'serial_pm'})
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/paulus/dev/python/home-assistant/lib/python3.4/site-packages/voluptuous-0.9.2-py3.4.egg/voluptuous/schema_builder.py", line 192, in __call__
  File "/Users/paulus/dev/python/home-assistant/lib/python3.4/site-packages/voluptuous-0.9.2-py3.4.egg/voluptuous/schema_builder.py", line 486, in validate_dict
  File "/Users/paulus/dev/python/home-assistant/lib/python3.4/site-packages/voluptuous-0.9.2-py3.4.egg/voluptuous/schema_builder.py", line 324, in validate_mapping
voluptuous.error.MultipleInvalid: required key not provided @ data['platform']

[balloob]

I think we've hit a bug in voluptuous.

import voluptuous as vol

schema1 = vol.Schema({vol.Required('platform'): str})
schema2 = schema1.extend({vol.Required('platform'): 'serial_pm'})

data = {'platform': 'serial_pm'}

print('schema1')
print(schema1(data))
print('schema2')
print(schema2(data))

results in the output:

› python3 pm_test.py
schema1
{'platform': 'serial_pm'}
schema2
Traceback (most recent call last):
  File "pm_test.py", line 20, in <module>
    print(schema2(data))
  File "/Users/paulus/dev/python/home-assistant/lib/python3.4/site-packages/voluptuous-0.9.2-py3.4.egg/voluptuous/schema_builder.py", line 192, in __call__
  File "/Users/paulus/dev/python/home-assistant/lib/python3.4/site-packages/voluptuous-0.9.2-py3.4.egg/voluptuous/schema_builder.py", line 486, in validate_dict
  File "/Users/paulus/dev/python/home-assistant/lib/python3.4/site-packages/voluptuous-0.9.2-py3.4.egg/voluptuous/schema_builder.py", line 324, in validate_mapping
voluptuous.error.MultipleInvalid: required key not provided @ data['platform']

[balloob]

I'll merge this and will include it in a hotfix if we end up doing one later this weekend.

Reported the bug here: alecthomas/voluptuous#192