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.

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

Add performance test #168

Merged
merged 15 commits into from
Feb 27, 2020
Merged

Add performance test #168

merged 15 commits into from
Feb 27, 2020

Conversation

nikolay-borzov
Copy link
Collaborator

@nikolay-borzov nikolay-borzov commented Feb 22, 2020 •
edited
Loading

  • Reorganize tests folder structure - split tests into end-to-end, unit and performance
  • Add performance test

Resolves #166

@nikolay-borzov
Add performance test (#166)
7742bdc 
- Reorganize tests folder structure - split tests into end-to-end, unit and performance
- Add performance test
@nikolay-borzov nikolay-borzov requested a review from danvk February 22, 2020 13:36
@nikolay-borzov nikolay-borzov self-assigned this Feb 22, 2020
@nikolay-borzov
Add performance test (#166)
f2c1b61 
- Use `PerformanceObserver` to get duration to make actual execution time visible in failed test
@nikolay-borzov nikolay-borzov changed the title Add performance test (#166) Add performance test Feb 22, 2020
@nikolay-borzov
Add performance test (#166)
a518367 
- Set performance test threshold to value actual for CI
@coveralls
Copy link

coveralls commented Feb 22, 2020 •
edited
Loading

Coverage Status

Coverage remained the same at 97.938% when pulling 7ca2f99 on add-performance-test into c78a5c7 on master.

@nikolay-borzov
Add performance test (#168)
b38e1e7 
- Measure performance with `format = html` to cover default use case
@nikolay-borzov
Add performance test (#168)
192594c 
- Increase threshold for CI
@nikolay-borzov
Add performance test (#168)
9499eaf 
- Add `windows` and `osx` to Travis build matrix
@nikolay-borzov
Add performance test (#168)
3c65f5a 
- Increase threshold more
@nikolay-borzov
Add performance test (#168)
d8d0a4b 
- Try to normalize EOLs on CI
@nikolay-borzov
Add performance test (#168)
a3e6ad0 
- Increase threshold further
@nikolay-borzov
Add performance test (#168)
1b60862 
- Remove obsolete key `sudo`
@nikolay-borzov
Add performance test (#168)
8d773a2 
- Force using LF on CI
@nikolay-borzov
Add performance test (#168)
d9960c1 
- Force LF for CI
@nikolay-borzov
Add performance test (#168)
5ee4802 
- Revert changes to `.travis.yml`. Will be added as a separate commit
- Increase threshold
danvk
danvk approved these changes Feb 25, 2020
View reviewed changes
@nikolay-borzov
Add performance test (#168)
f8548c0 
- Add `_MS` to constant name
@nikolay-borzov
Add performance test (#168)
7ca2f99 
- Set threshould to 6000 so that we have ~1000ms buffer from the worst result on CI
@nikolay-borzov nikolay-borzov merged commit 7ec1b38 into master Feb 27, 2020
@nikolay-borzov nikolay-borzov deleted the add-performance-test branch February 27, 2020 17:18
@nikolay-borzov nikolay-borzov added this to the 2.4.0 milestone Mar 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@danvk danvk danvk approved these changes

Assignees

@nikolay-borzov nikolay-borzov

Labels
🔧 internal
Projects
None yet
Milestone
2.4.0
Development

Successfully merging this pull request may close these issues.

Add performance test
3 participants
@nikolay-borzov @coveralls @danvk