Skip to content

Commit 6ca6360

Browse files
committed
utest fix
1 parent 593b450 commit 6ca6360

File tree

3 files changed

+2
-4
lines changed

3 files changed

+2
-4
lines changed

nodescraper/interfaces/analyzerargs.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,8 +33,6 @@
3333

3434
class AnalyzerArgs(BaseModel, ABC):
3535
data_model: Optional["DataModel"] = Field(default=None)
36-
# model_config = {"exclude_none": True}
37-
# model_config = ConfigDict(exclude_noe=True)
3836
model_config = {"extra": "forbid", "exclude_none": True}
3937

4038
@classmethod

nodescraper/plugins/inband/cmdline/analyzer_args.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,4 +75,4 @@ def build_from_model(cls, datamodel: CmdlineDataModel) -> "CmdlineAnalyzerArgs":
7575
Returns:
7676
CmdlineAnalyzerArgs: instance of analyzer args class
7777
"""
78-
return cls(required_cmdline=datamodel.cmdline, banned_cmdline=datamodel.cmdline)
78+
return cls(required_cmdline=datamodel.cmdline)

test/unit/framework/test_analyzerargs.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ def test_build_from_model(dummy_data_model):
1818
assert isinstance(args, MyArgs)
1919
assert args.args_foo == dummy.foo
2020
dump = args.model_dump(mode="json", exclude_none=True)
21-
assert dump == "{'args_foo' : 1}"
21+
assert dump == {"args_foo": 1}
2222

2323

2424
def test_base_build_from_model_not_implemented():

0 commit comments

Comments
 (0)