[PATCH] D51223: Update tests for new YAMLIO polymorphic traits
Scott Linder via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Oct 31 08:43:16 PDT 2018
scott.linder updated this revision to Diff 171936.
scott.linder added a comment.
Rebase
@klimek is there anyone I should add to take a look at this? As far as the YAML is concerned I believe this is a non-functional change.
https://reviews.llvm.org/D51223
Files:
unittests/Tooling/DiagnosticsYamlTest.cpp
unittests/Tooling/RefactoringActionRulesTest.cpp
Index: unittests/Tooling/RefactoringActionRulesTest.cpp
===================================================================
--- unittests/Tooling/RefactoringActionRulesTest.cpp
+++ unittests/Tooling/RefactoringActionRulesTest.cpp
@@ -117,8 +117,8 @@
"Key: 'input.cpp:30'\n"
"FilePath: input.cpp\n"
"Error: ''\n"
- "InsertedHeaders: \n"
- "RemovedHeaders: \n"
+ "InsertedHeaders: []\n"
+ "RemovedHeaders: []\n"
"Replacements: \n" // Extra whitespace here!
" - FilePath: input.cpp\n"
" Offset: 30\n"
Index: unittests/Tooling/DiagnosticsYamlTest.cpp
===================================================================
--- unittests/Tooling/DiagnosticsYamlTest.cpp
+++ unittests/Tooling/DiagnosticsYamlTest.cpp
@@ -82,7 +82,7 @@
" Message: 'message #3'\n"
" FileOffset: 72\n"
" FilePath: 'path/to/source2.cpp'\n"
- " Replacements: \n"
+ " Replacements: []\n"
"...\n",
YamlContentStream.str());
}
@@ -113,7 +113,7 @@
" Message: 'message #3'\n"
" FileOffset: 98\n"
" FilePath: path/to/source.cpp\n"
- " Replacements: \n"
+ " Replacements: []\n"
"...\n";
TranslationUnitDiagnostics TUDActual;
yaml::Input YAML(YamlContent);
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D51223.171936.patch
Type: text/x-patch
Size: 1689 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20181031/efdf39bf/attachment.bin>
More information about the cfe-commits
mailing list