[backend] Fix of creation with associated file in entities (#6075)

opencti-platform/opencti-front/tests_e2e/general/navigation.spec.ts

@@ -89,8 +89,8 @@ test('Check navigation on all pages', async ({ page }) => {
 
   // Other
   await leftBarPage.clickOnMenu('Investigations');
-  await expect(page.getByText('Investigations')).toBeVisible();
+  await expect(page.getByRole('paragraph')).toHaveText('Investigations');
   await leftBarPage.clickOnMenu('Dashboards');
-  await expect(page.locator('ol').getByText('Dashboards')).toBeVisible();
+  await expect(page.getByRole('paragraph')).toHaveText('Dashboards');
 
 });

opencti-platform/opencti-front/tests_e2e/report/createReport.spec.ts

@@ -2,6 +2,7 @@ import { expect, test } from '../fixtures/baseFixtures';
 import { ReportPage } from '../model/report.pageModel';
 import { ReportDetailsPage } from '../model/reportDetails.pageModel';
 import { ReportFormPage } from '../model/reportForm.pageModel';
+import * as path from 'path';
 
 test('Create a new report page', async ({ page }) => {
   const reportPage = new ReportPage(page);
@@ -11,6 +12,24 @@ test('Create a new report page', async ({ page }) => {
   await reportPage.addNewReport();
   await reportForm.fillNameInput('Test e2e');
   await reportPage.getCreateReportButton().click();
-  await reportPage.getItemFromList('Test e2e Unknown - admin No').click();
+  await reportPage.getItemFromList('Test e2e').click();
   await expect(reportDetailsPage.getReportDetailsPage()).toBeVisible();
 });
+
+test('Create a new report with associated file', async ({ page }) => {
+  const reportPage = new ReportPage(page);
+  const reportDetailsPage = new ReportDetailsPage(page);
+  const reportForm = new ReportFormPage(page);
+  await page.goto('/dashboard/analyses/reports');
+  await reportPage.addNewReport();
+  await reportForm.fillNameInput('Test e2e with file');
+  const fileChooserPromise = page.waitForEvent('filechooser');
+  await page.getByRole('button', { name: 'Select your file', exact: true }).click();
+  const fileChooser = await fileChooserPromise;
+  await fileChooser.setFiles(path.join(__dirname, 'createReport.spec.ts'));
+  await reportPage.getCreateReportButton().click();
+  await reportPage.getItemFromList('Test e2e with file').click();
+  await expect(reportDetailsPage.getReportDetailsPage()).toBeVisible();
+  await page.getByRole('tab', { name: 'Data' }).click();
+  await expect(page.getByRole('button', { name: 'createReport.spec.ts Launch' })).toBeVisible()
+});

opencti-platform/opencti-front/tests_e2e/report/updateReport.spec.ts

@@ -11,7 +11,7 @@ test('Create a new report page and test update', async ({ page }) => {
   await reportPage.addNewReport();
   await reportForm.fillNameInput('Test Update e2e');
   await reportPage.getCreateReportButton().click();
-  await reportPage.getItemFromList( 'Test Update e2e Unknown - admin No' ).click();
+  await reportPage.getItemFromList( 'Test Update e2e' ).click();
   await reportDetailsPage.getEditButton().click();
   await reportForm.fillNameInput('Modification Test Update e2e');
   await reportForm.getCloseButton().click();

opencti-platform/opencti-front/tests_e2e/settings/parameters.spec.ts

@@ -15,7 +15,6 @@ test('Check EE activation', async ({ page }) => {
   await page.getByRole('button', { name: 'Enable' }).click();
   expect(page.getByText(/^Enterprise$/)).toBeVisible();
   await page.getByRole('button', { name: 'Disable Enterprise Edition' }).click();
-  expect(page.getByText(/^Enterprise$/)).toBeVisible();
 });
 
 test('Check Logo replacement', async ({ page }) => {

opencti-platform/opencti-graphql/src/database/middleware.js

@@ -1495,7 +1495,7 @@ const updateAttributeRaw = async (context, user, instance, inputs, opts = {}) =>
         const preparedAliases = (aliasesInput.value ?? [])
           .filter((a) => isNotEmptyField(a))
           .filter((a) => normalizeName(a) !== normalizeName(instance.name)
-              && normalizeName(a) !== normalizeName(askedModificationName))
+            && normalizeName(a) !== normalizeName(askedModificationName))
           .map((a) => a.trim());
         aliasesInput.value = R.uniqBy((e) => normalizeName(e), preparedAliases);
       }
@@ -1667,12 +1667,12 @@ const updateAttributeRaw = async (context, user, instance, inputs, opts = {}) =>
   // Impact the updated_at only if stix data is impacted
   // In case of upsert, this addition will be supported by the parent function
   if (impactedInputs.length > 0 && isUpdatedAtObject(instance.entity_type)
-      && !impactedInputs.find((i) => i.key === 'updated_at')) {
+    && !impactedInputs.find((i) => i.key === 'updated_at')) {
     const updatedAtInput = { key: 'updated_at', value: [today] };
     impactedInputs.push(updatedAtInput);
   }
   if (impactedInputs.length > 0 && isModifiedObject(instance.entity_type)
-      && !impactedInputs.find((i) => i.key === 'modified')) {
+    && !impactedInputs.find((i) => i.key === 'modified')) {
     const modifiedAtInput = { key: 'modified', value: [today] };
     impactedInputs.push(modifiedAtInput);
   }
@@ -3005,7 +3005,7 @@ const buildEntityData = async (context, user, input, type, opts = {}) => {
   // If file directly attached
   if (!isEmptyField(input.file)) {
     const path = `import/${type}/${internalId}`;
-    const file = await upload(context, user, path, input.file, { entity: { internal_id: internalId } });
+    const file = await upload(context, user, path, input.file, { entity: data });
     data.x_opencti_files = [storeFileConverter(user, file)];
     // Add external references from files if necessary
     const entitySetting = await getEntitySettingFromCache(context, type);
@@ -3043,7 +3043,7 @@ const buildEntityData = async (context, user, input, type, opts = {}) => {
 
 const createEntityRaw = async (context, user, rawInput, type, opts = {}) => {
   // region confidence control
-  const input = structuredClone(rawInput);
+  const input = { ...rawInput };
   const { confidenceLevelToApply } = controlCreateInputWithUserConfidence(user, input);
   input.confidence = confidenceLevelToApply; // confidence of new entity will be capped to user's confidence
   // endregion