From 5423e6817b737a5f3026c720ede2afabc6aa6454 Mon Sep 17 00:00:00 2001
From: PIETRO MARCHIORATO
 <120272502+pietromarchiorato@users.noreply.github.com>
Date: Wed, 1 May 2024 21:52:09 +0200
Subject: [PATCH 1/4] Add Persistenza Embeddings e Relative Exceptions

Aggiunta Persistenza Semplice degli Embeddings
---
 .../persistance/EmbeddingRepositoryAdapter.py | 29 ++++++++++++++-----
 backend/chatsql/utils/Exceptions.py           | 15 ++++++++++
 2 files changed, 37 insertions(+), 7 deletions(-)

diff --git a/backend/chatsql/adapter/outcoming/persistance/EmbeddingRepositoryAdapter.py b/backend/chatsql/adapter/outcoming/persistance/EmbeddingRepositoryAdapter.py
index 262fea6..b7f61b0 100644
--- a/backend/chatsql/adapter/outcoming/persistance/EmbeddingRepositoryAdapter.py
+++ b/backend/chatsql/adapter/outcoming/persistance/EmbeddingRepositoryAdapter.py
@@ -1,8 +1,9 @@
 from typing import List
-
-from chatsql.application.port.outcoming.persistance.BaseEmbeddingRepository import BaseEmbeddingRepository
-
+import os
+import numpy as np
 from chatsql.domain.Embedding import Embedding
+from chatsql.application.port.outcoming.persistance.BaseEmbeddingRepository import BaseEmbeddingRepository
+from chatsql.utils import Exceptions
 
 class EmbeddingRepositoryAdapter(BaseEmbeddingRepository):
 
@@ -10,11 +11,25 @@ def __init__(self, folder: str) -> None:
         self._folder = folder
 
     def save(self, filename: str, embeddings: List[Embedding]) -> bool:
-        raise NotImplementedError()
+        try:
+            filepath = os.path.join(self._folder, filename)
+            np.save(filepath, embeddings)
+            return True
+        except Exception as e:
+            raise Exceptions.FileNotSaved(f"Error while saving embeddings to {filename}: {e}")
 
     def load(self, filename: str) -> List[Embedding]:
-        raise NotImplementedError()
+        try:
+            filepath = os.path.join(self._folder, filename)
+            embeddings = np.load(filepath, allow_pickle=True)
+            return embeddings.tolist()  # Convert to list of Embedding objects
+        except Exception as e:
+            raise Exceptions.EmbeddingsNotLoaded(f"Error while loading embeddings from {filename}: {e}")
 
     def remove(self, filename: str) -> bool:
-        raise NotImplementedError()
-        
\ No newline at end of file
+        try:
+            filepath = os.path.join(self._folder, filename)
+            os.remove(filepath)
+            return True
+        except Exception as e:
+            raise Exceptions.FileNotRemoved(f"Error while removing embeddings file {filename}: {e}. Check if permissions are correct.")
diff --git a/backend/chatsql/utils/Exceptions.py b/backend/chatsql/utils/Exceptions.py
index 662a213..1d45871 100644
--- a/backend/chatsql/utils/Exceptions.py
+++ b/backend/chatsql/utils/Exceptions.py
@@ -18,4 +18,19 @@ def __init__(self, message="Invalid file structure"):
 class FileAlreadyUploaded(CustomException):
 
     def __init__(self, message="File Already uploaded"):
+        super().__init__(message)
+
+class FileNotSaved(CustomException):
+
+    def __init__(self, message="Error saving file"):
+        super().__init__(message)
+
+class EmbeddingsNotLoaded(CustomException):
+
+    def __init__(self, message="Error during embeddings loading from file"):
+        super().__init__(message)
+
+class FileNotRemoved(CustomException):
+
+    def __init__(self, message="Error removing file. Check if permissions are correct"):
         super().__init__(message)
\ No newline at end of file

From 46c6470607337cf76c1ab9e107f4cca651d6ad0f Mon Sep 17 00:00:00 2001
From: Vullnet <vuvogli@gmail.com>
Date: Wed, 1 May 2024 22:16:56 +0200
Subject: [PATCH 2/4] fix

---
 .github/workflows/backend.yaml            | 47 +++++++++++++
 .github/workflows/frontend.yaml           | 61 +++++++++++++++++
 .github/workflows/validate.yml            | 83 -----------------------
 __init__.py                               |  0
 tests/__init__.py                         |  0
 tests/backend/test_backend.py             |  8 ---
 tests/{backend => }/test_jsonvalidator.py | 20 +++---
 7 files changed, 119 insertions(+), 100 deletions(-)
 create mode 100644 .github/workflows/backend.yaml
 create mode 100644 .github/workflows/frontend.yaml
 delete mode 100644 .github/workflows/validate.yml
 delete mode 100644 __init__.py
 delete mode 100644 tests/__init__.py
 delete mode 100644 tests/backend/test_backend.py
 rename tests/{backend => }/test_jsonvalidator.py (69%)

diff --git a/.github/workflows/backend.yaml b/.github/workflows/backend.yaml
new file mode 100644
index 0000000..d55e89c
--- /dev/null
+++ b/.github/workflows/backend.yaml
@@ -0,0 +1,47 @@
+name: Backend build
+
+on:
+  push:
+    branches: [ "main" ]
+  pull_request:
+    branches: [ "main" ]
+
+permissions:
+  contents: read
+
+jobs:
+  build-backend:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v3
+      - name: Set up Python 3.10
+        uses: actions/setup-python@v3
+        with:
+          python-version: "3.10"
+
+      - name: Install Python dependencies
+        run: |
+          python -m pip install --upgrade pip
+          pip install flake8 coverage
+          if [ -f requirements.txt ]; then 
+            pip install -r requirements.txt;
+          fi
+        working-directory: "backend"
+
+      - name: Linting Codice Python (Flake8)
+        run: |
+          flake8 backend/ --count --exit-zero --select=E9,F63,F7,F82 --show-source --max-line-length=127 --statistics
+          flake8 backend/ --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics
+
+      - name: Run Python tests and generate coverage report
+        run: |
+          coverage run -m unittest discover -s tests/ -p "test_*.py"
+          coverage xml -o tests/coverage.xml
+
+      - name: Upload backend coverage report to Codecov
+        uses: codecov/codecov-action@v4.0.1
+        with:
+          token: ${{ secrets.CODECOV_TOKEN }}
+          file: tests/coverage.xml
+          flags: python
+          name: backend-coverage
\ No newline at end of file
diff --git a/.github/workflows/frontend.yaml b/.github/workflows/frontend.yaml
new file mode 100644
index 0000000..9580a64
--- /dev/null
+++ b/.github/workflows/frontend.yaml
@@ -0,0 +1,61 @@
+name: Frontend
+
+on:
+  push:
+    branches: [ "main" ]
+  pull_request:
+    branches: [ "main" ]
+
+permissions:
+  contents: read
+
+jobs:
+
+  build:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v3
+      - name: Installazione Node.js and NPM
+        uses: actions/setup-node@v2
+        with:
+          node-version: '14'
+
+      - name: Installazione Vue CLI
+        run: |
+          npm install -g @vue/cli
+
+      - name: Installazione Dipendenze Bootstrap
+        run: |
+          npm install jquery popper.js
+      - name: Installazione Vue.js dependencies
+        run: |
+          cd frontend
+          npm install
+
+      - name: Build FrontEnd
+        run: |
+          cd frontend
+          npm run build
+          
+      - name: Linting Codice Vue.js
+        run: |
+          cd frontend
+          npm run lint
+
+      - name: Build Unit Test
+        run: |
+          cd frontend
+          npm run test:unit
+
+      - name: Run Vue.js tests and generate coverage report
+        run: |
+          cd frontend
+          npx jest --coverage
+
+      - name: Upload frontend coverage report to Codecov
+        uses: codecov/codecov-action@v4.0.1
+        with:
+          token: ${{ secrets.CODECOV_TOKEN }}
+          file: frontend/coverage/lcov.info
+          flags: frontend
+          name: frontend-coverage
\ No newline at end of file
diff --git a/.github/workflows/validate.yml b/.github/workflows/validate.yml
deleted file mode 100644
index 16b643a..0000000
--- a/.github/workflows/validate.yml
+++ /dev/null
@@ -1,83 +0,0 @@
-name: Build
-
-on:
-  push:
-    branches: [ "main" ]
-  pull_request:
-    branches: [ "main" ]
-
-permissions:
-  contents: read
-
-jobs:
-  build:
-
-    runs-on: ubuntu-latest
-
-    steps:
-    - uses: actions/checkout@v3
-    - name: Set up Python 3.10
-      uses: actions/setup-python@v3
-      with:
-        python-version: "3.10"
-    - name: Install dependencies
-      run: |
-        python -m pip install --upgrade pip
-        pip install flake8 coverage
-        if [ -f requirements.txt ]; then pip install -r requirements.txt; fi
-    - name: Run Python tests and generate coverage report
-      run: |
-        coverage run -m unittest
-        coverage xml -o tests/results/coverage.xml
-    - name: Installazione Node.js and NPM
-      uses: actions/setup-node@v2
-      with:
-        node-version: '14'
-    - name: Installazione Vue CLI
-      run: |
-        npm install -g @vue/cli
-    - name: Installazione Dipendenze Bootstrap
-      run: |
-        npm install jquery popper.js
-    - name: Installazione Vue.js dependencies
-      run: |
-        cd frontend
-        npm install
-        cd ..
-    - name: Linting Codice Python (Flake8)
-      run: |
-        flake8 . --count --select=E9,F63,F7,F82 --show-source --statistics
-        flake8 . --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics
-    - name: Test Python
-      run: |
-        cd tests/backend
-        python -m unittest discover
-        cd ..
-    - name: Build Unit Test
-      run: |
-        cd frontend
-        npm run test:unit
-    - name: Run Vue.js tests and generate coverage report
-      run: |
-        cd frontend
-        npx jest --coverage
-        cd ../..
-    - name: Build FrontEnd
-      run: |
-        cd frontend
-        npm run build
-        cd ..
-    - name: Linting Codice Vue.js
-      run: |
-        cd frontend
-        npm run lint
-        cd ..
-    - name: Upload coverage reports to Codecov
-      uses: codecov/codecov-action@v4.0.1
-      with:
-        token: ${{ secrets.CODECOV_TOKEN }}
-        file: |
-          backend/coverage.xml
-          frontend/coverage/lcov.info # Aggiungi il percorso del report di copertura del frontend
-        flags: python,frontend # Flags opzionali, utili per separare i report di copertura
-        name: backend-coverage,frontend-coverage # Nomi opzionali, utili per separare i report di copertura
diff --git a/__init__.py b/__init__.py
deleted file mode 100644
index e69de29..0000000
diff --git a/tests/__init__.py b/tests/__init__.py
deleted file mode 100644
index e69de29..0000000
diff --git a/tests/backend/test_backend.py b/tests/backend/test_backend.py
deleted file mode 100644
index b60742e..0000000
--- a/tests/backend/test_backend.py
+++ /dev/null
@@ -1,8 +0,0 @@
-import unittest
-
-class TestEverythingWorking(unittest.TestCase):
-    def test_everything_working(self):
-        self.assertEqual(True, True)
-
-if __name__ == '__main__':
-    unittest.main()
diff --git a/tests/backend/test_jsonvalidator.py b/tests/test_jsonvalidator.py
similarity index 69%
rename from tests/backend/test_jsonvalidator.py
rename to tests/test_jsonvalidator.py
index 6aefe29..8b465c8 100644
--- a/tests/backend/test_jsonvalidator.py
+++ b/tests/test_jsonvalidator.py
@@ -1,7 +1,8 @@
-from backend.chatsql.utils.JSONValidator import JSONValidator  # Assumi che JSONValidator sia nella stessa directory o importabile
+import unittest
+from unittest import TestCase
+from backend.chatsql.utils.JSONValidator import JSONValidator
 
