Make fields and rows Openops tables helpers use database token authentication helpers (#1663)

Part of OPS-3021.

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

* **Refactor**
* Updated token/context handling and public parameter types across
table-related APIs.
  * Reorganized request header utilities to centralize header creation.

* **Tests**
  * Enhanced test mocks with numeric/boolean/throw-or-return helpers.
* Adjusted mock initialization to improve module loading and alignment
with header mocking.

* **Deprecation**
* Added deprecation notice for the default database ID helper; slated
for future removal.

<sub>✏️ Tip: You can customize this high-level summary in your review
settings.</sub>
<!-- end of auto-generated comment: release notes by coderabbit.ai -->

---------

Co-authored-by: Marcelo Gonçalves <marcelo@openops.com>

Author: cezudas

packages/openops/src/index.ts

@@ -39,6 +39,7 @@ export * from './lib/openops-analytics';
 export * from './lib/openops-tables/applications-service';
 export * from './lib/openops-tables/auth-user';
 export * from './lib/openops-tables/context-helpers';
+export * from './lib/openops-tables/create-axios-headers';
 export * from './lib/openops-tables/fields';
 export * from './lib/openops-tables/filters';
 export * from './lib/openops-tables/openops-tables-common-properties';

packages/openops/src/lib/openops-tables/applications-service.ts

@@ -1,4 +1,5 @@
-import { createAxiosHeaders, makeOpenOpsTablesGet } from './requests-helpers';
+import { createAxiosHeaders } from './create-axios-headers';
+import { makeOpenOpsTablesGet } from './requests-helpers';
 import { Application } from './types';
 
 export const OPENOPS_DEFAULT_DATABASE_NAME = 'OpenOps Dataset';

packages/openops/src/lib/openops-tables/fields.ts

@@ -1,17 +1,15 @@
 import { IAxiosRetryConfig } from 'axios-retry';
-import {
-  createAxiosHeaders,
-  makeOpenOpsTablesGet,
-} from '../openops-tables/requests-helpers';
+import { makeOpenOpsTablesGet } from '../openops-tables/requests-helpers';
 import { TokenOrResolver } from './context-helpers';
+import { createAxiosHeaders } from './create-axios-headers';
 
 export async function getFields(
   tableId: number,
-  contextOrToken: TokenOrResolver,
+  tokenOrResolver: TokenOrResolver,
   userFieldNames = true,
   axiosRetryConfig?: IAxiosRetryConfig,
 ): Promise<OpenOpsField[]> {
-  const authenticationHeader = createAxiosHeaders(contextOrToken);
+  const authenticationHeader = createAxiosHeaders(tokenOrResolver);
   const fields = await makeOpenOpsTablesGet<any[]>(
     `api/database/fields/table/${tableId}/?user_field_names=${userFieldNames}`,
     authenticationHeader,

packages/openops/src/lib/openops-tables/requests-helpers.ts

@@ -4,7 +4,6 @@ import { AxiosError, AxiosHeaders, AxiosResponse, Method } from 'axios';
 import { IAxiosRetryConfig } from 'axios-retry';
 import { StatusCodes } from 'http-status-codes';
 import { makeHttpRequest } from '../axios-wrapper';
-export { createAxiosHeaders } from './create-axios-headers';
 
 const RETRY_DELAY_MS = 1000;
 

packages/openops/src/lib/openops-tables/rows.ts

@@ -6,14 +6,14 @@ import {
   ViewFilterTypesEnum,
 } from '../openops-tables/filters';
 import {
-  createAxiosHeaders,
   makeOpenOpsTablesDelete,
   makeOpenOpsTablesGet,
   makeOpenOpsTablesPatch,
   makeOpenOpsTablesPost,
   makeOpenOpsTablesPut,
 } from '../openops-tables/requests-helpers';
 import { TokenOrResolver } from './context-helpers';
+import { createAxiosHeaders } from './create-axios-headers';
 
 export interface OpenOpsRow {
   id: number;
@@ -22,7 +22,7 @@ export interface OpenOpsRow {
 
 export interface RowParams {
   tableId: number;
-  token: string;
+  token: TokenOrResolver;
 }
 
 type RowParamsWithTokenOrContext = Omit<RowParams, 'token'> & {

packages/openops/src/lib/openops-tables/tables.ts

@@ -1,9 +1,7 @@
 import { AxiosHeaders } from 'axios';
-import {
-  createAxiosHeaders,
-  makeOpenOpsTablesGet,
-} from '../openops-tables/requests-helpers';
-import { resolveTokenProvider, TablesServerContext } from './context-helpers';
+import { makeOpenOpsTablesGet } from '../openops-tables/requests-helpers';
+import { TablesServerContext, resolveTokenProvider } from './context-helpers';
+import { createAxiosHeaders } from './create-axios-headers';
 
 export interface OpenOpsTable {
   id: number;

packages/openops/test/openops-tables/application-service.test.ts

@@ -2,6 +2,9 @@ const makeOpenOpsTablesGetMock = jest.fn();
 const createAxiosHeadersMock = jest.fn();
 jest.mock('../../src/lib/openops-tables/requests-helpers', () => ({
   makeOpenOpsTablesGet: makeOpenOpsTablesGetMock,
+}));
+
+jest.mock('../../src/lib/openops-tables/create-axios-headers', () => ({
   createAxiosHeaders: createAxiosHeadersMock,
 }));
 

packages/openops/test/openops-tables/fields.test.ts

@@ -2,6 +2,9 @@ const makeOpenOpsTablesGetMock = jest.fn();
 const createAxiosHeadersMock = jest.fn();
 jest.mock('../../src/lib/openops-tables/requests-helpers', () => ({
   makeOpenOpsTablesGet: makeOpenOpsTablesGetMock,
+}));
+
+jest.mock('../../src/lib/openops-tables/create-axios-headers', () => ({
   createAxiosHeaders: createAxiosHeadersMock,
 }));
 

packages/openops/test/openops-tables/rows.test.ts

@@ -14,10 +14,13 @@ jest.mock('../../src/lib/openops-tables/requests-helpers', () => {
     makeOpenOpsTablesPatch: makeOpenOpsTablesPatchMock,
     makeOpenOpsTablesPost: makeOpenOpsTablesPostMock,
     makeOpenOpsTablesDelete: makeOpenOpsTablesDeleteMock,
-    createAxiosHeaders: createAxiosHeadersMock,
   };
 });
 
+jest.mock('../../src/lib/openops-tables/create-axios-headers', () => ({
+  createAxiosHeaders: createAxiosHeadersMock,
+}));
+
 jest.mock('@openops/server-shared', () => ({
   ...jest.requireActual('@openops/server-shared'),
   logger: {

packages/openops/test/openops-tables/tables.test.ts

@@ -4,6 +4,9 @@ const createAxiosHeadersMock = jest.fn();
 
 jest.mock('../../src/lib/openops-tables/requests-helpers', () => ({
   makeOpenOpsTablesGet: makeOpenOpsTablesGetMock,
+}));
+
+jest.mock('../../src/lib/openops-tables/create-axios-headers', () => ({
   createAxiosHeaders: createAxiosHeadersMock,
 }));