Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: includes/testcases.md

Issue 29755578: Issue 2148 - Rewrite the testpages test suite (Closed)
Patch Set: Addressed more feedback. Added README. Created June 7, 2018, 10:20 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: includes/testcases.md
===================================================================
--- a/includes/testcases.md
+++ b/includes/testcases.md
@@ -1,8 +1,9 @@
<section class="site-panel">
<h2>Testpages Filter Subscription</h2>
- <p>This subscription includes all of the filters required to verify the following test cases.</p><br>
+ <p>This subscription includes all of the filters required to verify the following test cases.</p>
+ <p>You can add this subscription and then click through to each test case page and check that the page state matches the test case description. Alternatively you could add a test cases filter(s) manually in order to check it in isolation.</p>
<a class="site-button" href="/abp-testcase-subscription.txt" target="_blank">View</a>
- <a class="site-button" href="abp:subscribe?location=http%3A%2F%2Ftestpages.adblockplus.org%2Fabp-testcase-subscription.txt&amp;title=ABP%20Testcase%20Subscription"">Subscribe</a>
+ <a id="subscribe-button" class="site-button" href="abp:subscribe?location=http%3A%2F%2Ftestpages.adblockplus.org%2Fabp-testcase-subscription.txt&amp;title=ABP%20Testcase%20Subscription">Subscribe</a>
</section>
<section class="site-panel">

Powered by Google App Engine
This is Rietveld