Skip to content

Commit 0fe3305

Browse files
committed
node: Use default_factory for ProviderFactorySpec.special
Replace direct SpecialSourceProvider.Options() default with dataclasses.field(default_factory=...) to satisfy rruff and avoid function call in dataclass default
1 parent 08566b2 commit 0fe3305

1 file changed

Lines changed: 12 additions & 10 deletions

File tree

node/tests/conftest.py

Lines changed: 12 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
import subprocess
66
import sys
77
from collections.abc import Iterable, Iterator
8-
from dataclasses import dataclass
8+
from dataclasses import dataclass, field
99
from pathlib import Path
1010
from typing import Any
1111

@@ -223,15 +223,17 @@ def add_to_manifest(self, gen: ManifestGenerator) -> None:
223223
class ProviderFactorySpec:
224224
datadir: Path
225225
type: ProviderFactoryType
226-
special: SpecialSourceProvider.Options = SpecialSourceProvider.Options(
227-
node_chromedriver_from_electron=None,
228-
electron_ffmpeg=None,
229-
electron_node_headers=False,
230-
nwjs_version=None,
231-
nwjs_node_headers=False,
232-
nwjs_ffmpeg=False,
233-
xdg_layout=True,
234-
node_sdk_extension=None,
226+
special: SpecialSourceProvider.Options = field(
227+
default_factory=lambda: SpecialSourceProvider.Options(
228+
node_chromedriver_from_electron=None,
229+
electron_ffmpeg=None,
230+
electron_node_headers=False,
231+
nwjs_version=None,
232+
nwjs_node_headers=False,
233+
nwjs_ffmpeg=False,
234+
xdg_layout=True,
235+
node_sdk_extension=None,
236+
)
235237
)
236238

237239
def create_factory(

0 commit comments

Comments
 (0)