Merge ~pieq/oem-qa-autosummary:fix-1930349-ignore-non-url-text into oem-qa-autosummary:master
Status: | Merged |
---|---|
Approved by: | Pierre Equoy |
Approved revision: | bd8c0385dd549d5f2ebbe92bc975c43e3b037347 |
Merged at revision: | 6327e7ff7aacdc70723836a4395b1d1de9508379 |
Proposed branch: | ~pieq/oem-qa-autosummary:fix-1930349-ignore-non-url-text |
Merge into: | oem-qa-autosummary:master |
Diff against target: |
285 lines (+100/-51) 5 files modified
autosummary/summary.py (+20/-11) logging_conf.py (+18/-18) tests/test_webapp.py (+30/-6) webapp/__init__.py (+27/-11) webapp/templates/index.html (+5/-5) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
StanleyHuang | Approve | ||
Review via email:
|
Description of the change
Now, when generating the summary:
- Full Test Reports and Stress Test Reports text fields from the HTML form are parsed to extract C3 submission URLs and nothing else. If user includes additional information, it will be discarded.
- HTML Report is now parsed to only return a URL and nothing else. If user has entered something like "blabla https:/
Tests:
- unit test (run $ python -m unittest discover -s tests -v)
- generated report using the carlsbad project with different content for the full, stress and HTML reports.
Note:
Please check commits individually, as one of them is a Black code formatting commit that is not related to this modification...
just a small suggestion about the regular expression pattern.