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

[language text] Remove auto language detection tests & update healthcare expectations #25891

Merged
merged 2 commits into from
May 16, 2023
Merged

[language text] Remove auto language detection tests & update healthcare expectations #25891

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
65e1c40
Remove auto language detection tests
minhanh-phan May 16, 2023
b0e4728
Update recordings
minhanh-phan May 16, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion sdk/cognitivelanguage/ai-language-text/assets.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,5 +2,5 @@
"AssetsRepo": "Azure/azure-sdk-assets",
"AssetsRepoPrefixPath": "js",
"TagPrefix": "js/cognitivelanguage/ai-language-text",
"Tag": "js/cognitivelanguage/ai-language-text_ccae2f3f11"
"Tag": "js/cognitivelanguage/ai-language-text_b611b37431"
}
44 changes: 3 additions & 41 deletions sdk/cognitivelanguage/ai-language-text/test/public/analyzeBatch.spec.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,6 @@ import {
expectation9,
expectation30,
expectation31,
expectation71,
} from "./expectations";
import { windows365ArticlePart1, windows365ArticlePart2 } from "./inputs";

Expand Down Expand Up @@ -804,43 +803,6 @@ matrix([["APIKey", "AAD"]] as const, async (authMethod: AuthMethod) => {
await assertActionsResults(await poller.pollUntilDone(), expectation15);
});

// TODO: Unskip when hear back from service team on 'isLanguageDefaulted' property
it.skip("whole batch input with auto language detection", async function () {
const docs = [
"I will go to the park.",
"Este es un document escrito en Español.",
"猫は幸せ",
];
const poller = await client.beginAnalyzeBatch(
[
{
kind: AnalyzeBatchActionNames.EntityRecognition,
},
{
kind: AnalyzeBatchActionNames.PiiEntityRecognition,
},
{
kind: AnalyzeBatchActionNames.SentimentAnalysis,
},
{
kind: AnalyzeBatchActionNames.KeyPhraseExtraction,
},
{
kind: AnalyzeBatchActionNames.EntityLinking,
},
{
kind: AnalyzeBatchActionNames.Healthcare,
},
],
docs,
"auto",
{
updateIntervalInMs: pollingInterval,
}
);
await assertActionsResults(await poller.pollUntilDone(), expectation71);
});

it("invalid language hint", async function () {
const docs = ["This should fail because we're passing in an invalid language hint"];
const poller = await client.beginAnalyzeBatch(
Expand Down Expand Up @@ -1005,7 +967,7 @@ matrix([["APIKey", "AAD"]] as const, async (authMethod: AuthMethod) => {
});

describe("stringIndexType", function () {
it("family emoji wit skin tone modifier", async function () {
it("family emoji with skin tone modifier", async function () {
const docs = ["👩🏻‍👩🏽‍👧🏾‍👦🏿 SSN: 859-98-0987"];
const poller = await client.beginAnalyzeBatch(
[
Expand All @@ -1023,7 +985,7 @@ matrix([["APIKey", "AAD"]] as const, async (authMethod: AuthMethod) => {
await assertActionsResults(await poller.pollUntilDone(), expectation18);
});

it("family emoji wit skin tone modifier with Utf16CodeUnit", async function () {
it("family emoji with skin tone modifier with Utf16CodeUnit", async function () {
const docs = ["👩🏻‍👩🏽‍👧🏾‍👦🏿 ibuprofen"];
const poller = await client.beginAnalyzeBatch(
[
Expand All @@ -1041,7 +1003,7 @@ matrix([["APIKey", "AAD"]] as const, async (authMethod: AuthMethod) => {
await assertActionsResults(await poller.pollUntilDone(), expectation22);
});

it("family emoji wit skin tone modifier with UnicodeCodePoint", async function () {
it("family emoji with skin tone modifier with UnicodeCodePoint", async function () {
const docs = ["👩🏻‍👩🏽‍👧🏾‍👦🏿 ibuprofen"];
const poller = await client.beginAnalyzeBatch(
[
Expand Down
Loading