test(linter): add more test cases for disable directives (#6047)

- related to #6041 - follow-up to #6045 While working on the original PR, I noticed that we didn't have any pathological test cases yet for disable directives. I've added some more complex test cases here to ensure we don't regress the disable/enable behavior in the future.
oxc-project · Sep 24, 2024 · 58d333a · 58d333a
1 parent 6f76ebe
commit 58d333a
Showing 1 changed file with 140 additions and 0 deletions.
diff --git a/crates/oxc_linter/src/disable_directives.rs b/crates/oxc_linter/src/disable_directives.rs
@@ -357,6 +357,84 @@ fn test() {
                 debugger;
             "
             ),
+            // Handles no spaces in comment
+            format!(
+                "debugger; //{prefix}-disable-line
+            debugger; //{prefix}-disable-line
+
+            //{prefix}-disable-next-line
+            debugger;
+
+            /*{prefix}-disable-next-line*/
+            debugger;
+
+            debugger; /*{prefix}-disable-line*/
+            
+            debugger; //{prefix}-disable-line no-debugger
+
+            //{prefix}-disable-next-line no-debugger
+            debugger;
+
+            debugger; /*{prefix}-disable-line no-debugger*/
+
+            /*{prefix}-disable-next-line no-debugger*/
+            debugger;
+        "
+            ),
+            // Handles extra spaces in comment
+            format!(
+                "debugger; //       {prefix}-disable-line
+            debugger; // \t\t {prefix}-disable-line
+
+            //         {prefix}-disable-next-line
+            debugger;
+
+            /*      {prefix}-disable-next-line        */
+            debugger;
+
+            debugger; /*    {prefix}-disable-line       */
+            
+            debugger; //            {prefix}-disable-line no-debugger
+
+            //          {prefix}-disable-next-line no-debugger
+            debugger;
+
+            debugger; /*     \t   {prefix}-disable-line no-debugger*/
+
+            /*    \t   {prefix}-disable-next-line no-debugger       */
+            debugger;
+        "
+            ),
+            // Extra commas
+            format!(
+                "
+            debugger // {prefix}-disable-line no-debugger,
+            debugger // {prefix}-disable-line ,no-debugger
+            debugger // {prefix}-disable-line no-debugger,,
+            debugger // {prefix}-disable-line ,,no-debugger,,
+            debugger // {prefix}-disable-line ,,no-debugger,,semi,,
+            debugger // {prefix}-disable-line ,,no-debugger,,no-debugger,,
+            debugger // {prefix}-disable-line ,  , ,,no-debugger, , ,
+
+            // {prefix}-disable-next-line no-debugger,
+            debugger
+            // {prefix}-disable-next-line ,no-debugger,
+            debugger
+            // {prefix}-disable-next-line no-debugger,,
+            debugger
+            // {prefix}-disable-next-line ,,no-debugger,,
+            debugger
+            // {prefix}-disable-next-line ,,no-debugger,,semi,,
+            debugger
+            // {prefix}-disable-next-line ,,no-debugger,,no-debugger,,
+            debugger
+            // {prefix}-disable-next-line ,  , ,,no-debugger, , ,
+        "
+            ),
+            format!("
+                /* {prefix}-disable , ,no-debugger, , */
+                debugger;
+            ")
         ];
 
         let fail = vec![
@@ -470,6 +548,68 @@ fn test() {
                 debugger;
             "
             ),
+            // Handles no spaces in comment
+            format!(
+                "
+            /*{prefix}-disable*/
+                debugger;
+            /*{prefix}-enable*/
+                debugger;
+            "
+            ),
+            format!(
+                "
+            /*{prefix}-disable no-debugger,no-console*/
+                debugger;
+            /*{prefix}-enable no-debugger,no-console*/
+                debugger;
+            "
+            ),
+            format!(
+                "debugger; //{prefix}-disable-line no-alert,quotes,semi
+            //{prefix}-disable-next-line no-alert,quotes,semi
+            debugger;
+            debugger; /*{prefix}-disable-line no-alert,quotes,semi */
+            /*{prefix}-disable-next-line no-alert,quotes,semi */
+            debugger;
+            /*{prefix}-disable-next-line
+no-alert,
+quotes,
+semi*/
+            debugger;
+        "
+            ),
+            // Handles extra spaces in comment
+            format!(
+                "
+            /*   \t\t {prefix}-disable   \t\t*/
+                debugger;
+            /*   \t\t {prefix}-enable   \t\t*/
+                debugger;
+            "
+            ),
+            format!(
+                "
+            /*   \t\t {prefix}-disable    \t\t no-debugger,   \t\t no-console   \t\t */
+                debugger;
+            /*   \t\t {prefix}-enable    \t\t no-debugger,   \t\t no-console   \t\t */
+                debugger;
+            "
+            ),
+            format!(
+                "debugger; //   \t\t {prefix}-disable-line   \t\t  no-alert,   \t\t quotes,   \t\t semi   \t\t 
+            //   \t\t {prefix}-disable-next-line   \t\t  no-alert,   \t\t quotes,   \t\t semi
+            debugger;
+            debugger; /*   \t\t {prefix}-disable-line    \t\t no-alert,   \t\t quotes,   \t\t semi   \t\t  */
+            /*   \t\t {prefix}-disable-next-line   \t\t  no-alert,   \t\t quotes,   \t\t semi */
+            debugger;
+            /*  \t\t {prefix}-disable-next-line
+  \t\t no-alert,  \t\t 
+  \t\t quotes,  \t\t 
+  \t\t semi  \t\t */
+            debugger;
+        "
+            ),
         ];
 
         Tester::new("no-debugger", pass, fail).test();