Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
forum.module
Priority:
Normal
Category:
Task
Assigned:
Issue tags:
Reporter:
Created:
10 Jul 2014 at 21:02 UTC
Updated:
22 Aug 2014 at 12:50 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
sjancich commentedComment #2
larowlanDo we need a test?
Comment #3
andypostWould be good to have a test
Comment #4
andypostSuppose the quick fix here is to change setting in
hook_install()Comment #5
mitsuroseba commentedComment #9
andyposttested this locally and no failures detected
Comment #10
andypostComment #12
jthorson commentedGET http://10.20.0.214/checkout/checkout/comment/reply/node/1/comment_forum/1 returned 404 (3.69 KB).Note the duplicate 'checkout' in the URL ... that should not be there. The testbot installs in a subdirectory (i.e. not at the web root), whereas I suspect your local tests which are passing are on a site installed at '/'.
The verbose output for the failed tests is a 'Page not found'.
Comment #13
mitsuroseba commentedThank you, @jthorson!
Comment #14
mitsuroseba commentedinterdiff.
Comment #15
andypostGreat! @larowlan are you agree?
Comment #16
larowlanFine by me
Comment #17
alexpottI don;t think we need to tests in the Forum module for non threaded comments - this is extensively tested elsewhere - we're just using another module's API here.
Comment #18
mitsuroseba commentedThank you, @alexpott!
Remove test.
Comment #19
andypostback to rtbc
Comment #20
alexpottCommitted dc11fb9 and pushed to 8.0.x. Thanks!