Webtests and CRM test fix

[civicrm-core.git] / tests / phpunit / WebTest / Contact / SignatureTest.php

diff --git a/tests/phpunit/WebTest/Contact/SignatureTest.php b/tests/phpunit/WebTest/Contact/SignatureTest.php
index a4ce36cae1eea40c3b5a96ba960ca9afb164d9c3..5967bc590a4b0e38b70a4c1c5cc927e57bc1733a 100644 (file)
--- a/tests/phpunit/WebTest/Contact/SignatureTest.php
+++ b/tests/phpunit/WebTest/Contact/SignatureTest.php
@@ -1,7 +1,7 @@
 <?php
 /*
  +--------------------------------------------------------------------+
- | CiviCRM version 4.5                                                |
+ | CiviCRM version 4.6                                                |
  +--------------------------------------------------------------------+
  | Copyright CiviCRM LLC (c) 2004-2014                                |
  +--------------------------------------------------------------------+
@@ -38,7 +38,7 @@ class WebTest_Contact_SignatureTest extends CiviSeleniumTestCase {
   /**
    * Test Signature in TinyMC.
    */
-  function testTinyMCE() {
+  public function testTinyMCE() {
     $this->webtestLogin();
 
     $this->openCiviPage('dashboard', 'reset=1', 'crm-recently-viewed');
@@ -71,8 +71,8 @@ class WebTest_Contact_SignatureTest extends CiviSeleniumTestCase {
 
     // Go for Ckeck Your Editor, Click on Send Mail
     $this->click("//a[@id='crm-contact-actions-link']/span");
-    //after clicking on 'Send an Email', wait for the text in tinymce editor to load
-    $this->clickLink('link=Send an Email', "xpath=//body[@id='tinymce']/p[2]", FALSE);
+    // the other test checks this in a popup, we'll try it full-page here
+    $this->clickLinkSuppressPopup('link=Send an Email', "xpath=//body[@id='tinymce']/p[2]");
 
     $this->click('subject');
     $subject = 'Subject_' . substr(sha1(rand()), 0, 8);
@@ -93,7 +93,7 @@ class WebTest_Contact_SignatureTest extends CiviSeleniumTestCase {
   /**
    *  Test Signature in CKEditor.
    */
-  function testCKEditor() {
+  public function testCKEditor() {
     $this->webtestLogin();
 
     $this->openCiviPage('dashboard', 'reset=1', 'crm-recently-viewed');
@@ -144,7 +144,7 @@ class WebTest_Contact_SignatureTest extends CiviSeleniumTestCase {
   /**
    * Helper function to select Editor.
    */
-  function _selectEditor($editor) {
+  public function _selectEditor($editor) {
     $this->openCiviPage('admin/setting/preferences/display', 'reset=1');
 
     // Change editor if not already selected
@@ -159,7 +159,7 @@ class WebTest_Contact_SignatureTest extends CiviSeleniumTestCase {
   /**
    * Helper function for Check Signature in Editor.
    */
-  function _checkSignature($fieldName, $signature, $editor) {
+  public function _checkSignature($fieldName, $signature, $editor) {
     if ($editor == 'CKEditor') {
       $this->waitForElementPresent("xpath=//div[@id='cke_{$fieldName}']//iframe");
       $this->selectFrame("xpath=//div[@id='cke_{$fieldName}']//iframe");
@@ -175,7 +175,7 @@ class WebTest_Contact_SignatureTest extends CiviSeleniumTestCase {
   /**
    * Helper function for Check Signature in Activity.
    */
-  function _checkActivity($subject, $signature) {
+  public function _checkActivity($subject, $signature) {
     $this->openCiviPage('activity/search', 'reset=1', '_qf_Search_refresh');
 
     $this->type('activity_subject', $subject);
@@ -189,4 +189,3 @@ class WebTest_Contact_SignatureTest extends CiviSeleniumTestCase {
     $this->assertTextPresent($signature);
   }
 }
-