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

Conversation

@dorpvom
Copy link
Member

@dorpvom dorpvom commented Mar 12, 2018

New compare plugin providing a "binwalk -Ww"-ish view on the first 512 byte block of all compared files

@codecov-io
Copy link

codecov-io commented Mar 12, 2018

Codecov Report

Merging #80 into master will increase coverage by 0.02%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #80      +/-   ##
==========================================
+ Coverage   93.75%   93.78%   +0.02%     
==========================================
  Files         269      271       +2     
  Lines       12933    13043     +110     
==========================================
+ Hits        12125    12232     +107     
- Misses        808      811       +3
Impacted Files Coverage Δ
src/compare/compare.py 98.46% <100%> (-1.54%) ⬇️
...ugins/compare/file_header/test/test_file_header.py 100% <100%> (ø)
src/test/unit/scheduler/test_compare.py 89.23% <100%> (-5.86%) ⬇️
...rc/plugins/compare/file_header/code/file_header.py 100% <100%> (ø)
...erface/components/additional_functions/hex_dump.py 100% <100%> (+2.38%) ⬆️
...st/unit/web_interface/test_additional_functions.py 100% <100%> (ø) ⬆️
src/test/unit/compare/test_compare.py 98.24% <100%> (ø) ⬆️
... and 2 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 ed1e8a2...69b7905. Read the comment docs.

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. But it should be shown, which file you are looking at.

from flask import Markup

from compare.PluginBase import CompareBasePlugin
from storage.binary_service import BinaryService
Copy link
Contributor

Choose a reason for hiding this comment

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

unused import


from flask import Markup

from storage.binary_service import BinaryService
Copy link
Contributor

Choose a reason for hiding this comment

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

unused import

@@ -0,0 +1,29 @@
from unittest.mock import patch

Copy link
Contributor

Choose a reason for hiding this comment

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

unused import

@weidenba weidenba merged commit eda5bb5 into master Mar 15, 2018
@weidenba weidenba deleted the add-file-header-plugin branch April 25, 2018 09:38
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants