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

Cypress Implemented for binary search on a secret array #276

Merged
merged 57 commits into from
Apr 26, 2024
Merged

Conversation

Snafkin547
Copy link
Contributor

No description provided.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This file needed to be updated corresponding to the changes made due to package update

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This file needed to be updated corresponding to the changes made due to package update

@Snafkin547
Copy link
Contributor Author

A Cypress test was implemented for the array binary search test.

  1. Since the cypress test does not allow multiple tabs to be used, I changed the UI and MPC operations (initialization and asynchronous operation).
  • For the UI, you can simply initiate two jiff instances by one click and can submit an array and an element from the same window/tab
  • On the MPC operation front, it uses Web Worker because running two jiffs in a single tab/window ends up with a deadlock. With Web Worker, we can emulate two threads.
  1. Two files in dist had to be updated because of the update in jiff code due to the package update.

  2. I've also had to modify the ts config file, creating separate tsconfig files for cypress and jest so their types do not conflict. Both of these tsconfig files reference the original tsconfig file in the root.

Screenshot 2024-04-17 at 3 03 16 PM

@Snafkin547 Snafkin547 requested review from gregfrasco and removed request for gregfrasco April 17, 2024 22:28
@Snafkin547 Snafkin547 merged commit 2ff5439 into master Apr 26, 2024
5 of 33 checks passed
@Snafkin547 Snafkin547 deleted the cypress branch April 26, 2024 00:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants