Skip to content

Commit e4333aa

Browse files
Jaspal SinghJaspal Singh
authored andcommitted
kernel and doc py 3.9 fixes
1 parent 759f425 commit e4333aa

File tree

4 files changed

+12
-7
lines changed

4 files changed

+12
-7
lines changed

docs/node-scraper-external/ext_nodescraper_plugins/sample/sample_collector.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
from typing import Optional
2+
13
from nodescraper.base import InBandDataCollector
24
from nodescraper.enums import ExecutionStatus
35
from nodescraper.models import TaskResult
@@ -9,7 +11,7 @@ class SampleCollector(InBandDataCollector[SampleDataModel, None]):
911

1012
DATA_MODEL = SampleDataModel
1113

12-
def collect_data(self, args=None) -> tuple[TaskResult, SampleDataModel | None]:
14+
def collect_data(self, args=None) -> tuple[TaskResult, Optional[SampleDataModel]]:
1315
sample_data = SampleDataModel(some_str="example123")
1416
self.result.message = "Collector ran successfully"
1517
self.result.status = ExecutionStatus.OK

nodescraper/plugins/inband/kernel/kernel_collector.py

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -81,9 +81,10 @@ def collect_data(
8181
if self.system_info.os_family == OSFamily.WINDOWS:
8282
res = self._run_sut_cmd(self.CMD_WINDOWS)
8383
if res.exit_code == 0:
84-
kernel_info = [line for line in res.stdout.splitlines() if "Version=" in line][
85-
0
86-
].split("=")[1]
84+
kernel_info = res.stdout
85+
kernel = [line for line in res.stdout.splitlines() if "Version=" in line][0].split(
86+
"="
87+
)[1]
8788
else:
8889
res = self._run_sut_cmd(self.CMD)
8990
if res.exit_code == 0:
@@ -107,7 +108,7 @@ def collect_data(
107108
console_log=True,
108109
)
109110

110-
if kernel_info:
111+
if kernel_info and kernel:
111112

112113
kernel_data = KernelDataModel(kernel_info=kernel_info, kernel_version=kernel)
113114
self._log_event(

nodescraper/plugins/inband/kernel/kerneldata.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,4 +29,4 @@
2929

3030
class KernelDataModel(DataModel):
3131
kernel_info: str
32-
kernel_version: str | None = None
32+
kernel_version: str

test/unit/plugin/test_kernel_collector.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,9 @@ def test_run_windows(collector, conn_mock):
5353

5454
result, data = collector.collect_data()
5555

56-
assert data == KernelDataModel(kernel_info="10.0.19041.1237", kernel_version=None)
56+
assert data == KernelDataModel(
57+
kernel_info="Version=10.0.19041.1237", kernel_version="10.0.19041.1237"
58+
)
5759
assert result.status == ExecutionStatus.OK
5860

5961

0 commit comments

Comments
 (0)