hohn/codeql
1
0
Fork 0
mirror of https://github.com/github/codeql.git synced 2025-12-16 16:53:25 +01:00
Code Issues Packages Projects Releases Wiki Activity

Ruby: improve wording of error messages

Browse Source
This commit is contained in:
Arthur Baars
2023-02-16 15:53:56 +01:00
parent 51f34eb3e9
commit 6fd836d3a9
3 changed files with 14 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
ruby/extractor/src/extractor.rs
View File

@@ -306,15 +306,20 @@ impl<'a> Visitor<'a> {
fn enter_node(&mut self, node: Node) -> bool {
if node.is_missing() {
self.record_parse_error_for_node(
"parse error: expecting {}",
&[node.kind()],
"A parse error occurred, expecting {} symbol. Check the syntax of the file using the {} command. If the file is indeed invalid, please correct the error or exclude the file from analysis.",
&[node.kind(), "ruby -c"],
node,
true,
);
return false;
}
if node.is_error() {
self.record_parse_error_for_node("parse error", &[], node, true);
self.record_parse_error_for_node(
"A parse error occurred. Check the syntax of the file using the {} command. If the file is indeed invalid, please correct the error or exclude the file from analysis.",
&["ruby -c"],
node,
true,
);
return false;
};

6
ruby/extractor/src/main.rs
View File

@@ -203,8 +203,8 @@ fn main() -> std::io::Result<()> {
)
.file(&path.to_string_lossy())
.message(
"could not decode the file contents as {}: {}",
&[&encoding_name, &msg],
"Could not decode the file contents as {}: {}. Validate that the contents of the file matches the character encoding specified in the {} directive at the top of the file.",
&[&encoding_name, &msg, "encoding:"],
)
.status_page()
.severity(diagnostics::Severity::Warning),
@@ -218,7 +218,7 @@ fn main() -> std::io::Result<()> {
.new_entry("unknown-character-encoding", "Unknown character encoding")
.file(&path.to_string_lossy())
.message(
"unknown character encoding {} in {} directive.",
"Unknown character encoding {} in {} directive. Validate that the specified name is a supported character set.",
&[&encoding_name, "#encoding:"],
)
.status_page()

6
ruby/ql/test/query-tests/diagnostics/ExtractionErrors.expected
View File

@@ -1,3 +1,3 @@
| src/not_ruby.rb:5:25:5:26 | parse error | Extraction failed in src/not_ruby.rb with error parse error | 2 |
| src/unsupported_feature.rb:2:1:2:4 | parse error | Extraction failed in src/unsupported_feature.rb with error parse error | 2 |
| src/unsupported_feature.rb:3:1:3:8 | parse error | Extraction failed in src/unsupported_feature.rb with error parse error | 2 |
| src/not_ruby.rb:5:25:5:26 | A parse error occured. Check the syntax of the file using the ruby -c command. If the file is invalid, please correct the error or exclude the file from analysis. | Extraction failed in src/not_ruby.rb with error A parse error occured. Check the syntax of the file using the ruby -c command. If the file is invalid, please correct the error or exclude the file from analysis. | 2 |
| src/unsupported_feature.rb:2:1:2:4 | A parse error occured. Check the syntax of the file using the ruby -c command. If the file is invalid, please correct the error or exclude the file from analysis. | Extraction failed in src/unsupported_feature.rb with error A parse error occured. Check the syntax of the file using the ruby -c command. If the file is invalid, please correct the error or exclude the file from analysis. | 2 |
| src/unsupported_feature.rb:3:1:3:8 | A parse error occured. Check the syntax of the file using the ruby -c command. If the file is invalid, please correct the error or exclude the file from analysis. | Extraction failed in src/unsupported_feature.rb with error A parse error occured. Check the syntax of the file using the ruby -c command. If the file is invalid, please correct the error or exclude the file from analysis. | 2 |