Refactor DefaultTargetModuleTest to use/test DefaultTargetModuleSetUpTrait and TargetModuleCollection::ensureDefaultTargetModuleIsSet() parameters

Created on 23 May 2023, almost 2 years ago

This is a follow up to ✨ Make it possible to set default target module machine name and path before creating Needs review and 📌 Add test trait to create randomly named, non-conflicting default target modules Needs review to refactor/adapt DefaultTargetModuleTest to make use of both the new parameters added to TargetModuleCollection::ensureDefaultTargetModuleIsSet() and the new test trait to avoid naming collisions with any existing default target module.

Look, I'm tired and it's late and you're smart enough.

I'll make a fork and merge request hang tight.

Do the thing.

None.

None.

None.

📌 Task

Status

Needs review

Version

1.0

Component

Code

Created by

🇨🇦Canada ambient.impact Toronto

Live updates comments and jobs are added and updated live.

Comments & Activities

Issue created by @ambient.impact
Open in Jenkins → Open on Drupal.org →
Core: 9.5.x + Environment: PHP 7.4 & MySQL 5.7
last update almost 2 years ago
3 pass
@ambientimpact opened merge request.
Issue was unassigned.
Status changed to Needs review almost 2 years ago7:00am 23 May 2023
Comment almost 2 years ago →
🇨🇦Canada ambient.impact Toronto

Production build 0.71.5 2024