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

feat(scan): provide a default scan factory #76

Merged
merged 5 commits into from
Dec 8, 2022

Conversation

ostridm
Copy link
Contributor

@ostridm ostridm commented Nov 29, 2022

closes #68

@ostridm ostridm added the Type: enhancement New feature or request. label Nov 29, 2022
@ostridm ostridm self-assigned this Nov 29, 2022
@codeclimate
Copy link

codeclimate bot commented Nov 29, 2022

Code Climate has analyzed commit 504062c and detected 1 issue on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 1

The test coverage on the diff in this pull request is 99.1% (50% is the threshold).

This pull request will bring the total coverage in the repository to 96.1% (0.2% change).

View more on Code Climate.

@ostridm ostridm force-pushed the feat_#68/provide-a-default-scan-factory branch 5 times, most recently from 6c21728 to 1fc57ec Compare December 2, 2022 09:44
@ostridm ostridm marked this pull request as ready for review December 2, 2022 09:45
@ostridm ostridm force-pushed the feat_#68/provide-a-default-scan-factory branch from 1fc57ec to 7ebc870 Compare December 2, 2022 13:00
Copy link
Member

@derevnjuk derevnjuk left a comment

Choose a reason for hiding this comment

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

BTW comment in Issue and comment in Har are not the same thing

src/SecTester.Scan/Target/Har/Response.cs Outdated Show resolved Hide resolved
src/SecTester.Scan/Target/Har/Entry.cs Outdated Show resolved Hide resolved
src/SecTester.Scan/Target/Har/Log.cs Outdated Show resolved Hide resolved
src/SecTester.Scan/Target/Har/Timings.cs Outdated Show resolved Hide resolved
src/SecTester.Scan/Target/Har/Timings.cs Outdated Show resolved Hide resolved
src/SecTester.Scan/Scan.cs Show resolved Hide resolved
src/SecTester.Scan/Target/Har/Response.cs Outdated Show resolved Hide resolved
src/SecTester.Scan/DefaultScanFactory.cs Outdated Show resolved Hide resolved
src/SecTester.Scan/DefaultScanFactory.cs Outdated Show resolved Hide resolved
src/SecTester.Scan/DefaultScanFactory.cs Outdated Show resolved Hide resolved
@ostridm ostridm force-pushed the feat_#68/provide-a-default-scan-factory branch from 7ebc870 to 155fba6 Compare December 5, 2022 08:17
@ostridm ostridm force-pushed the feat_#68/provide-a-default-scan-factory branch from 155fba6 to a273064 Compare December 5, 2022 08:28
@ostridm ostridm force-pushed the feat_#68/provide-a-default-scan-factory branch from 1f0d337 to fc3bce7 Compare December 5, 2022 11:49
Copy link
Member

@derevnjuk derevnjuk left a comment

Choose a reason for hiding this comment

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

Please describe how to use the project in README

test/SecTester.Scan.Tests/Mocks/MockLogger.cs Outdated Show resolved Hide resolved
@ostridm ostridm marked this pull request as draft December 6, 2022 08:16
@ostridm ostridm force-pushed the feat_#68/provide-a-default-scan-factory branch from fc3bce7 to a451e98 Compare December 6, 2022 09:58
@ostridm ostridm marked this pull request as ready for review December 6, 2022 10:09
test/SecTester.Scan.Tests/Commands/UploadHarTests.cs Outdated Show resolved Hide resolved
test/SecTester.Core.Tests/Utils/StringUtilsTests.cs Outdated Show resolved Hide resolved
test/SecTester.Core.Tests/Utils/StringUtilsTests.cs Outdated Show resolved Hide resolved
src/SecTester.Core/Utils/StringUtils.cs Outdated Show resolved Hide resolved
test/SecTester.Core.Tests/Utils/StringUtilsTests.cs Outdated Show resolved Hide resolved
src/SecTester.Scan/ScanSettings.cs Outdated Show resolved Hide resolved
src/SecTester.Scan/ScanSettings.cs Outdated Show resolved Hide resolved
src/SecTester.Scan/DefaultScanFactory.cs Outdated Show resolved Hide resolved
src/SecTester.Scan/DefaultScanFactory.cs Outdated Show resolved Hide resolved
src/SecTester.Scan/DefaultScanFactory.cs Outdated Show resolved Hide resolved
@ostridm ostridm force-pushed the feat_#68/provide-a-default-scan-factory branch from cc1ba55 to 70fee25 Compare December 7, 2022 14:40
@derevnjuk derevnjuk force-pushed the feat_#68/provide-a-default-scan-factory branch from 0d5d563 to bb1cc16 Compare December 8, 2022 17:45
@derevnjuk derevnjuk force-pushed the feat_#68/provide-a-default-scan-factory branch from bb1cc16 to 504062c Compare December 8, 2022 17:57
@derevnjuk derevnjuk merged commit 2c7ef66 into master Dec 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: enhancement New feature or request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Provide a default scan factory
2 participants