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

Conversation

@dorpvom
Copy link
Member

@dorpvom dorpvom commented Nov 21, 2018

Fixing mismatched naming of firmware version, where in database and object version was used and upload used firmware_version.

Side effect: Removed unused function convert_fw_obj_to_analysis_task from task conversion helper.

Name was already changed in wiki.

@dorpvom dorpvom requested a review from weidenba November 21, 2018 12:45
@dorpvom
Copy link
Member Author

dorpvom commented Nov 21, 2018

Should close #164

Copy link
Contributor

@weidenba weidenba left a comment

Choose a reason for hiding this comment

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

looks good to me, can be merged if tests pass

@codecov-io
Copy link

codecov-io commented Nov 21, 2018

Codecov Report

Merging #183 into master will decrease coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #183      +/-   ##
==========================================
- Coverage   95.38%   95.37%   -0.02%     
==========================================
  Files         326      326              
  Lines       16023    15994      -29     
==========================================
- Hits        15284    15254      -30     
- Misses        739      740       +1
Impacted Files Coverage Δ
src/test/acceptance/test_compare_firmwares.py 100% <ø> (ø) ⬆️
.../acceptance/test_upload_analyze_delete_firmware.py 99% <ø> (ø) ⬆️
...unit/helperFunctions/test_mongo_task_conversion.py 100% <ø> (ø) ⬆️
src/test/common_helper.py 95.19% <ø> (ø) ⬆️
...test/unit/web_interface/rest/test_rest_firmware.py 100% <ø> (ø) ⬆️
src/test/acceptance/rest/test_rest_compare.py 100% <ø> (ø) ⬆️
src/test/unit/web_interface/test_app_re_analyze.py 100% <ø> (ø) ⬆️
src/helperFunctions/mongo_task_conversion.py 87.75% <100%> (-1.34%) ⬇️
src/test/acceptance/test_misc.py 100% <100%> (ø) ⬆️
...tegration/web_interface/rest/test_rest_firmware.py 94.04% <100%> (ø) ⬆️
... and 3 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 2fd5676...cb9a106. Read the comment docs.

@weidenba weidenba merged commit 695b452 into master Nov 21, 2018
@weidenba weidenba deleted the align-version-naming branch November 21, 2018 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants