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.

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

fix: checkReturnType in ExpressionValidation throws unexpected error #4112

Merged
merged 4 commits into from
Sep 15, 2020

Conversation

lei9444
Copy link
Contributor

@lei9444 lei9444 commented Sep 14, 2020

Description

The Expression lib use bit field to store the return type list. so if the return type is 24, it means 16 + 8 (string and array).

  1. This PR use bitwise operation to fix the return type error.
  2. refactor and clean the validate flow.
    • check the expression, return the final return type.
    • use schema types to check the final return type.
    • filter the custom functions error. (Todo: support the functions defined in the runtime).

Task Item

closes #4069
refs microsoft/botbuilder-js#2780

Screenshots

@lei9444 lei9444 changed the title fix: dialog validate return type error fix: checkReturnType in ExpressionValidation throws unexpected error Sep 14, 2020
@coveralls
Copy link

coveralls commented Sep 14, 2020

Coverage Status

Coverage remained the same at 55.5% when pulling 1aeec00 on lei9444:validate into b48b37d on microsoft:main.

@a-b-r-o-w-n a-b-r-o-w-n added the Approved to merge approved, waiting to be merged label Sep 15, 2020
@a-b-r-o-w-n a-b-r-o-w-n merged commit 834f8df into microsoft:main Sep 15, 2020
@a-b-r-o-w-n a-b-r-o-w-n deleted the validate branch September 15, 2020 16:08
alanlong9278 added a commit to alanlong9278/BotFramework-Composer that referenced this pull request Sep 16, 2020
* main:
  fix: add more missing formatMessage calls (microsoft#4144)
  feat: electron splash screen (microsoft#4119)
  feat: Add QnA files to dispatch model in skill manifest (microsoft#3985)
  fix: check whether operation is under current project folder (microsoft#4078)
  update localization and add missing "example" string (microsoft#4138)
  fix: defense invocation of `value.match()` (microsoft#4110)
  fix: checkReturnType in ExpressionValidation throws unexpected error (microsoft#4112)
  chore(deps): Bump tree-kill from 1.2.1 to 1.2.2 in /Composer (microsoft#4035)
  build: fix docker builds in ACR (microsoft#3986)
  fix: security: downgrade node-forge to 0.9.0 (microsoft#4133)
  chore: update archiver to fix security warning (microsoft#4116)
@cwhitten cwhitten mentioned this pull request Nov 13, 2020
lei9444 added a commit to lei9444/BotFramework-Composer-1 that referenced this pull request Jun 15, 2021
…icrosoft#4112)

* fix: dialog validate return type error

* fix e2e test

Co-authored-by: Chris Whitten <christopher.whitten@microsoft.com>
Co-authored-by: Andy Brown <asbrown002@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approved to merge approved, waiting to be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

checkReturnType in ExpressionValidation throws unexpected error
4 participants