-
Notifications
You must be signed in to change notification settings - Fork 170
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
[JS] Unit tests #54
Open
24 of 35 tasks
Labels
Milestone
Comments
Merged
corinagum
added a commit
that referenced
this issue
Aug 15, 2023
#54 - Finish setting up nyc coverage - All current tests are passing Coverage report: ![image](https://github.com/microsoft/teams-ai/assets/14900841/28cc7b0e-9e93-4dc6-a932-db95799c1bda) --------- Co-authored-by: Corina Gum <>
Merged
corinagum
added a commit
that referenced
this issue
Aug 16, 2023
1. Resolves #424 (as a stop-gap between updates) 1. Add information for `defaultModel` in `AzureOpenAIPlannerOptions` so users know what deployment/model name to use. 2. This is a mitigation until #394 is resolved. 1. #54 1. Minor cleanup of test scripts 2. !! Add questions for discussion on `ConversationHistory` implementation for `clear()` behavior and `replaceLastSay()` 1. (no work item) Update `dotnet/samples/README.md` for clarity --------- Co-authored-by: Corina Gum <>
Closed
80% goal |
Closed
7 tasks
This was referenced Dec 11, 2023
corinagum
pushed a commit
that referenced
this issue
Dec 13, 2023
…1056) ## Linked issues closes: #54 ## Details Unit tests added for: - [x] SequenceAugmentation.ts - [x] DefaultAugmentation.ts ## Attestation Checklist - [x] My code follows the style guidelines of this project - I have checked for/fixed spelling, linting, and other errors - I have commented my code for clarity - I have made corresponding changes to the documentation (we use [TypeDoc](https://typedoc.org/) to document our code) - My changes generate no new warnings - I have added tests that validates my changes, and provides sufficient test coverage. I have tested with: - Local testing - E2E testing in Teams - New and existing unit tests pass locally with my changes
1 task
corinagum
added a commit
that referenced
this issue
Dec 19, 2023
…ere switched (#1081) ## Linked issues closes: #1063 Related to: #54 ## Details - Fixed an issue where `config/submit` and 'config/fetch` invoke names were switched in `TaskModules.ts` - Adds unit tests to Task Modules <img width="793" alt="image" src="https://github.com/microsoft/teams-ai/assets/14900841/8a9c0894-6b5e-45a2-8e77-577fdfedec8e"> - Add unit tests for `TextDataSource.ts` <img width="567" alt="image" src="https://github.com/microsoft/teams-ai/assets/14900841/438373c8-1a35-4094-b14a-f91e1ea18646"> - Add unit tests for `JSONResponseParser.ts` <img width="624" alt="image" src="https://github.com/microsoft/teams-ai/assets/14900841/402d5001-2f4e-4ba3-927e-096b55f9d7bd"> ## Attestation Checklist - [x] My code follows the style guidelines of this project - I have checked for/fixed spelling, linting, and other errors - I have commented my code for clarity - I have made corresponding changes to the documentation (we use [TypeDoc](https://typedoc.org/) to document our code) - My changes generate no new warnings - I have added tests that validates my changes, and provides sufficient test coverage. I have tested with: - Local testing - E2E testing in Teams - New and existing unit tests pass locally with my changes --------- Co-authored-by: Corina Gum <>
1 task
1 task
corinagum
added a commit
that referenced
this issue
Jan 23, 2024
## Linked issues Fixes: #1163, closes #1181 related to: #239 #54 ## Details - Fixes MemoryFork doc strings - Fixes TurnState doc strings - Adds missing docs to several classes/methods - Adaptive Cards - TurnState - OpenAIEmbeddings - Azure OpenAI Client - testing folder - Layout Engine - System Message - User Input Message - User Message - Action Response Validator - Default Response Validator - JSON Response Validator - Response - Teams Chef Bot files - Fix `api-extractor` script to generate an updated version of `teams-ai.api.json` -- it was very out of date - Fix `typedoc`script to work. I did not merge in generated docs though; we have never used the docs site and I'm not convinced we need it. But at least the script works for if we decide to keep it. - Move `Test____.ts` files into `packages/src/internals/testing` -- these are mini-modules used exclusively for testing, and for better findability I decided to move them. - Updated eslintrc for Teams Chef Bot to allow `@remarks` - dev packages: - remove typedoc (unused) - remove unused typedoc script ## Attestation Checklist - [x] My code follows the style guidelines of this project - I have checked for/fixed spelling, linting, and other errors - I have commented my code for clarity - I have made corresponding changes to the documentation (we use [TypeDoc](https://typedoc.org/) to document our code) - My changes generate no new warnings - I have added tests that validates my changes, and provides sufficient test coverage. I have tested with: - Local testing - E2E testing in Teams - New and existing unit tests pass locally with my changes --------- Co-authored-by: Corina Gum <>
1 task
corinagum
added a commit
that referenced
this issue
Feb 1, 2024
1 task
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
%age goal?
This will also depend on when we believe breaking changes will settle down
12/7 list of JS unit tests BELOW 80%
Tasks
The text was updated successfully, but these errors were encountered: