Problem/Motivation

The hole TestSuite of ETrackerFunctionalTests is not working. There are some API failures and tiny wrong expectations in the test.

Steps to reproduce

Running the FunctionalTests inside you IDE and see the red bulbs popping up.

Proposed resolution

Fix the TestSuite.

Issue fork etracker-3605475

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

sunlix created an issue. See original summary.

sunlix opened merge request !48

sunlix’s picture

sunlix
Primary language English
Location Wesel
commented
Status: Active » Needs work

All test except testExclusionByPathWithSlashUrl are running again.

With the config in this test we do not get a route title by the title_resolver.

Exception: Deprecated function: rawurlencode(): Passing null to parameter #1 ($string) of type string is deprecated
etracker_page_attachments_alter()() (Line: 204)
sunlix’s picture

sunlix
Primary language English
Location Wesel
commented

The route user.logout.confirm from cores user module does not have a _title default value. Because of that, the title_resolver service does not get any title.

Should that fix in core?!
I think we have to test around that and handle it in eTracker if no title can be resolved, et_pagename should not printed.

But I do not understand why this only happens in this specific test.
All other tests doing the same with different settings to test.

sunlix’s picture

sunlix
Primary language English
Location Wesel
commented
Status: Needs work » Fixed

So the tests are running through now and the issue with the title_resolver is fixed.

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

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

Maintainers, credit people who helped resolve this issue.

  • sunlix committed c3f5fa6f on 8.x-3.x 
    fix: #3605475 ETrackerFunctionalTests are not working

By: sunlix