There are currently no SimpleTest web tests for this module. I suggest some are added to ensure that future additions or refactoring doesn't cause regressions and failures in the code.

Assigning to myself for now, and tagging for the next Microserve sprint day.

CommentFileSizeAuthor
#6 interdiff.txt3.78 KBopdavies
#6 2865481-6.patch3.49 KBopdavies
#5 node_display_title-tests-2865481-4-D7.patch3.87 KBdarrenwh
#3 node_display_title-tests-2865481-3-D7.patch3.48 KBdarrenwh

Comments

opdavies created an issue. See original summary.

darrenwh’s picture

darrenwh commented
Assigned: opdavies » darrenwh

OK going to work on this today and set up some tests

darrenwh’s picture

darrenwh commented
Status: Active » Needs review
StatusFileSize
new node_display_title-tests-2865481-3-D7.patch3.48 KB

Patch file attached with tests

darrenwh’s picture

darrenwh commented
Assigned: darrenwh » Unassigned
darrenwh’s picture

darrenwh commented
StatusFileSize
new node_display_title-tests-2865481-4-D7.patch3.87 KB

Updated added include

opdavies’s picture

opdavies
He/him
Primary language English
Location Wales
commented
StatusFileSize
new 2865481-6.patch3.49 KB
new interdiff.txt3.78 KB

+1

I've made some small changes, but nothing major.

I'm surprised that an admin user needs to be created and logged in within the setUp method to perform the initial tasks, but without it the number of passes drops from 34 to 20, though with no failures or exceptions.

astonvictor’s picture

astonvictor commented
Status: Needs review » Closed (outdated)

D7 reached its EOL back in January 2025, and there is no active release for D7 for this module anymore.
Development or support is not planned for D7. All D7-related issues are marked as outdated in a bunch.

Now that this issue is closed, please review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, please credit people who helped resolve this issue.