Skip to content

Add testnet client init process test. Add a param 'fast' for GNTE test#226

Merged
leventeliu merged 17 commits intodevelopfrom
feature/test_cmd_script
Jan 22, 2019
Merged

Add testnet client init process test. Add a param 'fast' for GNTE test#226
leventeliu merged 17 commits intodevelopfrom
feature/test_cmd_script

Conversation

@laodouya
Copy link
Member

@laodouya laodouya commented Jan 22, 2019 •
edited
Loading

Add testnet client init test:

  1. create a client and key pair
  2. generate a wallet address
  3. transfer some coin to new address
  4. create db
  5. connect db and create table.

Add a param 'fast' for GNTE test

laodouya added 16 commits January 21, 2019 16:04
@laodouya
Add flag skipMasterKey in cql-utils for using empty master key
a0749dd
@laodouya
Add testnet create account test.
4ed6780
@laodouya
Increase Particle balance for testnet create db test, add sleep for t…
c0f2278 
…ransaction done.
@laodouya
Add testnet client test to gitlab ci
e065452
@laodouya
Split gitlab ci test work to three sub jobs for parallel test.
1d4e980
@laodouya
Add gitlab ci build stage.
1b6f208
@laodouya
Reset GOPATH for every job at gitlab ci.
79d0af4
@laodouya
Make local miner tests in one gitlabci jobs.
6f0b542
@laodouya
Copy gitlabci build stage bins to test stage.
9f1a965
@laodouya
Fix gitlab ci artifacts path wrong
8ae2f3d
@laodouya
Change gitlab ci stages from artifacts to cache
a488ab0
@laodouya
Add git commit id as gitlabci cache key.
6df0f83
@laodouya
Remove gitlab ci cache. Remove stage for gitlabci. Split make in diff…
51d4953 
…erent jobs
@laodouya
Fix testnet config missing params. add wait-tx-confirm in transfer to…
cc299f3 
…ken test.
@laodouya
Increase waitTxConfirmationMaxDuration to 20 times BPPeriod. Add wait…
5caa487 
…TxConfirmation in createdb.
@laodouya
Add a param 'fast' for GNTE test.
12f28af
@laodouya laodouya changed the title [WIP] Add testnet client init process test. Add testnet client init process test. Jan 22, 2019
@laodouya laodouya changed the title Add testnet client init process test. Add testnet client init process test. Add a param 'fast' for GNTE test Jan 22, 2019
@codecov
Copy link

codecov bot commented Jan 22, 2019 •
edited
Loading

Codecov Report

Merging #226 into develop will decrease coverage by 0.08%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #226      +/-   ##
===========================================
- Coverage    76.03%   75.95%   -0.09%     
===========================================
  Files          136      136              
  Lines        11281    11281              
===========================================
- Hits          8578     8568      -10     
- Misses        2046     2052       +6     
- Partials       657      661       +4

@codecov
Copy link

codecov bot commented Jan 22, 2019

Codecov Report

Merging #226 into develop will increase coverage by 0.03%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #226      +/-   ##
===========================================
+ Coverage    75.99%   76.03%   +0.03%     
===========================================
  Files          136      136              
  Lines        11281    11281              
===========================================
+ Hits          8573     8577       +4     
+ Misses        2049     2045       -4     
  Partials       659      659

auxten
auxten approved these changes Jan 22, 2019
View reviewed changes
Copy link
Contributor

@auxten auxten left a comment

Choose a reason for hiding this comment

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

LGTM

leventeliu
leventeliu approved these changes Jan 22, 2019
View reviewed changes
Copy link
Contributor

@leventeliu leventeliu left a comment

Choose a reason for hiding this comment

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

Approve.

@leventeliu leventeliu merged commit fc865ea into develop Jan 22, 2019
@leventeliu leventeliu deleted the feature/test_cmd_script branch January 22, 2019 07:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@auxten auxten auxten approved these changes

@leventeliu leventeliu leventeliu approved these changes

@xq262144 xq262144 Awaiting requested review from xq262144

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@laodouya @auxten @leventeliu