Add automated test coverage

Open on Drupal.org →

Created on 21 May 2021, over 3 years ago

Updated 9 August 2023, over 1 year ago

Problem/Motivation

This module has no automated test coverage. There is some suspicion that a Drupal core patch ( #2716019: View titles in breadcrumb and metatag title don't get properly translated → ) will cause issues for this module - allow this is not proven. Automated test coverage would help prove or disprove this notion.

FWIW a module with so many users deserves automated test coverage to prove its features are not broken by bugfixes or changes to its dependencies especially Drupal core.

Steps to reproduce

Proposed resolution

Follow instructions at https://www.drupal.org/docs/develop/managing-a-drupalorg-theme-module-or... →

Remaining tasks

- [x] Activate testing for this module
- [x] Add a composer.json
- [wip] Write some tests
- [ ] Get tests to run and pass
- [ ] Review
- [ ] Merge

User interface changes

None

API changes

None

Data model changes

None

📌 Task

Status

Needs review

Version

1.0

Component

Code

Created by

🇬🇧United Kingdom alexpott 🇪🇺🌍

Live updates comments and jobs are added and updated live.

Incomplete comments

Sign in to follow issues

Merge Requests

!19Add automated test coverage
Merged
🇳🇿New Zealand xurizaemon
updated 8 months ago
!16Add automated test coverage
Open
🇳🇿New Zealand xurizaemon
updated 8 months ago
!3Add automated test coverage
Merged
🇳🇿New Zealand xurizaemon
updated 8 months ago
!4Add automated test coverage
Merged
🇳🇿New Zealand xurizaemon
updated 8 months ago

Comments & Activities

Not all content is available!

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

Merge request !3Issue #3215110: Add automated test coverage → (Merged) created by xurizaemon
Merge request !4Resolve #3215110 "Add automated test" → (Merged) created by xurizaemon
Comment over 1 year ago →
🇺🇸United States yesct
This is listed as a blocker for a full release with D10 compatibility.

Is the next step here to help get the MR passing tests?
Comment over 1 year ago →
🇳🇿New Zealand xurizaemon Ōtepoti, Aotearoa 🏝
Yes. I think at this point, merging even some tests would be better than letting it block D10 release. Help welcome!
Open on Drupal.org →
Core: 9.5.x + Environment: PHP 8.0 & MySQL 5.7
last update 9 months ago
Waiting for branch to pass
Open in Jenkins → Open on Drupal.org →
Core: 10.2.1 + Environment: PHP 8.1 & MySQL 5.7
last update 9 months ago
Composer require failure
Open on Drupal.org →
Core: 10.2.1 + Environment: PHP 8.1 & MySQL 5.7
last update 9 months ago
Waiting for branch to pass
Open on Drupal.org →
Core: 10.2.1 + Environment: PHP 8.1 & MySQL 5.7
last update 9 months ago
Waiting for branch to pass
Open on Drupal.org →
Core: 10.2.1 + Environment: PHP 8.1 & MySQL 5.7
last update 9 months ago
Waiting for branch to pass
Open on Drupal.org →
Core: 10.2.1 + Environment: PHP 8.1 & MySQL 5.7
last update 9 months ago
Waiting for branch to pass
Open on Drupal.org →
Core: 10.2.1 + Environment: PHP 8.1 & MySQL 5.7
last update 9 months ago
Waiting for branch to pass
Open on Drupal.org →
Core: 10.2.1 + Environment: PHP 8.1 & MySQL 5.7
last update 9 months ago
Waiting for branch to pass
Open on Drupal.org →
Core: 10.2.1 + Environment: PHP 8.1 & MySQL 5.7
last update 9 months ago
Waiting for branch to pass
Open on Drupal.org →
Core: 10.2.1 + Environment: PHP 8.1 & MySQL 5.7
last update 9 months ago
Waiting for branch to pass
Open on Drupal.org →
Core: 10.2.1 + Environment: PHP 8.1 & MySQL 5.7
last update 9 months ago
Waiting for branch to pass
Comment 8 months ago →
🇳🇿New Zealand xurizaemon Ōtepoti, Aotearoa 🏝
Comment 8 months ago →
System Message

xurizaemon → committed d27a7baf on 2.0.x
Resolve #3215110 "Add automated test"
Comment 8 months ago →
System Message

xurizaemon → committed 423724f0 on 8.x-1.x
Issue #3215110: Add automated test coverage
Merge request !16Issue #3215110: More tests! → (Open) created by xurizaemon
Merge request !19Issue #3215110: Expand test coverage → (Merged) created by xurizaemon
Comment 8 months ago →
System Message

xurizaemon → committed 9c900e72 on 2.0.x
Issue #3215110: Expand test coverage
Status changed to Fixed about 1 month ago8:15pm 30 October 2024
Comment about 1 month ago →
🇳🇿New Zealand xurizaemon Ōtepoti, Aotearoa 🏝
There's more to do here, but I'll move that to a new issue. This one has been around long enough and (highly speculative!) might even discourage smaller contribution efforts.
Comment 20 days ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024