test: handling of rating null values as 0

Signed-off-by: Raju Udava <86527202+dstala@users.noreply.github.com>

packages/nocodb/tests/unit/rest/tests/filter.test.ts

@@ -96,19 +96,25 @@ async function retrieveRecordsAndValidate(
       );
       break;
     case 'lt':
-      expectedRecords = unfilteredRecords.filter(
+      expectedRecords = unfilteredRecords.filter((record) =>
-        (record) =>
+        title === 'Rating'
-          (toFloat ? parseFloat(record[title]) : record[title]) <
+          ? (toFloat ? parseFloat(record[title]) : record[title]) <
-            (toFloat ? parseFloat(filter.value) : filter.value) &&
+              (toFloat ? parseFloat(filter.value) : filter.value) ||
-          record[title] !== null
+            record[title] === null
+          : (toFloat ? parseFloat(record[title]) : record[title]) <
+              (toFloat ? parseFloat(filter.value) : filter.value) &&
+            record[title] !== null
       );
       break;
     case 'lte':
-      expectedRecords = unfilteredRecords.filter(
+      expectedRecords = unfilteredRecords.filter((record) =>
-        (record) =>
+        title === 'Rating'
-          (toFloat ? parseFloat(record[title]) : record[title]) <=
+          ? (toFloat ? parseFloat(record[title]) : record[title]) <=
-            (toFloat ? parseFloat(filter.value) : filter.value) &&
+              (toFloat ? parseFloat(filter.value) : filter.value) ||
-          record[title] !== null
+            record[title] === null
+          : (toFloat ? parseFloat(record[title]) : record[title]) <=
+              (toFloat ? parseFloat(filter.value) : filter.value) &&
+            record[title] !== null
       );
       break;
     case 'anyof':

tests/playwright/tests/filters.spec.ts

@@ -144,13 +144,18 @@ test.describe('Filter Tests: Numerical', () => {
       {
         op: '<',
         value: isLikeString,
-        rowCount: records.list.filter(r => r[dataType] < parseFloat(isLikeStringDerived) && r[dataType] != null).length,
+        rowCount:
+          dataType === 'Rating'
+            ? records.list.filter(r => r[dataType] < parseFloat(isLikeStringDerived)).length
+            : records.list.filter(r => r[dataType] < parseFloat(isLikeStringDerived) && r[dataType] != null).length,
       },
       {
         op: '<=',
         value: isLikeString,
-        rowCount: records.list.filter(r => r[dataType] <= parseFloat(isLikeStringDerived) && r[dataType] != null)
+        rowCount:
-          .length,
+          dataType === 'Rating'
+            ? records.list.filter(r => r[dataType] <= parseFloat(isLikeStringDerived)).length
+            : records.list.filter(r => r[dataType] <= parseFloat(isLikeStringDerived) && r[dataType] != null).length,
       },
     ];