Fix misleading test description for filesize field 0 bytes behavior

[Copilot]

The test for 0 bytes filesize claimed the empty string was returned "because isVisible will hide it", but the actual reason is the truthy check in getValue:

getValue: ( { item }: { item: MediaItem } ) =>
    item?.media_details?.filesize
        ? formatFileSize( item?.media_details?.filesize )
        : '',

Since 0 is falsy in JavaScript, getValue returns '' before formatFileSize is ever called. The formatFileSize function would actually return "0 B" if it received 0 (line 47-49), but it never reaches that code path.

Changes

• Updated test description to "returns empty string for 0 bytes due to truthy check"
• Clarified comment to explain the truthy check behavior
• Added note that formatFileSize would return "0 B" for 0, but is never called

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[Copilot]

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• translate.wordpress.org
  • Triggering command: /usr/local/bin/node node i18n-cache/index.js (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

[github-actions]

Warning: Type of PR label mismatch

To merge this PR, it requires exactly 1 label indicating the type of PR. Other labels are optional and not being checked here.

• Required label: Any label starting with [Type].
• Labels found: .

Read more about Type labels in Gutenberg. Don't worry if you don't have the required permissions to add labels; the PR reviewer should be able to help with the task.

[github-actions]

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

Unlinked Accounts

The following contributors have not linked their GitHub and WordPress.org accounts: @Copilot.

Contributors, please read how to link your accounts to ensure your work is properly credited in WordPress releases.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Unlinked contributors: Copilot.

Co-authored-by: ramonjd <[email protected]>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

[github-actions]

Flaky tests detected in be44b67.
Some tests passed with failed attempts. The failures may not be related to this commit but are still reported for visibility. See the documentation for more information.

🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/20254480079
📝 Reported issues:

• [Flaky Test] should allow user to add and remove multiple local font files #62862 in /test/e2e/specs/site-editor/font-library.spec.js
• [Flaky Test] should open and edit templates correctly when active_templates experiment is disabled #73743 in /test/e2e/specs/site-editor/template-id-format.spec.js

[andrewserong]

Wow, looks like Copilot is trying to improve its merged PRs stats 😆