MatomoBasicTest::testMatomoPageVisibility() fails in merge requests

Created on 7 March 2025, about 1 month ago

Problem/Motivation

The test looks for /matomo/js/matomo.js in the page.
But when the pipeline runs in a merge request, the path of the module contains the MR number.
For example:

<script src="/web/modules/custom/matomo-3376581/js/matomo.js?v=11.1.2"></script>

Steps to reproduce

See for example: https://git.drupalcode.org/project/matomo/-/merge_requests/48

Proposed resolution

The test could simply look for /js/matomo.js.

Remaining tasks

User interface changes

API changes

Data model changes

🐛 Bug report

Status

Active

Version

2.0

Component

Code

Created by

🇫🇷France prudloff Lille

Live updates comments and jobs are added and updated live.

Merge Requests

!72MatomoBasicTest::testMatomoPageVisibility() fails in merge requests
Open
🇮🇳India divyansh.gupta
updated about 1 month ago

Comments & Activities

Issue created by @prudloff
Comment about 1 month ago →
🇮🇳India divyansh.gupta Jaipur
Working on it.
Merge request !72Issue #3258071 by ressa, Grimreaper: Drupal 10 compatibility and do a slight... → (Open) created by divyansh.gupta
Pipeline finished with Success
about 1 month ago
#444434
Pipeline finished with Success
about 1 month ago
Total: 350s
#444436
Comment about 1 month ago →
🇮🇳India divyansh.gupta Jaipur
I have made changes in the tests and the pipeline is also working green after the changes,
Please review.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024