Add comments to test cases

stackernews · Feb 14, 2025 · 2222c8f · 2222c8f
1 parent cb8cb12
commit 2222c8f
Showing 1 changed file with 8 additions and 4 deletions.
diff --git a/lib/url.spec.js b/lib/url.spec.js
@@ -31,15 +31,19 @@ describe('internal links', () => {
 })
 
 const misleadingLinkCases = [
+  // if text is the same as the link, it's not misleading
   ['https://stacker.news/items/1234', 'https://stacker.news/items/1234', false],
+  // same origin is not misleading
+  ['https://stacker.news/items/1235', 'https://stacker.news/items/1234', false],
+  ['www.google.com', 'https://www.google.com', false],
+  ['stacker.news', 'https://stacker.news', false],
+  // if text is obviously not a link, it's not misleading
   ['innocent text', 'https://stacker.news/items/1234', false],
   ['innocenttext', 'https://stacker.news/items/1234', false],
+  // if text might be a link to a different origin, it's misleading
   ['innocent.text', 'https://stacker.news/items/1234', true],
-  ['https://stacker.news/items/1235', 'https://stacker.news/items/1234', false],
   ['https://google.com', 'https://bing.com', true],
-  ['www.google.com', 'https://bing.com', true],
-  ['www.google.com', 'https://www.google.com', false],
-  ['stacker.news', 'https://stacker.news', false]
+  ['www.google.com', 'https://bing.com', true]
 ]
 
 describe('misleading links', () => {