Skip to content

Fix course list block unit tests failing#7875

Merged
m1r0 merged 1 commit intotrunkfrom
fix/unit-tests
Dec 9, 2025
Merged

Fix course list block unit tests failing#7875
m1r0 merged 1 commit intotrunkfrom
fix/unit-tests

Conversation

@m1r0
Copy link
Member

@m1r0 m1r0 commented Dec 9, 2025

Proposed Changes

  • This fixes the PHP unit tests failing.

Testing Instructions

  1. The PHP unit tests should be passing in CI.

@m1r0 m1r0 added this to the 4.25.2 milestone Dec 9, 2025
@m1r0 m1r0 self-assigned this Dec 9, 2025
Copilot AI review requested due to automatic review settings December 9, 2025 16:43
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes failing PHP unit tests by correcting a filter callback that was not returning the required value.

  • Added missing return statement in the get_block_instance filter callback
  • Minor code formatting improvements with blank line additions

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@m1r0 m1r0 merged commit 0668150 into trunk Dec 9, 2025
20 of 21 checks passed
@m1r0 m1r0 deleted the fix/unit-tests branch December 9, 2025 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants