llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang Author: Robin Caloudis (robincaloudis) <details> <summary>Changes</summary> ## Why In https://github.com/llvm/llvm-project/pull/113612/files#diff-ada12e18f3e902b41b6989b46455c4e32656276e59907026e2464cf57d10d583, the parameter `qual_name` was introduced. However, the tests have not been adopted accordingly and hence cannot be executed. ## What Fix the execution of tests by providing the missing argument. --- Full diff: https://github.com/llvm/llvm-project/pull/139463.diff 1 Files Affected: - (modified) clang/tools/include-mapping/test.py (+9-5) ``````````diff diff --git a/clang/tools/include-mapping/test.py b/clang/tools/include-mapping/test.py index eef328381f2bb..81803855dac8f 100755 --- a/clang/tools/include-mapping/test.py +++ b/clang/tools/include-mapping/test.py @@ -52,7 +52,7 @@ def testParseSymbolPage_SingleHeader(self): </tr> </tbody></table> """ - self.assertEqual(_ParseSymbolPage(html, "foo"), set(["<cmath>"])) + self.assertEqual(_ParseSymbolPage(html, "foo", "foo"), set(["<cmath>"])) def testParseSymbolPage_MulHeaders(self): # Defined in header <cstddef> @@ -92,7 +92,9 @@ def testParseSymbolPage_MulHeaders(self): </tr> </tbody></table> """ - self.assertEqual(_ParseSymbolPage(html, "foo"), set(["<cstdio>", "<cstdlib>"])) + self.assertEqual( + _ParseSymbolPage(html, "foo", "foo"), set(["<cstdio>", "<cstdlib>"]) + ) def testParseSymbolPage_MulHeadersInSameDiv(self): # Multile <code> blocks in a Div. @@ -118,7 +120,7 @@ def testParseSymbolPage_MulHeadersInSameDiv(self): </tbody></table> """ self.assertEqual( - _ParseSymbolPage(html, "foo"), set(["<algorithm>", "<utility>"]) + _ParseSymbolPage(html, "foo", "foo"), set(["<algorithm>", "<utility>"]) ) def testParseSymbolPage_MulSymbolsInSameTd(self): @@ -142,8 +144,10 @@ def testParseSymbolPage_MulSymbolsInSameTd(self): </tr> </tbody></table> """ - self.assertEqual(_ParseSymbolPage(html, "int8_t"), set(["<cstdint>"])) - self.assertEqual(_ParseSymbolPage(html, "int16_t"), set(["<cstdint>"])) + self.assertEqual(_ParseSymbolPage(html, "int8_t", "int8_t"), set(["<cstdint>"])) + self.assertEqual( + _ParseSymbolPage(html, "int16_t", "int16_t"), set(["<cstdint>"]) + ) if __name__ == "__main__": `````````` </details> https://github.com/llvm/llvm-project/pull/139463 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
