Visibility Service: Tests

Open on Drupal.org →

Created on 8 March 2025, about 2 months ago

Problem/Motivation

The page method of the Visibility Service is difficult to test

Steps to reproduce

Proposed resolution

Remove the static variable. The service is only called from one function and does not need to cache the results.

static $page_match = NULL;

Remaining tasks

User interface changes

API changes

Data model changes

📌 Task

Status

Active

Version

2.0

Component

Code

Created by

🇺🇸United States bluegeek9

Live updates comments and jobs are added and updated live.

Sign in to follow issues

Merge Requests

!227Visibility Service: Tests
Merged
🇺🇸United States bluegeek9
updated about 2 months ago
!228Visibility Service: Tests
Merged
🇺🇸United States bluegeek9
updated about 2 months ago

Comments & Activities

Issue created by @bluegeek9
Merge request !227Issue #3511793 by bluegeek9: Visibility Service: Tests → (Merged) created by bluegeek9
Merge request !228Resolve #3511793 "3 visibility service tests" → (Merged) created by bluegeek9
Pipeline finished with Skipped
about 2 months ago
#443765
Comment about 2 months ago →
System Message

bluegeek9 → committed 7bb4e119 on 3.0.x
Issue #3511793: Visibility Service: Tests
Pipeline finished with Skipped
about 2 months ago
#443767
Comment about 2 months ago →
System Message

bluegeek9 → committed 8243dda6 on 8.x-2.x
Issue #3511793: Visibility Service: Tests
Comment about 2 months ago →
🇺🇸United States bluegeek9
Comment about 1 month ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.
Comment 14 days ago →
🇺🇸United States bluegeek9

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024