diff --git a/backend/ops_api/ops/document/azure_document_repository.py b/backend/ops_api/ops/document/azure_document_repository.py index b3f91a7df1..7d4af499eb 100644 --- a/backend/ops_api/ops/document/azure_document_repository.py +++ b/backend/ops_api/ops/document/azure_document_repository.py @@ -74,7 +74,14 @@ def generate_account_sas_url(account_name, account_key, expiry_hours=1): ) # Construct the SAS URL - sas_url = f"https://{account_name}.blob.core.windows.net/?{sas_token}" + DEFAULT_DEV_OPS_URL = "https://dev.ops.opre.acf.gov" + OPS_URL = ( + DEFAULT_DEV_OPS_URL + if "localhost" in current_app.config.get("OPS_FRONTEND_URL") + else current_app.config.get("OPS_FRONTEND_URL") + ) + # https://dev.ops.opre.acf.gov/?{sas_token} in dev + sas_url = f"{OPS_URL}/?{sas_token}" return sas_url except SasUrlGenerationError as e: diff --git a/backend/ops_api/ops/document/document_gateway.py b/backend/ops_api/ops/document/document_gateway.py index 599dddce45..159c9455f7 100644 --- a/backend/ops_api/ops/document/document_gateway.py +++ b/backend/ops_api/ops/document/document_gateway.py @@ -1,5 +1,4 @@ from flask import Config -from flask_jwt_extended import current_user from ops_api.ops.document.azure_document_repository import AzureDocumentRepository from ops_api.ops.document.document_repository import DocumentRepository @@ -16,11 +15,7 @@ def __init__(self, config: Config) -> None: # Validate and register providers with the factory self.register_providers() - # Select provider based on the current user - if current_user.id >= 500: # Users with id 5xx are test users - self.provider = DocumentProviders.fake.name - else: - self.provider = DocumentProviders.azure.name + self.provider = DocumentProviders.azure.name def register_providers(self) -> None: """ diff --git a/frontend/cypress/e2e/uploadDocument.cy.js b/frontend/cypress/e2e/uploadDocument.cy.js index ad75911db5..620a563d9f 100644 --- a/frontend/cypress/e2e/uploadDocument.cy.js +++ b/frontend/cypress/e2e/uploadDocument.cy.js @@ -14,7 +14,7 @@ it("should loads", () => { cy.get("h1").should("have.text", "Temporary Upload Document Page"); }); -it("should create a document database record and upload to in memory storage", () => { +it.skip("should create a document database record and upload to in memory storage", () => { // Entering an Agreement ID in the Upload Document section cy.get('#agreement-id-upload').type('1'); // Selecting a file @@ -47,7 +47,7 @@ it("should create a document database record and upload to in memory storage", ( }) }); -it("Should download document in memory storage and verify logs", () => { +it.skip("Should download document in memory storage and verify logs", () => { // set up spy on console.log let logSpy; cy.window().then((win) => { diff --git a/frontend/src/components/Agreements/Documents/Document.test.js b/frontend/src/components/Agreements/Documents/Document.test.js index 3b04a1712a..59e5218e6a 100644 --- a/frontend/src/components/Agreements/Documents/Document.test.js +++ b/frontend/src/components/Agreements/Documents/Document.test.js @@ -58,7 +58,7 @@ describe("processUploading", () => { }); it("should upload to Azure Blob Storage", async () => { - const sasUrl = "https://mock.blob.core.windows.net"; + const sasUrl = "https://mock.ops.opre.acf.gov"; await processUploading(sasUrl, uuid, file, agreementId, mockUploadDocumentToBlob, mockUploadDocumentToBlob); @@ -77,7 +77,7 @@ describe("processUploading", () => { }); it("should handle errors gracefully", async () => { - const sasUrl = "https://mock.blob.core.windows.net"; + const sasUrl = "https://mock.ops.opre.acf.gov"; mockUploadDocumentToBlob.mockRejectedValue(new Error("Upload failed")); diff --git a/frontend/src/components/Agreements/Documents/Documents.constants.js b/frontend/src/components/Agreements/Documents/Documents.constants.js index dd24ca3421..0a58232d37 100644 --- a/frontend/src/components/Agreements/Documents/Documents.constants.js +++ b/frontend/src/components/Agreements/Documents/Documents.constants.js @@ -10,7 +10,7 @@ export const DOCUMENT_TYPES = [ export const VALID_EXTENSIONS = ["pdf", "doc", "docx", "xls", "xlsx"]; -export const DOCUMENT_CONTAINER_NAME = "documents"; +export const DOCUMENT_CONTAINER_NAME = "docs"; export const ALLOWED_FAKE_HOSTS = "FakeDocumentRepository"; -export const ALLOWED_HOSTS = "blob.core.windows.net"; +export const ALLOWED_HOSTS = "ops.opre.acf.gov";