Create kernel test class for test of token replacement

Open on Drupal.org →

Created on 27 May 2025, 12 days ago

Problem/Motivation

A kernel test can be used to test the new token.

Steps to reproduce

See parent issue.

Proposed resolution

Add new test class for Kernel test to validate token replacement. Test will be added by a follow up issue.

Remaining tasks

Create a new class file in tests/src/Kernel/TokenTest.php.
Specify the namespace, Drupal\Tests\verify_rmail.
Define a protected static class variable, modules, which should be an array containing "verify_email".
This class does not need a setup() function.

User interface changes

None.

API changes

None.

Data model changes

None.

📌 Task

Status

Active

Version

1.1

Component

Code

Created by

🇮🇪Ireland lostcarpark

Live updates comments and jobs are added and updated live.

Sign in to follow issues

Merge Requests

!27Create kernel test class for test of token replacement
Merged
🇮🇳India nidhi27
updated 9 days ago

Comments & Activities

Issue created by @lostcarpark
Comment 11 days ago →
🇮🇪Ireland lostcarpark
Changed merge target to 1.1.x.

Unfortunately, as 📌 Add token info hook Active modified the same file, this creates a merge conflict. Could you please rebase?
Comment 11 days ago →
🇮🇪Ireland lostcarpark
Comment 11 days ago →
🇮🇪Ireland lostcarpark
Accidently changed status on wrong issue. Setting back to active.
Comment 11 days ago →
🇮🇪Ireland lostcarpark
Comment 9 days ago →
🇮🇳India nidhi27
Merge request !27Resolve #3526678 "Create kernel test" → (Merged) created by nidhi27
Pipeline finished with Failed
9 days ago
Total: 187s
#509689
Pipeline finished with Failed
9 days ago
Total: 199s
#509690
Comment 9 days ago →
🇮🇳India nidhi27
Hii,

I have created the MR !27 for this. Let me know if any changes are needed.
Thanks!
Comment 9 days ago →
🇮🇪Ireland lostcarpark
This looks great to me.

The tests are producing a warning because there are no tests in the class, which is getting marked as a failure in PHPUnit. However, this should be resolved in the next issue.

Moving to RTBC.
Comment 9 days ago →
System Message

lostcarpark → committed 5e40138e on 1.1.x authored by nidhi27 →
Resolve #3526678 "Create kernel test"
Comment 9 days ago →
🇮🇪Ireland lostcarpark
Merged, though we have test failures. Will be fixed by 📌 Add test of token replacement Active .

Thank you for working on this.
Comment 9 days ago →
🇮🇪Ireland lostcarpark

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024