Port xbstream_fifo_test.sh to pytest (xbstream_fifo_tests.py) - #12

reviewdog: Too many results (annotations) in diff. You may miss some annotations due to GitHub limitation for annotation created by logging command. Please check GitHub Actions log console to see all results. Limitation: - 10 warning annotations and 10 error annotations per step - 50 annotations per job (sum of annotations from all the steps) - 50 annotations per run (separate from the job annotations, these annotations aren't created by users) Source: https://github.com/orgs/community/discussions/26680#discussioncomment-3252835

Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24: actions/checkout@v4. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/

[rdjson] reported by reviewdog 🐶 Use `X | None` for type annotations Raw Output: message:"Use `X | None` for type annotations" location:{path:"/home/runner/work/server-qa/server-qa/test_scripts/pxb/test_helper.py" range:{start:{line:1996 column:19} end:{line:1996 column:32}}} severity:WARNING source:{name:"ruff" url:"https://docs.astral.sh/ruff"} code:{value:"UP045" url:"https://docs.astral.sh/ruff/rules/non-pep604-annotation-optional"} suggestions:{range:{start:{line:1996 column:19} end:{line:1996 column:32}} text:"str | None"}

[rdjson] reported by reviewdog 🐶 Use `X | None` for type annotations Raw Output: message:"Use `X | None` for type annotations" location:{path:"/home/runner/work/server-qa/server-qa/test_scripts/pxb/test_helper.py" range:{start:{line:1995 column:19} end:{line:1995 column:32}}} severity:WARNING source:{name:"ruff" url:"https://docs.astral.sh/ruff"} code:{value:"UP045" url:"https://docs.astral.sh/ruff/rules/non-pep604-annotation-optional"} suggestions:{range:{start:{line:1995 column:19} end:{line:1995 column:32}} text:"str | None"}

[rdjson] reported by reviewdog 🐶 Use `X | None` for type annotations Raw Output: message:"Use `X | None` for type annotations" location:{path:"/home/runner/work/server-qa/server-qa/test_scripts/pxb/test_helper.py" range:{start:{line:1994 column:23} end:{line:1994 column:36}}} severity:WARNING source:{name:"ruff" url:"https://docs.astral.sh/ruff"} code:{value:"UP045" url:"https://docs.astral.sh/ruff/rules/non-pep604-annotation-optional"} suggestions:{range:{start:{line:1994 column:23} end:{line:1994 column:36}} text:"int | None"}

[rdjson] reported by reviewdog 🐶 Use `X | None` for type annotations Raw Output: message:"Use `X | None` for type annotations" location:{path:"/home/runner/work/server-qa/server-qa/test_scripts/pxb/test_helper.py" range:{start:{line:1992 column:30} end:{line:1992 column:43}}} severity:WARNING source:{name:"ruff" url:"https://docs.astral.sh/ruff"} code:{value:"UP045" url:"https://docs.astral.sh/ruff/rules/non-pep604-annotation-optional"} suggestions:{range:{start:{line:1992 column:30} end:{line:1992 column:43}} text:"str | None"}

[rdjson] reported by reviewdog 🐶 `datetime.datetime.now()` called without a `tz` argument Raw Output: message:"`datetime.datetime.now()` called without a `tz` argument" location:{path:"/home/runner/work/server-qa/server-qa/test_scripts/pxb/test_helper.py" range:{start:{line:1575 column:70} end:{line:1575 column:84}}} severity:WARNING source:{name:"ruff" url:"https://docs.astral.sh/ruff"} code:{value:"DTZ005" url:"https://docs.astral.sh/ruff/rules/call-datetime-now-without-tzinfo"}

[rdjson] reported by reviewdog 🐶 Use `X | None` for type annotations Raw Output: message:"Use `X | None` for type annotations" location:{path:"/home/runner/work/server-qa/server-qa/test_scripts/pxb/seaweedfs_helper.py" range:{start:{line:37 column:36} end:{line:37 column:49}}} severity:WARNING source:{name:"ruff" url:"https://docs.astral.sh/ruff"} code:{value:"UP045" url:"https://docs.astral.sh/ruff/rules/non-pep604-annotation-optional"} suggestions:{range:{start:{line:37 column:36} end:{line:37 column:49}} text:"str | None"}

[rdjson] reported by reviewdog 🐶 Use `X | None` for type annotations Raw Output: message:"Use `X | None` for type annotations" location:{path:"/home/runner/work/server-qa/server-qa/test_scripts/pxb/seaweedfs_helper.py" range:{start:{line:31 column:34} end:{line:31 column:47}}} severity:WARNING source:{name:"ruff" url:"https://docs.astral.sh/ruff"} code:{value:"UP045" url:"https://docs.astral.sh/ruff/rules/non-pep604-annotation-optional"} suggestions:{range:{start:{line:31 column:34} end:{line:31 column:47}} text:"str | None"}

[rdjson] reported by reviewdog 🐶 Import block is un-sorted or un-formatted Raw Output: message:"Import block is un-sorted or un-formatted" location:{path:"/home/runner/work/server-qa/server-qa/test_scripts/pxb/seaweedfs_helper.py" range:{start:{line:12 column:1} end:{line:18 column:28}}} severity:WARNING source:{name:"ruff" url:"https://docs.astral.sh/ruff"} code:{value:"I001" url:"https://docs.astral.sh/ruff/rules/unsorted-imports"} suggestions:{range:{start:{line:12 column:1} end:{line:20 column:1}} text:"import json\nimport os\nimport subprocess\nimport time\nimport urllib.error\nimport urllib.request\nfrom typing import Optional\n\n"}

[rdjson] reported by reviewdog 🐶 Shebang is present but file is not executable Raw Output: message:"Shebang is present but file is not executable" location:{path:"/home/runner/work/server-qa/server-qa/test_scripts/pxb/seaweedfs_helper.py" range:{start:{line:1 column:1} end:{line:1 column:23}}} severity:WARNING source:{name:"ruff" url:"https://docs.astral.sh/ruff"} code:{value:"EXE001" url:"https://docs.astral.sh/ruff/rules/shebang-not-executable"}

[rdjson] reported by reviewdog 🐶 Mutable default value for class attribute Raw Output: message:"Mutable default value for class attribute" location:{path:"/home/runner/work/server-qa/server-qa/test_scripts/pxb/kmip_helper.py" range:{start:{line:38 column:28} end:{line:43 column:6}}} severity:WARNING source:{name:"ruff" url:"https://docs.astral.sh/ruff"} code:{value:"RUF012" url:"https://docs.astral.sh/ruff/rules/mutable-class-default"}