Skip to content
This repository was archived by the owner on Jan 15, 2025. It is now read-only.

Commit 5c86839

Browse files
authored
fix some corner case of prebuild entities in luis:test cli (#1148)
1 parent 436a699 commit 5c86839

File tree

4 files changed

+8
-9
lines changed

4 files changed

+8
-9
lines changed

packages/lu/src/parser/test/testhelper.ts

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -118,9 +118,11 @@ function ParseEntitiyResult(entities: any, result: any) {
118118
}
119119
}
120120
} else {
121-
for (const subEntity of entities[name]) {
122-
if (typeof subEntity === 'object') {
123-
ParseEntitiyResult(subEntity, result)
121+
if (typeof entities[name][Symbol.iterator] === 'function') {
122+
for (const subEntity of entities[name]) {
123+
if (typeof subEntity === 'object') {
124+
ParseEntitiyResult(subEntity, result)
125+
}
124126
}
125127
}
126128
}

packages/luis/test/commands/luis/test.test.ts

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ describe('luis:test cli entity test', () => {
7575
.nock('https://westus.api.cognitive.microsoft.com', api => api
7676
.post(uri => uri.includes('apps'))
7777
.reply(200, {
78-
"query":"accept all meetings for christmas party next week.","prediction":{"topIntent":"AcceptEventEntry","intents":{"AcceptEventEntry":{"score":0.948831439},"FindCalendarEntry":{"score":0.0371829346},"None":{"score":0.00728923827},"CreateCalendarEntry":{"score":0.007234955}},"entities":{"Subject":["christmas party"],"FromDate":["next week"],"$instance":{"Subject":[{"type":"Subject","text":"christmas party","startIndex":24,"length":15,"score":0.9657892,"modelTypeId":1,"modelType":"Entity Extractor","recognitionSources":["model"]}],"FromDate":[{"type":"FromDate","text":"next week","startIndex":40,"length":9,"score":0.966946542,"modelTypeId":1,"modelType":"Entity Extractor","recognitionSources":["model"]}]}}}
78+
"query":"accept all meetings for christmas party next week.","prediction":{"topIntent":"AcceptEventEntry","intents":{"AcceptEventEntry":{"score":0.948831439},"FindCalendarEntry":{"score":0.0371829346},"None":{"score":0.00728923827},"CreateCalendarEntry":{"score":0.007234955}},"entities":{"Subject":["christmas party"],"FromDate":["next week"],"ordinalV2": [{"offset": 1, "relativeTo": "current"}], "$instance":{"Subject":[{"type":"Subject","text":"christmas party","startIndex":24,"length":15,"score":0.9657892,"modelTypeId":1,"modelType":"Entity Extractor","recognitionSources":["model"]}],"FromDate":[{"type":"FromDate","text":"next week","startIndex":40,"length":9,"score":0.966946542,"modelTypeId":1,"modelType":"Entity Extractor","recognitionSources":["model"]}]}}}
7979
})
8080
)
8181
.nock('https://westus.api.cognitive.microsoft.com', api => api
@@ -127,7 +127,6 @@ describe('luis:test cli entity test', () => {
127127
})
128128
})
129129

130-
131130
describe('luis:test cli role test', () => {
132131
before(async function(){
133132
await fs.ensureDir(path.join(__dirname, './../../../results/'))
@@ -222,8 +221,6 @@ describe('luis:test cli Hierarchical entity test', () => {
222221
})
223222
})
224223

225-
226-
227224
describe('luis:test normal test', () => {
228225
before(async function(){
229226
await fs.ensureDir(path.join(__dirname, './../../../results/'))

packages/luis/test/fixtures/testcases/lutest/input/AllEntity.lu

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ $ToTime:simple
5050

5151
> # PREBUILT Entity definitions
5252

53-
$PREBUILT:ordinal
53+
@ PREBUILT ordinalV2
5454

5555
$PREBUILT:personName Roles=Female, Male
5656

packages/luis/test/fixtures/testcases/lutest/output/AllEntity.lu

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@
6767

6868
> # PREBUILT Entity definitions
6969

70-
@ prebuilt ordinal
70+
@ prebuilt ordinalV2
7171

7272
@ prebuilt personName hasRoles Female,Male
7373

0 commit comments

Comments
 (0)