-# Test di unità per JSONValidator
-class TestJSONValidator:
+class TestJSONValidator(TestCase):
     def test_is_valid_structure_valid(self):
         json_data = {
             "tables_info": {
@@ -15,7 +16,7 @@ def test_is_valid_structure_valid(self):
     def test_is_valid_structure_invalid(self):
         # Test caso in cui mancano chiavi
         json_data_missing_keys = {"tables_info": {}, "primary_key": {}, "foreign_keys": {}}
-        assert JSONValidator.is_valid_structure(json_data_missing_keys) == False
+        TestCase.assertTrue(self, JSONValidator.is_valid_structure(json_data_missing_keys))
 
         # Test caso in cui le tabelle non hanno tutte le informazioni necessarie
         json_data_invalid_tables_info = {
@@ -23,7 +24,7 @@ def test_is_valid_structure_invalid(self):
             "primary_key": {"table1": ["column1"]},
             "foreign_keys": []
         }
-        assert JSONValidator.is_valid_structure(json_data_invalid_tables_info) == False
+        TestCase.assertFalse(self, JSONValidator.is_valid_structure(json_data_invalid_tables_info))
 
         # Test caso in cui le chiavi primarie non sono correttamente strutturate
         json_data_invalid_primary_key = {
@@ -31,14 +32,15 @@ def test_is_valid_structure_invalid(self):
             "primary_key": {"table1": "column1"},
             "foreign_keys": []
         }
-        assert JSONValidator.is_valid_structure(json_data_invalid_primary_key) == False
-
+        TestCase.assertFalse(self, JSONValidator.is_valid_structure(json_data_invalid_primary_key))
+        
         # Test caso in cui le chiavi esterne non sono correttamente strutturate
         json_data_invalid_foreign_keys = {
             "tables_info": {"table1": {"table_description": "Description", "columns": [], "attribute_types": []}},
             "primary_key": {"table1": ["column1"]},
             "foreign_keys": [{"table": "table1"}]
         }
-        assert JSONValidator.is_valid_structure(json_data_invalid_foreign_keys) == False
+        TestCase.assertFalse(self, JSONValidator.is_valid_structure(json_data_invalid_foreign_keys))
 
-    # Aggiungi altri test per i metodi di validazione delle tabelle, delle chiavi primarie e delle chiavi esterne se necessario
+if __name__ == '__main__':
+    unittest.main()

From bf5e535e1b7b6489b72f8ae4f562d17b92219fa0 Mon Sep 17 00:00:00 2001
From: unknown <y.lovato01@gmail.com>
Date: Thu, 2 May 2024 16:11:19 +0200
Subject: [PATCH 3/4] Corretto e aggiunto test

---
 frontend/jest.config.js                       |   6 +-
 frontend/src/components/InputFile.vue         |  16 +--
 frontend/src/components/LoadButton.vue        |   4 +-
 frontend/src/components/ToastPopup.vue        |  76 ++++++------
 .../sendrequestbutton.spec.js.snap            |  10 --
 .../viewgeneratedprompts.spec.js.snap         |  65 ----------
 frontend/tests/unit/inputfile.spec.js         | 114 ++++++++++++------
 frontend/tests/unit/loadbutton.spec.js        |   6 +-
 frontend/tests/unit/redirectbutton.spec.js    |  90 ++++----------
 frontend/tests/unit/requestviewmodel.spec.js  |  55 ---------
 frontend/tests/unit/toastpopup.spec.js        |  20 +++
 frontend/tests/unit/uploadbutton.spec.js      |  40 ++++++
 frontend/tests/unit/viewdictionary.spec.js    | 114 ++++++++++++++----
 13 files changed, 293 insertions(+), 323 deletions(-)
 delete mode 100644 frontend/tests/unit/__snapshots__/sendrequestbutton.spec.js.snap
 delete mode 100644 frontend/tests/unit/__snapshots__/viewgeneratedprompts.spec.js.snap
 delete mode 100644 frontend/tests/unit/requestviewmodel.spec.js
 create mode 100644 frontend/tests/unit/toastpopup.spec.js
 create mode 100644 frontend/tests/unit/uploadbutton.spec.js

diff --git a/frontend/jest.config.js b/frontend/jest.config.js
index 98fe091..4853be3 100644
--- a/frontend/jest.config.js
+++ b/frontend/jest.config.js
@@ -12,5 +12,9 @@ module.exports = {
    // Directory dove Jest deve mettere i risultati della copertura
    coverageDirectory: '<rootDir>/coverage',
 
-   transformIgnorePatterns: ['node_modules/(?!(axios)/)'],
+   transformIgnorePatterns: [
+    'node_modules/(?!(axios)/)',
+    '/node_modules/(?!(bootstrap)/)',
+    '\\.css$'
+  ],
 }
diff --git a/frontend/src/components/InputFile.vue b/frontend/src/components/InputFile.vue
index f682073..7146969 100644
--- a/frontend/src/components/InputFile.vue
+++ b/frontend/src/components/InputFile.vue
@@ -4,7 +4,6 @@
       <input type="file" class="form-control" ref="fileInput" @change="handleFileUpload" />
       <upload-button ref="uploadbtn" :class="uploadButtonClass" @upload-click="emitFile" :disabled="!file" :isuploading="isUploading"/>
     </div>
-    <p>{{ message }}</p>
   </div>
 </template>
 
@@ -33,7 +32,6 @@ export default {
      * @property {ref} isUploading - A reference to a boolean indicating if the file is currently being uploaded.
      */
     const file = ref(null);
-    const message = ref(' ');
     const isUploading = ref(false);
 
     /**
@@ -43,7 +41,6 @@ export default {
      * @param {Event} event - The file upload event.
      */
     function handleFileUpload(event) {
-      this.changeMessage('');
       file.value = event.target.files[0];
     }
 
@@ -54,21 +51,12 @@ export default {
      * @emits file-selected - Event emitted when a file is selected
      */
     function emitFile() {
-      if (file.value) {
+      if (file) {
         isUploading.value = true;
         emit('file-selected', file.value);
       }
     }
 
-    /**
-     * Updates the value of the message.
-     *
-     * @param {string} newMessage - The new message value.
-     */
-    function changeMessage(newMessage) {
-      message.value = newMessage;
-    }
-
     /**
      * Sets the value of the isUploading reactive variable.
      *
@@ -81,10 +69,8 @@ export default {
 
     return {
       file,
-      message,
       handleFileUpload,
       emitFile,
-      changeMessage,
       isUploading,
       setIsUploading
     };
diff --git a/frontend/src/components/LoadButton.vue b/frontend/src/components/LoadButton.vue
index e505090..a8ebf9f 100644
--- a/frontend/src/components/LoadButton.vue
+++ b/frontend/src/components/LoadButton.vue
@@ -33,8 +33,8 @@ export default {
      *
      * @param {number} id - The ID of the dictionary to load.
      */
-    function loadDictionary(id) {
-      emit('load-click', id);
+    function loadDictionary() {
+      emit('load-click', props.id);
     }
 
     return {
diff --git a/frontend/src/components/ToastPopup.vue b/frontend/src/components/ToastPopup.vue
index 2fe9f82..e42f766 100644
--- a/frontend/src/components/ToastPopup.vue
+++ b/frontend/src/components/ToastPopup.vue
@@ -1,50 +1,52 @@
-<template ref="Toast">
-<div class="toast align-items-center text-white bg-primary border-0" role="alert" aria-live="assertive" aria-atomic="true">
-  <div class="d-flex">
-    <div class="toast-body">
-      {{ Message }}
+<template>
+  <div ref="toast" class="toast align-items-center text-white bg-primary border-0" role="alert" aria-live="assertive" aria-atomic="true">
+    <div class="d-flex">
+      <div class="toast-body">
+        {{ Message }}
+      </div>
+      <button type="button" class="btn-close btn-close-white me-2 m-auto" aria-label="Close" data-bs-dismiss="toast"></button>
     </div>
-    <button type="button" class="btn-close btn-close-white me-2 m-auto" data-bs-dismiss="toast" aria-label="Close"></button>
   </div>
-</div>  
 </template>
+
 <script>
 import { ref } from 'vue';
-import 'bootstrap/dist/css/bootstrap.min.css';
 import * as bootstrap from 'bootstrap';
+
 export default {
-    name: 'ToastPopup',
-    setup() {
-      
-      /**
-       * Represents a reactive reference to a message.
-       * @type {Ref<string>}
-       */
-      const Message = ref('');
+  name: 'ToastPopup',
+  setup() {
+    /**
+     * Represents a reactive reference to a message.
+     * @type {Ref<string>}
+     */
+    const Message = ref('');
 
-      /**
-       * Shows a toast notification.
-       */
-      function showToast() {
-        const toastEl = document.querySelector('.toast');
+    /**
+     * Shows a toast message.
+     */
+    function showToast() {
+      const toastEl = document.querySelector('.toast');
+      if (toastEl) {
         const toast = new bootstrap.Toast(toastEl);
         toast.show();
       }
+    }
 
-      /**
-       * Sets the value of the Message variable to the specified message.
-       *
-       * @param {string} message - The message to set.
-       */
-      function setTest(message){
-        Message.value=message;
-      }
-      
-      return{
-        showToast,
-        setTest,
-        Message
-      }
+    /**
+     * Sets the value of the Message variable to the specified message.
+     *
+     * @param {string} message - The message to set the Message variable to.
+     */
+    function setTest(message) {
+      Message.value = message;
     }
-}
-</script>
\ No newline at end of file
+
+    return {
+      showToast,
+      setTest,
+      Message
+    };
+  }
+};
+</script>
diff --git a/frontend/tests/unit/__snapshots__/sendrequestbutton.spec.js.snap b/frontend/tests/unit/__snapshots__/sendrequestbutton.spec.js.snap
deleted file mode 100644
index d27d9af..0000000
--- a/frontend/tests/unit/__snapshots__/sendrequestbutton.spec.js.snap
+++ /dev/null
@@ -1,10 +0,0 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
-
-exports[`SendRequestButton.vue renders correctly 1`] = `
-<button
-  class=""
-  type="button"
->
-  Invia
-</button>
-`;
diff --git a/frontend/tests/unit/__snapshots__/viewgeneratedprompts.spec.js.snap b/frontend/tests/unit/__snapshots__/viewgeneratedprompts.spec.js.snap
deleted file mode 100644
index 9712c12..0000000
--- a/frontend/tests/unit/__snapshots__/viewgeneratedprompts.spec.js.snap
+++ /dev/null
@@ -1,65 +0,0 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
-
-exports[`ViewGeneratedPrompts.vue renders correctly 1`] = `
-<div
-  class="d-flex flex-column justify-content-center align-items-center"
->
-  <div
-    class="w-100 text-black"
-  >
-    
-    <div
-      class="border-bottom"
-    >
-      <div
-        class="w-100 p-3 text-black"
-      >
-        <i
-          class="bi bi-person-fill me-2"
-        />
-        <strong>
-          User
-        </strong>
-        <p>
-          Test message
-        </p>
-      </div>
-    </div>
-    <div
-      class="border-bottom"
-    >
-      <div
-        class="w-100 p-3 text-black bg-light"
-      >
-        <i
-          class="bi bi-robot me-2"
-        />
-        <strong>
-          Response
-        </strong>
-        <p>
-          Test response
-        </p>
-        <i
-          class="bi bi-clipboard me-2 copy-icon"
-        />
-      </div>
-    </div>
-    
-    <div
-      class="w-100 text-center text-black"
-    >
-      <div
-        class="spinner-border text-primary mt-2"
-        role="status"
-      >
-        <span
-          class="visually-hidden"
-        >
-          Loading...
-        </span>
-      </div>
-    </div>
-  </div>
-</div>
-`;
diff --git a/frontend/tests/unit/inputfile.spec.js b/frontend/tests/unit/inputfile.spec.js
index e9a4501..e44939e 100644
--- a/frontend/tests/unit/inputfile.spec.js
+++ b/frontend/tests/unit/inputfile.spec.js
@@ -1,50 +1,84 @@
-import { shallowMount } from '@vue/test-utils';
+import { mount } from '@vue/test-utils';
 import InputFile from '@/components/InputFile.vue';
 
-// Funzione per creare un oggetto FileList simulato
-function createFileList(files) {
-  return {
-    length: files.length,
-    item(index) {
-      return files[index];
-    },
-  };
-}
-
 describe('InputFile', () => {
-  it('does not emit "file-selected" event when an invalid file is uploaded', () => {
-    const wrapper = shallowMount(InputFile);
+  it('verifies that the file field contains the selected file when a file is uploaded', async () => {
+    const wrapper = mount(InputFile);
+    
+    // Simuliamo l'upload di un file
     const file = new File(['file content'], 'test.txt', { type: 'text/plain' });
-    const fileList = createFileList([file]);
-
-    wrapper.vm.isExtensionAllowed = jest.fn().mockReturnValue(false);
-    wrapper.vm.isSizeValid = jest.fn().mockReturnValue(true);
-
-    const input = wrapper.find('input[type="file"]');
-    Object.defineProperty(input.element, 'files', {
-      value: fileList,
-      writable: false,
-    });
-    input.trigger('change');
+    await wrapper.vm.handleFileUpload({ target: { files: [file] } });
+  
+    // Attendi un breve periodo di tempo per consentire l'aggiornamento del valore di 'file'
+    await wrapper.vm.$nextTick();
+  
+    // Verifichiamo che il campo file contenga effettivamente il file caricato
+    expect(wrapper.vm.file).toBe(file);
+  });
 
+  it('does not emit file when no file is selected', async () => {
+    const wrapper = mount(InputFile);
+    
+    // Simuliamo il click sul pulsante di upload senza selezionare un file
+    await wrapper.findComponent({ ref: 'uploadbtn' }).trigger('upload-click');
+  
+    // Verifichiamo che non sia stato emesso l'evento "file-selected"
     expect(wrapper.emitted('file-selected')).toBeFalsy();
+  
+    // Verifichiamo che isUploading rimanga false
+    expect(wrapper.vm.isUploading).toBe(false);
   });
- /* it('emits "file-selected" event when a valid file is uploaded', () => {
-    const wrapper = shallowMount(InputFile);
-    const file = new File(['file content'], 'test.json', { type: 'application/json' });
-    const fileList = createFileList([file]);
-
-    wrapper.vm.isExtensionAllowed = jest.fn().mockReturnValue(true);
-    wrapper.vm.isSizeValid = jest.fn().mockReturnValue(true);
-
-    const input = wrapper.find('input[type="file"]');
-    Object.defineProperty(input.element, 'files', {
-      value: fileList,
-      writable: false,
-    });
-    input.trigger('change');
 
+  it('emits the selected file when upload button is clicked', async () => {
+    const wrapper = mount(InputFile);
+
+    // Simuliamo l'upload di un file
+    const file = new File(['file content'], 'test.txt', { type: 'text/plain' });
+    await wrapper.vm.handleFileUpload({ target: { files: [file] } });
+
+    // Simuliamo il click sul pulsante di upload
+    await wrapper.findComponent({ ref: 'uploadbtn' }).trigger('upload-click');
+
+    // Verifichiamo che sia stato emesso l'evento "file-selected" con il file selezionato
     expect(wrapper.emitted('file-selected')).toBeTruthy();
+    expect(wrapper.emitted('file-selected')[0][0]).toBe(file);
+  });
+
+  it('assigns the passed boolean true to isUploading when setIsUploading is called', async () => {
+    const wrapper = mount(InputFile);
+    
+    // Chiamiamo la funzione setIsUploading con un valore booleano
+    await wrapper.vm.setIsUploading(true);
+  
+    // Attendi un breve periodo di tempo per consentire l'aggiornamento del valore di 'isUploading'
+    await wrapper.vm.$nextTick();
+  
+    // Verifichiamo che il valore booleano sia stato assegnato correttamente a isUploading
+    expect(wrapper.vm.isUploading).toBe(true);
+  });
+
+  it('assigns the passed boolean false to isUploading when setIsUploading is called', async () => {
+    const wrapper = mount(InputFile);
+    
+    // Chiamiamo la funzione setIsUploading con un valore booleano
+    await wrapper.vm.setIsUploading(false);
+  
+    // Attendi un breve periodo di tempo per consentire l'aggiornamento del valore di 'isUploading'
+    await wrapper.vm.$nextTick();
+  
+    // Verifichiamo che il valore booleano sia stato assegnato correttamente a isUploading
+    expect(wrapper.vm.isUploading).toBe(false);
+  });
+
+
+  it('correctly assigns the provided props', async () => {
+    const wrapper = mount(InputFile, {
+      props: {
+        uploadButtonClass: 'custom-class'
+      }
+    });
+    
+    // Verifichiamo che il valore di uploadButtonClass sia stato assegnato correttamente
+    expect(wrapper.vm.uploadButtonClass).toBe('custom-class');
   });
-  */
-});
+});
\ No newline at end of file
diff --git a/frontend/tests/unit/loadbutton.spec.js b/frontend/tests/unit/loadbutton.spec.js
index df6e8ae..f488edd 100644
--- a/frontend/tests/unit/loadbutton.spec.js
+++ b/frontend/tests/unit/loadbutton.spec.js
@@ -4,12 +4,8 @@ import LoadButton from '@/components/LoadButton.vue';
 describe('LoadButton', () => {
   it('emits a "load-click" event when clicked', async () => {
     const id = '123';
-    const LoadButtonClass = 'custom-class';
     const wrapper = mount(LoadButton, {
-      props: {
-        id,
-        LoadButtonClass
-      }
+      props: { id }
     });
     await wrapper.find('button').trigger('click');
     expect(wrapper.emitted('load-click')).toBeTruthy();
diff --git a/frontend/tests/unit/redirectbutton.spec.js b/frontend/tests/unit/redirectbutton.spec.js
index 6cd98e1..c41f098 100644
--- a/frontend/tests/unit/redirectbutton.spec.js
+++ b/frontend/tests/unit/redirectbutton.spec.js
@@ -1,80 +1,32 @@
-import { createRouter, createWebHistory } from 'vue-router';
 import { mount } from '@vue/test-utils';
 import RedirectButton from '@/components/RedirectButton.vue';
-import HomePage from '@/views/HomePage.vue';
-import RequestPage from '@/views/RequestPage.vue';
-import ManagerPage from '@/views/ManagerPage.vue';
+import { useRouter } from 'vue-router';
 
-describe('RedirectButton', () => {
-  let router;
-
-  beforeEach(() => {
-    router = createRouter({
-      history: createWebHistory(),
-      routes: [
-        { path: '/', component: HomePage },
-        { path: '/request', component: RequestPage },
-        { path: '/manager', component: ManagerPage },
-      ],
-    });
-  });
-
-  it('redirects to the home page on button click', async () => {
-    const wrapper = mount(RedirectButton, {
-      props: { destination: '/' },
-      global: { plugins: [router] },
-    });
-
-    await wrapper.find('button').trigger('click');
-    await router.isReady();
+jest.mock('vue-router', () => ({
+  useRouter: jest.fn()
+}));
 
-    expect(router.currentRoute.value.path).toBe('/');
-  });
-
-  it('redirects to the request page on button click', async () => {
-    const wrapper = mount(RedirectButton, {
-      props: { destination: '/request' },
-      global: { plugins: [router] },
+describe('RedirectButton', () => {
+  it('navigates to the specified destination when clicked', async () => {
+    const mockPush = jest.fn();
+    useRouter.mockReturnValue({
+      push: mockPush
     });
 
-    await wrapper.find('button').trigger('click');
-    await router.isReady();
-
-    expect(router.currentRoute.value.path).toBe('/request');
-  });
-
-  it('redirects to the manager page on button click', async () => {
+    const destination = '/example';
+    const buttonClass = 'btn-primary';
     const wrapper = mount(RedirectButton, {
-      props: { destination: '/manager' },
-      global: { plugins: [router] },
+      props: {
+        destination,
+        buttonClass
+      },
+      slots: {
+        default: 'Click me'
+      }
     });
-
     await wrapper.find('button').trigger('click');
-    await router.isReady();
-
-    expect(router.currentRoute.value.path).toBe('/manager');
-  });
-/*
-  it('throws an error when an invalid destination is provided', async () => {
-    const routerOrigPush = router.push;
-    const pushMock = jest.fn().mockImplementation(() => {
-      throw new Error('Invalid route');
-    });
-  
-    router.push = pushMock;
-  
-    const wrapper = mount(RedirectButton, {
-      props: { destination: '/invalid' },
-      global: { plugins: [router] },
-    });
-  
-    try {
-      await wrapper.find('button').trigger('click');
-    } catch (error) {
-      expect(error).toEqual(new Error('Invalid route'));
-    }
-  
-    router.push = routerOrigPush;
+    expect(useRouter).toHaveBeenCalled();
+    expect(mockPush).toHaveBeenCalledWith({ path: destination });
+    expect(wrapper.find('button').classes()).toContain(buttonClass);
   });
-  */
 });
\ No newline at end of file
diff --git a/frontend/tests/unit/requestviewmodel.spec.js b/frontend/tests/unit/requestviewmodel.spec.js
deleted file mode 100644
index c9a8c27..0000000
--- a/frontend/tests/unit/requestviewmodel.spec.js
+++ /dev/null
@@ -1,55 +0,0 @@
-import { createPinia, setActivePinia } from 'pinia'
-import RequestPageViewModel from '@/viewmodel/VMRequest'
-
-describe('VMRequest', () => {
-
-  beforeEach(() => {
-    setActivePinia(createPinia())
-
-  })
-  /*
-  it('should call testApiCall and add messages on submitForm', async () => {
-    const vm = RequestPageViewModel()
-    const testApiCallMock = jest.spyOn(vm.requestStore, 'testApiCall')
-    const addMessageMock = jest.spyOn(vm.requestStore, 'addMessage')
-
-    await vm.submitForm()
-
-    expect(testApiCallMock).toHaveBeenCalled()
-    expect(addMessageMock).toHaveBeenCalledTimes(2)
-
-    testApiCallMock.mockRestore()
-    addMessageMock.mockRestore()
-  })
-
-  it('should handle error on submitForm', async () => {
-    const vm = RequestPageViewModel()
-    jest.spyOn(vm.requestStore, 'testApiCall').mockImplementation(() => {
-      throw new Error('Test error')
-    })
-    const addMessageMock = jest.spyOn(vm.requestStore, 'addMessage')
-
-    await vm.submitForm()
-
-    expect(addMessageMock).toHaveBeenCalledWith('user', vm.requestStore.requestMessage)
-    expect(addMessageMock).toHaveBeenCalledWith('response', 'errore o stoppato')
-  })
-
-  it('should call cancelRequest on stopSending', () => {
-    const vm = RequestPageViewModel()
-    const cancelRequestMock = jest.spyOn(vm.requestStore, 'cancelRequest')
-
-    vm.stopSending()
-
-    expect(cancelRequestMock).toHaveBeenCalled()
-  })
-  */
-  it('should call clearMessages on clearMessages', () => {
-    const vm = RequestPageViewModel()
-    const clearMessagesMock = jest.spyOn(vm.requestStore, 'clearMessages')
-
-    vm.clearMessages()
-
-    expect(clearMessagesMock).toHaveBeenCalled()
-  })
-})
\ No newline at end of file
diff --git a/frontend/tests/unit/toastpopup.spec.js b/frontend/tests/unit/toastpopup.spec.js
new file mode 100644
index 0000000..fc82125
--- /dev/null
+++ b/frontend/tests/unit/toastpopup.spec.js
@@ -0,0 +1,20 @@
+import { mount } from '@vue/test-utils';
+import ToastPopup from '@/components/ToastPopup.vue';
+
+describe('ToastPopup', () => {
+    
+  it('renders toast with message', async () => {
+    const wrapper = mount(ToastPopup);    
+    await wrapper.vm.setTest('Test Message');
+    await wrapper.vm.showToast();
+    expect(wrapper.find('.toast').isVisible()).toBe(true);
+  });
+
+  it('sets message correctly when setTest is called', async () => {
+    const wrapper = mount(ToastPopup);
+    const testMessage = 'Test Message';
+    await wrapper.vm.setTest(testMessage);
+    await wrapper.vm.showToast();
+    expect(wrapper.find('.toast-body').text()).toContain(testMessage);
+  });
+});
\ No newline at end of file
diff --git a/frontend/tests/unit/uploadbutton.spec.js b/frontend/tests/unit/uploadbutton.spec.js
new file mode 100644
index 0000000..703b7c9
--- /dev/null
+++ b/frontend/tests/unit/uploadbutton.spec.js
@@ -0,0 +1,40 @@
+import { mount } from '@vue/test-utils';
+import UploadButton from '@/components/UploadButton.vue';
+
+describe('UploadButton', () => {
+
+  it('emits an "upload-click" event when clicked', async () => {
+    const wrapper = mount(UploadButton);
+    await wrapper.find('button').trigger('click');
+    expect(wrapper.emitted('upload-click')).toBeTruthy();
+  });
+
+  it('correctly assigns props', () => {
+    const UploadButtonClass = 'custom-class';
+    const disabled = true;
+    const isuploading = false;
+
+    const wrapper = mount(UploadButton, {
+      props: {
+        UploadButtonClass,
+        disabled,
+        isuploading
+      }
+    });
+    expect(wrapper.props('UploadButtonClass')).toBe(UploadButtonClass);
+    expect(wrapper.props('disabled')).toBe(disabled);
+    expect(wrapper.props('isuploading')).toBe(isuploading);
+  });
+
+  it('renders upload button with spinner when uploading', async () => {
+    const wrapper = mount(UploadButton, {
+      props: {
+        isuploading: true
+      }
+    });
+
+    // Verifica che il pulsante sia renderizzato correttamente
+    expect(wrapper.find('.bi-cloud-arrow-up-fill').exists()).toBe(false);
+    expect(wrapper.find('.spinner-grow').exists()).toBe(true);
+  });
+});
\ No newline at end of file
diff --git a/frontend/tests/unit/viewdictionary.spec.js b/frontend/tests/unit/viewdictionary.spec.js
index 7a37eea..4c6b74d 100644
--- a/frontend/tests/unit/viewdictionary.spec.js
+++ b/frontend/tests/unit/viewdictionary.spec.js
@@ -2,28 +2,94 @@ import { mount } from '@vue/test-utils';
 import ViewDictionary from '@/components/ViewDictionary.vue';
 
 describe('ViewDictionary', () => {
-    it('emits a "load-button-clicked" event when LoadButton is clicked', async () => {
-        const wrapper = mount(ViewDictionary);
-        await wrapper.vm.addNewEntry('Nuova voce', '2');
-        expect(wrapper.emitted('load-button-clicked')).toBeFalsy();
-        await wrapper.findComponent({ name: 'LoadButton' }).trigger('load-click');
-        await wrapper.vm.$nextTick();
-        expect(wrapper.emitted('load-button-clicked')).toBeTruthy();
-      });
-    
-      it('emits a "delete-button-clicked" event when DeleteButton is clicked', async () => {
-        const wrapper = mount(ViewDictionary);
-        await wrapper.vm.addNewEntry('Nuova voce', '2');
-        expect(wrapper.emitted('delete-button-clicked')).toBeFalsy();
-        await wrapper.findComponent({ name: 'DeleteButton' }).trigger('delete-click');
-        await wrapper.vm.$nextTick();
-        expect(wrapper.emitted('delete-button-clicked')).toBeTruthy();
-      });
-      it('adds a new entry to the table when newEntry is called', async () => {
-        const wrapper = mount(ViewDictionary);
-        const initialRowCount = wrapper.findAll('tbody tr').length;
-        await wrapper.vm.addNewEntry('New Entry', '1');
-        const updatedRowCount = wrapper.findAll('tbody tr').length;
-        expect(updatedRowCount).toBeGreaterThan(initialRowCount);
-      });
+  it('adds a new entry to the dictionary', () => {
+    const wrapper = mount(ViewDictionary);
+    wrapper.vm.addNewEntry(1, 'example', 'txt', '2024-05-03', 1024, true);
+    expect(wrapper.vm.dictionaryEntries.length).toBe(1);
+  });
+
+  it('does not add a new entry when one or more parameters are missing', () => {
+    const wrapper = mount(ViewDictionary);
+    wrapper.vm.addNewEntry(1, 'example', 'txt', '2024-05-03'); // Missing size and load
+    expect(wrapper.vm.dictionaryEntries.length).toBe(0);
+  });
+
+  it('does not add a new entry when one or more parameters are undefined', () => {
+    const wrapper = mount(ViewDictionary);
+    wrapper.vm.addNewEntry(undefined, 'example', 'txt', '2024-05-03', 1024, true); // Missing id
+    expect(wrapper.vm.dictionaryEntries.length).toBe(0);
+  });
+
+  it('does not add a new entry when one or more parameters are empty or falsy', () => {
+    const wrapper = mount(ViewDictionary);
+    wrapper.vm.addNewEntry(0, '', '', '', 0, false); // Empty or falsy parameters
+    expect(wrapper.vm.dictionaryEntries.length).toBe(0);
+  });
+
+  it('does not add a new entry when one or more parameters are null', () => {
+    const wrapper = mount(ViewDictionary);
+    wrapper.vm.addNewEntry(null, 'example', 'txt', '2024-05-03', 1024, true); // Null id
+    expect(wrapper.vm.dictionaryEntries.length).toBe(0);
+  });
+
+  it('calls loadButtonClick with the provided id', () => {
+    const wrapper = mount(ViewDictionary);
+    const id = 'example-id';
+    wrapper.vm.addNewEntry(1, 'example', 'txt', '2024-05-03', 1024, true); // Aggiungi manualmente un'entry
+    wrapper.vm.loadButtonClick(id);
+    expect(wrapper.emitted('load-button-clicked')).toBeTruthy();
+    expect(wrapper.emitted('load-button-clicked')[0][0]).toBe(id);
+  });
+
+  it('emits deleteButtonClick with the provided id', () => {
+    const wrapper = mount(ViewDictionary);
+    const id = 'example-id';
+    wrapper.vm.addNewEntry(1, 'example', 'txt', '2024-05-03', 1024, true); // Aggiungi manualmente un'entry
+    wrapper.vm.deleteButtonClick(id);
+    expect(wrapper.emitted('delete-button-clicked')).toBeTruthy();
+    expect(wrapper.emitted('delete-button-clicked')[0][0]).toBe(id);
+  });
+
+  it('updates the entry in the dictionary and emits update-entry event', () => {
+    const wrapper = mount(ViewDictionary);
+    wrapper.vm.updateEntry();
+    expect(wrapper.vm.alertMessage.value).toBe('');
+    expect(wrapper.vm.isRefreshing).toBe(true);
+    expect(wrapper.emitted('update-entry')).toBeTruthy();
+  });
+
+  it('resets the dictionary entries', () => {
+    const wrapper = mount(ViewDictionary);
+    wrapper.vm.addNewEntry(1, 'example', 'txt', '2024-05-03', 1024, true);
+    wrapper.vm.resetEntry();
+    expect(wrapper.vm.dictionaryEntries.length).toBe(0);
+  });
+
+  it('sets the alert message', () => {
+    const wrapper = mount(ViewDictionary);
+    const message = 'This is an alert message';
+    wrapper.vm.setAlertMessage(message);
+    expect(wrapper.vm.alertMessage.value).toBe(message);
+  });
+
+  it('stops the refreshing process', () => {
+    const wrapper = mount(ViewDictionary);
+    wrapper.vm.isRefreshingStop();
+    expect(wrapper.vm.isRefreshing).toBe(false);
+  });
+
+  it('correctly assigns loadButtonClass and deleteButtonClass props', () => {
+    const loadButtonClass = 'load-btn';
+    const deleteButtonClass = 'delete-btn';
+    const wrapper = mount(ViewDictionary, {
+      props: {
+        loadButtonClass: loadButtonClass,
+        deleteButtonClass: deleteButtonClass
+      }
+    });
+
+    // Verifica l'assegnazione dei props loadButtonClass e deleteButtonClass
+    expect(wrapper.vm.loadButtonClass).toBe(loadButtonClass);
+    expect(wrapper.vm.deleteButtonClass).toBe(deleteButtonClass);
+  });
 });

From 5a5eefb92146822f1d3091d2413912b2c1e15f32 Mon Sep 17 00:00:00 2001
From: unknown <y.lovato01@gmail.com>
Date: Thu, 2 May 2024 16:15:44 +0200
Subject: [PATCH 4/4] Fix

---
 frontend/src/components/InputFile.vue | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/frontend/src/components/InputFile.vue b/frontend/src/components/InputFile.vue
index 7146969..b095c70 100644
--- a/frontend/src/components/InputFile.vue
+++ b/frontend/src/components/InputFile.vue
@@ -51,7 +51,7 @@ export default {
      * @emits file-selected - Event emitted when a file is selected
      */
     function emitFile() {
-      if (file) {
+      if (file.value) {
         isUploading.value = true;
         emit('file-selected', file.value);
       }