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

docs: remove mentions of Hedera in documentation #286

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 7 additions & 7 deletions DevelopmentProcess.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# TCK Development Process

This document is meant to describe and outline the process by which TCK tests are written and developed for a specific Hedera request type. Since the work for these TCK tests will not only be contained to this repo but also spread out amongst all the SDK repos, there will be a good amount of coordination required between all developers to make sure TCK and SDK work stays in parity with each other. The outline below will describe how tests should be documented, developed, and tested to keep this parity. This process is always open to changes at any point if the developers find better or more optimal ways to accomplish tasks.
This document is meant to describe and outline the process by which TCK tests are written and developed for a specific request type. Since the work for these TCK tests will not only be contained to this repo but also spread out amongst all the SDK repos, there will be a good amount of coordination required between all developers to make sure TCK and SDK work stays in parity with each other. The outline below will describe how tests should be documented, developed, and tested to keep this parity. This process is always open to changes at any point if the developers find better or more optimal ways to accomplish tasks.

## SDK Leads

Expand All @@ -17,18 +17,18 @@ Since the process outlined will require coordination across all SDKs, having a d

## Process

The TCK development process encompasses all work done for a Hedera request type, including documentation, development of tests, the actual testing of the tests, and finally synchronously merging all the work.
The TCK development process encompasses all work done for a request type, including documentation, development of tests, the actual testing of the tests, and finally synchronously merging all the work.

### Step 1: Documentation

Before any development takes place, the tests to be written need to be thought out and put in a markdown file in the `test-specifications` folder. A new file should use the `test-specifications/testSpecificationsTemplate.md` file as a template. The new markdown file should also be placed within a folder within the `test-specifications` folder that contains all the tests for the particular Hedera service that services the request for which tests are being written. For example, if tests are being written for `AccountCreateTransaction`, the `accountCreateTransaction.md` file should be placed in `test-specifications/crypto-service`.
Before any development takes place, the tests to be written need to be thought out and put in a markdown file in the `test-specifications` folder. A new file should use the `test-specifications/testSpecificationsTemplate.md` file as a template. The new markdown file should also be placed within a folder within the `test-specifications` folder that contains all the tests for the particular service that services the request for which tests are being written. For example, if tests are being written for `AccountCreateTransaction`, the `accountCreateTransaction.md` file should be placed in `test-specifications/crypto-service`.

