search

onflow / kitty-items

Kitty Items: CryptoKitties Sample App
https://kitty-items.onflow.org
Apache License 2.0
408 stars 213 forks source link

[Testing] Add basic tests to demonstrate JS testing library #71

Closed srinjoyc closed 2 years ago

srinjoyc commented 3 years ago

Why

Currently, we don't have any usage of our JS testing library in this keystone example to demonstrate how the integration works.

What (Definition of Done)

Add the following basic tests:

  1. Account creation: Multiple accounts can be created
  2. Contract deploy: Deploys all contracts one by one in the cadence folders.
  3. Transaction to mint kibble.
  4. Transaction to mint kitty-item
  5. Transaction to transfer kitty-item

Add documentation to the README explaining the testing and link out to remaining testing docs on the testing repo. https://github.com/onflow/flow-js-testing

10thfloor commented 3 years ago

@MaxStalker

MaxStalker commented 3 years ago

@10thfloor I think we covered this one. The only problem we have right now is failing tests in CI environment. I was not able to find a stable solution, but one of the community members reported that they might have a clue :) So I'm waiting for their response to fix it everywhere.

10thfloor commented 2 years ago

@MaxStalker Any word on this?

MaxStalker commented 2 years ago

@10thfloor we've identified a couple of solutions to said problem here: https://github.com/onflow/flow-js-testing/issues/61

I have started with custom runner path, but no tangible result yet. Still WIP 🙏

MaxStalker commented 2 years ago

@10thfloor can you check #150 for errors with Cypress tests? They block check for that PR 😓

10thfloor commented 2 years ago

Closing because stale. Tests have been added.