这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@jstucke
Copy link
Collaborator

@jstucke jstucke commented Dec 1, 2017

No description provided.

@jstucke jstucke requested review from dorpvom and weidenba December 1, 2017 11:16
self.assertEqual(self.db_frontend_interface.get_number_of_total_matches(query, only_parent_firmwares=True), 1)

def test_get_other_versions_of_firmware(self):
parent_fw1 = create_test_firmware(version="1")
Copy link
Contributor

Choose a reason for hiding this comment

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

Coding convention violated!
" -> '



def create_test_firmware(device_class="Router", device_name="test_router", vendor="test_vendor", bin_path="container/test.zip", all_files_included_set=False):
def create_test_firmware(device_class="Router", device_name="test_router", vendor="test_vendor", bin_path="container/test.zip",
Copy link
Contributor

Choose a reason for hiding this comment

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

Coding convention violation
" -> '

@codecov-io
Copy link

Codecov Report

Merging #35 into master will increase coverage by <.01%.
The diff coverage is 96%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #35      +/-   ##
==========================================
+ Coverage      90%   90.01%   +<.01%     
==========================================
  Files         247      247              
  Lines       11569    11517      -52     
==========================================
- Hits        10413    10367      -46     
+ Misses       1156     1150       -6
Impacted Files Coverage Δ
src/helperFunctions/fileSystem.py 91.78% <100%> (+0.23%) ⬆️
src/web_interface/components/analysis_routes.py 94.94% <100%> (+0.45%) ⬆️
.../integration/storage/test_db_interface_frontend.py 100% <100%> (ø) ⬆️
src/test/unit/helperFunctions/test_file_system.py 100% <100%> (ø) ⬆️
src/storage/db_interface_frontend.py 85.49% <85.71%> (-0.15%) ⬇️
src/test/common_helper.py 90.57% <89.65%> (ø) ⬆️
src/scheduler/Analysis.py 85.38% <0%> (-0.75%) ⬇️
src/intercom/front_end_binding.py 88.67% <0%> (-0.42%) ⬇️
src/web_interface/rest/rest_firmware.py 96.25% <0%> (-0.35%) ⬇️
src/intercom/back_end_binding.py 94.73% <0%> (-0.23%) ⬇️
... and 8 more

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 e9ffbd4...1054647. Read the comment docs.

@weidenba weidenba merged commit 09516dc into master Dec 4, 2017
@weidenba weidenba deleted the 18-Version-Relation branch December 18, 2017 14:21
weidenba added a commit that referenced this pull request Oct 22, 2018
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.

4 participants