Skip to content

Add previous version compatibility test #269

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 31 commits into from
Feb 28, 2019
Merged

Add previous version compatibility test #269

merged 31 commits into from
Feb 28, 2019

Conversation

laodouya
Copy link
Member

  1. Add old bp/miner/client compatibility test
  2. Use common volumes in host for build bins

@laodouya
Add param for test old client.
2c7b406
@laodouya
Add start_bp_miner for makefile, Add test old clinet task.
bb59b5f
@laodouya
Fix test old client case missing param set bug.
572160f
@laodouya
Try using build docker with docker command.
a5e6baa
@laodouya
Revert compatibility test docker build.
5061e5c
@laodouya
Let testnet_client only run testnet.
b767b94
@laodouya
Add test old clinet to current bp miner test.
459bd53
@laodouya
Make bp miner bin before compatibility test.
b0972cc
@laodouya
Make compatibility test bp and miner running background
5435557
@laodouya
Fix bp miner not running background.
b5232b8
@laodouya
Fix compatibility test, wait for miner start.
223975c
@laodouya
Increase integration miner client init stable coin count.
24a2a9e
@laodouya
Increase compatibility test wait bp miner time.
cdb9222
@laodouya
Disable other ci test.
5a00d66
@laodouya
Fail compatibility test if dsn not created.
8349f4c
@laodouya
Fix compatibility test miner config.
70d1914
@laodouya
Revert "Disable other ci test."
2978a94 
This reverts commit 5a00d66.
@laodouya
Add ci build stage, reuse build result in test stage.
bf1eafd
@laodouya
Add old bp, old miner case for compatibility test.
fbdbda4
@laodouya
Rename gitlab ci task name.
9197878
@laodouya
Fix compatibility miner bin name wrong.
1112e67
@laodouya
Seperate logs for different compatibility test case.
61085a6
@laodouya
Update prev bin version to v0.4.0
2d080b5
@codecov
Copy link

codecov bot commented Feb 28, 2019 •
edited
Loading

Codecov Report

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

@@             Coverage Diff             @@
##           develop     #269      +/-   ##
===========================================
- Coverage    80.96%   80.45%   -0.51%     
===========================================
  Files          159      159              
  Lines        11914    11914              
===========================================
- Hits          9646     9586      -60     
- Misses        1580     1635      +55     
- Partials       688      693       +5

leventeliu
leventeliu reviewed Feb 28, 2019
View reviewed changes
xq262144
xq262144 reviewed Feb 28, 2019
View reviewed changes
@laodouya laodouya force-pushed the feature/compatibility branch 2 times, most recently from b657eb2 to 13b1951 Compare February 28, 2019 09:36
@laodouya laodouya force-pushed the feature/compatibility branch from 13b1951 to 7aef69d Compare February 28, 2019 09:39
xq262144
xq262144 reviewed Feb 28, 2019
View reviewed changes
xq262144
xq262144 approved these changes Feb 28, 2019
View reviewed changes
Copy link
Contributor

@xq262144 xq262144 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 approved these changes Feb 28, 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.

LGTM.

@leventeliu leventeliu merged commit 895721b into develop Feb 28, 2019
@leventeliu leventeliu deleted the feature/compatibility branch February 28, 2019 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xq262144 xq262144 xq262144 approved these changes

@leventeliu leventeliu leventeliu approved these changes

@auxten auxten Awaiting requested review from auxten

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 @xq262144 @leventeliu