Site Health: If 0 active plugins (themes), they can't be "all up to date"

[Presskopp]

If 0 plugins are active it states:

Your site has 0 active plugins, and they are all up to date.

In my opinion we should strip the "up to date" part in that case.

Same for themes.

[costdev]

I wonder if: Your site does not have any installed themes.
might be better as: Your site does not have any themes installed.

The latter reads a little more natural to me, though it's a minor thing that I'm not precious about.

Nevermind - Just realised this would create a discrepancy with the other strings used.

The patch looks good to me! Adding needs-testing.

@Presskopp or @SergeyBiryukov, could you add some ​testing instructions and the has-testing-info keyword for the Test Team?

[Presskopp]

Testing Instructions

These steps define how to test the feature or enhancement, and indicates the expected behavior or results.

Steps to Test

1. Rename plugins folder and themes folder
2. Open "Tools" > "Site Health"
3. Open "Passed Tests" > "Your plugins are all up to date"
4. Read "Your site has 0 active plugins, and they are all up to date."
5. Open "recommended improvements"
6. Open "Have a default theme available"
7. Read "Your site has 0 installed themes, and they are all up to date."

Expected Results

Output:

• ✅ Your site does not have any active plugins.
• ✅ Your site does not have any installed themes.

[martin.krcho]

@SergeyBiryukov, did you consider adding unit tests to the patch?

I can see that Tests_Site_Health tests cover only the site health check cron at the moment. Adding more specific tests will probably require considerable effort. Has this been already discussed somewhere?

[SergeyBiryukov]

Replying to martin.krcho:

@SergeyBiryukov, did you consider adding unit tests to the patch?

Thanks for bringing this up! Not really, I would for any functionality changes, but this is only a string change, so I don't think it requires unit tests.

I can see that Tests_Site_Health tests cover only the site health check cron at the moment. Adding more specific tests will probably require considerable effort. Has this been already discussed somewhere?

Yes, there's a ticket for that: #47880.

[martin.krcho]

Sounds good to me, @SergeyBiryukov. I will follow #47880 and see if I can help at some point.

[Clorith]

Quick and simple change, looks good to go.

[SergeyBiryukov]

In 53816:

Site Health: Improve the wording in plugin and theme version tests.

This displays a more appropriate message in case the site does not have any active plugins or installed themes.

Follow-up to [44986], [45099], [45336].

Props Presskopp, costdev, martin.krcho, Clorith, SergeyBiryukov.
Fixes #56134.

[SergeyBiryukov]

In 53817:

Coding Standards: Remove extra space in two conditionals.

This fixes WPCS errors in WP_Site_Health::get_test_plugin_version() and ::get_test_theme_version():

• Expected exactly one space between closing parenthesis and opening control structure; " " found.
• Expected 1 space(s) after closing parenthesis; found 2

Follow-up to [53816].

See #56134.