Skip to content

Expose MockMicrogridApiClient and MockComponent#28

Merged
simonvoelcker merged 4 commits into
frequenz-floss:v0.x.xfrom
simonvoelcker:expose_mocks
May 11, 2026
Merged

Expose MockMicrogridApiClient and MockComponent#28
simonvoelcker merged 4 commits into
frequenz-floss:v0.x.xfrom
simonvoelcker:expose_mocks

Conversation

@simonvoelcker
Copy link
Copy Markdown
Contributor

No description provided.

@simonvoelcker simonvoelcker force-pushed the expose_mocks branch 9 times, most recently from 3c91554 to 87e41b8 Compare April 1, 2026 16:15
@simonvoelcker
Copy link
Copy Markdown
Contributor Author

@shsms any idea why this PR trips over all sorts of clippy errors in unchanged code while all other PRs pass? I don't see any tags that would control this either.

@shsms
Copy link
Copy Markdown
Collaborator

shsms commented Apr 21, 2026

@simonvoelcker it is because the tests are full of unwraps and panics, which we disallow in source code.

@shsms
Copy link
Copy Markdown
Collaborator

shsms commented Apr 21, 2026

Let me see if I can clean that up and expose it.

@simonvoelcker simonvoelcker force-pushed the expose_mocks branch 3 times, most recently from 2be9305 to 2873bd1 Compare May 11, 2026 12:18
Signed-off-by: Simon Völcker <simon.voelcker@frequenz.com>
Signed-off-by: Simon Völcker <simon.voelcker@frequenz.com>
Signed-off-by: Simon Völcker <simon.voelcker@frequenz.com>
Signed-off-by: Simon Völcker <simon.voelcker@frequenz.com>
@simonvoelcker simonvoelcker marked this pull request as ready for review May 11, 2026 12:45
@simonvoelcker simonvoelcker enabled auto-merge May 11, 2026 12:46
@simonvoelcker simonvoelcker requested a review from shsms May 11, 2026 12:46
@simonvoelcker simonvoelcker added this pull request to the merge queue May 11, 2026
Merged via the queue into frequenz-floss:v0.x.x with commit 18aa009 May 11, 2026
3 checks passed
@simonvoelcker simonvoelcker deleted the expose_mocks branch May 11, 2026 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants