[upgrade] add tests to build process - Githubissues

microsoft / sample-app-aoai-chatGPT

Sample code for a simple web chat experience through Azure OpenAI, including Azure OpenAI On Your Data.

MIT License

1.45k stars 2.16k forks source link

[upgrade] add tests to build process #787

Closed iseabock closed 2 months ago

iseabock commented 2 months ago

Motivation and Context

Since frontend test have been included in the repo they should be included into the build process.

Description

This was added to the last process in the build process and a polyfill was added to support modern Javascript in Node 14.x

Contribution Checklist

[x] I have built and tested the code locally and in a deployed app
[x] For frontend changes, I have pulled the latest code from main, built the frontend, and committed all static files.
[x] This is a change for all users of this app. No code or asset is specific to my use case or my organization.
[x] I didn't break any existing functionality :smile:

abhahn commented 2 months ago

Thanks for setting up the tests!

I noticed a few static files were removed. Was that intentional? Just checking if any regenerated files need to be readded.