Add basic ui tests for admin and user websites. #503

superhindupur · 2024-03-20T17:22:33Z

Purpose

This PR adds Cypress UI tests - one for the user website and one for the admin website.
These tests are not configured to run from Github actions yet - that will be covered in the next PR.
Required by Add E2E test suite #419.

Does this introduce a breaking change?

[ ] Yes
[x ] No

Pull Request Type

What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[x] Other - Adds Cypress UI tests.

How to Test

Open VSCode from within the dev container.

git clone https://github.com/Azure-Samples/chat-with-your-data-solution-accelerator.git
cd chat-with-your-data-solution-accelerator
git checkout bhindupur/ui-tests
npx cypress run --env BASE_URL_ADMIN_WEBSITE=https://example-admin.com,BASE_URL_USER_WEBSITE=https://example.com

What to Check

Verify that the tests pass.

ross-p-smith · 2024-03-20T18:14:59Z

Hi @superhindupur - am I missing a step? I ran npm install beforehand and it asked me to install Cypress.

❯ npx cypress run --env BASE_URL_ADMIN_WEBSITE=https://example-admin.com,BASE_URL_USER_WEBSITE=https://example.com
Need to install the following packages:
[email protected]
Ok to proceed? (y) y
It looks like this is your first time using Cypress: 13.7.0


Cypress failed to start.

This may be due to a missing library or dependency. https://on.cypress.io/required-dependencies

Please refer to the error below for more details.

----------

/home/vscode/.cache/Cypress/13.7.0/Cypress/Cypress: error while loading shared libraries: libatk-1.0.so.0: cannot open shared object file: No such file or directory

----------

Platform: linux-x64 (Debian - 11)
Cypress Version: 13.7.0

If env was meant to be a real website then we should output the two website addresses from the main.bicep and then reference them from the MakeFile so I could do something like make uitest

code/frontend/cypress.config.ts

code/frontend/cypress/fixtures/ingest.txt

code/frontend/cypress/support/e2e.ts

adamdougal

Can we add a task to the makefile to make this easier to run? Thanks!

code/frontend/cypress/e2e/admin.spec.cy.ts

code/frontend/cypress/fixtures/ingest.txt

code/frontend/cypress/support/commands.ts

code/frontend/package-lock.json

code/tests/integration/ui/cypress/README.md

package-lock.json

code/tests/integration/ui/package.json

code/tests/integration/ui/cypress/support/e2e.ts

github-actions · 2024-03-25T15:54:06Z

Coverage Report •

File	Stmts	Miss	Cover	Missing
TOTAL	1633	977	40%

report-only-changed-files is enabled. No files were changed during this commit :)

Tests	Skipped	Failures	Errors	Time
40	0 💤	0 ❌	0 🔥	10.123s ⏱️

.devcontainer/postCreate.sh

cecheta · 2024-03-25T16:31:35Z

azure.yaml

I don't think we can make these changes as we are still deploying from ./code

Makefile

ross-p-smith · 2024-03-25T18:54:48Z

I think you need to rebase ontop of Ren's changes so you can get the docker file you need.

ross-p-smith · 2024-03-25T19:09:58Z

I can get this to run - I think the final must have is moving the postCreate into the Dockerfile

