Skip to content
This repository has been archived by the owner on Dec 18, 2018. It is now read-only.

Commit

Permalink
Ensure manifest tool flags manual HTTPS tests as manual (#123)
Browse files Browse the repository at this point in the history
Manifest tool missed the `manual` in `mytest-manual.https.html` and thus
incorreclty listed such tests in the automated section of the manifest file.
  • Loading branch information
tidoust authored and Ms2ger committed Oct 5, 2016
1 parent 5b4013f commit cb4fb70
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 1 addition & 1 deletion manifest/sourcefile.py
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ def __init__(self, tests_root, rel_path, url_base, use_committed=False,

self.type_flag = None
if "-" in self.name:
self.type_flag = self.name.rsplit("-", 1)[1]
self.type_flag = self.name.rsplit("-", 1)[1].split(".")[0]

self.meta_flags = self.name.split(".")[1:]

Expand Down
2 changes: 2 additions & 0 deletions manifest/tests/test_sourcefile.py
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,8 @@ def test_name_is_manual():
manual_tests = [
"html/test-manual.html",
"html/test-manual.xhtml",
"html/test-manual.https.html",
"html/test-manual.https.xhtml"
]

for rel_path in manual_tests:
Expand Down

0 comments on commit cb4fb70

Please sign in to comment.