Move support for NOCONTENT search option from client.search to client.searchNoContent

Author: brettburch

packages/search/lib/commands/SEARCH.spec.ts

@@ -12,13 +12,6 @@ describe('SEARCH', () => {
             );
         });
 
-        it('with NOCONTENT', () => {
-            assert.deepEqual(
-                transformArguments('index', 'query', { NOCONTENT: true }),
-                ['FT.SEARCH', 'index', 'query', 'NOCONTENT']
-            );
-        });
-
         it('with VERBATIM', () => {
             assert.deepEqual(
                 transformArguments('index', 'query', { VERBATIM: true }),

packages/search/lib/commands/SEARCH.ts

@@ -6,7 +6,6 @@ export const FIRST_KEY_INDEX = 1;
 export const IS_READ_ONLY = true;
 
 export interface SearchOptions {
-    NOCONTENT?: true;
     VERBATIM?: true;
     NOSTOPWORDS?: true;
     // WITHSCORES?: true;

packages/search/lib/commands/SEARCH_NOCONTENT.spec.ts

@@ -0,0 +1,36 @@
+import { strict as assert } from 'assert';
+import { SchemaFieldTypes } from '.';
+import testUtils, { GLOBAL } from '../test-utils';
+import { transformArguments } from './SEARCH_NOCONTENT';
+
+describe('SEARCH_NOCONTENT', () => {
+    describe('transformArguments', () => {
+        it('without options', () => {
+            assert.deepEqual(
+                transformArguments('index', 'query'),
+                ['FT.SEARCH', 'index', 'query', 'NOCONTENT']
+            );
+        });
+    });
+
+    describe('client.ft.searchNoContent', () => {
+        testUtils.testWithClient('returns total and keys', async client => {
+            await Promise.all([
+                client.ft.create('index', {
+                    field: SchemaFieldTypes.NUMERIC
+                }),
+                client.hSet('1', 'field', 'field1'),
+                client.hSet('2', 'field', 'field2'),
+                client.hSet('3', 'field', 'field3')
+            ]);
+
+            assert.deepEqual(
+                await client.ft.searchNoContent('index', '*'),
+                {
+                    total: 3,
+                    documents: ['1','2','3']
+                }
+            );
+        }, GLOBAL.SERVERS.OPEN);
+    });
+});

packages/search/lib/commands/SEARCH_NOCONTENT.ts

@@ -0,0 +1,30 @@
+import { RedisCommandArguments } from "@redis/client/dist/lib/commands";
+import { pushSearchOptions } from ".";
+import { SearchOptions, SearchRawReply } from "./SEARCH";
+
+export const FIRST_KEY_INDEX = 1;
+
+export const IS_READ_ONLY = true;
+
+export function transformArguments(
+    index: string,
+    query: string,
+    options?: SearchOptions
+): RedisCommandArguments {
+    return pushSearchOptions(
+        ['FT.SEARCH', index, query, 'NOCONTENT'],
+        options
+    );
+}
+
+export interface SearchNonContentReply {
+    total: number;
+    documents: Array<string>;
+};
+
+export function transformReply(reply: SearchRawReply): SearchNonContentReply {
+    return {
+        total: reply[0],
+        documents: reply.splice(0)
+    };
+}

packages/search/lib/commands/index.ts

@@ -20,6 +20,7 @@ import * as INFO from './INFO';
 import * as PROFILESEARCH from './PROFILE_SEARCH';
 import * as PROFILEAGGREGATE from './PROFILE_AGGREGATE';
 import * as SEARCH from './SEARCH';
+import * as SEARCH_NOCONTENT from './SEARCH_NOCONTENT';
 import * as SPELLCHECK from './SPELLCHECK';
 import * as SUGADD from './SUGADD';
 import * as SUGDEL from './SUGDEL';
@@ -80,6 +81,8 @@ export default {
     profileAggregate: PROFILEAGGREGATE,
     SEARCH,
     search: SEARCH,
+    SEARCH_NOCONTENT,
+    searchNoContent: SEARCH_NOCONTENT,
     SPELLCHECK,
     spellCheck: SPELLCHECK,
     SUGADD,
@@ -395,10 +398,6 @@ export function pushSearchOptions(
     args: RedisCommandArguments,
     options?: SearchOptions
 ): RedisCommandArguments {
-    if (options?.NOCONTENT) {
-        args.push('NOCONTENT');
-    }
-
     if (options?.VERBATIM) {
         args.push('VERBATIM');
     }