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

feat: Implement verbose output for transactions pipelines #148

Merged
merged 50 commits into from
May 17, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
50 commits
Select commit Hold shift + click to select a range
b6ff821
chore: add jest setup to coverage all files (#101)
Brunonascdev Mar 26, 2024
e78148d
refactor: fix circular import
fazzatti Mar 26, 2024
e9497cb
test: contract engine initial structure
fazzatti Mar 26, 2024
1d20ca7
Merge pull request #102 from CheesecakeLabs/unit-test
Brunonascdev Mar 26, 2024
a4a6bde
Contract Engine unit tests (#103)
fazzatti Mar 28, 2024
f66a40c
refactor: contract engine errors (#104)
fazzatti Mar 28, 2024
47ccf16
Update docs and minor adjustments to jsdocs (#105)
fazzatti Mar 28, 2024
c30ab86
packaging: bump version to 0.7.0 (#108)
fazzatti Mar 28, 2024
81b25eb
test: add build transaction unit tests
Brunonascdev Apr 1, 2024
1a17047
Merge branch 'develop' of github.com:CheesecakeLabs/stellar-plus into…
Brunonascdev Apr 1, 2024
961ad28
chore: merge with develop
Brunonascdev Apr 1, 2024
7249e02
test: add build transaction failure tests
Brunonascdev Apr 1, 2024
44561cb
refactor: make networkConfig variable private
Brunonascdev Apr 1, 2024
ec59df0
Merge pull request #110 from CheesecakeLabs/test/build-transaction-un…
Brunonascdev Apr 1, 2024
d8924db
test: add unit test for fee bump pipeline (#113)
Brunonascdev Apr 1, 2024
88a8b0a
test: add unit tests for classic signature requirements pipeline (#115)
fazzatti Apr 1, 2024
619b65f
Add unit tests for simulate transaction pipeline (#117)
fazzatti Apr 2, 2024
9712847
Add unit tests for classic transaction pipeline (#118)
fazzatti Apr 2, 2024
e2d382a
test: add unit tests for soroban transaction pipeline (#119)
fazzatti Apr 2, 2024
0da0a45
test: add unit test for sign transaction pipeline
Brunonascdev Apr 2, 2024
85c2044
Merge branch 'develop' of github.com:CheesecakeLabs/stellar-plus into…
Brunonascdev Apr 2, 2024
13e354a
Merge pull request #120 from CheesecakeLabs/test/sign-transaction-pip…
Brunonascdev Apr 2, 2024
72d7b93
test: add unit test for submit transaction pipeline
Brunonascdev Apr 3, 2024
ef9c214
Merge pull request #122 from CheesecakeLabs/test/submit-transaction-p…
Brunonascdev Apr 3, 2024
0d57031
Add unit tests soroban auth (#123)
fazzatti Apr 3, 2024
882cd85
Complete unit test coverage for account handlers (#125)
fazzatti Apr 5, 2024
10ace5d
Create test-coverage.yml action (#126)
fazzatti Apr 5, 2024
89405b9
Add unit test for base account handler (#127)
fazzatti Apr 5, 2024
4ab013b
Add unit test for soroban get transaction pipeline (#124)
fazzatti Apr 5, 2024
228a50f
Add unit tests for the classic asset handler (#128)
fazzatti Apr 5, 2024
4823037
feat: remove issuer requirement for classic assets (#129)
fazzatti Apr 8, 2024
210cbd7
test: add unit test to soroban token handler (#131)
fazzatti Apr 24, 2024
cf4b2e9
feat: remove helper classes for account (#132)
fazzatti Apr 24, 2024
c008ca6
Standardize default network configs and custom (#133)
fazzatti Apr 25, 2024
20abb09
packaging: bump version to v0.8.0
fazzatti Apr 25, 2024
158165a
Merge branch 'main' into develop
fazzatti Apr 25, 2024
5bf176a
Fix merge issues (#137)
fazzatti Apr 25, 2024
9eff855
Update stellar sdk (#139)
fazzatti Apr 30, 2024
39af9af
packaging: bump version to 0.8.1
fazzatti Apr 30, 2024
2fb5cee
Merge branch 'main' into develop
fazzatti Apr 30, 2024
2f367fa
Minor fixes (#141)
fazzatti May 1, 2024
75d6f4b
Merge branch 'main' into develop
fazzatti May 1, 2024
364ca4b
Contract engine improvements (#144)
fazzatti May 7, 2024
25c35aa
chore: bump version to 0.8.3
fazzatti May 7, 2024
aecd297
Merge branch 'main' into develop
fazzatti May 7, 2024
b6b97bb
fix: soroban auth simulation fails if partial auth entries are present
fazzatti May 10, 2024
703c935
chore: bump version to 0.8.4
fazzatti May 10, 2024
6d5ed86
Merge branch 'main' into develop
fazzatti May 10, 2024
3385918
feat: Implement verbose output for transactions pipelines (#147)
fazzatti May 16, 2024
91cc0f8
Merge branch 'main' into develop
fazzatti May 16, 2024
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
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "stellar-plus",
"version": "0.8.4",
"version": "0.9.0",
"description": "beta version of stellar-plus, an all-in-one sdk for the Stellar blockchain",
"main": "./lib/index.js",
"types": "./lib/index.d.ts",
Expand Down
17 changes: 10 additions & 7 deletions src/stellar-plus/asset/classic/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,10 @@ import {
} from 'stellar-plus/asset/classic/types'
import { AssetTypes } from 'stellar-plus/asset/types'
import { ClassicTransactionPipeline } from 'stellar-plus/core/pipelines/classic-transaction'
import { ClassicTransactionPipelineOptions } from 'stellar-plus/core/pipelines/classic-transaction/types'
import {
ClassicTransactionPipelineOptions,
ClassicTransactionPipelineOutputSimple,
} from 'stellar-plus/core/pipelines/classic-transaction/types'
import { TransactionInvocation } from 'stellar-plus/core/types'
import { HorizonHandlerClient as HorizonHandler } from 'stellar-plus/horizon'

Expand Down Expand Up @@ -231,10 +234,10 @@ export class ClassicAssetHandler implements IClassicAssetHandler {
source: this.asset.getIssuer(),
})

const result = await this.classicTransactionPipeline.execute({
const result = (await this.classicTransactionPipeline.execute({
txInvocation: updatedTxInvocation,
operations: [mintOp],
})
})) as ClassicTransactionPipelineOutputSimple

return result.response
}
Expand Down Expand Up @@ -290,10 +293,10 @@ export class ClassicAssetHandler implements IClassicAssetHandler {
source: this.asset.getIssuer(),
})

const result = await this.classicTransactionPipeline.execute({
const result = (await this.classicTransactionPipeline.execute({
txInvocation: updatedTxInvocation,
operations: [addTrustlineOp, mintOp],
})
})) as ClassicTransactionPipelineOutputSimple

return result.response
}
Expand Down Expand Up @@ -323,10 +326,10 @@ export class ClassicAssetHandler implements IClassicAssetHandler {
asset: this.asset,
})

const result = await this.classicTransactionPipeline.execute({
const result = (await this.classicTransactionPipeline.execute({
txInvocation,
operations: [addTrustlineOp],
})
})) as ClassicTransactionPipelineOutputSimple

return result.response
}
Expand Down
75 changes: 45 additions & 30 deletions src/stellar-plus/asset/soroban-token/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ import { spec as defaultSpec, methods } from 'stellar-plus/asset/soroban-token/c
import { SorobanTokenHandlerConstructorArgs, SorobanTokenInterface } from 'stellar-plus/asset/soroban-token/types'
import { AssetTypes } from 'stellar-plus/asset/types'
import { ContractEngine } from 'stellar-plus/core/contract-engine'
import { SorobanTransactionPipelineOutputSimple } from 'stellar-plus/core/pipelines/soroban-transaction/types'
import { SorobanSimulationInvocation, TransactionInvocation } from 'stellar-plus/core/types'
import { i128, u32 } from 'stellar-plus/types'

Expand Down Expand Up @@ -51,11 +52,11 @@ export class SorobanTokenHandler extends ContractEngine implements SorobanTokenI
*
* @description Initialize the contract instance with the given parameters.
*
* @returns {Promise<void>}
* @returns {Promise<SorobanTransactionPipelineOutputSimple>}
*/
public async initialize(
args: { admin: string; decimal: u32; name: string; symbol: string } & TransactionInvocation
): Promise<void> {
): Promise<SorobanTransactionPipelineOutputSimple> {
return (await this.invokeContract({
method: methods.initialize,
methodArgs: {
Expand All @@ -67,7 +68,7 @@ export class SorobanTokenHandler extends ContractEngine implements SorobanTokenI
signers: args.signers,
header: args.header,
feeBump: args.feeBump,
})) as void
})) as SorobanTransactionPipelineOutputSimple
}

//==========================================
Expand All @@ -86,11 +87,13 @@ export class SorobanTokenHandler extends ContractEngine implements SorobanTokenI
*
* @description Set a new admin account for the contract.
*
* @returns {Promise<void>}
* @returns {Promise<SorobanTransactionPipelineOutputSimple>}
*
* @requires {args.id} to be the current admin account
*/
public async setAdmin(args: { id: string; new_admin: string } & TransactionInvocation): Promise<void> {
public async setAdmin(
args: { id: string; new_admin: string } & TransactionInvocation
): Promise<SorobanTransactionPipelineOutputSimple> {
return (await this.invokeContract({
method: methods.set_admin,
methodArgs: {
Expand All @@ -100,7 +103,7 @@ export class SorobanTokenHandler extends ContractEngine implements SorobanTokenI
signers: args.signers,
header: args.header,
feeBump: args.feeBump,
})) as void
})) as SorobanTransactionPipelineOutputSimple
}

/**
Expand Down Expand Up @@ -133,11 +136,13 @@ export class SorobanTokenHandler extends ContractEngine implements SorobanTokenI
*
* @description Authorize or deauthorize an account to interact with the contract.
*
* @returns {Promise<void>}
* @returns {Promise<SorobanTransactionPipelineOutputSimple>}
*
* @requires - Authorization from the admin account
*/
public async setAuthorized(args: { id: string; authorize: boolean } & TransactionInvocation): Promise<void> {
public async setAuthorized(
args: { id: string; authorize: boolean } & TransactionInvocation
): Promise<SorobanTransactionPipelineOutputSimple> {
return (await this.invokeContract({
method: methods.set_authorized,
methodArgs: {
Expand All @@ -147,7 +152,7 @@ export class SorobanTokenHandler extends ContractEngine implements SorobanTokenI
signers: args.signers,
header: args.header,
feeBump: args.feeBump,
})) as void
})) as SorobanTransactionPipelineOutputSimple
}

/**
Expand All @@ -162,11 +167,13 @@ export class SorobanTokenHandler extends ContractEngine implements SorobanTokenI
*
* @description Mint tokens to an account.
*
* @returns {Promise<void>}
* @returns {Promise<SorobanTransactionPipelineOutputSimple>}
*
* @requires - Authorization from the admin account
*/
public async mint(args: { to: string; amount: i128 } & TransactionInvocation): Promise<void> {
public async mint(
args: { to: string; amount: i128 } & TransactionInvocation
): Promise<SorobanTransactionPipelineOutputSimple> {
return (await this.invokeContract({
method: methods.mint,
methodArgs: {
Expand All @@ -176,7 +183,7 @@ export class SorobanTokenHandler extends ContractEngine implements SorobanTokenI
signers: args.signers,
header: args.header,
feeBump: args.feeBump,
})) as void
})) as SorobanTransactionPipelineOutputSimple
}

/**
Expand All @@ -191,11 +198,13 @@ export class SorobanTokenHandler extends ContractEngine implements SorobanTokenI
*
* @description Clawback tokens from an account.
*
* @returns {Promise<void>}
* @returns {Promise<SorobanTransactionPipelineOutputSimple>}
*
* @requires - Authorization from the admin account
*/
public async clawback(args: { from: string; amount: i128 } & TransactionInvocation): Promise<void> {
public async clawback(
args: { from: string; amount: i128 } & TransactionInvocation
): Promise<SorobanTransactionPipelineOutputSimple> {
return (await this.invokeContract({
method: methods.clawback,
methodArgs: {
Expand All @@ -205,7 +214,7 @@ export class SorobanTokenHandler extends ContractEngine implements SorobanTokenI
signers: args.signers,
header: args.header,
feeBump: args.feeBump,
})) as void
})) as SorobanTransactionPipelineOutputSimple
}

//==========================================
Expand Down Expand Up @@ -253,11 +262,11 @@ export class SorobanTokenHandler extends ContractEngine implements SorobanTokenI
*
* @description Approve a spender to spend tokens on behalf of the account.
*
* @returns {Promise<void>}
* @returns {Promise<SorobanTransactionPipelineOutputSimple>}
*/
public async approve(
args: { from: string; spender: string; amount: i128; expiration_ledger: u32 } & TransactionInvocation
): Promise<void> {
): Promise<SorobanTransactionPipelineOutputSimple> {
return (await this.invokeContract({
method: methods.approve,
methodArgs: {
Expand All @@ -269,7 +278,7 @@ export class SorobanTokenHandler extends ContractEngine implements SorobanTokenI
signers: args.signers,
header: args.header,
feeBump: args.feeBump,
})) as void
})) as SorobanTransactionPipelineOutputSimple
}

/**
Expand Down Expand Up @@ -327,9 +336,11 @@ export class SorobanTokenHandler extends ContractEngine implements SorobanTokenI
*
* @description Transfer tokens from the sender to the recipient.
*
* @returns {Promise<void>}
* @returns {Promise<SorobanTransactionPipelineOutputSimple>}
*/
public async transfer(args: { from: string; to: string; amount: i128 } & TransactionInvocation): Promise<void> {
public async transfer(
args: { from: string; to: string; amount: i128 } & TransactionInvocation
): Promise<SorobanTransactionPipelineOutputSimple> {
return (await this.invokeContract({
method: methods.transfer,
methodArgs: {
Expand All @@ -340,7 +351,7 @@ export class SorobanTokenHandler extends ContractEngine implements SorobanTokenI
signers: args.signers,
header: args.header,
feeBump: args.feeBump,
})) as void
})) as SorobanTransactionPipelineOutputSimple
}

/**
Expand All @@ -357,11 +368,11 @@ export class SorobanTokenHandler extends ContractEngine implements SorobanTokenI
*
* @description Transfer tokens from the sender to the recipient on behalf of the spender.
*
* @returns {Promise<void>}
* @returns {Promise<SorobanTransactionPipelineOutputSimple>}
*/
public async transferFrom(
args: { spender: string; from: string; to: string; amount: i128 } & TransactionInvocation
): Promise<void> {
): Promise<SorobanTransactionPipelineOutputSimple> {
return (await this.invokeContract({
method: methods.transfer_from,
methodArgs: {
Expand All @@ -373,7 +384,7 @@ export class SorobanTokenHandler extends ContractEngine implements SorobanTokenI
signers: args.signers,
header: args.header,
feeBump: args.feeBump,
})) as void
})) as SorobanTransactionPipelineOutputSimple
}

/**
Expand All @@ -388,9 +399,11 @@ export class SorobanTokenHandler extends ContractEngine implements SorobanTokenI
*
* @description Burn tokens from an account.
*
* @returns {Promise<void>}
* @returns {Promise<SorobanTransactionPipelineOutputSimple>}
*/
public async burn(args: { from: string; amount: i128 } & TransactionInvocation): Promise<void> {
public async burn(
args: { from: string; amount: i128 } & TransactionInvocation
): Promise<SorobanTransactionPipelineOutputSimple> {
return (await this.invokeContract({
method: methods.burn,
methodArgs: {
Expand All @@ -400,7 +413,7 @@ export class SorobanTokenHandler extends ContractEngine implements SorobanTokenI
signers: args.signers,
header: args.header,
feeBump: args.feeBump,
})) as void
})) as SorobanTransactionPipelineOutputSimple
}

/**
Expand All @@ -416,9 +429,11 @@ export class SorobanTokenHandler extends ContractEngine implements SorobanTokenI
*
* @description Burn tokens from an account on behalf of the spender.
*
* @returns {Promise<void>}
* @returns {Promise<SorobanTransactionPipelineOutputSimple>}
*/
public async burnFrom(args: { spender: string; from: string; amount: i128 } & TransactionInvocation): Promise<void> {
public async burnFrom(
args: { spender: string; from: string; amount: i128 } & TransactionInvocation
): Promise<SorobanTransactionPipelineOutputSimple> {
return (await this.invokeContract({
method: methods.burn_from,
methodArgs: {
Expand All @@ -429,7 +444,7 @@ export class SorobanTokenHandler extends ContractEngine implements SorobanTokenI
signers: args.signers,
header: args.header,
feeBump: args.feeBump,
})) as void
})) as SorobanTransactionPipelineOutputSimple
}

/**
Expand Down
27 changes: 17 additions & 10 deletions src/stellar-plus/asset/soroban-token/types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ import { ContractSpec } from '@stellar/stellar-sdk'

import { AssetType } from 'stellar-plus/asset/types'
import { Options } from 'stellar-plus/core/contract-engine/types'
import { SorobanTransactionPipelineOutputSimple } from 'stellar-plus/core/pipelines/soroban-transaction/types'
import { SorobanSimulationInvocation, TransactionInvocation } from 'stellar-plus/core/types'
import { NetworkConfig, i128, u32 } from 'stellar-plus/types'

Expand All @@ -21,13 +22,13 @@ export type SorobanTokenInterface = AssetType & SorobanTokenInterfaceManagement
export type SorobanTokenInterfaceManagement = {
initialize: (
args: { admin: string; decimal: u32; name: string; symbol: string } & TransactionInvocation
) => Promise<void>
) => Promise<SorobanTransactionPipelineOutputSimple>
setAdmin: (
args: {
id: string
new_admin: string
} & TransactionInvocation
) => Promise<void>
) => Promise<SorobanTransactionPipelineOutputSimple>

admin: (args: TransactionInvocation) => Promise<string>

Expand All @@ -36,36 +37,42 @@ export type SorobanTokenInterfaceManagement = {
id: string
authorize: boolean
} & TransactionInvocation
) => Promise<void>
) => Promise<SorobanTransactionPipelineOutputSimple>

mint: (
args: {
to: string
amount: i128
} & TransactionInvocation
) => Promise<void>
) => Promise<SorobanTransactionPipelineOutputSimple>

clawback: (
args: {
from: string
amount: i128
} & TransactionInvocation
) => Promise<void>
) => Promise<SorobanTransactionPipelineOutputSimple>
}

export type SorobanTokenInterfaceUser = {
allowance: (args: { from: string; spender: string } & SorobanSimulationInvocation) => Promise<i128>
approve: (
args: { from: string; spender: string; amount: i128; expiration_ledger: u32 } & TransactionInvocation
) => Promise<void>
) => Promise<SorobanTransactionPipelineOutputSimple>
balance: (args: { id: string } & SorobanSimulationInvocation) => Promise<i128>
spendableBalance: (args: { id: string } & SorobanSimulationInvocation) => Promise<i128>
transfer: (args: { from: string; to: string; amount: i128 } & TransactionInvocation) => Promise<void>
transfer: (
args: { from: string; to: string; amount: i128 } & TransactionInvocation
) => Promise<SorobanTransactionPipelineOutputSimple>
transferFrom: (
args: { spender: string; from: string; to: string; amount: i128 } & TransactionInvocation
) => Promise<void>
burn: (args: { from: string; amount: i128 } & TransactionInvocation) => Promise<void>
burnFrom: (args: { spender: string; from: string; amount: i128 } & TransactionInvocation) => Promise<void>
) => Promise<SorobanTransactionPipelineOutputSimple>
burn: (
args: { from: string; amount: i128 } & TransactionInvocation
) => Promise<SorobanTransactionPipelineOutputSimple>
burnFrom: (
args: { spender: string; from: string; amount: i128 } & TransactionInvocation
) => Promise<SorobanTransactionPipelineOutputSimple>
decimals: (args: SorobanSimulationInvocation) => Promise<u32>
name: (args: SorobanSimulationInvocation) => Promise<string>
symbol: (args: SorobanSimulationInvocation) => Promise<string>
Expand Down
Loading
Loading