Fix branch issue

brother/__init__.py

@@ -14,7 +14,7 @@
 
 _LOGGER = logging.getLogger(__name__)
 
-REGEX_MODEL_PATTERN = re.compile(r"MODEL=\"(?P<model>[\w\-]+)( series)?\"")
+REGEX_MODEL_PATTERN = re.compile(r"MDL:(?P<model>[\w\-]+)")
 
 
 class Brother:  # pylint:disable=too-many-instance-attributes
@@ -55,13 +55,16 @@ async def async_update(self):
             self.model = re.search(
                 REGEX_MODEL_PATTERN, raw_data[OIDS[ATTR_MODEL]]
             ).group("model")
+            data[ATTR_MODEL] = self.model
             self.serial = raw_data[OIDS[ATTR_SERIAL]]
+            data[ATTR_SERIAL] = self.serial
         except (TypeError, AttributeError):
             raise UnsupportedModel(
                 "It seems that this printer model is not supported. Sorry."
             )
         try:
             self.firmware = raw_data[OIDS[ATTR_FIRMWARE]]
+            data[ATTR_FIRMWARE] = self.firmware
             code_page = chardet.detect(raw_data[OIDS[ATTR_STATUS]].encode("latin1"))[
                 "encoding"
             ]

brother/const.py

@@ -72,7 +72,7 @@
     ATTR_COUNTERS: "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.5.10.0",
     ATTR_FIRMWARE: "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.5.17.0",
     ATTR_MAINTENANCE: "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.5.8.0",
-    ATTR_MODEL: "1.3.6.1.4.1.2435.2.4.3.99.3.1.6.1.2.1",
+    ATTR_MODEL: "1.3.6.1.4.1.2435.2.3.9.1.1.7.0",
     ATTR_NEXTCARE: "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.5.11.0",
     ATTR_PAGE_COUNT: "1.3.6.1.2.1.43.10.2.1.4.1.1",
     ATTR_SERIAL: "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.5.1.0",

pytest.ini

@@ -1,2 +1,2 @@
 [pytest]
-addopts=--cov --cov-report term-missing
+addopts=--cov --cov-report term-missing --disable-pytest-warnings

tests/data/dcp-9020cdw.json

