544299 - Change Selenium PHP tests to work with Hudson

Status: RESOLVED INVALID

(support.mozilla.org :: General, defect)

Description

[Laura Thomson :laura]

We need a config file, basically.  Patch coming in comment #1

Comment 1

[Laura Thomson :laura]

Attachment: patch for tests/frontend/php_tests

I followed what clouserw did for moz.com but I'd just like you to check that I haven't broken anything...

Comment 2

[Vishal Kamdar [:vish_moz]]

Laura,
Please make the following changes:
1. leave the line 
   $this->selenium->open("http://support:[e-mail]");
   as it is in TC_rewrites.php

2. change
   if (!defined('SERVER_NAME'))
    define('SERVER_NAME', 'support:[e-mail]');
   
   to
   if (!defined('SERVER_NAME'))
    define('SERVER_NAME', 'support-stage.mozilla.org');
   
3. Also, the test cases files will have to be renamed. Just prepend the names with
   test_

Comment 3

[Laura Thomson :laura]

Attachment: patch with vish_moz changes

Comment 4

[Vishal Kamdar [:vish_moz]]

The changes look good.

Comment 5

[Stephen Donner [:stephend] Not actively reading bugmail]

(In reply to comment #4)
> The changes look good.

Vishal, please click on "Details" for Laura's patch and mark it as r+; thanks!

Comment 6

[Vishal Kamdar [:vish_moz]]

Comment on attachment 425485 [details] [diff] [review]
patch with vish_moz changes

marking it as 'Reviewed'.

Comment 7

[Laura Thomson :laura]

in r61845

Comment 8

[Stephen Donner [:stephend] Not actively reading bugmail]

Vishal, are you able to verify this?

Comment 9

[Laura Thomson :laura]

Stephen: I don't think he'll be able to...in fact I need to reopen this

Comment 10

[Vishal Kamdar [:vish_moz]]

Made changes to the rewrite script.
1. renamed config.php to configuration.php
2. removed moz_testCase inheritance
3. added code to varify as=u parameter in redirect URL's

Comment 11

[Vishal Kamdar [:vish_moz]]

Attachment: fix for the rewrite script.

Laura, sorry, I have already checked-in the fix at http://viewvc.svn.mozilla.org/vc/projects/sumo/tests/frontend/php_tests/test_TC_rewrites.php?view=log

Comment 12

[Kadir Topal [:atopal]]

I totally don't understand what this is about, but assuming this is about testing php code in some way, I'm closing this bug, since we are on python now.