search

hyperledger-cacti / cacti

Hyperledger Cacti is a new approach to the blockchain interoperability problem
https://wiki.hyperledger.org/display/cactus
Apache License 2.0
344 stars 286 forks source link

test(connector-xdai): jestify openapi-validation.test.ts #3569

Open petermetz opened 1 month ago

petermetz commented 1 month ago

Description

The task is to migrate a specific test case (relative path based below) to Jest from TAP: packages/cactus-plugin-ledger-connector-xdai/src/test/typescript/integration/openapi/openapi-validation.test.ts

An example pull request showing how these changes are to be done. Pay attention to the config file changes and the coding style of imports, describe() calls, it()/test() calls, the location of the variables storing the environment specific configuration that is created at runtime during the test execution, etc.

The example PR to use as a reference: https://github.com/hyperledger/cacti/pull/3206/files

Acceptance Criteria

  1. Test is only using Jest for assertions.
  2. Config files updated (.taprc, ci.yaml, jest config, etc) so that the test has no hardcoded references to it in these files and instead gets picked up by the correct glob of the test job in ci.yaml related to the package housing the test.
  3. Test is passing
  4. The way the changes were done to the test case matches all the established patterns. See example PR linked above for the patterns of refactoring.
Shubhamkunwar10 commented 1 month ago

Accepting 1st open source project

petermetz commented 1 month ago

@Shubhamkunwar10 Thank you very much in advance for the contribution!

petermetz commented 1 month ago

@Shubhamkunwar10 Just checking: Are you still working on this? Do you need any help?

Shubhamkunwar10 commented 3 weeks ago

Got busy. Had to work on erc4337 project. Today I am back will update you soon @petermetz

petermetz commented 3 weeks ago

@Shubhamkunwar10 No worries, thank you again for the contribution in advance!