fix: lots of tests

ForestAdmin · Oct 18, 2024 · abf22c3 · abf22c3
1 parent b16a2ec
commit abf22c3
Show file tree

Hide file tree

Showing 21 changed files with 359 additions and 62 deletions.
diff --git a/packages/_example/docker-compose.yml b/packages/_example/docker-compose.yml
@@ -28,7 +28,6 @@ services:
   mssql:
     image: mcr.microsoft.com/azure-sql-edge
     container_name: forest_agent_example_mssql
-    user: root
     ports:
       - '1433:1433'
     environment:

diff --git a/packages/_example/src/forest/customizations/card.ts b/packages/_example/src/forest/customizations/card.ts
@@ -4,16 +4,14 @@ export default (collection: CardCustomizer) =>
   collection
     .addManyToOneRelation('customer', 'customer', { foreignKey: 'customer_id' })
     .addAction('Create new card', {
-      scope: 'Single',
+      scope: 'Global',
       execute: (context, resultBuilder) => {
         return resultBuilder.success('ok');
       },
       form: [
         {
           type: 'Layout',
           component: 'Page',
-          nextButtonLabel: 'go to next page',
-          previousButtonLabel: 'back',
           elements: [
             {
               type: 'String',
@@ -61,8 +59,8 @@ export default (collection: CardCustomizer) =>
         {
           type: 'Layout',
           component: 'Page',
-          nextButtonLabel: 'go to next page (finalisation, setup of card properties)',
-          previousButtonLabel: 'return to previous page (selection of the card plan)',
+          nextButtonLabel: 'go to next page',
+          previousButtonLabel: 'go to previous page',
           elements: [
             {
               type: 'Layout',
@@ -79,7 +77,6 @@ export default (collection: CardCustomizer) =>
         {
           type: 'Layout',
           component: 'Page',
-          previousButtonLabel: 'return to previous page (useless empty page)',
           elements: [
             {
               type: 'Number',

diff --git a/packages/agent/test/routes/access/api-chart-collection.test.ts b/packages/agent/test/routes/access/api-chart-collection.test.ts
@@ -49,7 +49,12 @@ describe('CollectionApiChartRoute', () => {
         await route.handleApiChart(context);
 
         expect(dataSource.getCollection('books').renderChart).toHaveBeenCalledWith(
-          { email: 'marty@doclabs.com', requestId: expect.any(String), timezone: 'Europe/Paris' },
+          {
+            email: 'marty@doclabs.com',
+            requestId: expect.any(String),
+            request: { ip: expect.any(String) },
+            timezone: 'Europe/Paris',
+          },
           'myChart',
           [123],
         );
@@ -72,7 +77,12 @@ describe('CollectionApiChartRoute', () => {
         await route.handleSmartChart(context);
 
         expect(dataSource.getCollection('books').renderChart).toHaveBeenCalledWith(
-          { email: 'marty@doclabs.com', requestId: expect.any(String), timezone: 'Europe/Paris' },
+          {
+            email: 'marty@doclabs.com',
+            requestId: expect.any(String),
+            request: { ip: expect.any(String) },
+            timezone: 'Europe/Paris',
+          },
           'myChart',
           [123],
         );

diff --git a/packages/agent/test/routes/access/api-chart-datasource.test.ts b/packages/agent/test/routes/access/api-chart-datasource.test.ts
@@ -42,7 +42,12 @@ describe('DataSourceApiChartRoute', () => {
         await route.handleApiChart(context);
 
         expect(dataSource.renderChart).toHaveBeenCalledWith(
-          { email: 'marty@doclabs.com', requestId: expect.any(String), timezone: 'Europe/Paris' },
+          {
+            email: 'marty@doclabs.com',
+            requestId: expect.any(String),
+            request: { ip: expect.any(String) },
+            timezone: 'Europe/Paris',
+          },
           'myChart',
         );
         expect(context.response.body).toMatchObject({
@@ -64,7 +69,12 @@ describe('DataSourceApiChartRoute', () => {
         await route.handleSmartChart(context);
 
         expect(dataSource.renderChart).toHaveBeenCalledWith(
-          { email: 'marty@doclabs.com', requestId: expect.any(String), timezone: 'Europe/Paris' },
+          {
+            email: 'marty@doclabs.com',
+            requestId: expect.any(String),
+            request: { ip: expect.any(String) },
+            timezone: 'Europe/Paris',
+          },
           'myChart',
         );
         expect(context.response.body).toStrictEqual({ countCurrent: 12 });

diff --git a/packages/agent/test/routes/access/chart.test.ts b/packages/agent/test/routes/access/chart.test.ts
@@ -126,7 +126,12 @@ describe('ChartRoute', () => {
         await chart.handleChart(context);
 
         expect(dataSource.getCollection('books').aggregate).toHaveBeenCalledWith(
-          { ...user, requestId: expect.any(String), timezone: 'Europe/Paris' },
+          {
+            ...user,
+            requestId: expect.any(String),
+            request: { ip: expect.any(String) },
+            timezone: 'Europe/Paris',
+          },
           { conditionTree: null, search: null, searchExtended: false, segment: null },
           { field: undefined, groups: undefined, operation: 'Count' },
         );
@@ -259,7 +264,12 @@ describe('ChartRoute', () => {
 
             expect(dataSource.getCollection('books').aggregate).toHaveBeenCalledTimes(1);
             expect(dataSource.getCollection('books').aggregate).toHaveBeenCalledWith(
-              { ...user, requestId: expect.any(String), timezone: 'Europe/Paris' },
+              {
+                ...user,
+                requestId: expect.any(String),
+                request: { ip: expect.any(String) },
+                timezone: 'Europe/Paris',
+              },
               {
                 conditionTree: { field: 'name', operator: 'Present', value: null },
                 search: null,
@@ -308,6 +318,7 @@ describe('ChartRoute', () => {
               {
                 email: 'john.doe@domain.com',
                 requestId: expect.any(String),
+                request: { ip: expect.any(String) },
                 timezone: 'Europe/Paris',
               },
               {
@@ -393,7 +404,12 @@ describe('ChartRoute', () => {
         await chart.handleChart(context);
 
         expect(dataSource.getCollection('books').aggregate).toHaveBeenCalledWith(
-          { ...user, requestId: expect.any(String), timezone: 'Europe/Paris' },
+          {
+            ...user,
+            requestId: expect.any(String),
+            request: { ip: expect.any(String) },
+            timezone: 'Europe/Paris',
+          },
           { conditionTree: null, search: null, searchExtended: false, segment: null },
           { field: undefined, groups: undefined, operation: 'Count' },
         );
@@ -434,7 +450,12 @@ describe('ChartRoute', () => {
         await chart.handleChart(context);
 
         expect(dataSource.getCollection('books').aggregate).toHaveBeenCalledWith(
-          { email: 'john.doe@domain.com', requestId: expect.any(String), timezone: 'Europe/Paris' },
+          {
+            email: 'john.doe@domain.com',
+            requestId: expect.any(String),
+            request: { ip: expect.any(String) },
+            timezone: 'Europe/Paris',
+          },
           {
             conditionTree: { field: 'title', operator: 'NotContains', value: '[test]' },
             search: null,
@@ -478,7 +499,12 @@ describe('ChartRoute', () => {
         await chart.handleChart(context);
 
         expect(dataSource.getCollection('books').aggregate).toHaveBeenCalledWith(
-          { ...user, requestId: expect.any(String), timezone: 'Europe/Paris' },
+          {
+            ...user,
+            requestId: expect.any(String),
+            request: { ip: expect.any(String) },
+            timezone: 'Europe/Paris',
+          },
           { conditionTree: null, search: null, searchExtended: false, segment: null },
           { field: undefined, groups: [{ field: 'author:firstName' }], operation: 'Count' },
         );
@@ -530,7 +556,12 @@ describe('ChartRoute', () => {
         await chart.handleChart(context);
 
         expect(dataSource.getCollection('books').aggregate).toHaveBeenCalledWith(
-          { email: 'john.doe@domain.com', requestId: expect.any(String), timezone: 'Europe/Paris' },
+          {
+            email: 'john.doe@domain.com',
+            requestId: expect.any(String),
+            request: { ip: expect.any(String) },
+            timezone: 'Europe/Paris',
+          },
           {
             conditionTree: { field: 'title', operator: 'NotContains', value: '[test]' },
             search: null,
@@ -584,7 +615,12 @@ describe('ChartRoute', () => {
         await chart.handleChart(context);
 
         expect(dataSource.getCollection('books').aggregate).toHaveBeenCalledWith(
-          { ...user, requestId: expect.any(String), timezone: 'Europe/Paris' },
+          {
+            ...user,
+            requestId: expect.any(String),
+            request: { ip: expect.any(String) },
+            timezone: 'Europe/Paris',
+          },
           {
             conditionTree: factories.conditionTreeLeaf.build({
               operator: 'Present',
@@ -649,7 +685,12 @@ describe('ChartRoute', () => {
         await chart.handleChart(context);
 
         expect(dataSource.getCollection('books').aggregate).toHaveBeenCalledWith(
-          { ...user, requestId: expect.any(String), timezone: 'Europe/Paris' },
+          {
+            ...user,
+            requestId: expect.any(String),
+            request: { ip: expect.any(String) },
+            timezone: 'Europe/Paris',
+          },
           {
             conditionTree: factories.conditionTreeBranch.build({
               aggregator: 'And',
@@ -723,7 +764,12 @@ describe('ChartRoute', () => {
         await chart.handleChart(context);
 
         expect(dataSource.getCollection('books').aggregate).toHaveBeenCalledWith(
-          { ...user, requestId: expect.any(String), timezone: 'Europe/Paris' },
+          {
+            ...user,
+            requestId: expect.any(String),
+            request: { ip: expect.any(String) },
+            timezone: 'Europe/Paris',
+          },
           { conditionTree: null, search: null, searchExtended: false, segment: null },
           { field: 'id', groups: [{ field: 'author:id' }], operation: 'Sum' },
           2,
@@ -769,7 +815,12 @@ describe('ChartRoute', () => {
         await chart.handleChart(context);
 
         expect(dataSource.getCollection('books').aggregate).toHaveBeenCalledWith(
-          { ...user, requestId: expect.any(String), timezone: 'Europe/Paris' },
+          {
+            ...user,
+            requestId: expect.any(String),
+            request: { ip: expect.any(String) },
+            timezone: 'Europe/Paris',
+          },
           { conditionTree: null, search: null, searchExtended: false, segment: null },
           { operation: 'Count', field: null, groups: [{ field: 'publisher:id' }] },
           2,
@@ -822,7 +873,12 @@ describe('ChartRoute', () => {
         await chart.handleChart(context);
 
         expect(dataSource.getCollection('books').aggregate).toHaveBeenCalledWith(
-          { email: 'john.doe@domain.com', requestId: expect.any(String), timezone: 'Europe/Paris' },
+          {
+            email: 'john.doe@domain.com',
+            requestId: expect.any(String),
+            request: { ip: expect.any(String) },
+            timezone: 'Europe/Paris',
+          },
           {
             conditionTree: {
               field: 'author:name',
@@ -879,7 +935,12 @@ describe('ChartRoute', () => {
           await chart.handleChart(context);
 
           expect(dataSource.getCollection('books').aggregate).toHaveBeenCalledWith(
-            { ...user, requestId: expect.any(String), timezone: 'Europe/Paris' },
+            {
+              ...user,
+              requestId: expect.any(String),
+              request: { ip: expect.any(String) },
+              timezone: 'Europe/Paris',
+            },
             { conditionTree: null, search: null, searchExtended: false, segment: null },
             { groups: [{ field: 'author:id' }], operation: 'Count' },
             2,

diff --git a/packages/agent/test/routes/access/count-related.test.ts b/packages/agent/test/routes/access/count-related.test.ts
@@ -124,7 +124,12 @@ describe('CountRelatedRoute', () => {
           dataSource.getCollection('books'),
           ['2d162303-78bf-599e-b197-93590ac3d315'],
           'myBookPersons',
-          { email: 'john.doe@domain.com', requestId: expect.any(String), timezone: 'Europe/Paris' },
+          {
+            email: 'john.doe@domain.com',
+            requestId: expect.any(String),
+            request: { ip: expect.any(String) },
+            timezone: 'Europe/Paris',
+          },
           new Filter({
             search: 'searched argument',
             searchExtended: false,
@@ -223,7 +228,12 @@ describe('CountRelatedRoute', () => {
           dataSource.getCollection('books'),
           ['2d162303-78bf-599e-b197-93590ac3d315'],
           'myBookPersons',
-          { email: 'john.doe@domain.com', requestId: expect.any(String), timezone: 'Europe/Paris' },
+          {
+            email: 'john.doe@domain.com',
+            requestId: expect.any(String),
+            request: { ip: expect.any(String) },
+            timezone: 'Europe/Paris',
+          },
           new Filter({
             search: 'searched argument',
             searchExtended: false,

diff --git a/packages/agent/test/routes/access/count.test.ts b/packages/agent/test/routes/access/count.test.ts
@@ -33,7 +33,12 @@ describe('CountRoute', () => {
         await count.handleCount(context);
 
         expect(aggregateSpy).toHaveBeenCalledWith(
-          { email: 'john.doe@domain.com', requestId: expect.any(String), timezone: 'Europe/Paris' },
+          {
+            email: 'john.doe@domain.com',
+            requestId: expect.any(String),
+            request: { ip: expect.any(String) },
+            timezone: 'Europe/Paris',
+          },
           { conditionTree: null, search: null, searchExtended: false, segment: null },
           { operation: 'Count' },
         );
@@ -75,7 +80,12 @@ describe('CountRoute', () => {
         await count.handleCount(context);
 
         expect(aggregateSpy).toHaveBeenCalledWith(
-          { email: 'john.doe@domain.com', requestId: expect.any(String), timezone: 'Europe/Paris' },
+          {
+            email: 'john.doe@domain.com',
+            requestId: expect.any(String),
+            request: { ip: expect.any(String) },
+            timezone: 'Europe/Paris',
+          },
           {
             conditionTree: {
               field: 'title',

diff --git a/packages/agent/test/routes/access/csv-related.test.ts b/packages/agent/test/routes/access/csv-related.test.ts
@@ -122,7 +122,12 @@ describe('CsvRelatedRoute', () => {
 
       await readCsv(context.response.body as AsyncGenerator<string>);
       expect(csvGenerator).toHaveBeenCalledWith(
-        { email: 'john.doe@domain.com', requestId: expect.any(String), timezone: 'Europe/Paris' },
+        {
+          email: 'john.doe@domain.com',
+          requestId: expect.any(String),
+          request: { ip: expect.any(String) },
+          timezone: 'Europe/Paris',
+        },
         new Projection('id', 'name'),
         'id,name',
         paginatedFilter,
@@ -134,7 +139,12 @@ describe('CsvRelatedRoute', () => {
         dataSource.getCollection('books'),
         ['123e4567-e89b-12d3-a456-111111111111'],
         'myPersons',
-        { email: 'john.doe@domain.com', requestId: expect.any(String), timezone: 'Europe/Paris' },
+        {
+          email: 'john.doe@domain.com',
+          requestId: expect.any(String),
+          request: { ip: expect.any(String) },
+          timezone: 'Europe/Paris',
+        },
         expect.any(PaginatedFilter),
         expect.any(Projection),
       );
@@ -216,7 +226,12 @@ describe('CsvRelatedRoute', () => {
 
       await readCsv(context.response.body as AsyncGenerator<string>);
       expect(csvGenerator).toHaveBeenCalledWith(
-        { email: 'john.doe@domain.com', requestId: expect.any(String), timezone: 'Europe/Paris' },
+        {
+          email: 'john.doe@domain.com',
+          requestId: expect.any(String),
+          request: { ip: expect.any(String) },
+          timezone: 'Europe/Paris',
+        },
         new Projection('id', 'name'),
         'id,name',
         paginatedFilter,
@@ -228,7 +243,12 @@ describe('CsvRelatedRoute', () => {
         dataSource.getCollection('books'),
         ['123e4567-e89b-12d3-a456-111111111111'],
         'myPersons',
-        { email: 'john.doe@domain.com', requestId: expect.any(String), timezone: 'Europe/Paris' },
+        {
+          email: 'john.doe@domain.com',
+          requestId: expect.any(String),
+          request: { ip: expect.any(String) },
+          timezone: 'Europe/Paris',
+        },
         expect.any(PaginatedFilter),
         expect.any(Projection),
       );