Skip to content

Siege integration with Travis CI #50

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 25 commits into from
Feb 22, 2018
Merged

Siege integration with Travis CI #50

merged 25 commits into from
Feb 22, 2018

Conversation

@ghost
Copy link

@ghost ghost commented Feb 21, 2018

close #25

Marc-Antoine-Leconte and others added 22 commits February 21, 2018 14:56
@Marc-Antoine-Leconte
HttpParserTooStrict(debug): add debug prints to fix too strict httppa…
094f796 
…rser
siege(test): Travis jobs tests
d282803
siege(test): Siege integration with docker
be5d7dd
siege(fix): Change localhost port
e81e337
siege(fix): Change localhost port to 80
6c26397
siege(test): Launch and kill zia in background
aaf8747
siege(test): Siege process in one job
f74e08c
siege(fix): Zia executable path in Travis job
7319705
siege(test): Try to find cpp_zia exe
b1cb290
siege(test): Add project build
075f3a6
siege(test): Redirect zia input to fifo file
49ec6b4
siege(fix): Fix input file path
d67c83f
siege(test): Zia background execution with screen
0da4e81
siege(test): Zia background execution with screen
e8401c4
siege(feat): Replace CLI for background execution
a6dbed3
siege(fix): Switch to port 7000
af2d5b2
siege(fix): Remove screen command for Travis
6ffea23
siege(fix): Change URL to siege
306bd9b
siege(test): Change Travis tests order
545c725
siege(test): Change Travis tests order
2861615
siege(test): Switch to two jobs
91e8066
siege(feat): Travis integration of siege done
3f5a2f7
@ghost ghost requested a review from AEnguerrand February 21, 2018 17:54
@ghost ghost added this to the Delivery milestone Feb 21, 2018
@ghost ghost self-assigned this Feb 21, 2018
@codecov
Copy link

codecov bot commented Feb 21, 2018 •
edited
Loading

Codecov Report

Merging #50 into master will increase coverage by 0.1%.
The diff coverage is 50%.

Impacted file tree graph

@@           Coverage Diff            @@
##           master     #50     +/-   ##
========================================
+ Coverage    5.66%   5.76%   +0.1%     
========================================
  Files          18      18             
  Lines        1854    1856      +2     
========================================
+ Hits          105     107      +2     
  Misses       1749    1749
Impacted Files Coverage Δ
src/zia.cpp 0% <0%> (ø) ⬆️
src/HttpParser/HttpParser.cpp 42.55% <100%> (+0.82%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a707730...0a057a5. Read the comment docs.

@AEnguerrand AEnguerrand merged commit 2e37f9f into master Feb 22, 2018
@AEnguerrand AEnguerrand deleted the siege branch February 22, 2018 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

enhancement units tests

Projects

None yet

Milestone

Delivery

Development

Successfully merging this pull request may close these issues.

Siége testing

2 participants

@AEnguerrand @Marc-Antoine-Leconte