Skip to content

Conversation

@katieblomquist
Copy link
Contributor

@katieblomquist katieblomquist commented May 30, 2024

Introduces Front End E2E tests to Clark and includes 1 sample test.

Notes:

  • Node V16 enforcement in package.json needed to be removed in order for this to work. I don't believe this check has any real effect on the client. If you run into problems, let me know.
  • Front End tests were removed from CircleCI because Cypress charges for CI, but just running it locally is free.

Documentation : https://github.com/Cyber4All/cyber4all-docs/pull/16#issue-2326065763

@katieblomquist
Copy link
Contributor Author

Note: I added Some of the questions asked throughout this PR to an FAQ section in the Front End Testing Documentation.

@katieblomquist katieblomquist requested review from mfranz1 and pzalep1 May 31, 2024 13:05
Copy link
Contributor

@JosueMolinaMorales JosueMolinaMorales left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Love it. Ship it

@mfranz1 mfranz1 requested review from johngonz0 and ksmartin301 July 22, 2024 13:34
@mfranz1 mfranz1 marked this pull request as draft August 1, 2024 21:20
@mfranz1
Copy link
Contributor

mfranz1 commented Aug 1, 2024

Passing the buck to the one, the only, our leader, she whom we will go over a cliff for.... Dr. Mrs. Professor Flores previously Zaleppa, Paige A.

@mfranz1
Copy link
Contributor

mfranz1 commented Aug 1, 2024

Passing the buck to the one, the only, our leader, she whom we will go over a cliff for.... Dr. Mrs. Professor Flores previously Zaleppa, Paige A.

@pzalep1

Comment on lines -15 to -19
- cyber4all/test-app:
docker-image: cimg/node@sha256:9e7dff70aee20446a146e374367c3fddc58df1e39cca318dbd4993e625892880
node-test: test:unit
isNode: true

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The orb just needs to be updated

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants