-
Notifications
You must be signed in to change notification settings - Fork 371
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
fix plist parsing for skip files #2195
Conversation
8980c42
to
4d3483a
Compare
codechecker_common/plist_parser.py
Outdated
@@ -97,7 +97,7 @@ def parse(self, fileobj): | |||
return self.root | |||
|
|||
|
|||
def read_plist(path): | |||
def load_plist(plist_file): |
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.
@gyorb The argument of this function is not a plist file. It is the content of a plist file.
Also this function will not load the given plist file, it will only parse it.
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.
Actually it is a file object and not the file content, I've made some renaming to make it more explicit.
0c8d4fa
to
7fe4a78
Compare
The introduced new plist xml parsing method with lxml was not introduced for the use cases where a skip file was used. In case of a skip file the report plist files can be rewritten (the reports based on the skip file content) are removed. Previous tests did not fail because a second skip is done at the server side during storage and the server skip tests worked on the stored reports.
The intoduced new plist xml parsing method with lxml
was not introduced for the use cases where the a skip file
was used. In case of a skip file the report plist files
can be rewritten (the reports based on the skip file content)
are removed.
Previous tests did not fail because a second skip is done
at the server side during storage and the server skip tests
worked on the stored reports.