Commit Graph

55 Commits

Author SHA1 Message Date
zach%zachlipton.com
d35ceb8627 Fix for bug 143124, Fix warning messages about *::TESTOUT and clean up
test code. Patch makes the tests much better now.

r=zach x2
2002-05-08 23:49:32 +00:00
gerv%gerv.net
2743a43bf1 Bug 140435 - Templatise GetCommandMenu. Patch by gerv; r=bbaetz, justdave. 2002-05-07 21:28:52 +00:00
bbaetz%student.usyd.edu.au
09a9c41e7a Bug 135543 - @Support::Templates::testitems does not list all templates
Patch by ddkilzer@theracingworld.com (David D. Kilzer), r=gerv, justdave
2002-05-07 09:45:32 +00:00
gerv%gerv.net
25f5d8eef4 Bug 136180 - use uri/url_quote filters correctly. Patch by ddk; 2xr=gerv. 2002-04-24 18:27:43 +00:00
justdave%syndicomm.com
6ef0e88937 Fix tinderbox bustage caused by the template location code change. The tests are now looking in the right place for the
templates.  4 templates in the new location now come up with syntax errors however.
2002-04-24 09:50:53 +00:00
gerv%gerv.net
0a5ed8dad9 Bug 137589 - fix test 4 not to hang on the "use CGI" in the template. 2002-04-17 21:10:43 +00:00
zach%zachlipton.com
1c03ca5b01 Fix for bug 135291: add version string to new templates and fix the string for non
xml templates.

r=bbaetz, r2=ddk
2002-04-04 07:01:17 +00:00
zach%zachlipton.com
895b099b56 Fix for bug 98658: Let administrator know which customised templates have been updated by Bugzilla team.
Patch adds a version string to every template and a check in
t/004template.t to check for version strings in templates.
Note that two templates that were not included in the initial patch
now have version strings added.

r=justdave, r2=myk
2002-04-03 20:01:42 +00:00
bbaetz%student.usyd.edu.au
34a9262f08 Bug 130254 - Template params don't have to exist, since they may be
subject to interpolation. Ignore names with $ in them as a workaround
r=myk x 2
2002-03-12 13:33:55 +00:00
jake%acutex.net
effc112ada Update the test to use the same basic format as the handling code from globals.pl 2002-02-13 19:24:05 +00:00
justdave%syndicomm.com
e24c04e22a Fix for bug 108982: enable taint mode for all user-facing CGI files.
Patch by Brad Baetz <bbaetz@student.usyd.edu.au>
r= jake, justdave
2002-01-20 01:44:52 +00:00
dkl%redhat.com
be9e066b76 Fix for bug 119060 - Use of Template.pm filters for url and html encoding. r=jake,dkl 2002-01-18 16:50:08 +00:00
zach%zachlipton.com
479c20a58b Additional fix for bug 112914: "Test should not cat $file" Previous
patch fixed the problem, but this patch stops once it finds strict so
it will not take as long to run.

Patch by ddkilzer@theracingworld.com. R=zach@zachlipton.com though review
is not required for tests.
2002-01-11 03:53:18 +00:00
jake%acutex.net
21bfc9968e Bug 112914 - This test was relying on the existance of the unix "cat" command instead of using perl's standard open() function. This caused a test failed when running the tinderbox script on win32. 2002-01-04 16:04:09 +00:00
jake%acutex.net
8429b5989b Get the file list of files to scan for templates from Support::Files instead of glob('*') 2001-11-30 21:49:47 +00:00
zach%zachlipton.com
68a56e9d2b Fix the double testing of .pm files.
No review needed for tests at this time.
2001-11-17 17:29:06 +00:00
jake%acutex.net
cca68c9a93 We don't really need to look for fragments that are pulled in by [% INCLUDE %] or [% PROCESS %]. While removing this code bit doesn't allow us to seperatly check that those fragments exist and compile, they'll be checked atomatically when the the template that wants them is run through the process() routine by the 004template.t test. This issue was raised because bug 98707 introduced a [% BLOCK %] element and the syntax for using that is the same as for including a template fragment. 2001-11-14 03:44:07 +00:00
zach%zachlipton.com
5669c51e6b Have the tests test .pm files as well, yes this creates bustage, this is
intentional!

No review needed for tests at this time.
2001-10-31 01:06:31 +00:00
jake%acutex.net
c30466ca7c A few enhancements to the template test:
* If there's a compilation error, report what it is
 * Don't try to compile a template if it doesn't exist
   - We already tested for that and issued an ERROR
 * Define the 'url' FILTER