@@ -1,5 +1,5 @@
 {
-  "1.3.6.1.4.1.2435.2.4.3.99.3.1.6.1.2.1": "MODEL=\"DCP-9020CDW series\"",
+  "1.3.6.1.4.1.2435.2.3.9.1.1.7.0": "MFG:Brother;CMD:PJL,HBP,URF;MDL:DCP-9020CDW series;CLS:PRINTER;CID:Brother Laser Type1;",
   "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.5.10.0": [
     "0001040000134b",
     "02010400000c69",

tests/data/dcp-j132w.json

@@ -1,7 +1,7 @@
 {
   "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.5.1.0": "serial_number",
   "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.4.5.2.0": "TRYB U¦PIENIA      ",
-  "1.3.6.1.4.1.2435.2.4.3.99.3.1.6.1.2.1": "MODEL=\"DCP-J132W series\"",
+  "1.3.6.1.4.1.2435.2.3.9.1.1.7.0": "MFG:Brother;CMD:PJL,HBP,URF;MDL:DCP-J132W;CLS:PRINTER;",
   "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.5.17.0": "Q1906110144",
   "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.5.10.0": ["0001040000036f"],
   "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.5.8.0": [

tests/data/dcp-l2540dn.json

@@ -10,7 +10,7 @@
     "8101040000003c",
     "86010400000010"
   ],
-  "1.3.6.1.4.1.2435.2.4.3.99.3.1.6.1.2.1": "MODEL=\"DCP-L2540DN series\"",
+  "1.3.6.1.4.1.2435.2.3.9.1.1.7.0": "MFG:Brother;CMD:PJL,HBP,URF;MDL:DCP-L2540DN series;CLS:PRINTER;CID:Brother Laser Type1;",
   "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.5.11.0": ["82010400002d93"],
   "1.3.6.1.2.1.43.10.2.1.4.1.1": "333",
   "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.5.1.0": "serial_number",

tests/data/dcp-l3550cdw.json

@@ -1,7 +1,7 @@
 {
   "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.5.1.0": "serial_number",
   "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.4.5.2.0": "Ma³o toneru (Y) ",
-  "1.3.6.1.4.1.2435.2.4.3.99.3.1.6.1.2.1": "MODEL=\"DCP-L3550CDW series\"",
+  "1.3.6.1.4.1.2435.2.3.9.1.1.7.0": "MFG:Brother;CMD:PJL,HBP,URF;MDL:DCP-L3550CDW;CLS:PRINTER;CID:Brother Laser Type1;",
   "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.5.17.0": "J1906051424",
   "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.5.10.0": [
     "0001040000064b",

tests/data/hl-l2340dw.json

@@ -11,7 +11,7 @@
     "81010400000050",
     "8601040000000a"
   ],
-  "1.3.6.1.4.1.2435.2.4.3.99.3.1.6.1.2.1": "MODEL=\"HL-L2340DW\"",
+  "1.3.6.1.4.1.2435.2.3.9.1.1.7.0": "MFG:Brother;CMD:PJL,HBP,URF;MDL:HL-L2340DW series;CLS:PRINTER;CID:Brother Laser Type1;URF:W8,CP1,IS4-1,MT1-3-4-5-8,OB10,PQ4,RS300-600,V1.3,DM1;",
   "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.5.11.0": ["82010400002b06"],
   "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.5.1.0": "serial_number",
   "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.4.5.2.0": "OCZEKIWANIE     "

tests/data/incomplete.json

@@ -1,7 +1,7 @@
 {
   "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.5.1.0": "serial_number",
   "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.4.5.2.0": null,
-  "1.3.6.1.4.1.2435.2.4.3.99.3.1.6.1.2.1": "MODEL=\"DCP-L3550CDW series\"",
+  "1.3.6.1.4.1.2435.2.3.9.1.1.7.0": "MFG:Brother;CMD:PJL,HBP,URF;MDL:DCP-L3550CDW;CLS:PRINTER;",
   "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.5.17.0": null,
   "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.5.10.0": [],
   "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.5.8.0": [

tests/data/invalid.json

@@ -1,7 +1,7 @@
 {
   "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.5.1.0": null,
   "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.4.5.2.0": null,
-  "1.3.6.1.4.1.2435.2.4.3.99.3.1.6.1.2.1": null,
+  "1.3.6.1.4.1.2435.2.3.9.1.1.7.0": null,
   "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.5.17.0": null,
   "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.5.10.0": [],
   "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.5.8.0": [],

tests/data/mfc-j680dw.json

@@ -2,7 +2,7 @@
     "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.5.10.0": ["010104000001cd", "020104000001eb"],
     "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.5.17.0": "U1804191714VER.J",
     "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.5.8.0": ["31010400000001", "32010400000001", "33010400000001", "34010400000001", "6f01040000125c", "70010400000dac", "71010400002134", "72010400001194", "8101040000002f", "82010400000023", "83010400000055", "8401040000002d", "86010400000012", "87010400000012", "88010400000012", "89010400000012"],
-    "1.3.6.1.4.1.2435.2.4.3.99.3.1.6.1.2.1": "MODEL=\"MFC-J680DW\"",
+    "1.3.6.1.4.1.2435.2.3.9.1.1.7.0": "MFG:Brother;CMD:PJL,HBP,URF;MDL:MFC-J680DW;CLS:PRINTER;",
     "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.5.11.0": [],
     "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.5.1.0": "serial_number",
     "1.3.6.1.4.1.2435.2.3.9.4.2.1.5.4.5.2.0": "Ready"

tests/test_base.py

@@ -39,7 +39,7 @@ async def test_dcp_l3550cdw_model():
 
     with patch("brother.Brother._get_data", return_value=data):
 
-        brother = Brother(INVALID_HOST)
+        brother = Brother(HOST)
         await brother.async_update()
 
         assert brother.available == True