[meta] Convert remaining tests using Standard profile to use Testing profile instead

grep -lr "\$profile = 'standard'" core | nl 1 core/tests/Drupal/FunctionalTests/Installer/StandardInstallerTest.php 2 core/profiles/standard/tests/src/FunctionalJavascript/StandardJavascriptTest.php 3 core/profiles/standard/tests/src/Functional/StandardTest.php 4 core/modules/config/tests/src/Functional/ConfigImportAllTest.php 5 core/modules/config_translation/tests/src/Functional/ConfigTranslationInstallTest.php 6 core/modules/content_translation/tests/src/Functional/ContentTranslationStandardFieldsTe st.php 7 core/modules/content_translation/tests/src/Functional/ContentTranslationMetadataFieldsTe st.php 8 core/modules/migrate_drupal_ui/tests/src/Functional/MigrateUpgradeTestBase.php 9 core/modules/media/tests/src/FunctionalJavascript/MediaStandardProfileTest.php 10 core/modules/page_cache/tests/src/Functional/PageCacheTagsIntegrationTest.php 11 core/modules/rdf/tests/src/Functional/StandardProfileTest.php 12 core/modules/system/tests/src/Functional/Theme/EntityFilteringThemeTest.php 13 core/modules/system/tests/src/Functional/Menu/BreadcrumbTest.php 14 core/modules/system/tests/src/Functional/Common/NoJavaScriptAnonymousTest.php 15 core/modules/comment/tests/src/Functional/Views/CommentEditTest.php 16 core/modules/comment/tests/src/Functional/CommentPreviewTest.php 17 core/modules/help/tests/src/Functional/HelpTest.php 18 core/modules/workspaces/tests/src/Functional/WorkspacesUninstallTest.php 19 core/modules/node/tests/src/Functional/NodeTranslationUITest.php 20 core/modules/node/tests/src/Functional/NodeAccessBaseTableTest.php 21 core/modules/user/tests/src/FunctionalJavascript/UserPasswordResetTest.php 22 core/modules/user/tests/src/Functional/UserPictureTest.php

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Comment over 1 year ago →
🇳🇿New Zealand quietone
Grouped the remaining issues in the IS.
Comment over 1 year ago →
🇳🇿New Zealand quietone
Comment over 1 year ago →
🇺🇸United States smustgrave
Thanks for bringing this back up. Some of these were my first core contribution attempts so glad to be able to circle back
Comment over 1 year ago →
🇳🇿New Zealand quietone
Moved two tests that are testing the Standard profile to the list of the one not to be converted.
Comment over 1 year ago →
🇳🇿New Zealand quietone
Comment over 1 year ago →
🇺🇸United States smustgrave
@quietone can help on these but want to get one reviewed before moving on to the next. Been having trouble with that part.
Comment over 1 year ago →
🇳🇿New Zealand quietone
Converted another one and updated the IS.
Comment over 1 year ago →
🇳🇿New Zealand quietone
Comment over 1 year ago →
🇳🇿New Zealand quietone
Comment about 1 year ago →
🇳🇿New Zealand quietone
Just 3 more to do.
Comment about 1 year ago →
🇺🇸United States smustgrave
With regards to MediaStandardProfileTest it appears to only be testing the media bundles that ship with standard. We could move those config to a test module thoughts?
Comment about 1 year ago →
🇳🇿New Zealand quietone
The remaining test is core/modules/media/tests/src/FunctionalJavascript/MediaStandardProfileTest.php. I asked in Slack #media if this needs to use the standard profile. marcoscano, a maintainer, replied that it does, stating "In principle that's testing the config that ships with the standard profile, so I'd say yes, I think so". It is good I asked because when I first looked at the file I didn't think it needed to use the standard profile.

I did some digging and found the comment → where the test was added. In that comment seanB states,

Added MediaStandardProfileTest to test the standard profile form configuration, creation, field mapping and display configuration. We could expand on that in the future. See if we can move/remove the media specific code in Drupal\Tests\standard\Functional\StandardTest and see what we want to do with Drupal\Tests\media\FunctionalJavascript\MediaDisplayTest since there is some overlap.

I spent a few minutes comparing the two tests and I found that MediaDisplayTest is testing the standard profile configuration. So, this needs a followup up issue to remove that overlap.
Status changed to Fixed about 1 year ago10:24pm 10 January 2024
Comment about 1 year ago →
🇳🇿New Zealand quietone
Made a follow up to look at the overlap mentioned in #25, 📌 Remove overlap in StandardTest and MediaDisplayTest Active .

All the children are done and the followup is made. I am setting this to Fixed

Thanks to everyone who worked in the child issues to speed up the tests.

Woohoo!
Comment about 1 year ago →
🇳🇿New Zealand quietone
The follow up was made, see previous comment
Comment about 1 year ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

[meta] Convert remaining tests using Standard profile to use Testing profile instead

Problem/Motivation

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Comments & Activities