2001-10-27 15:27:31 +00:00
jake%acutex.net
d6fb901800 Check templates for tabs, also. 2001-10-26 13:21:38 +00:00
jake%acutex.net
6747fcb289 Don't rely on the TEST_VERBOSE environment variable (no longer exported from runtests.sh) and instead print to the TESTOUT file handle pulled in from Test::More. This will allow the testing backend to check for verbosity rather than having to handle it in the .t files. 2001-10-24 18:41:53 +00:00
jake%acutex.net
670ce08714 Bug 106424 - We weren't going orange on warnings such as "used only once". This script now properly outputs the --WARNING and fails on such a condition. 2001-10-24 13:29:49 +00:00
jake%acutex.net
e2b7472ba4 Bug 103664 - Tests should "use strict;" and not contain any tabs. We should also use the TEST_VERBOSE environment variable instead of VERBOSE.
Patch by David D. Kilzer <ddkilzer@theracingworld.com>
Additional edits by myself to add the emacs mode line.  Also, the change to runtests.sh was done by me.
2001-10-19 22:39:51 +00:00
justdave%syndicomm.com
57dc85bacc yet another meaningless whitespace change to test tinderbox with 2001-10-19 13:03:52 +00:00
justdave%syndicomm.com
84b6128d34 another useless whitespace change to test tinderbox with 2001-10-19 06:19:52 +00:00
justdave%syndicomm.com
95bb6c3a2f whitespace change to test tinderbox again. 2001-10-19 05:33:23 +00:00
justdave%syndicomm.com
eba7113fc2 whitespace change to do a test checkin to see how tinderbox reacts 2001-10-19 04:36:19 +00:00
jake%acutex.net
3d8a2fdf39 Tabs are not allowed in Bugzilla source... if we find one, issue a warning.
No review needed for tests at this time. NOT PART OF BUILD
2001-10-13 01:44:06 +00:00
jake%acutex.net
7bdf2856ba If a template doesn't exist, this should be considered a compile failure.
No review needed for tests at this time. NOT PART OF BUILD
2001-10-12 20:35:37 +00:00
jake%acutex.net
9e3be8f1c7 It's also possible to PROCESS another attachment fragment, not just INCLUDE.
No review needed for tests at this time. NOT PART OF BUILD.
2001-10-09 19:14:59 +00:00
jake%acutex.net
b351c9eaa8 Allow compile tests for files that run in taint mode
No review needed for tests at this time. NOT PART OF BUILD
2001-10-06 23:22:48 +00:00
jake%acutex.net
90ef6cf6a4 We also need to look for templates that are INCLUDEd in other templates.
No review needed for tests at this time. NOT PART OF BUILD
2001-10-06 21:37:22 +00:00
jake%acutex.net
f503702088 Test that scans Bugzilla's code looking for used templates then checks to make sure that:
a) All the required templates exist
 b) They all have good syntax

No review needed for tests at this time. NOT PART OF BUILD
2001-10-06 18:59:36 +00:00
zach%zachlipton.com
8d4abc6e4a Fixing issue where excluded files aren't excluded if they end in .pl or
.cgi.
2001-10-06 03:49:06 +00:00
zach%zachlipton.com
e2a1e61fca Don't test importxml.pl since you need XML::Parser which is not required,
creating a false fail.

No review needed for tests. NOT PART OF BUILD
2001-10-06 03:25:43 +00:00
zach%zachlipton.com
1898fb6cc6 Get rid of our which causes errors on 5.005.
No review needed for tests at this time. NOT PART OF BUILD
2001-10-06 01:23:38 +00:00
zach%zachlipton.com
2f3b70d35e Renaming *.t to be 00*.t to solve any sorting issues down the road as per
bug 97976.

No review required for tests at this time. NOT_PART_OF_BUILD
2001-09-23 04:08:44 +00:00
zach%zachlipton.com
88ae8c099f *** empty log message *** 2001-09-23 04:02:19 +00:00
zach%zachlipton.com
f19ab6c7a1 Fix to make the tests generate the list of files to test instead of having to update the list manually whenever a file is added
or removed. The main logic is in t/Support/Files.pm, changes to the test files are needed to allow it to function properly.

NOT_PART_OF_BUILD. No review needed for tests at this time.
2001-09-16 18:35:18 +00:00
zach%zachlipton.com
d4b24ad2a9 Make this file test for exec calls in addition to system calls that are
one-argument and are therefore insecure.

NOT_PART_OF_BUILD, no review needed.
2001-09-16 17:52:19 +00:00
zach%zachlipton.com
7c0ea78940 Oops. Forgot to check this is. 2001-09-09 19:48:15 +00:00
zach%zachlipton.com
d1851dde5d Adding a test for safe system and exec calls. Part of this code by Dave. 2001-09-05 01:43:25 +00:00
zach%zachlipton.com
4bc671ac36 Add a little 1; to the end of this. 2001-09-04 04:28:24 +00:00
zach%zachlipton.com
07ee72b713 Actually make the strict checking work. 2001-09-03 22:11:43 +00:00
zach%zachlipton.com
4c3cc4a6a2 add tests for use strict; 2001-09-03 22:04:09 +00:00
zach%zachlipton.com
995690cb67 Enforce -w. This file will also be used to enforce use strict; 2001-09-03 21:43:22 +00:00
zach%zachlipton.com
13e7d849ce Rename 1.t to 1compile.t as per bug 97976. 2001-09-03 20:17:40 +00:00
zach%zachlipton.com
7af1301bfc Remove processmail from the list. It was causing problems with taint. 2001-09-03 19:39:37 +00:00
zach%zachlipton.com
b708e0e513 Ok, really this time. This should work :) 2001-09-03 19:26:58 +00:00
zach%zachlipton.com
c500ba418f Backout of this to get things back to normal 2001-09-03 18:50:56 +00:00