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

Search Test Runs via Env properties and display progress #109

Merged
merged 8 commits into from
Nov 6, 2017

Conversation

atodorov
Copy link
Member

@atodorov atodorov commented Nov 6, 2017

This change makes it possible to search Test Runs by environment properties and also improves the progress bar shown next to each run in the search results. We show the completion result in % and also show a red bar if there are failed cases.

100% means everything was tested, all blue means none of the tested cases failed!

this commit makes it possible to specify environment properties
when searching for Test Runs in both regular and advanced search
mode. There is a new text field where you can specify property
values split by comma.
NOTE: this needs to be on a single line because it is inlcuded
in a JSON template which fails when there are new lines b/c we can't
escapejs included content!
use the model method instead of calculating this every time and
also show a red bar if there are failed cases to be able to easily
distinguish failed test runs.
@coveralls
Copy link

coveralls commented Nov 6, 2017

Coverage Status

Coverage decreased (-0.1%) to 59.24% when pulling 28b53ae on next_release into b69b217 on master.

2 similar comments
@coveralls
Copy link

coveralls commented Nov 6, 2017

Coverage Status

Coverage decreased (-0.1%) to 59.24% when pulling 28b53ae on next_release into b69b217 on master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.1%) to 59.24% when pulling 28b53ae on next_release into b69b217 on master.

this commit removes the custom calculation of the progress bar
for the home page dashboard and reuses the stats method and template
that we made earlier.

Also removes a leftover RawSQL query.
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.1%) to 59.251% when pulling f4e9a30 on next_release into b69b217 on master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.1%) to 59.251% when pulling f4e9a30 on next_release into b69b217 on master.

@coveralls
Copy link

coveralls commented Nov 6, 2017

Pull Request Test Coverage Report for Build 346

  • 32 of 50 (64.0%) changed or added relevant lines in 5 files are covered.
  • 90 unchanged lines in 2 files lost coverage.
  • Overall coverage decreased (-0.1%) to 59.24%

Changes Missing Coverage Covered Lines Changed/Added Lines %
tcms/search/forms.py 4 5 80.0%
tcms/testruns/models.py 22 39 56.41%
Files with Coverage Reduction New Missed Lines %
tcms/search/init.py 1 17.89%
tcms/testruns/models.py 89 65.18%
Totals Coverage Status
Change from base Build 344: -0.1%
Covered Lines: 7152
Relevant Lines: 11263

💛 - Coveralls

@atodorov atodorov merged commit be47ea8 into master Nov 6, 2017
@atodorov atodorov deleted the next_release branch November 6, 2017 20:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants