module: use yapf formatter, reduce pylint warnings

.pylintrc

@@ -1,560 +1,9 @@
-[MASTER]
-
-# A comma-separated list of package or module names from where C extensions may
-# be loaded. Extensions are loading into the active Python interpreter and may
-# run arbitrary code
-extension-pkg-whitelist=
-
-# Add files or directories to the blacklist. They should be base names, not
-# paths.
-ignore=CVS
-
-# Add files or directories matching the regex patterns to the blacklist. The
-# regex matches against base names, not paths.
-ignore-patterns=.*_test.py,setup.py
-
-# Python code to execute, usually for sys.path manipulation such as
-# pygtk.require().
-#init-hook=
-
-# Use multiple processes to speed up Pylint.
-jobs=1
-
-# List of plugins (as comma separated values of python modules names) to load,
-# usually to register additional checkers.
-load-plugins=
-
-# Pickle collected data for later comparisons.
-persistent=no
-
-# Specify a configuration file.
-#rcfile=
-
-# When enabled, pylint would attempt to guess common misconfiguration and emit
-# user-friendly hints instead of false-positive error messages
-suggestion-mode=yes
-
-# Allow loading of arbitrary C extensions. Extensions are imported into the
-# active Python interpreter and may run arbitrary code.
-unsafe-load-any-extension=no
-
-
-[MESSAGES CONTROL]
-
-# Only show warnings with the listed confidence levels. Leave empty to show
-# all. Valid levels: HIGH, INFERENCE, INFERENCE_FAILURE, UNDEFINED
-confidence=
-
-# Disable the message, report, category or checker with the given id(s). You
-# can either give multiple identifiers separated by comma (,) or put this
-# option multiple times (only on the command line, not in the configuration
-# file where it should appear only once).You can also use "--disable=all" to
-# disable everything first and then reenable specific checks. For example, if
-# you want to run only the similarities checker, you can use "--disable=all
-# --enable=similarities". If you want to run only the classes checker, but have
-# no Warning level messages displayed, use"--disable=all --enable=classes
-# --disable=W"
-disable=print-statement,
-        parameter-unpacking,
-        unpacking-in-except,
-        old-raise-syntax,
-        backtick,
-        long-suffix,
-        old-ne-operator,
-        old-octal-literal,
-        import-star-module-level,
-        non-ascii-bytes-literal,
-        raw-checker-failed,
-        bad-inline-option,
-        locally-disabled,
-        locally-enabled,
-        file-ignored,
-        suppressed-message,
-        useless-suppression,
-        deprecated-pragma,
-        apply-builtin,
-        basestring-builtin,
-        buffer-builtin,
-        cmp-builtin,
-        coerce-builtin,
-        execfile-builtin,
-        file-builtin,
-        long-builtin,
-        raw_input-builtin,
-        reduce-builtin,
-        standarderror-builtin,
-        unicode-builtin,
-        xrange-builtin,
-        coerce-method,
-        delslice-method,
-        getslice-method,
-        setslice-method,
-        no-absolute-import,
-        old-division,
-        dict-iter-method,
-        dict-view-method,
-        next-method-called,
-        metaclass-assignment,
-        indexing-exception,
-        raising-string,
-        reload-builtin,
-        oct-method,
-        hex-method,
-        nonzero-method,
-        cmp-method,
-        input-builtin,
-        round-builtin,
-        intern-builtin,
-        unichr-builtin,
-        map-builtin-not-iterating,
-        zip-builtin-not-iterating,
-        range-builtin-not-iterating,
-        filter-builtin-not-iterating,
-        using-cmp-argument,
-        eq-without-hash,
-        div-method,
-        idiv-method,
-        rdiv-method,
-        exception-message-attribute,
-        invalid-str-codec,
-        sys-max-int,
-        bad-python3-import,
-        deprecated-string-function,
-        deprecated-str-translate-call,
-        deprecated-itertools-function,
-        deprecated-types-field,
-        next-method-defined,
-        dict-items-not-iterating,
-        dict-keys-not-iterating,
-        dict-values-not-iterating,
-        missing-docstring,
-        line-too-long,
-        c-extension-no-member,
-        no-self-use
-
-# Enable the message, report, category or checker with the given id(s). You can
-# either give multiple identifier separated by comma (,) or put this option
-# multiple time (only on the command line, not in the configuration file where
-# it should appear only once). See also the "--disable" option for examples.
-enable=
-
-
-[REPORTS]
-
-# Python expression which should return a note less than 10 (10 is the highest
-# note). You have access to the variables errors warning, statement which
-# respectively contain the number of errors / warnings messages and the total
-# number of statements analyzed. This is used by the global evaluation report
-# (RP0004).
-evaluation=10.0 - ((float(5 * error + warning + refactor + convention) / statement) * 10)
-
-# Template used to display messages. This is a python new-style format string
-# used to format the message information. See doc for all details
-#msg-template=
-
-# Set the output format. Available formats are text, parseable, colorized, json
-# and msvs (visual studio).You can also give a reporter class, eg
-# mypackage.mymodule.MyReporterClass.
-output-format=colorized
-
-# Tells whether to display a full report or only the messages
-reports=no
-
-# Activate the evaluation score.
-score=yes
-
-
-[REFACTORING]
-
-# Maximum number of nested blocks for function / method body
-max-nested-blocks=5
-
-# Complete name of functions that never returns. When checking for
-# inconsistent-return-statements if a never returning function is called then
-# it will be considered as an explicit return statement and no message will be
-# printed.
-never-returning-functions=optparse.Values,sys.exit
-
-
-[BASIC]
-
-# Naming style matching correct argument names
-argument-naming-style=snake_case
-
-# Regular expression matching correct argument names. Overrides argument-
-# naming-style
-#argument-rgx=
-
-# Naming style matching correct attribute names
-attr-naming-style=snake_case
-
-# Regular expression matching correct attribute names. Overrides attr-naming-
-# style
-#attr-rgx=
-
-# Bad variable names which should always be refused, separated by a comma
-bad-names=foo,
-          bar,
-          baz,
-          toto,
-          tutu,
-          tata
-
-# Naming style matching correct class attribute names
-class-attribute-naming-style=any
-
-# Regular expression matching correct class attribute names. Overrides class-
-# attribute-naming-style
-#class-attribute-rgx=
-
-# Naming style matching correct class names
-class-naming-style=PascalCase
-
-# Regular expression matching correct class names. Overrides class-naming-style
-#class-rgx=
-
-# Naming style matching correct constant names
-const-naming-style=UPPER_CASE
-
-# Regular expression matching correct constant names. Overrides const-naming-
-# style
-#const-rgx=
-
-# Minimum line length for functions/classes that require docstrings, shorter
-# ones are exempt.
-docstring-min-length=-1
-
-# Naming style matching correct function names
-function-naming-style=snake_case
-
-# Regular expression matching correct function names. Overrides function-
-# naming-style
-#function-rgx=
-
-# Good variable names which should always be accepted, separated by a comma
-good-names=i,
-           j,
-           k,
-           ex,
-           Run,
-           _,
-           ho,
-           po,
-           to,
-           t,
-           p,
-           h,
-           digT,
-           digP,
-           digH,
-           adc_T,
-           adc_P,
-           adc_H,
-           var_H,
-           is_status_image_register_updating
-
-# Include a hint for the correct naming format with invalid-name
-include-naming-hint=no
-
-# Naming style matching correct inline iteration names
-inlinevar-naming-style=any
-
-# Regular expression matching correct inline iteration names. Overrides
-# inlinevar-naming-style
-#inlinevar-rgx=
-
-# Naming style matching correct method names
-method-naming-style=snake_case
-
-# Regular expression matching correct method names. Overrides method-naming-
-# style
-#method-rgx=
-
-# Naming style matching correct module names
-module-naming-style=snake_case
-
-# Regular expression matching correct module names. Overrides module-naming-
-# style
-#module-rgx=
-
-# Colon-delimited sets of names that determine each other's naming style when
-# the name regexes allow several styles.
-name-group=
-
-# Regular expression which should only match function or class names that do
-# not require a docstring.
-no-docstring-rgx=^_
-
-# List of decorators that produce properties, such as abc.abstractproperty. Add
-# to this list to register other decorators that produce valid properties.
-property-classes=abc.abstractproperty
-
-# Naming style matching correct variable names
-variable-naming-style=snake_case
-
-# Regular expression matching correct variable names. Overrides variable-
-# naming-style
-#variable-rgx=
-
-
-[FORMAT]
-
-# Expected format of line ending, e.g. empty (any line ending), LF or CRLF.
-expected-line-ending-format=
-
-# Regexp for a line that is allowed to be longer than the limit.
-ignore-long-lines=^\s*(# )?<?https?://\S+>?$
-
-# Number of spaces of indent required inside a hanging  or continued line.
-indent-after-paren=4
-
-# String used as indentation unit. This is usually "    " (4 spaces) or "\t" (1
-# tab).
-indent-string='    '
-
-# Maximum number of characters on a single line.
-max-line-length=100
-
-# Maximum number of lines in a module
-max-module-lines=1000
-
-# List of optional constructs for which whitespace checking is disabled. `dict-
-# separator` is used to allow tabulation in dicts, etc.: {1  : 1,\n222: 2}.
-# `trailing-comma` allows a space between comma and closing bracket: (a, ).
-# `empty-line` allows space-only lines.
-no-space-check=trailing-comma,
-               dict-separator
-
-# Allow the body of a class to be on the same line as the declaration if body
-# contains single statement.
-single-line-class-stmt=no
-
-# Allow the body of an if to be on the same line as the test if there is no
-# else.
-single-line-if-stmt=no
-
-
-[MISCELLANEOUS]
-
-# List of note tags to take in consideration, separated by a comma.
-notes=FIXME,
-      XXX,
-      TODO
-
-
-[LOGGING]
-
-# Logging modules to check that the string format arguments are in logging
-# function parameter format
-logging-modules=logging
-
-
-[SIMILARITIES]
-
-# Ignore comments when computing similarities.
-ignore-comments=yes
-
-# Ignore docstrings when computing similarities.
-ignore-docstrings=yes
-
-# Ignore imports when computing similarities.
-ignore-imports=no
-
-# Minimum lines number of a similarity.
-min-similarity-lines=4
-
-
-[SPELLING]
-
-# Limits count of emitted suggestions for spelling mistakes
-max-spelling-suggestions=4
-
-# Spelling dictionary name. Available dictionaries: none. To make it working
-# install python-enchant package.
-spelling-dict=
-
-# List of comma separated words that should not be checked.
-spelling-ignore-words=
-
-# A path to a file that contains private dictionary; one word per line.
-spelling-private-dict-file=
-
-# Tells whether to store unknown words to indicated private dictionary in
-# --spelling-private-dict-file option instead of raising a message.
-spelling-store-unknown-words=no
-
-
-[TYPECHECK]
-
-# List of decorators that produce context managers, such as
-# contextlib.contextmanager. Add to this list to register other decorators that
-# produce valid context managers.
-contextmanager-decorators=contextlib.contextmanager
-
-# List of members which are set dynamically and missed by pylint inference
-# system, and so shouldn't trigger E1101 when accessed. Python regular
-# expressions are accepted.
-generated-members=
-
-# Tells whether missing members accessed in mixin class should be ignored. A
-# mixin class is detected if its name ends with "mixin" (case insensitive).
-ignore-mixin-members=yes
-
-# This flag controls whether pylint should warn about no-member and similar
-# checks whenever an opaque object is returned when inferring. The inference
-# can return multiple potential results while evaluating a Python object, but
-# some branches might not be evaluated, which results in partial inference. In
-# that case, it might be useful to still emit no-member and other checks for
-# the rest of the inferred objects.
-ignore-on-opaque-inference=yes
-
-# List of class names for which member attributes should not be checked (useful
-# for classes with dynamically set attributes). This supports the use of
-# qualified names.
-ignored-classes=optparse.Values,thread._local,_thread._local
-
-# List of module names for which member attributes should not be checked
-# (useful for modules/projects where namespaces are manipulated during runtime
-# and thus existing member attributes cannot be deduced by static analysis. It
-# supports qualified module names, as well as Unix pattern matching.
-ignored-modules=
-
-# Show a hint with possible names when a member name was not found. The aspect
-# of finding the hint is based on edit distance.
-missing-member-hint=yes
-
-# The minimum edit distance a name should have in order to be considered a
-# similar match for a missing member name.
-missing-member-hint-distance=1
-
-# The total number of similar names that should be taken in consideration when
-# showing a hint for a missing member.
-missing-member-max-choices=1
-
-
-[VARIABLES]
-
-# List of additional names supposed to be defined in builtins. Remember that
-# you should avoid to define new builtins when possible.
-additional-builtins=
-
-# Tells whether unused global variables should be treated as a violation.
-allow-global-unused-variables=yes
-
-# List of strings which can identify a callback function by name. A callback
-# name must start or end with one of those strings.
-callbacks=cb_,
-          _cb
-
-# A regular expression matching the name of dummy variables (i.e. expectedly
-# not used).
-dummy-variables-rgx=_+$|(_[a-zA-Z0-9_]*[a-zA-Z0-9]+?$)|dummy|^ignored_|^unused_
-
-# Argument names that match this expression will be ignored. Default to name
-# with leading underscore
-ignored-argument-names=_.*|^ignored_|^unused_
-
-# Tells whether we should check for unused import in __init__ files.
-init-import=no
-
-# List of qualified module names which can have objects that can redefine
-# builtins.
-redefining-builtins-modules=six.moves,past.builtins,future.builtins
-
-
-[IMPORTS]
-
-# Allow wildcard imports from modules that define __all__.
-allow-wildcard-with-all=no
-
-# Analyse import fallback blocks. This can be used to support both Python 2 and
-# 3 compatible code, which means that the block might have code that exists
-# only in one or another interpreter, leading to false positives when analysed.
-analyse-fallback-blocks=no
-
-# Deprecated modules which should not be used, separated by a comma
-deprecated-modules=regsub,
-                   TERMIOS,
-                   Bastion,
-                   rexec
-
-# Create a graph of external dependencies in the given file (report RP0402 must
-# not be disabled)
-ext-import-graph=
-
-# Create a graph of every (i.e. internal and external) dependencies in the
-# given file (report RP0402 must not be disabled)
-import-graph=
-
-# Create a graph of internal dependencies in the given file (report RP0402 must
-# not be disabled)
-int-import-graph=
-
-# Force import order to recognize a module as part of the standard
-# compatibility libraries.
-known-standard-library=
-
-# Force import order to recognize a module as part of a third party library.
-known-third-party=enchant
-
-
-[CLASSES]
-
-# List of method names used to declare (i.e. assign) instance attributes.
-defining-attr-methods=__init__,
-                      __new__,
-                      setUp
-
-# List of member names, which should be excluded from the protected access
-# warning.
-exclude-protected=_asdict,
-                  _fields,
-                  _replace,
-                  _source,
-                  _make
-
-# List of valid names for the first argument in a class method.
-valid-classmethod-first-arg=cls
-
-# List of valid names for the first argument in a metaclass class method.
-valid-metaclass-classmethod-first-arg=mcs
-
-
 [DESIGN]
-
-# Maximum number of arguments for function / method
-max-args=5
-
-# Maximum number of attributes for a class (see R0902).
 max-attributes=11
-
-# Maximum number of boolean expressions in a if statement
-max-bool-expr=5
-
-# Maximum number of branch for function / method body
-max-branches=12
-
-# Maximum number of locals for function / method body
-max-locals=15
-
-# Maximum number of parents for a class (see R0901).
-max-parents=7
-
-# Maximum number of public methods for a class (see R0904).
 max-public-methods=25
 
-# Maximum number of return / yield for function / method body
+[FORMAT]
-max-returns=6
+max-line-length=121
 
-# Maximum number of statements in function / method body
+[MESSAGES CONTROL]
-max-statements=50
+disable=invalid-name,missing-docstring
-
-# Minimum number of public methods for a class (see R0903).
-min-public-methods=2
-
-
-[EXCEPTIONS]
-
-# Exceptions that will emit a warning when being caught. Defaults to
-# "Exception"
-overgeneral-exceptions=Exception

.style.yapf

@@ -0,0 +1,3 @@
+[style]
+based_on_style = pep8
+column_limit = 120

Makefile

@@ -13,7 +13,8 @@ all: lint test ## Run lint and test (default goal)
 
 .PHONY: lint
 lint: ## Lint all source code
-	poetry run pylint bme280
+	poetry run yapf -q -r bme280
+	poetry run pylint --ignore=snapshots bme280 tests
 
 .PHONY: test
 test: ## Run all tests

bme280/__init__.py

@@ -1,6 +1,8 @@
-
 from .bme280 import Bme280
-from .bme280 import HO_SKIPPED, HO_1, HO_2, HO_4, HO_8, HO_16, PO_SKIPPED, PO_1, PO_2, PO_4, PO_8, PO_16, TO_SKIPPED, TO_1, TO_2, TO_4, TO_8, TO_16
+from .bme280 import HO_SKIPPED, HO_1, HO_2, HO_4, HO_8, HO_16
+from .bme280 import PO_SKIPPED, PO_1, PO_2, PO_4, PO_8, PO_16
+from .bme280 import TO_SKIPPED, TO_1, TO_2, TO_4, TO_8, TO_16
 from .bme280 import MODE_SLEEP, MODE_FORCED, MODE_NORMAL
-from .bme280 import TSTANDBY_0_5, TSTANDBY_62_5, TSTANDBY_125, TSTANDBY_250, TSTANDBY_500, TSTANDBY_1000, TSTANDBY_10, TSTANDBY_20
+from .bme280 import TSTANDBY_0_5, TSTANDBY_62_5, TSTANDBY_125, TSTANDBY_250, TSTANDBY_500, TSTANDBY_1000
+from .bme280 import TSTANDBY_10, TSTANDBY_20
 from .bme280 import FILTER_OFF, FILTER_2, FILTER_4, FILTER_8, FILTER_16

bme280/bme280.py

@@ -1,4 +1,3 @@
-
 import time
 
 import smbus  # pylint: disable=import-error
@@ -52,8 +51,8 @@ FILTER_4 = 0x02
 FILTER_8 = 0x03
 FILTER_16 = 0x04  # and all higher
 
-class Bme280(object): # pylint: disable=bad-option-value,useless-object-inheritance
 
+class Bme280(object):  # pylint: disable=bad-option-value,useless-object-inheritance
     def __init__(self, i2c_bus=1, sensor_address=ADDR):
         self.bus = smbus.SMBus(i2c_bus)
         self.sensor_address = sensor_address
@@ -233,7 +232,8 @@ class Bme280(object): # pylint: disable=bad-option-value,useless-object-inherita
         var2 = (adc_T / 131072.0 - self.digT[0] / 8192.0) * (adc_T / 131072.0 - self.digT[0] / 8192.0) * self.digT[2]
         return var1 + var2
 
-    def calc_compensated_temperature(self, t_fine):
+    @staticmethod
+    def calc_compensated_temperature(t_fine):
         return t_fine / 5120.0
 
     def calc_compensated_pressure(self, t_fine, adc_P):
@@ -253,7 +253,10 @@ class Bme280(object): # pylint: disable=bad-option-value,useless-object-inherita
 
     def calc_compensated_humidity(self, t_fine, adc_H):
         var_H = t_fine - 76800.0
+        # yapf: disable
+        # pylint: disable=line-too-long
         var_H = (adc_H - (self.digH[3] * 64.0 + self.digH[4] / 16384.0 * var_H)) * (self.digH[1] / 65536.0 * (1.0 + self.digH[5] / 67108864.0 * var_H * (1.0 + self.digH[2] / 67108864.0 * var_H)))
+        # yapf: enable
         var_H = var_H * (1.0 - self.digH[0] * var_H / 524288.0)
         if var_H > 100.0:
             var_H = 100.0

demo.py

@@ -1,12 +1,16 @@
 #!/usr/bin/env python
-# -*- coding: UTF-8 -*-
 
 import bme280
 
-if __name__ == '__main__':
+
+def main():
     bme = bme280.Bme280()
     bme.set_mode(bme280.MODE_FORCED)
     t, p, h = bme.get_data()
-    print "Temperature: %f °C" % t
+    print("Temperature: %f °C" % t)
-    print "Pressure: %f P" % p
+    print("Pressure: %f P" % p)
-    print "Humidity: %f %%" % h
+    print("Humidity: %f %%" % h)
+
+
+if __name__ == '__main__':
+    main()

tests/bme280_test.py

@@ -1,4 +1,3 @@
-
 # import unittest
 import sys
 import time
@@ -11,10 +10,11 @@ import snapshottest
 MOCKED_SMBUS_MODULE = mock.Mock()
 sys.modules['smbus'] = MOCKED_SMBUS_MODULE
 time.sleep = lambda s: None
-import bme280
+import bme280  # pylint: disable=wrong-import-position
+
 
 # inspired by https://github.com/adafruit/Adafruit_Python_GPIO/blob/master/Adafruit_GPIO/I2C.py
-class MockSMBus(object):
+class MockSMBus(object):  # pylint: disable=bad-option-value,useless-object-inheritance
     def __init__(self, initial_read=None):
         self._log = []
         self.initial_read = initial_read or {}
@@ -37,45 +37,40 @@ class MockSMBus(object):
         self._log.append(('w', addr, cmd))
         self._log.append(('w', addr, val))
 
+
 def setup_mockbus(**kwargs):
     mockbus = MockSMBus(**kwargs)
     MOCKED_SMBUS_MODULE.SMBus.return_value = mockbus
     return mockbus
 
+
 def calibration_read():
     return [0x00] * (24 + 1 + 7)
 
-class TestBme280(snapshottest.TestCase):
 
+# pylint: disable=protected-access
+class TestBme280(snapshottest.TestCase):
     def test_setup(self):
-        mockbus = setup_mockbus(initial_read={
+        mockbus = setup_mockbus(initial_read={0x76: calibration_read()})
-            0x76: calibration_read()
-        })
         bme = bme280.Bme280()
         MOCKED_SMBUS_MODULE.SMBus.assert_called_with(1)
         self.assertIsNotNone(bme)
         self.assertMatchSnapshot(mockbus._log)
 
     def test_chip_id(self):
-        mockbus = setup_mockbus(initial_read={
+        mockbus = setup_mockbus(initial_read={0x76: calibration_read() + [0x23]})
-            0x76: calibration_read() + [0x23]
-        })
         bme = bme280.Bme280()
         self.assertEqual(bme.get_chip_id(), 0x23)
         self.assertMatchSnapshot(mockbus._log)
 
     def test_reset(self):
-        mockbus = setup_mockbus(initial_read={
+        mockbus = setup_mockbus(initial_read={0x76: calibration_read()})
-            0x76: calibration_read()
-        })
         bme = bme280.Bme280()
         bme.reset()
         self.assertMatchSnapshot(mockbus._log)
 
     def test_status(self):
-        mockbus = setup_mockbus(initial_read={
+        mockbus = setup_mockbus(initial_read={0x76: calibration_read() + [0x08, 0x00, 0x01, 0x00]})
-            0x76: calibration_read() + [0x08, 0x00, 0x01, 0x00]
-        })
         bme = bme280.Bme280()
         self.assertTrue(bme.is_status_measuring())
         self.assertFalse(bme.is_status_measuring())
@@ -84,9 +79,7 @@ class TestBme280(snapshottest.TestCase):
         self.assertMatchSnapshot(mockbus._log)
 
     def test_oversampling(self):
-        mockbus = setup_mockbus(initial_read={
+        mockbus = setup_mockbus(initial_read={0x76: calibration_read()})
-            0x76: calibration_read()
-        })
         bme = bme280.Bme280()
         self.assertEqual(bme.get_humidity_oversampling(), bme280.HO_1)
         bme.set_humidity_oversampling(bme280.HO_16)
@@ -100,9 +93,7 @@ class TestBme280(snapshottest.TestCase):
         self.assertMatchSnapshot(mockbus._log)
 
     def test_mode(self):
-        mockbus = setup_mockbus(initial_read={
+        mockbus = setup_mockbus(initial_read={0x76: calibration_read()})
-            0x76: calibration_read()
-        })
         bme = bme280.Bme280()
         self.assertEqual(bme.get_mode(), bme280.MODE_SLEEP)
         bme.set_mode(bme280.MODE_FORCED)
@@ -110,9 +101,7 @@ class TestBme280(snapshottest.TestCase):
         self.assertMatchSnapshot(mockbus._log)
 
     def test_config(self):
-        mockbus = setup_mockbus(initial_read={
+        mockbus = setup_mockbus(initial_read={0x76: calibration_read()})
-            0x76: calibration_read()
-        })
         bme = bme280.Bme280()
         self.assertEqual(bme.get_tstandy(), bme280.TSTANDBY_1000)
         bme.set_tstandy(bme280.TSTANDBY_20)
@@ -123,11 +112,9 @@ class TestBme280(snapshottest.TestCase):
         self.assertMatchSnapshot(mockbus._log)
 
     def test_data(self):
-        mockbus = setup_mockbus(initial_read={
+        mockbus = setup_mockbus(initial_read={0x76: calibration_read() + [0x00] * 8})
-            0x76: calibration_read() + [0x00] * 8
-        })
         bme = bme280.Bme280()
         bme.set_mode(bme280.MODE_FORCED)
-        t, p, h = bme.get_data()
+        t, p, h = bme.get_data()  # pylint: disable=unused-variable
         # self.assertEqual(t, 0x00)
         self.assertMatchSnapshot(mockbus._log)