Closed (outdated)
Project:
Tour UI
Version:
8.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
22 Aug 2013 at 06:43 UTC
Updated:
7 Oct 2018 at 14:17 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
clemens.tolboomPath on */edit are listed in test results as 404
Test on ie #71: admin/config/user-interface/tour/add
Current OK-ed Test
GET http://drupal.d8/admin/config/user-interface/tour/manage/fj0awosg/edit returned 404 (4.82 KB).
So this needs some fixes.
Comment #2
clemens.tolboom#2026239: Display the tour file name on the Tour UI overview page added the tour file short name (without tour.tour. prefix).
Other tests are broken with mostly XPath errors.
- the result page is empty (I don't get these)
- the create new tour fails completely :-/
Comment #3
clemens.tolboomThe exceptions are about
Comment #4
clemens.tolboomI've added two fixes but the exception from #3 needs more work.
From the tipTest() output I get a
Drupal\Component\Plugin\Exception\PluginException: Unknown plugin ID 'tour-ui-test-image-tip'. in Drupal\Component\Plugin\DefaultPluginBag->initializePlugin() (line 79 of /Users/clemens/Sites/drupal/d8/www/core/lib/Drupal/Component/Plugin/DefaultPluginBag.php).Attached patch should reduce
from
FAILED: [[SimpleTest]]: [MySQL] 39 pass(es), 47 fail(s), and 4 exception(s).to
73 passes, 16 fails, 8 exceptions, and 22 debug messagesSo patch will be committed shortly.
Comment #5
clemens.tolboomThis is weird. Both patches from #2and #4 are now scheduled for testing.
#2 https://qa.drupal.org/pifr/test/619643
#4 https://qa.drupal.org/pifr/test/619653
Next time I'll commit immediately I should not add a patch or one with donottest ?
Comment #6
clemens.tolboomIn our tests we enable tour_test module which provides an image plugin style. But we do not have an editor for this.
I'm not sure whether this is a core bug as TipPluginImage has a different signature then TipPluginText.
Comment #7
clemens.tolboom8.x-1.x Failed FAILED: [[SimpleTest]]: [MySQL] 44 pass(es), 19 fail(s), and 42 exception(s). 22/09/2013 - 12:06
Comment #8
nick_schuch commentedThe "tour_test" TipPluginImage is a super basic plugin to test extendability of tour. It doesn't have an admin interface for the image plugin because no core tip types should provide an admin form. We should provide that in contrib. We want to be able to innovate on the admin interface.
If you are ok with this I'll take this one and test.
Comment #9
clemens.tolboomTag this with needs tests to find it back later
Comment #10
clemens.tolboom[edit]moved to #3004840: Fix testbot errors
Comment #11
clemens.tolboomLet's see whether we can kick the test with a removed attribute.
Trying to reimport the wrong tour file shows the same failure. Applying the patch makes manually re-import work. Yeah.
Comment #12
clemens.tolboomBack to the work done before the temp patch from #11.
Comment #13
clemens.tolboomPatch #4 is fixed through #2182095: Fix Tour UI to work on latest core