Skip to content

Conversation

@sujata-m
Copy link
Contributor

@sujata-m sujata-m commented Dec 26, 2023

  • Updated Request and Response Payload
  • Fixed Unit, Integration and Harness according to new request and response payload.

Request Payload -

{
  "messageId": "c8c76e89f30944d2b2abd2491bd95337",
  "messageType": "workflow_identifier",
  "data": {
    "file_upload_path": "https://tdeisamplestorage.blob.core.windows.net/osw/test_upload/valid.zip",
    "user_id": "c59d29b6-a063-4249-943f-d320d15ac9ab",
    "tdei_project_group_id": "0b41ebc5-350c-42d3-90af-3af4ad3628fb"
  }
}

Response Payload -

{
  "messageId": "c8c76e89f30944d2b2abd2491bd95337",
  "messageType": "workflow_identifier",
  "data": {
    "file_upload_path": "https://tdeisamplestorage.blob.core.windows.net/osw/test_upload/valid.zip",
    "user_id": "c59d29b6-a063-4249-943f-d320d15ac9ab",
    "tdei_project_group_id": "0b41ebc5-350c-42d3-90af-3af4ad3628fb",
    "success": true,
    "message": ""
  }
}

@sujata-m sujata-m requested a review from MashB December 26, 2023 09:49
@susrisha
Copy link
Collaborator

was able to run the unit tests and running properly. Approving

Copy link
Contributor

@sureshgaussian sureshgaussian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving changes. didnt run the integration tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants