CRM-16617, fixed checkstyle jenkin error
authorPradeep Nayak <pradpnayak@gmail.com>
Mon, 8 Feb 2016 15:12:48 +0000 (20:42 +0530)
committerPradeep Nayak <pradpnayak@gmail.com>
Sat, 13 Feb 2016 14:43:32 +0000 (20:13 +0530)
----------------------------------------
* CRM-16617:
  https://issues.civicrm.org/jira/browse/CRM-16617

tests/phpunit/CRM/Core/DAOTest.php

index 6d8c0614163b4ebfbad17d656160d9c2a12c45f0..d44b23b95344c327611bb9a2b11d6fdb5219c7e8 100644 (file)
@@ -142,13 +142,17 @@ class CRM_Core_DAOTest extends CiviUnitTestCase {
     $this->assertEquals($expectSql, $actualSql);
   }
 
-  // CASE: Two params where the %2 is already present in the query
-  // NOTE: This case should rightly FAIL, as using strstr in the replace mechanism will turn
-  // the query into: SELECT * FROM whatever WHERE name = 'Alice' AND title = 'Bob' AND year LIKE ''Bob'012'
-  // So, to avoid such ERROR, the query should be framed like:
-  // 'SELECT * FROM whatever WHERE name = %1 AND title = %3 AND year LIKE '%2012'
-  // $params[3] = array('Bob', 'String');
-  // i.e. the place holder should be unique and should not contain in any other operational use in query
+  /**
+   *
+   * CASE: Two params where the %2 is already present in the query
+   * NOTE: This case should rightly FAIL, as using strstr in the replace mechanism will turn
+   * the query into: SELECT * FROM whatever WHERE name = 'Alice' AND title = 'Bob' AND year LIKE ''Bob'012'
+   * So, to avoid such ERROR, the query should be framed like:
+   * 'SELECT * FROM whatever WHERE name = %1 AND title = %3 AND year LIKE '%2012'
+   * $params[3] = array('Bob', 'String');
+   * i.e. the place holder should be unique and should not contain in any other operational use in query
+   *
+   */
   public function testComposeQueryFailure() {
     $cases[] = array(
       'SELECT * FROM whatever WHERE name = %1 AND title = %2 AND year LIKE \'%2012\' ',
@@ -173,25 +177,25 @@ class CRM_Core_DAOTest extends CiviUnitTestCase {
         'this is an even longer string which is exactly 60 character',
         60,
         FALSE,
-        'this is an even longer string which is exactly 60 character'
+        'this is an even longer string which is exactly 60 character',
       ),
       array(
         'this is an even longer string which is exactly 60 character',
         60,
         TRUE,
-        'this is an even longer string which is exactly 60 character'
+        'this is an even longer string which is exactly 60 character',
       ),
       array(
         'this is an even longer string which is a bit more than 60 character',
         60,
         FALSE,
-        'this is an even longer string which is a bit more than 60 ch'
+        'this is an even longer string which is a bit more than 60 ch',
       ),
       array(
         'this is an even longer string which is a bit more than 60 character',
         60,
         TRUE,
-        'this is an even longer string which is a bit more th_c1cbd519'
+        'this is an even longer string which is a bit more th_c1cbd519',
       ),
     );
   }