24443277181
99%

Ran 15 Apr 2026 08:03AM UTC

Jobs 1

Files 102

Run time 1min

Badge

Embed ▾

Committed 15 Apr 2026 08:00AM UTC coverage: 99.583% (+0.001%) from 99.582%

Build # 24443277181

Build Type

push

github

Committed by

web-flow

Commit Message

fix: broadcast CHAIN_UPDATE hook to all registered services (#1519)

* fix: broadcast CHAIN_UPDATE hook to all registered services when no specific service key is provided

Update ChainUpdateWebhookService.notify() to query all Service records
and fire per-service hooks when called without service_keys (Chain,
GasPrice, Wallet signals). Falls back to service=None when no services
exist in the DB.

Fix Feature signal callers that coerced empty service lists to None via
`or None`, which would have silently broadcast to all services under the
new semantics. They now pass [] to mean "no services".

* test: Add explicit no-services fallback test for CHAIN_UPDATE hook

Verify that when no Service records exist in the DB, chain creation
fires a single hook with service=None rather than silently skipping.

* test: Add delete-with-services tests for GasPrice and Wallet hooks

Cover the delete signal path for GasPrice and Wallet when Service
records exist, verifying per-service hooks fire on deletion.

* docs: add inline comment explaining [] vs None semantics in signals.py

Clarify at the first Feature signal call site that passing []
(not None) to notify() is intentional — None means "broadcast to
all registered services".

Coverage Stats

1434 of 1440 relevant lines covered (99.58%)

1.0 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	24443277181.1	15 Apr 2026 08:03AM UTC	102	99.58	GitHub Action Run

safe-global / safe-config-service / 24443277181
99%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 24443277181

safe-global / safe-config-service / 24443277181 99%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 24443277181

safe-global / safe-config-service / 24443277181
99%

README BADGES
x