From 1165d973f88ecbe0516a2c0253bcbbe1c559c629 Mon Sep 17 00:00:00 2001
From: luciajanikova <19lucia99@gmail.com>
Date: Thu, 14 Nov 2024 14:40:02 +0100
Subject: [PATCH] User can upload a message draft only to the tenant's boxes

---
 app/models/box.rb                     |  2 ++
 app/models/fs/box.rb                  |  2 --
 app/models/fs/message_draft.rb        |  2 +-
 test/fixtures/api_connections.yml     |  8 ++++++++
 test/fixtures/boxes.yml               |  9 +++++++++
 test/fixtures/tenants.yml             |  1 +
 test/system/fs/message_drafts_test.rb | 23 +++++++++++++++++++++++
 7 files changed, 44 insertions(+), 3 deletions(-)

diff --git a/app/models/box.rb b/app/models/box.rb
index 45492b1cd..246e8b691 100644
--- a/app/models/box.rb
+++ b/app/models/box.rb
@@ -27,6 +27,8 @@ class Box < ApplicationRecord
   has_many :message_drafts_imports, dependent: :destroy
   has_many :automation_conditions, as: :condition_object
 
+  scope :with_enabled_message_drafts_import, -> { where("(settings ->> 'message_drafts_import_enabled')::boolean = ?", true) }
+
   after_destroy do |box|
     api_connection.destroy if api_connection.destroy_with_box?
     EventBus.publish(:box_destroyed, box.id)
diff --git a/app/models/fs/box.rb b/app/models/fs/box.rb
index dcbf94eac..40a0c4b70 100644
--- a/app/models/fs/box.rb
+++ b/app/models/fs/box.rb
@@ -18,8 +18,6 @@
 class Fs::Box < Box
   DISABLED_MESSAGE_DRAFTS_IMPORT_KEYWORDS = ['(oblasť SPD)']
 
-  scope :with_enabled_message_drafts_import, -> { where("(settings ->> 'message_drafts_import_enabled')::boolean = ?", true) }
-
   validates_uniqueness_of :name, :short_name, scope: :tenant_id
 
   def self.policy_class
diff --git a/app/models/fs/message_draft.rb b/app/models/fs/message_draft.rb
index b3a8d8ac6..c18c0978c 100644
--- a/app/models/fs/message_draft.rb
+++ b/app/models/fs/message_draft.rb
@@ -35,7 +35,7 @@ def self.create_and_validate_with_fs_form(form_files: [], author:, fs_client: Fs
       dic = form_information['subject']&.strip
       fs_form_identifier = form_information['form_identifier']
 
-      box = Fs::Box.with_enabled_message_drafts_import.find_by("settings ->> 'dic' = ?", dic)
+      box = author.tenant.boxes.with_enabled_message_drafts_import.find_by("settings ->> 'dic' = ?", dic)
       fs_form = Fs::Form.find_by(identifier: fs_form_identifier)
 
       unless box && fs_form
diff --git a/test/fixtures/api_connections.yml b/test/fixtures/api_connections.yml
index 7632cc068..aa6607339 100644
--- a/test/fixtures/api_connections.yml
+++ b/test/fixtures/api_connections.yml
@@ -63,3 +63,11 @@ sk_api_api_connection_with_obo_support:
   sub: sub4
   api_token_private_key: private_key
   type: 'SkApi::ApiConnectionWithOboSupport'
+
+fs_api_connection2:
+  sub: fs_sub2
+  api_token_private_key: private_key
+  type: 'Fs::ApiConnection'
+  tenant: solver
+  settings: {"username": "7654321", "password": "fs_password"}
+
diff --git a/test/fixtures/boxes.yml b/test/fixtures/boxes.yml
index 93b17002e..87c00c55e 100644
--- a/test/fixtures/boxes.yml
+++ b/test/fixtures/boxes.yml
@@ -87,3 +87,12 @@ fs_accountants2:
   api_connection: fs_api_connection1
   settings: {"dic": "1122334456", "subject_id": "387cf7da-2cff-40e0-8ad2-650030429994", "message_drafts_import_enabled": true}
   type: 'Fs::Box'
+
+fs_solver:
+  name: Solver main FS
+  uri: dic://sk/9988776655
+  tenant: solver
+  short_name: SFS
+  api_connection: fs_api_connection2
+  settings: {"dic": "1122334456", "subject_id": SecureRandom.uuid, "message_drafts_import_enabled": true}
+  type: 'Fs::Box'
diff --git a/test/fixtures/tenants.yml b/test/fixtures/tenants.yml
index 1ad955a4d..67893b680 100644
--- a/test/fixtures/tenants.yml
+++ b/test/fixtures/tenants.yml
@@ -12,6 +12,7 @@ solver:
   name: Solver
   feature_flags:
     - api
+    - fs_api
   api_token_public_key: "-----BEGIN PUBLIC KEY-----\nMFwwDQYJKoZIhvcNAQEBBQADSwAwSAJBAMIpyOBmP6R5hkFzBlNIbhX4t2CpAaoE\nyDDRzog4rvycPQNXnvZXuIm1shVq0XCrx+2nMWNHFrKqH3QViQR0QGMCAwEAAQ==\n-----END PUBLIC KEY-----"
 
 google:
diff --git a/test/system/fs/message_drafts_test.rb b/test/system/fs/message_drafts_test.rb
index 16926cdc0..97c7d25ee 100644
--- a/test/system/fs/message_drafts_test.rb
+++ b/test/system/fs/message_drafts_test.rb
@@ -96,4 +96,27 @@ class Fs::MessageDraftsTest < ApplicationSystemTestCase
     click_button "Vytvoriť novú správu"
     assert_not has_link? "Vytvoriť novú správu na finančnú správu"
   end
+
+  test "user can upload a message draft only to the tenant's boxes" do
+    sign_in_as(:solver_other)
+
+    visit message_threads_path
+
+    click_button "Vytvoriť novú správu"
+    click_link "Vytvoriť novú správu na finančnú správu"
+
+    fs_api = Minitest::Mock.new
+    fs_api.expect :parse_form, {
+      "subject" => "1122334455",
+      "form_identifier" => "3055_781"
+    },
+    [file_fixture("fs/dic1122334455_fs3055_781__sprava_dani_2023.xml").read]
+
+    FsEnvironment.fs_client.stub :api, fs_api do
+      attach_file "content[]", file_fixture("fs/dic1122334455_fs3055_781__sprava_dani_2023.xml")
+      click_button "Nahrať správy"
+
+      assert_text "Nahratie správ nebolo úspešné"
+    end
+  end
 end