Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Updating tests #37847

Closed
wants to merge 0 commits into from
Closed

Updating tests #37847

wants to merge 0 commits into from

Conversation

hamshavathimunibyraiah
Copy link
Member

Updating tests:

  1. Authenticating storage accounts using MI
  2. SAS urls are no longer needed to access containers using DefaultAzureCredential instead
  3. Updated test-resource.json inline with what we have for other langs
  4. Updating resource output variable names accordingly.
  5. Updated text translation test cases: updated resource output names
  6. Added test cases for the overridden models and operations (startTranslation using start_translation_details)

@azure-sdk
Copy link
Collaborator

API change check

API changes are not detected in this pull request.

@hamshavathimunibyraiah
Copy link
Member Author

/azp run python - translation - tests

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@hamshavathimunibyraiah
Copy link
Member Author

/azp run python - translation - tests

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

mccoyp
mccoyp reviewed Oct 14, 2024
View reviewed changes
sdk/translation/tests.yml Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mccoyp mccoyp mccoyp left review comments

@weshaggard weshaggard weshaggard left review comments

@MikeyMCZ MikeyMCZ Awaiting requested review from MikeyMCZ

@kristapratico kristapratico Awaiting requested review from kristapratico

Assignees
No one assigned
Labels
Cognitive - Translator
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@hamshavathimunibyraiah @azure-sdk @weshaggard @mccoyp