Скорректировал разделитель в трейс логах
This commit is contained in:
+1
-1
@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
|
|||||||
|
|
||||||
[project]
|
[project]
|
||||||
name = "plba"
|
name = "plba"
|
||||||
version = "0.3.16"
|
version = "0.3.17"
|
||||||
description = "Platform runtime for business applications"
|
description = "Platform runtime for business applications"
|
||||||
readme = "README.md"
|
readme = "README.md"
|
||||||
requires-python = ">=3.11"
|
requires-python = ">=3.11"
|
||||||
|
|||||||
@@ -71,7 +71,7 @@ class TraceResponseRenderer:
|
|||||||
f"trace_id: {trace_view.trace_id}",
|
f"trace_id: {trace_view.trace_id}",
|
||||||
f"parent_id: {trace_view.parent_id or ''}",
|
f"parent_id: {trace_view.parent_id or ''}",
|
||||||
*self._child_id_lines(trace_view.child_ids),
|
*self._child_id_lines(trace_view.child_ids),
|
||||||
"--------------------------------------------------",
|
"------------------------------",
|
||||||
]
|
]
|
||||||
previous_step: str | None = None
|
previous_step: str | None = None
|
||||||
for record in trace_view.records:
|
for record in trace_view.records:
|
||||||
@@ -79,7 +79,7 @@ class TraceResponseRenderer:
|
|||||||
if previous_step is None:
|
if previous_step is None:
|
||||||
lines.append(f"step: {current_step}")
|
lines.append(f"step: {current_step}")
|
||||||
elif current_step != previous_step:
|
elif current_step != previous_step:
|
||||||
lines.append("--------------------------------------------------")
|
lines.append("------------------------------")
|
||||||
lines.append(f"step: {current_step}")
|
lines.append(f"step: {current_step}")
|
||||||
previous_step = current_step
|
previous_step = current_step
|
||||||
lines.append(self._text_message(record, request.include_attrs_json))
|
lines.append(self._text_message(record, request.include_attrs_json))
|
||||||
@@ -177,7 +177,7 @@ class TraceResponseRenderer:
|
|||||||
self._html_plain_line(f"parent_id: {self._optional_trace_link(trace_view.parent_id, request)}"),
|
self._html_plain_line(f"parent_id: {self._optional_trace_link(trace_view.parent_id, request)}"),
|
||||||
self._html_plain_line("child_ids:"),
|
self._html_plain_line("child_ids:"),
|
||||||
*(self._html_plain_line(f" - {self._trace_link(child_id, request)}") for child_id in trace_view.child_ids),
|
*(self._html_plain_line(f" - {self._trace_link(child_id, request)}") for child_id in trace_view.child_ids),
|
||||||
self._html_plain_line("--------------------------------------------------"),
|
self._html_plain_line("------------------------------"),
|
||||||
]
|
]
|
||||||
previous_step: str | None = None
|
previous_step: str | None = None
|
||||||
for record in trace_view.records:
|
for record in trace_view.records:
|
||||||
@@ -187,7 +187,7 @@ class TraceResponseRenderer:
|
|||||||
lines.append(self._html_plain_line(""))
|
lines.append(self._html_plain_line(""))
|
||||||
elif current_step != previous_step:
|
elif current_step != previous_step:
|
||||||
lines.append(self._html_plain_line(""))
|
lines.append(self._html_plain_line(""))
|
||||||
lines.append(self._html_plain_line("--------------------------------------------------"))
|
lines.append(self._html_plain_line("------------------------------"))
|
||||||
lines.append(self._html_step_line(current_step))
|
lines.append(self._html_step_line(current_step))
|
||||||
lines.append(self._html_plain_line(""))
|
lines.append(self._html_plain_line(""))
|
||||||
previous_step = current_step
|
previous_step = current_step
|
||||||
|
|||||||
@@ -100,7 +100,7 @@ def test_trace_endpoint_returns_text_when_requested() -> None:
|
|||||||
"child_ids:\n"
|
"child_ids:\n"
|
||||||
" - child-1\n"
|
" - child-1\n"
|
||||||
" - child-2\n"
|
" - child-2\n"
|
||||||
"--------------------------------------------------\n"
|
"------------------------------\n"
|
||||||
"step: process\n"
|
"step: process\n"
|
||||||
"first error\n"
|
"first error\n"
|
||||||
"second warning"
|
"second warning"
|
||||||
@@ -129,7 +129,7 @@ def test_trace_endpoint_appends_attrs_json_in_text_mode() -> None:
|
|||||||
"trace_id: trace-1\n"
|
"trace_id: trace-1\n"
|
||||||
"parent_id: \n"
|
"parent_id: \n"
|
||||||
"child_ids:\n"
|
"child_ids:\n"
|
||||||
"--------------------------------------------------\n"
|
"------------------------------\n"
|
||||||
"step: process\n"
|
"step: process\n"
|
||||||
'failure, {"attempt":2,"source":"crm"}'
|
'failure, {"attempt":2,"source":"crm"}'
|
||||||
)
|
)
|
||||||
@@ -159,11 +159,11 @@ def test_trace_endpoint_separates_messages_by_step_in_text_mode() -> None:
|
|||||||
"trace_id: trace-1\n"
|
"trace_id: trace-1\n"
|
||||||
"parent_id: \n"
|
"parent_id: \n"
|
||||||
"child_ids:\n"
|
"child_ids:\n"
|
||||||
"--------------------------------------------------\n"
|
"------------------------------\n"
|
||||||
"step: load_stocks\n"
|
"step: load_stocks\n"
|
||||||
"load first\n"
|
"load first\n"
|
||||||
"load second\n"
|
"load second\n"
|
||||||
"--------------------------------------------------\n"
|
"------------------------------\n"
|
||||||
"step: filter_stocks\n"
|
"step: filter_stocks\n"
|
||||||
"filter first"
|
"filter first"
|
||||||
)
|
)
|
||||||
@@ -241,7 +241,7 @@ def test_trace_endpoint_returns_html_page_with_related_links() -> None:
|
|||||||
assert '<div class="line"> - <a href="/traces/child-1?format=html&levels=ERROR%2CWARNING%2CINFO&attrs_json=true">child-1</a></div>' in response.text
|
assert '<div class="line"> - <a href="/traces/child-1?format=html&levels=ERROR%2CWARNING%2CINFO&attrs_json=true">child-1</a></div>' in response.text
|
||||||
assert '<div class="line"> - <a href="/traces/child-2?format=html&levels=ERROR%2CWARNING%2CINFO&attrs_json=true">child-2</a></div>' in response.text
|
assert '<div class="line"> - <a href="/traces/child-2?format=html&levels=ERROR%2CWARNING%2CINFO&attrs_json=true">child-2</a></div>' in response.text
|
||||||
assert '<div class="line" style="color: var(--step);">load_stocks</div>' in response.text
|
assert '<div class="line" style="color: var(--step);">load_stocks</div>' in response.text
|
||||||
assert '<div class="line">--------------------------------------------------</div>' in response.text
|
assert '<div class="line">------------------------------</div>' in response.text
|
||||||
assert '<div class="line" style="color: var(--step);">filter_stocks</div>' in response.text
|
assert '<div class="line" style="color: var(--step);">filter_stocks</div>' in response.text
|
||||||
assert "loaded prices" in response.text
|
assert "loaded prices" in response.text
|
||||||
assert "filtered suspicious ticker" in response.text
|
assert "filtered suspicious ticker" in response.text
|
||||||
|
|||||||
Reference in New Issue
Block a user