Skip to content
This repository

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse code

Fix: fnRender unit tests need to be updated for the change to the sec…

…ond argument being passed
  • Loading branch information...
commit dae243eb7a9000d89e89921758ad21d710aca438 1 parent 8c9f0b6
Allan Jardine DataTables authored
9 media/unit_testing/tests_onhold/1_dom/aoColumns.fnRender.js
@@ -25,16 +25,17 @@ $(document).ready( function () {
25 25 );
26 26
27 27 oTest.fnTest(
28   - "Confirm that fnRender passes one argument (an object) with three parameters",
  28 + "Confirm that fnRender passes two arguments with four parameters",
29 29 function () {
30 30 mTmp = true;
31 31 oSession.fnRestore();
32 32 oTable = $('#example').dataTable( {
33 33 "aoColumns": [
34 34 null,
35   - { "fnRender": function (a) {
36   - if ( arguments.length != 1 || typeof a.iDataRow=='undefined' ||
37   - typeof a.iDataColumn=='undefined' || typeof a.aData=='undefined' )
  35 + { "fnRender": function (a, v) {
  36 + if ( arguments.length != 2 || typeof a.iDataRow=='undefined' ||
  37 + typeof a.iDataColumn=='undefined' || typeof a.aData=='undefined' ||
  38 + typeof a.mDataProp=='undefined' )
38 39 {
39 40 mTmp = false;
40 41 }
7 media/unit_testing/tests_onhold/2_js/aoColumns.fnRender.js
@@ -26,7 +26,7 @@ $(document).ready( function () {
26 26 );
27 27
28 28 oTest.fnTest(
29   - "Confirm that fnRender passes one argument (an object) with three parameters",
  29 + "Confirm that fnRender passes two arguments with four parameters",
30 30 function () {
31 31 mTmp = true;
32 32 oSession.fnRestore();
@@ -35,8 +35,9 @@ $(document).ready( function () {
35 35 "aoColumns": [
36 36 null,
37 37 { "fnRender": function (a) {
38   - if ( arguments.length != 1 || typeof a.iDataRow=='undefined' ||
39   - typeof a.iDataColumn=='undefined' || typeof a.aData=='undefined' )
  38 + if ( arguments.length != 2 || typeof a.iDataRow=='undefined' ||
  39 + typeof a.iDataColumn=='undefined' || typeof a.aData=='undefined' ||
  40 + typeof a.mDataProp=='undefined' )
40 41 {
41 42 mTmp = false;
42 43 }
7 media/unit_testing/tests_onhold/3_ajax/aoColumns.fnRender.js
@@ -26,7 +26,7 @@ $(document).ready( function () {
26 26 );
27 27
28 28 oTest.fnWaitTest(
29   - "Confirm that fnRender passes one argument (an object) with three parameters",
  29 + "Confirm that fnRender passes two arguments with four parameters",
30 30 function () {
31 31 mTmp = true;
32 32 oSession.fnRestore();
@@ -35,8 +35,9 @@ $(document).ready( function () {
35 35 "aoColumns": [
36 36 null,
37 37 { "fnRender": function (a) {
38   - if ( arguments.length != 1 || typeof a.iDataRow=='undefined' ||
39   - typeof a.iDataColumn=='undefined' || typeof a.aData=='undefined' )
  38 + if ( arguments.length != 2 || typeof a.iDataRow=='undefined' ||
  39 + typeof a.iDataColumn=='undefined' || typeof a.aData=='undefined' ||
  40 + typeof a.mDataProp=='undefined' )
40 41 {
41 42 mTmp = false;
42 43 }
7 media/unit_testing/tests_onhold/4_server-side/aoColumns.fnRender.js
@@ -27,7 +27,7 @@ $(document).ready( function () {
27 27 );
28 28
29 29 oTest.fnWaitTest(
30   - "Confirm that fnRender passes one argument (an object) with three parameters",
  30 + "Confirm that fnRender passes two arguments with four parameters",
31 31 function () {
32 32 mTmp = true;
33 33 oSession.fnRestore();
@@ -37,8 +37,9 @@ $(document).ready( function () {
37 37 "aoColumns": [
38 38 null,
39 39 { "fnRender": function (a) {
40   - if ( arguments.length != 1 || typeof a.iDataRow=='undefined' ||
41   - typeof a.iDataColumn=='undefined' || typeof a.aData=='undefined' )
  40 + if ( arguments.length != 2 || typeof a.iDataRow=='undefined' ||
  41 + typeof a.iDataColumn=='undefined' || typeof a.aData=='undefined' ||
  42 + typeof a.mDataProp=='undefined' )
42 43 {
43 44 mTmp = false;
44 45 }
7 media/unit_testing/tests_onhold/5_ajax_objects/aoColumns.fnRender.js
@@ -29,7 +29,7 @@ $(document).ready( function () {
29 29 );
30 30
31 31 oTest.fnWaitTest(
32   - "Confirm that fnRender passes one argument (an object) with three parameters",
  32 + "Confirm that fnRender passes two arguments with four parameters",
33 33 function () {
34 34 mTmp = true;
35 35 oSession.fnRestore();
@@ -39,8 +39,9 @@ $(document).ready( function () {
39 39 { "mDataProp": "engine" },
40 40 {
41 41 "fnRender": function (a) {
42   - if ( arguments.length != 1 || typeof a.iDataRow=='undefined' ||
43   - typeof a.iDataColumn=='undefined' || typeof a.aData=='undefined' )
  42 + if ( arguments.length != 2 || typeof a.iDataRow=='undefined' ||
  43 + typeof a.iDataColumn=='undefined' || typeof a.aData=='undefined' ||
  44 + typeof a.mDataProp=='undefined' )
44 45 {
45 46 mTmp = false;
46 47 }
7 media/unit_testing/tests_onhold/6_delayed_rendering/aoColumns.fnRender.js
@@ -27,7 +27,7 @@ $(document).ready( function () {
27 27 );
28 28
29 29 oTest.fnWaitTest(
30   - "Confirm that fnRender passes one argument (an object) with three parameters",
  30 + "Confirm that fnRender passes two arguments with four parameters",
31 31 function () {
32 32 mTmp = true;
33 33 oSession.fnRestore();
@@ -37,8 +37,9 @@ $(document).ready( function () {
37 37 "aoColumns": [
38 38 null,
39 39 { "fnRender": function (a) {
40   - if ( arguments.length != 1 || typeof a.iDataRow=='undefined' ||
41   - typeof a.iDataColumn=='undefined' || typeof a.aData=='undefined' )
  40 + if ( arguments.length != 2 || typeof a.iDataRow=='undefined' ||
  41 + typeof a.iDataColumn=='undefined' || typeof a.aData=='undefined' ||
  42 + typeof a.mDataProp=='undefined' )
42 43 {
43 44 mTmp = false;
44 45 }

0 comments on commit dae243e

Please sign in to comment.
Something went wrong with that request. Please try again.