Skip to content

Commit

Permalink
Log correct signal names on CRUDEvent creation exception
Browse files Browse the repository at this point in the history
  • Loading branch information
mschoettle authored and jheld committed Sep 17, 2024
1 parent 5e51a80 commit f903c2a
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions easyaudit/signals/crud_flows.py
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ def post_save_crud_flow(instance, object_json_repr):
object_json_repr,
)
except Exception:
handle_flow_exception(instance, "pre_save")
handle_flow_exception(instance, "post_save")


def m2m_changed_crud_flow( # noqa: PLR0913
Expand All @@ -106,7 +106,7 @@ def m2m_changed_crud_flow( # noqa: PLR0913
changed_fields=changed_fields,
)
except Exception:
handle_flow_exception(instance, "pre_save")
handle_flow_exception(instance, "m2m_changed")


def post_delete_crud_flow(instance, object_json_repr):
Expand All @@ -118,4 +118,4 @@ def post_delete_crud_flow(instance, object_json_repr):
)

except Exception:
handle_flow_exception(instance, "pre_save")
handle_flow_exception(instance, "post_delete")

4 comments on commit f903c2a

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Coverage

Coverage Report •
FileStmtsMissCoverMissing
easyaudit/signals
   crud_flows.py61690%76–77, 108–109, 120–121
TOTAL83122373% 

Tests Skipped Failures Errors Time
50 0 💤 0 ❌ 0 🔥 3.435s ⏱️

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Coverage

Coverage Report •
FileStmtsMissCoverMissing
easyaudit/signals
   crud_flows.py61690%76–77, 108–109, 120–121
TOTAL82922273% 

Tests Skipped Failures Errors Time
50 0 💤 0 ❌ 0 🔥 3.480s ⏱️

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Coverage

Coverage Report •
FileStmtsMissCoverMissing
easyaudit/signals
   crud_flows.py61690%76–77, 108–109, 120–121
TOTAL83122373% 

Tests Skipped Failures Errors Time
50 0 💤 0 ❌ 0 🔥 3.987s ⏱️

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Coverage

Coverage Report •
FileStmtsMissCoverMissing
easyaudit/signals
   crud_flows.py61690%76–77, 108–109, 120–121
TOTAL82922373% 

Tests Skipped Failures Errors Time
50 0 💤 0 ❌ 0 🔥 3.592s ⏱️

Please sign in to comment.