-
Notifications
You must be signed in to change notification settings - Fork 2
issue-64: Changes max_length for LogEntry.path #65
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #65 +/- ##
========================================
Coverage 98.33% 98.33%
========================================
Files 5 5
Lines 120 120
Branches 4 4
========================================
Hits 118 118
Misses 2 2
Continue to review full report at Codecov.
|
http_stubs/tests/tests.py
Outdated
client.post('/default_path/', 'test', content_type=content_type) | ||
http_body = http_stub_factory( | ||
method=HTTPMethod.POST.name, | ||
path='/regex/.*', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: .*
part in the regexp is unnecessary: regexp /regex/
will match same set of inputs as /regex/.*
but will do it more efficiently as no time would be wasted for searching for the throwaway .*
part
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're right) Thank you. Fixed
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
closes #64