The items that should be included in a request's test documentation can be seen by looking at the `testSpecificationsTemplate.md` file. These items include:
- Description: A description of the test specification. This, for the most part, should be copy-paste between test files with slight changes for names, links, etc.
- Design: A brief rundown of what is being tested, how it's being tested, and how the test results can be verified.
- Request properties: A link to the Hedera documentation for the request.
- Request properties: A link to the documentation for the request.
- Request protobuf: A link to the protobuf file located in the [hedera-protobufs](https://github.com/hashgraph/hedera-protobufs) repository for the request type being tested.
- Response codes: A link to the protobuf file that contains the response codes for gRPC requests sent to a Hedera network.
- Response codes: A link to the protobuf file that contains the response codes for gRPC requests sent to a network.
- Mirror Node APIs: A link to the mirror node APIs that can be used to verify test results.
- Code snippet: A code snippet that shows how to use the request type in code.
- JSON-RPC API documentation: The information on how the JSON-RPC servers should implement the method being used to test. This should include:
Expand All @@ -49,7 +49,7 @@ Once the tests have been written, they can be put up for in a pull request. The

### Step 2a: TCK Test Driver

Once the tests are written, approved, and merged, they can then be developed. The approved and merged test documentation should be used to discern what tests to write and how they should operate. Much like the test documentation, the file that contains the code for the tests should be placed within a folder within the `test` folder that contains all the tests for the particular Hedera service that services the request for which the tests are being written. For example, if tests are being written for `AccountCreateTransaction`, the `test_accountCreateTransaction.js` file should be placed in `test/crypto-service`.
Once the tests are written, approved, and merged, they can then be developed. The approved and merged test documentation should be used to discern what tests to write and how they should operate. Much like the test documentation, the file that contains the code for the tests should be placed within a folder within the `test` folder that contains all the tests for the particular service that services the request for which the tests are being written. For example, if tests are being written for `AccountCreateTransaction`, the `test_accountCreateTransaction.js` file should be placed in `test/crypto-service`.

A few guidelines for developing the tests:
- The name of the test file should match the name of the documentation, with a `test_` prepended to the file name, and obviously the different file extension. So a documentation markdown file named `accountCreateTransaction.md` would have its test implementation file named `test_accountCreateTransaction.js`.
Expand Down Expand Up @@ -108,4 +108,4 @@ Once the development of the SDK server is complete, it should be tested against

### Step 3: Merge Pull Requests

Once the development work for the TCK and the SDK servers is complete, three approvals from the SDK leads will be required for any pull requests to be merged. This is a new improvement, to merge PRs early enough while waiting for all the tests to pass on all SDK servers simultaneously. This will then mark the completion of the development work for this specific Hedera request.
Once the development work for the TCK and the SDK servers is complete, three approvals from the SDK leads will be required for any pull requests to be merged. This is a new improvement, to merge PRs early enough while waiting for all the tests to pass on all SDK servers simultaneously. This will then mark the completion of the development work for this specific request.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ In near future hedera-local-node will be transfered to Hiero (see our [transitio

### Configure usage of Hedera Testnet

- Get a Hedera testnet account ID and private key from Hedera [here](https://portal.hedera.com/register)
- Get a Hedera testnet account ID and private key [here](https://portal.hedera.com/register)
- rename `.env.testnet` to `.env`
- Add ECDSA account ID and private key to `.env`

Expand Down
8 changes: 3 additions & 5 deletions docs/design.md
Original file line number Diff line number Diff line change
Expand Up @@ -30,9 +30,8 @@ The SDK's JSON-RPC server returns responses that originate from the consensus no

### Requirements

1. The hedera network must support at least 4 nodes, so one can be shut down
without affecting consensus.
2. The hedera network will be a local network setup by the TCK
1. The test network must support at least 4 nodes, so one can be shut down without affecting consensus.
2. The test network will be a local network setup by the TCK
3. The TCK will be an executable NPM module.
4. The JSON-RPC server for the SDK must be started prior to running the TCK.
5. The TCK must take configuration, requiring the endpoint of the JSON-RPC
Expand All @@ -43,8 +42,7 @@ The SDK's JSON-RPC server returns responses that originate from the consensus no

### Guidance

The hedera network ideally would be a hedera-local-node, which would mean adding
support for multiple nodes to the hedera-local-node (Issue is already filed)
The network ideally would be a hedera-local-node, which would mean adding support for multiple nodes to the hedera-local-node (Issue is already filed)

### JSON-RPC API Examples

Expand Down
2 changes: 1 addition & 1 deletion test-specifications/commonTransactionParameters.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# Common Transaction Parameters

There are common parameters that can be set for all Hedera transaction types. This document specifies a common JSON object that should be added to all transactions that encapsulates these common parameters.
There are common parameters that can be set for all Hiero transaction types. This document specifies a common JSON object that should be added to all transactions that encapsulates these common parameters.

## Transaction Parameter Object Definition

Expand Down
6 changes: 3 additions & 3 deletions test-specifications/errorCodes.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ The JSON-RPC 2.0 specification that the TCK and SDK servers use allow for implem

## Errors

### Hedera Error
### Hiero Error

#### Error code

Expand All @@ -26,10 +26,10 @@ The `data` object in the JSON-RPC 2.0 `error` object should contain the status o
"id": <ID>,
"error": {
"code": -32001,
"message": "Hedera error",
"message": "Hiero error",
"data": {
"status": "INVALID_SIGNATURE",
"message": "Hedera transaction [email protected] failed precheck with status INVALID_SIGNATURE"
"message": "Hiero transaction [email protected] failed precheck with status INVALID_SIGNATURE"
}
}
}
Expand Down
2 changes: 1 addition & 1 deletion test-specifications/utility.md
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,7 @@ Method used to close the TCK network connections. Network connections can be ree

#### Description

Method used to generate a Hedera Key.
Method used to generate a Hiero Key.

#### Input Parameters

Expand Down