Bug 874180 - The test suite fails in 110.CreateBrand.t and 920.WebSite.t
Summary: The test suite fails in 110.CreateBrand.t and 920.WebSite.t
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Publican
Classification: Community
Component: publican
Version: 3.0
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: 3.1
Assignee: Jeff Fearn 🐞
QA Contact: tools-bugs
URL:
Whiteboard:
Depends On: 661946
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-11-07 16:19 UTC by Raphaël Hertzog
Modified: 2013-02-06 03:40 UTC (History)
3 users (show)

Fixed In Version: 3.1.0
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-02-06 03:40:45 UTC
Embargoed:


Attachments (Terms of Use)
Patch fixing test-suite (1.62 KB, application/octet-stream)
2012-11-07 16:19 UTC, Raphaël Hertzog
no flags Details
Patch v2 fixing the test suite (5.45 KB, patch)
2012-11-07 21:30 UTC, Raphaël Hertzog
no flags Details | Diff

Description Raphaël Hertzog 2012-11-07 16:19:00 UTC
Created attachment 640176 [details]
Patch fixing test-suite

While packaging publican 3.0 I noticed that the test suite doesn't work because it uses some unavailable files in /usr/share/publican instead of using in-tree files the failing tests are 110.CreateBrand.t (because it doesn't override common_config / common_content) and 920.WebSite.t (because it doesn't override tmpl_path).

Please find attached a tested patch.

Comment 1 Raphaël Hertzog 2012-11-07 21:30:24 UTC
Created attachment 640401 [details]
Patch v2 fixing the test suite

I attach a new version of the patch because I missed one problem. /usr/share/publican/sitetemplate is hardcoded in bin/publican. I modified the script and Publican::WebSite so that it's a parameter now and I improved the test suite to use that new parameter.

Comment 2 Jeff Fearn 🐞 2012-11-25 23:23:36 UTC
Hi Raphaël, I'm going to tie this in to https://bugzilla.redhat.com/show_bug.cgi?id=661946#c1 which should make the whole 'where are things" a lot easier.

Comment 3 Jeff Fearn 🐞 2013-01-07 04:26:34 UTC
This fix has been committed to the devel branch for inclusion in Publican 3.1.

Comment 4 xuezhi ma 2013-01-31 08:20:48 UTC
Hi Jeff, 

I remember that QE won't test this bug, and we have add this bug in "Features not to be tested" in test plan, so please help to verify this bug and change the bug status, thanks very much


Note You need to log in before you can comment on or make changes to this bug.