* Update minor/patch depedencies * @vitejs/plugin-react major update * Update vite

Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.15.5 to 1.15.6. - [Release notes](https://github.com/follow-redirects/follow-redirects/releases) - [Commits](follow-redirects/follow-redirects@v1.15.5...v1.15.6) --- updated-dependencies: - dependency-name: follow-redirects dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bumps [nodemon](https://github.com/remy/nodemon) from 2.0.22 to 3.1.0. - [Release notes](https://github.com/remy/nodemon/releases) - [Commits](remy/nodemon@v2.0.22...v3.1.0) --- updated-dependencies: - dependency-name: nodemon dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bumps [ts-node](https://github.com/TypeStrong/ts-node) from 10.9.1 to 10.9.2. - [Release notes](https://github.com/TypeStrong/ts-node/releases) - [Changelog](https://github.com/TypeStrong/ts-node/blob/main/development-docs/release-template.md) - [Commits](TypeStrong/ts-node@v10.9.1...v10.9.2) --- updated-dependencies: - dependency-name: ts-node dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bumps [botbuilder](https://github.com/Microsoft/botbuilder-js) from 4.21.4 to 4.22.1. - [Release notes](https://github.com/Microsoft/botbuilder-js/releases) - [Commits](microsoft/botbuilder-js@4.21.4...4.22.1) --- updated-dependencies: - dependency-name: botbuilder dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bumps [typescript](https://github.com/Microsoft/TypeScript) from 4.9.5 to 5.4.3. - [Release notes](https://github.com/Microsoft/TypeScript/releases) - [Changelog](https://github.com/microsoft/TypeScript/blob/main/azure-pipelines.release.yml) - [Commits](microsoft/TypeScript@v4.9.5...v5.4.3) --- updated-dependencies: - dependency-name: typescript dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bumps [restify](https://github.com/restify/node-restify) from 10.0.0 to 11.1.0. - [Release notes](https://github.com/restify/node-restify/releases) - [Changelog](https://github.com/restify/node-restify/blob/master/CHANGELOG.md) - [Commits](restify/node-restify@v10.0.0...v11.1.0) --- updated-dependencies: - dependency-name: restify dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Add functional test for /conversation/custom endpoint - Allow the disabling of app insights and loading config from blob storage to make testing easier - Switch to `Thread` from `Process` for running the app - This is to stop pytest calling the session scoped fixure multiple times - It also fixes an issue where we were not seeing app logs - Setup the mock server as https due to the azure libraries enforcing it - This introduced additional fixtures and the `trustme` dependency - Commit the encoding file for tiktoken - This is to stop it making a http call over the internet to retrive it Required by #420 * Parameterize tests with pytest.mark.parametrize * Improve test logging * Extract boolean env var behaviour * Add note about tiktoken issue * Allow tests to be run from anywhere

Bumps [postcss](https://github.com/postcss/postcss) from 8.4.37 to 8.4.38. - [Release notes](https://github.com/postcss/postcss/releases) - [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md) - [Commits](postcss/postcss@8.4.37...8.4.38) --- updated-dependencies: - dependency-name: postcss dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bumps [@types/react](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/react) from 18.2.67 to 18.2.70. - [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases) - [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/react) --- updated-dependencies: - dependency-name: "@types/react" dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bumps [@fluentui/react](https://github.com/microsoft/fluentui) from 8.116.0 to 8.117.0. - [Release notes](https://github.com/microsoft/fluentui/releases) - [Changelog](https://github.com/microsoft/fluentui/blob/master/azure-pipelines.release-fluentui.yml) - [Commits](https://github.com/microsoft/fluentui/compare/@fluentui/react_v8.116.0...@fluentui/react_v8.117.0) --- updated-dependencies: - dependency-name: "@fluentui/react" dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bumps [typescript](https://github.com/Microsoft/TypeScript) from 5.4.2 to 5.4.3. - [Release notes](https://github.com/Microsoft/TypeScript/releases) - [Changelog](https://github.com/microsoft/TypeScript/blob/main/azure-pipelines.release.yml) - [Commits](microsoft/TypeScript@v5.4.2...v5.4.3) --- updated-dependencies: - dependency-name: typescript dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Add adminweb docker image * Build off my branch * Create Requirements. * Seperate process * Dont traverse paths * Use poetry in docker build * dont install the root app * Add frontend image * Revert docker changes * Install the root again * Use requirements rather than toml * Use Requirements for images again * Rename docker images * Frontend use app directory * Add python path * Copy the whole code folder * Frontend runs locally * Frontend and Admin working * Use containers as default hosting model * Dont build from my branch anymore * Only build docker images - after test * Change trigger workflow * Only run when tests succeed * Trigger builds only if tests pass * Only changed App Service names * Reverting

Bumps [pytest-cov](https://github.com/pytest-dev/pytest-cov) from 4.1.0 to 5.0.0. - [Changelog](https://github.com/pytest-dev/pytest-cov/blob/master/CHANGELOG.rst) - [Commits](pytest-dev/pytest-cov@v4.1.0...v5.0.0) --- updated-dependencies: - dependency-name: pytest-cov dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bumps [pre-commit](https://github.com/pre-commit/pre-commit) from 3.6.2 to 3.7.0. - [Release notes](https://github.com/pre-commit/pre-commit/releases) - [Changelog](https://github.com/pre-commit/pre-commit/blob/main/CHANGELOG.md) - [Commits](pre-commit/pre-commit@v3.6.2...v3.7.0) --- updated-dependencies: - dependency-name: pre-commit dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

.devcontainer/postCreate.sh

ross-p-smith · 2024-03-26T13:27:34Z

This was against containers: -

superhindupur · 2024-03-26T15:51:58Z

Since we eventually want to move all tests outside the code/ folder, I took the first step by placing the UI tests at tests/integration/ui. This means that the azd build config in azure.yaml can stay as is. I created #541 to cover the work for moving the rest of the tests out into this new folder.

ross-p-smith

All works. #541 will fix the folder structure going forward

cecheta

Great stuff :)

* Add basic ui tests for admin and user websites. * Move ui tests to tests folder and create uitest target on makefile. * Update UI tests README to use latest env variable names. * Delete package-lock.json file added in error. * Update wrong command in ui tests README.md. * Remove unnecessary fixture file. * Update all UI dependencies (Azure-Samples#502) * Update minor/patch depedencies * @vitejs/plugin-react major update * Update vite * Bump follow-redirects from 1.15.5 to 1.15.6 in /extensions/teams (Azure-Samples#442) Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.15.5 to 1.15.6. - [Release notes](https://github.com/follow-redirects/follow-redirects/releases) - [Commits](follow-redirects/follow-redirects@v1.15.5...v1.15.6) --- updated-dependencies: - dependency-name: follow-redirects dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump nodemon from 2.0.22 to 3.1.0 in /extensions/teams (Azure-Samples#493) Bumps [nodemon](https://github.com/remy/nodemon) from 2.0.22 to 3.1.0. - [Release notes](https://github.com/remy/nodemon/releases) - [Commits](remy/nodemon@v2.0.22...v3.1.0) --- updated-dependencies: - dependency-name: nodemon dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Update tasks.json for func start (Azure-Samples#511) * Bump ts-node from 10.9.1 to 10.9.2 in /extensions/teams (Azure-Samples#496) Bumps [ts-node](https://github.com/TypeStrong/ts-node) from 10.9.1 to 10.9.2. - [Release notes](https://github.com/TypeStrong/ts-node/releases) - [Changelog](https://github.com/TypeStrong/ts-node/blob/main/development-docs/release-template.md) - [Commits](TypeStrong/ts-node@v10.9.1...v10.9.2) --- updated-dependencies: - dependency-name: ts-node dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump botbuilder from 4.21.4 to 4.22.1 in /extensions/teams (Azure-Samples#491) Bumps [botbuilder](https://github.com/Microsoft/botbuilder-js) from 4.21.4 to 4.22.1. - [Release notes](https://github.com/Microsoft/botbuilder-js/releases) - [Commits](microsoft/botbuilder-js@4.21.4...4.22.1) --- updated-dependencies: - dependency-name: botbuilder dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump typescript from 4.9.5 to 5.4.3 in /extensions/teams (Azure-Samples#509) Bumps [typescript](https://github.com/Microsoft/TypeScript) from 4.9.5 to 5.4.3. - [Release notes](https://github.com/Microsoft/TypeScript/releases) - [Changelog](https://github.com/microsoft/TypeScript/blob/main/azure-pipelines.release.yml) - [Commits](microsoft/TypeScript@v4.9.5...v5.4.3) --- updated-dependencies: - dependency-name: typescript dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump restify from 10.0.0 to 11.1.0 in /extensions/teams (Azure-Samples#489) Bumps [restify](https://github.com/restify/node-restify) from 10.0.0 to 11.1.0. - [Release notes](https://github.com/restify/node-restify/releases) - [Changelog](https://github.com/restify/node-restify/blob/master/CHANGELOG.md) - [Commits](restify/node-restify@v10.0.0...v11.1.0) --- updated-dependencies: - dependency-name: restify dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Add functional test for /conversation/custom endpoint (Azure-Samples#504) * Add functional test for /conversation/custom endpoint - Allow the disabling of app insights and loading config from blob storage to make testing easier - Switch to `Thread` from `Process` for running the app - This is to stop pytest calling the session scoped fixure multiple times - It also fixes an issue where we were not seeing app logs - Setup the mock server as https due to the azure libraries enforcing it - This introduced additional fixtures and the `trustme` dependency - Commit the encoding file for tiktoken - This is to stop it making a http call over the internet to retrive it Required by Azure-Samples#420 * Parameterize tests with pytest.mark.parametrize * Improve test logging * Extract boolean env var behaviour * Add note about tiktoken issue * Allow tests to be run from anywhere * Update @types/node to v18 (Azure-Samples#513) * force solution to build for amd64 architecture as arm64 does not supp… (Azure-Samples#506) * force solution to build for amd64 architecture as arm64 does not support azure functions * installing git in container * Update .devcontainer/Dockerfile --------- Co-authored-by: Ross Smith <[email protected]> * Add test for failing downstream (Azure-Samples#520) Required by Azure-Samples#420 Co-authored-by: Ross Smith <[email protected]> * Add destroy to CI workflow (Azure-Samples#521) * Bump @types/restify from 8.5.11 to 8.5.12 in /extensions/teams (Azure-Samples#533) Bumps [@types/restify](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/restify) from 8.5.11 to 8.5.12. - [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases) - [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/restify) --- updated-dependencies: - dependency-name: "@types/restify" dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Verify the downstream calls being made by /conversations/custom (Azure-Samples#519) * Verify the downstream calls being made by /conversations/custom - To ensure we are making the correct calls, with the correct api keys, index information and request bodies - Implemented using a generic request matcher for now - This should be added to the mock server library by csernazs/pytest-httpserver#292 Required by Azure-Samples#420 * Improve error message * Add fixture comments for the https mock server * Fix test comments --------- Co-authored-by: Ross Smith <[email protected]> * Bump @fluentui/react-icons from 2.0.232 to 2.0.233 in /code/frontend (Azure-Samples#527) Bumps [@fluentui/react-icons](https://github.com/microsoft/fluentui-system-icons) from 2.0.232 to 2.0.233. - [Commits](https://github.com/microsoft/fluentui-system-icons/commits) --- updated-dependencies: - dependency-name: "@fluentui/react-icons" dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump vite from 5.2.2 to 5.2.6 in /code/frontend (Azure-Samples#528) Bumps [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite) from 5.2.2 to 5.2.6. - [Release notes](https://github.com/vitejs/vite/releases) - [Changelog](https://github.com/vitejs/vite/blob/main/packages/vite/CHANGELOG.md) - [Commits](https://github.com/vitejs/vite/commits/v5.2.6/packages/vite) --- updated-dependencies: - dependency-name: vite dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump postcss from 8.4.37 to 8.4.38 in /code/frontend (Azure-Samples#531) Bumps [postcss](https://github.com/postcss/postcss) from 8.4.37 to 8.4.38. - [Release notes](https://github.com/postcss/postcss/releases) - [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md) - [Commits](postcss/postcss@8.4.37...8.4.38) --- updated-dependencies: - dependency-name: postcss dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump @types/react from 18.2.67 to 18.2.70 in /code/frontend (Azure-Samples#532) Bumps [@types/react](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/react) from 18.2.67 to 18.2.70. - [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases) - [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/react) --- updated-dependencies: - dependency-name: "@types/react" dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump @fluentui/react from 8.116.0 to 8.117.0 in /code/frontend (Azure-Samples#530) Bumps [@fluentui/react](https://github.com/microsoft/fluentui) from 8.116.0 to 8.117.0. - [Release notes](https://github.com/microsoft/fluentui/releases) - [Changelog](https://github.com/microsoft/fluentui/blob/master/azure-pipelines.release-fluentui.yml) - [Commits](https://github.com/microsoft/fluentui/compare/@fluentui/react_v8.116.0...@fluentui/react_v8.117.0) --- updated-dependencies: - dependency-name: "@fluentui/react" dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump typescript from 5.4.2 to 5.4.3 in /code/frontend (Azure-Samples#529) Bumps [typescript](https://github.com/Microsoft/TypeScript) from 5.4.2 to 5.4.3. - [Release notes](https://github.com/Microsoft/TypeScript/releases) - [Changelog](https://github.com/microsoft/TypeScript/blob/main/azure-pipelines.release.yml) - [Commits](microsoft/TypeScript@v5.4.2...v5.4.3) --- updated-dependencies: - dependency-name: typescript dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * AZD can use containers instead of code for deployment (Azure-Samples#505) * Add adminweb docker image * Build off my branch * Create Requirements. * Seperate process * Dont traverse paths * Use poetry in docker build * dont install the root app * Add frontend image * Revert docker changes * Install the root again * Use requirements rather than toml * Use Requirements for images again * Rename docker images * Frontend use app directory * Add python path * Copy the whole code folder * Frontend runs locally * Frontend and Admin working * Use containers as default hosting model * Dont build from my branch anymore * Only build docker images - after test * Change trigger workflow * Only run when tests succeed * Trigger builds only if tests pass * Only changed App Service names * Reverting * Bump pytest-cov from 4.1.0 to 5.0.0 (Azure-Samples#526) Bumps [pytest-cov](https://github.com/pytest-dev/pytest-cov) from 4.1.0 to 5.0.0. - [Changelog](https://github.com/pytest-dev/pytest-cov/blob/master/CHANGELOG.rst) - [Commits](pytest-dev/pytest-cov@v4.1.0...v5.0.0) --- updated-dependencies: - dependency-name: pytest-cov dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump pre-commit from 3.6.2 to 3.7.0 (Azure-Samples#524) Bumps [pre-commit](https://github.com/pre-commit/pre-commit) from 3.6.2 to 3.7.0. - [Release notes](https://github.com/pre-commit/pre-commit/releases) - [Changelog](https://github.com/pre-commit/pre-commit/blob/main/CHANGELOG.md) - [Commits](pre-commit/pre-commit@v3.6.2...v3.7.0) --- updated-dependencies: - dependency-name: pre-commit dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Set AZURE_APP_SERVICE_HOSTING_MODEL for CI (Azure-Samples#536) * Create env before deploying (Azure-Samples#537) * Add basic ui tests for admin and user websites. * Move ui tests to tests folder and create uitest target on makefile. * Commit updated main.json. * Move apt-get install for libraries required by cypress to dockerfile. * Remove newline at the end of postCreate.sh. * Conditional website address and install npm * Move UI tests out of code folder and revert changes to azure.yaml * Update ui tests location. * Update dependabot.yml with new location of ui tests package.json. --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: Chinedum Echeta <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Adam Dougal <[email protected]> Co-authored-by: Ren Silva <[email protected]> Co-authored-by: Ross Smith <[email protected]> Co-authored-by: ross-p-smith <[email protected]>

ross-p-smith reviewed Mar 20, 2024

View reviewed changes

code/frontend/cypress.config.ts Outdated Show resolved Hide resolved

ross-p-smith reviewed Mar 20, 2024

View reviewed changes

code/frontend/cypress/fixtures/ingest.txt Outdated Show resolved Hide resolved

ross-p-smith reviewed Mar 20, 2024

View reviewed changes

code/frontend/cypress/support/e2e.ts Outdated Show resolved Hide resolved

ross-p-smith assigned superhindupur Mar 20, 2024

adamdougal reviewed Mar 21, 2024

View reviewed changes

code/frontend/cypress/e2e/admin.spec.cy.ts Outdated Show resolved Hide resolved

code/frontend/cypress/fixtures/ingest.txt Outdated Show resolved Hide resolved

code/frontend/cypress/support/commands.ts Outdated Show resolved Hide resolved

Add basic ui tests for admin and user websites.

2f7c764

superhindupur force-pushed the bhindupur/ui-tests branch from 107e83b to 2f7c764 Compare March 22, 2024 11:03

cecheta reviewed Mar 25, 2024

View reviewed changes

Move ui tests to tests folder and create uitest target on makefile.

b09bb90

superhindupur added 4 commits March 25, 2024 15:54

Update UI tests README to use latest env variable names.

de71448

Delete package-lock.json file added in error.

6ab81c9

Update wrong command in ui tests README.md.

18fce4f

Remove unnecessary fixture file.

52fb765

ross-p-smith reviewed Mar 25, 2024

View reviewed changes

.devcontainer/postCreate.sh Outdated Show resolved Hide resolved

cecheta reviewed Mar 25, 2024

View reviewed changes

ross-p-smith reviewed Mar 25, 2024

View reviewed changes

Makefile Outdated Show resolved Hide resolved

ross-p-smith reviewed Mar 25, 2024

View reviewed changes

Makefile Outdated Show resolved Hide resolved

cecheta and others added 9 commits March 26, 2024 09:43

Update all UI dependencies (#502)

46a55ca

* Update minor/patch depedencies * @vitejs/plugin-react major update * Update vite

Update tasks.json for func start (#511)

aeaacb3

dependabot bot and others added 14 commits March 26, 2024 09:43

Set AZURE_APP_SERVICE_HOSTING_MODEL for CI (#536)

94690b1

Create env before deploying (#537)

b3bad81

Add basic ui tests for admin and user websites.

adcc843

Move ui tests to tests folder and create uitest target on makefile.

29baf9b

Commit updated main.json.

90c2f2a

Merge branch 'main' into bhindupur/ui-tests

f328746

Move apt-get install for libraries required by cypress to dockerfile.

4fc2320

ross-p-smith reviewed Mar 26, 2024

View reviewed changes

.devcontainer/postCreate.sh Show resolved Hide resolved

superhindupur and others added 3 commits March 26, 2024 12:37

Remove newline at the end of postCreate.sh.

45dc8f5

Conditional website address and install npm

2fe78e3

Merge branch 'main' into bhindupur/ui-tests

50c8883

superhindupur added 3 commits March 26, 2024 14:58

Move UI tests out of code folder and revert changes to azure.yaml

3404253

Update ui tests location.

59f8cb1

Update dependabot.yml with new location of ui tests package.json.

008ccc6

ross-p-smith approved these changes Mar 26, 2024

View reviewed changes

cecheta approved these changes Mar 26, 2024

View reviewed changes

superhindupur added this pull request to the merge queue Mar 26, 2024

Merged via the queue into main with commit b759a0d Mar 26, 2024
3 checks passed

superhindupur deleted the bhindupur/ui-tests branch March 26, 2024 16:19

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add basic ui tests for admin and user websites. #503

Add basic ui tests for admin and user websites. #503

superhindupur commented Mar 20, 2024

ross-p-smith commented Mar 20, 2024 •

edited

Loading

adamdougal left a comment

github-actions bot commented Mar 25, 2024 •

edited

Loading

cecheta Mar 25, 2024

ross-p-smith commented Mar 25, 2024

ross-p-smith commented Mar 25, 2024

ross-p-smith commented Mar 26, 2024

superhindupur commented Mar 26, 2024 •

edited

Loading

ross-p-smith left a comment

cecheta left a comment

Add basic ui tests for admin and user websites. #503

Add basic ui tests for admin and user websites. #503

Conversation

superhindupur commented Mar 20, 2024

Purpose

Does this introduce a breaking change?

Pull Request Type

How to Test

What to Check

ross-p-smith commented Mar 20, 2024 • edited Loading

adamdougal left a comment

Choose a reason for hiding this comment

github-actions bot commented Mar 25, 2024 • edited Loading

cecheta Mar 25, 2024

Choose a reason for hiding this comment

ross-p-smith commented Mar 25, 2024

ross-p-smith commented Mar 25, 2024

ross-p-smith commented Mar 26, 2024

superhindupur commented Mar 26, 2024 • edited Loading

ross-p-smith left a comment

Choose a reason for hiding this comment

cecheta left a comment

Choose a reason for hiding this comment

ross-p-smith commented Mar 20, 2024 •

edited

Loading

github-actions bot commented Mar 25, 2024 •

edited

Loading

superhindupur commented Mar 26, 2024 •

edited

Loading