================ @@ -276,6 +286,62 @@ // HTML-CIRCLE: <div>return</div> // HTML-CIRCLE: <p> double The perimeter of the circle.</p> +// HTML-SHAPE-PREFIX: <h1>class Shape</h1> +// HTML-SHAPE-PREFIX-NEXT: <p> +// HTML-SHAPE-PREFIX-NEXT: Defined at line +// HTML-SHAPE-PREFIX-NEXT: <a href="https://repository.com/./include/Shape.h#L8">8</a> +// HTML-SHAPE-PREFIX-NEXT: of file +// HTML-SHAPE-PREFIX-NEXT: <a href="https://repository.com/./include/Shape.h">Shape.h</a> +// HTML-SHAPE-PREFIX-NEXT: </p> +// HTML-SHAPE-PREFIX: <h3 id="{{([0-9A-F]{40})}}">~Shape</h3> +// HTML-SHAPE-PREFIX: <p>public void ~Shape()</p> +// HTML-SHAPE-PREFIX: Defined at line +// HTML-SHAPE-PREFIX-NEXT: <a href="https://repository.com/./include/Shape.h#L13">13</a> +// HTML-SHAPE-PREFIX-NEXT: of file +// HTML-SHAPE-PREFIX-NEXT: <a href="https://repository.com/./include/Shape.h">Shape.h</a> ---------------- ilovepi wrote:
Isn't this the exact same checks as done for HTML-SHAPE, except that the URL for the line is different? In those cases, we usually have a common set of checks that can serve for both, and then a separate check for the lines that differ. This is how we handled it for the REPOSITORY/NO-REPOSITORY checks. If you look above, we use HTML-CIRCLE to test the common logic, and then only use CIRCLE-NO-REPOSITORY and CIRCLE-REPOSITORY prefixes when the behavior diverges. You should follow that pattern instead of duplicating the entire test. Since we already independently check the behavior w/ repository, you could just use the HTML-CIRCLE + HTML-CIRCLE-PREFIX and ignore matching additional lines beyond the lines w/ actual prefixes. https://llvm.org/docs/TestingGuide.html has a lot of info on our testing practices. https://maskray.me/blog/2021-08-08-toolchain-testing is a great post about testing toolchains. https://github.com/llvm/llvm-project/pull/131280 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits