Skip to content
This repository has been archived by the owner on Jul 8, 2024. It is now read-only.

Add cypress e2e tests for emulator #286

Merged
merged 12 commits into from
Aug 26, 2022

Conversation

codingone21
Copy link
Contributor

@codingone21 codingone21 commented Aug 18, 2022

Testing in Emulator

Scenarios that do not interact with kitty items:

  • Visits header buttons
  • Logs in as admin
  • Creates new account

Scenarios that involve minting kitty items:

  • Mints an item as from a service account + remove from store
  • Mints an item + funds an account + purchases item from the funded account
  • Log in and out from user accounts handled in 'beforeEach' and 'afterEach'

Activities Coverage

Accounts

  • Routing between header buttons
  • Logging in/out
  • Creating account

KI

  • Minting
  • Funding
  • Removing from Store
  • Purchasing

Testing in Testnet

Scenarios that do not require calling external apps:

  • Visits header buttons
  • Logs in as admin + mint an item
  • Connects to Blocto wallet with testnet account

Activities Coverage

  • Routing between header buttons
  • Connecting to wallet
  • Logging in/out - This is partially covered by connecting to wallet
  • Minting KI

Activities Requiring External Services

Next Issue: #289

Next Steps

@codingone21 codingone21 changed the title Add assertions to emulator tests Add cypress e2e tests for emulator Aug 22, 2022
@codingone21 codingone21 marked this pull request as ready for review August 25, 2022 20:29
it('Visit main page', () => {
cy.visit('http://localhost:3001/')
})
cy.get('[data-cy="header mint"]').should('exist')
Copy link
Contributor

Choose a reason for hiding this comment

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

I think it's better to use dashes instead spaced to separate words in a single-value attributes.
For example "header mint" -> "header-mint"
With spaces there is some ambiguity around whether it's one value or an array of 2 values

@codingone21 codingone21 merged commit 0751e2f into master Aug 26, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants