rte commit: run just for arrays

Author: pawelangelow

redisinsight/api/test/api/array/POST-databases-id-array-get_count.test.ts

@@ -5,6 +5,7 @@ import {
   deps,
   Joi,
   requirements,
+  tag,
   generateInvalidDataTestCases,
   validateInvalidDataTestCase,
   validateApiCall,
@@ -38,6 +39,7 @@ const responseSchema = Joi.object()
 const mainCheckFn = getMainCheckFn(endpoint);
 
 describe('POST /databases/:instanceId/array/get-count', () => {
+  tag('array');
   requirements('rte.version>=8.8');
   beforeEach(async () => rte.data.truncate());
 

redisinsight/api/test/api/array/POST-databases-id-array-get_element.test.ts

@@ -6,6 +6,7 @@ import {
   deps,
   Joi,
   requirements,
+  tag,
   generateInvalidDataTestCases,
   validateInvalidDataTestCase,
   validateApiCall,
@@ -58,6 +59,7 @@ const seedSparse = (key: string) =>
   rte.client.call('ARMSET', key, '0', '20.1', '1', '20.4', '5', '21.4');
 
 describe('POST /databases/:instanceId/array/get-element', () => {
+  tag('array');
   requirements('rte.version>=8.8');
   beforeEach(async () => rte.data.truncate());
 

redisinsight/api/test/api/array/POST-databases-id-array-get_elements.test.ts

@@ -6,6 +6,7 @@ import {
   deps,
   Joi,
   requirements,
+  tag,
   generateInvalidDataTestCases,
   validateInvalidDataTestCase,
   validateApiCall,
@@ -57,6 +58,7 @@ const seedSparse = (key: string) =>
   rte.client.call('ARMSET', key, '0', '20.1', '1', '20.4', '5', '21.4');
 
 describe('POST /databases/:instanceId/array/get-elements', () => {
+  tag('array');
   requirements('rte.version>=8.8');
   beforeEach(async () => rte.data.truncate());
 

redisinsight/api/test/api/array/POST-databases-id-array-get_length.test.ts

@@ -6,6 +6,7 @@ import {
   deps,
   Joi,
   requirements,
+  tag,
   generateInvalidDataTestCases,
   validateInvalidDataTestCase,
   validateApiCall,
@@ -42,6 +43,7 @@ const responseSchema = Joi.object()
 const mainCheckFn = getMainCheckFn(endpoint);
 
 describe('POST /databases/:instanceId/array/get-length', () => {
+  tag('array');
   // Array is a Redis 8.8 preview type; skip where the server lacks ARLEN.
   requirements('rte.version>=8.8');
   beforeEach(async () => rte.data.truncate());

redisinsight/api/test/api/array/POST-databases-id-array-get_next_index.test.ts

@@ -6,6 +6,7 @@ import {
   deps,
   Joi,
   requirements,
+  tag,
   generateInvalidDataTestCases,
   validateInvalidDataTestCase,
   validateApiCall,
@@ -42,6 +43,7 @@ const responseSchema = Joi.object()
 const mainCheckFn = getMainCheckFn(endpoint);
 
 describe('POST /databases/:instanceId/array/get-next-index', () => {
+  tag('array');
   requirements('rte.version>=8.8');
   beforeEach(async () => rte.data.truncate());
 

redisinsight/api/test/api/array/POST-databases-id-array-get_range.test.ts

@@ -6,6 +6,7 @@ import {
   deps,
   Joi,
   requirements,
+  tag,
   generateInvalidDataTestCases,
   validateInvalidDataTestCase,
   validateApiCall,
@@ -59,6 +60,7 @@ const seedSparse = (key: string) =>
   rte.client.call('ARMSET', key, '0', '20.1', '1', '20.4', '5', '21.4');
 
 describe('POST /databases/:instanceId/array/get-range', () => {
+  tag('array');
   requirements('rte.version>=8.8');
   beforeEach(async () => rte.data.truncate());
 

redisinsight/api/test/api/array/POST-databases-id-array-scan.test.ts

@@ -6,6 +6,7 @@ import {
   deps,
   Joi,
   requirements,
+  tag,
   generateInvalidDataTestCases,
   validateInvalidDataTestCase,
   validateApiCall,
@@ -66,6 +67,7 @@ const seedSparse = (key: string) =>
   rte.client.call('ARMSET', key, '0', '20.1', '1', '20.4', '5', '21.4');
 
 describe('POST /databases/:instanceId/array/scan', () => {
+  tag('array');
   requirements('rte.version>=8.8');
   beforeEach(async () => rte.data.truncate());
 

redisinsight/api/test/helpers/test/conditionalIgnore.ts

@@ -30,6 +30,31 @@ export const requirements = function (...conditions) {
   });
 };
 
+/**
+ * Tag a suite. When the TEST_TAGS env var is set (e.g. TEST_TAGS=array on
+ * the oss-st-8 RTE), only suites whose declared tags intersect that list
+ * run; all others skip. When TEST_TAGS is unset, tags are a no-op — every
+ * suite runs as before.
+ *
+ * Mirrors requirements(); call inside a describe() block.
+ * note: add support for "it" if needed
+ *
+ * @param tags
+ */
+export const tag = function (...tags: string[]) {
+  const filter = process.env.TEST_TAGS?.split(',')
+    .map((s) => s.trim())
+    .filter(Boolean);
+
+  if (!filter?.length) return;
+
+  if (!tags.some((t) => filter.includes(t))) {
+    before(function () {
+      this.skip();
+    });
+  }
+};
+
 const cmdReg = /^([?!\w\.]+)(\s?[=<>]+)?(\s?[\w\.]+)?$/;
 const processConditionString = (condition: string): boolean => {
   if (!cmdReg.test(condition)) {

redisinsight/api/test/test-runs/oss-st-8/docker-compose.yml

@@ -3,3 +3,12 @@ version: '3.4'
 services:
   redis:
     image: redis:8.8-alpine
+
+  # Scope the run to suites that declared `tag('array')` (the new Array data
+  # type read endpoints). redis:8.8-alpine has no modules and ships Redis 8.8
+  # semantics (per-field hash TTL, RediSearch flag changes, etc.) that other
+  # untagged suites don't tolerate, so leaving them in would produce noise
+  # unrelated to the suite this RTE was added for.
+  test:
+    environment:
+      TEST_TAGS: array