8279: Fix the not operator use and test case fix.

Author: chetankhilosiya

crates/ide/src/syntax_highlighting/highlight.rs

@@ -169,16 +169,13 @@ pub(super) fn element(
                 HlTag::Operator(HlOperator::Other).into()
             }
             T![!] if element.parent().and_then(ast::MacroCall::cast).is_some() => {
-                eprintln!("in macro call: {}", element);
                 HlTag::Symbol(SymbolKind::Macro).into()
             }
             T![!] if element.parent().and_then(ast::NeverType::cast).is_some() => {
-                eprintln!("in never type : {}", element);
                 HlTag::BuiltinType.into()
             }
             T![!] if element.parent().and_then(ast::PrefixExpr::cast).is_some() => {
-                eprintln!("pre expr for : {}", element);
-                HlTag::Operator(HlOperator::Bitwise).into()
+                HlTag::Operator(HlOperator::Logical).into()
             }
             T![*] if element.parent().and_then(ast::PtrType::cast).is_some() => {
                 HlTag::Keyword.into()
@@ -197,7 +194,6 @@ pub(super) fn element(
                 }
             }
             T![-] if element.parent().and_then(ast::PrefixExpr::cast).is_some() => {
-                eprintln!("the - operator: {}", element);
                 let prefix_expr = element.parent().and_then(ast::PrefixExpr::cast)?;
 
                 let expr = prefix_expr.expr()?;
@@ -208,7 +204,6 @@ pub(super) fn element(
                 .into()
             }
             _ if element.parent().and_then(ast::PrefixExpr::cast).is_some() => {
-                eprintln!("the prefix expr block: {}", element);
                 HlTag::Operator(HlOperator::Other).into()
             }
             T![+] | T![-] | T![*] | T![/] | T![+=] | T![-=] | T![*=] | T![/=]
@@ -230,11 +225,9 @@ pub(super) fn element(
                 HlTag::Operator(HlOperator::Comparision).into()
             }
             _ if element.parent().and_then(ast::BinExpr::cast).is_some() => {
-                eprintln!("the bin expr : {}", element);
                 HlTag::Operator(HlOperator::Other).into()
             }
             _ if element.parent().and_then(ast::RangeExpr::cast).is_some() => {
-                eprintln!("the range expr block: {}", element);
                 HlTag::Operator(HlOperator::Other).into()
             }
             _ if element.parent().and_then(ast::RangePat::cast).is_some() => {

crates/ide/src/syntax_highlighting/test_data/highlight_doctest.html

@@ -76,7 +76,7 @@
     <span class="comment documentation">/// </span><span class="comment injected">// calls bar on foo</span>
     <span class="comment documentation">/// </span><span class="macro injected">assert!</span><span class="parenthesis injected">(</span><span class="none injected">foo</span><span class="operator injected">.</span><span class="none injected">bar</span><span class="parenthesis injected">(</span><span class="parenthesis injected">)</span><span class="parenthesis injected">)</span><span class="semicolon injected">;</span>
     <span class="comment documentation">///</span>
-    <span class="comment documentation">/// </span><span class="keyword injected">let</span><span class="none injected"> </span><span class="variable declaration injected">bar</span><span class="none injected"> </span><span class="operator injected">=</span><span class="none injected"> </span><span class="variable injected">foo</span><span class="operator injected">.</span><span class="field injected">bar</span><span class="none injected"> </span><span class="operator injected">||</span><span class="none injected"> </span><span class="struct injected">Foo</span><span class="operator injected">::</span><span class="constant injected">bar</span><span class="semicolon injected">;</span>
+    <span class="comment documentation">/// </span><span class="keyword injected">let</span><span class="none injected"> </span><span class="variable declaration injected">bar</span><span class="none injected"> </span><span class="operator injected">=</span><span class="none injected"> </span><span class="variable injected">foo</span><span class="operator injected">.</span><span class="field injected">bar</span><span class="none injected"> </span><span class="logical injected">||</span><span class="none injected"> </span><span class="struct injected">Foo</span><span class="operator injected">::</span><span class="constant injected">bar</span><span class="semicolon injected">;</span>
     <span class="comment documentation">///</span>
     <span class="comment documentation">/// </span><span class="comment injected">/* multi-line</span>
     <span class="comment documentation">/// </span><span class="comment injected">       comment */</span>

crates/ide/src/syntax_highlighting/test_data/highlighting.html

@@ -213,7 +213,7 @@
     <span class="keyword">let</span> <span class="variable declaration">baz</span> <span class="operator">=</span> <span class="numeric_literal">-</span><span class="numeric_literal">42</span><span class="semicolon">;</span>
     <span class="keyword">let</span> <span class="variable declaration">baz</span> <span class="operator">=</span> <span class="operator">-</span><span class="variable">baz</span><span class="semicolon">;</span>
 
-    <span class="keyword">let</span> <span class="punctuation">_</span> <span class="operator">=</span> <span class="operator">!</span><span class="bool_literal">true</span><span class="semicolon">;</span>
+    <span class="keyword">let</span> <span class="punctuation">_</span> <span class="operator">=</span> <span class="logical">!</span><span class="bool_literal">true</span><span class="semicolon">;</span>
 
     <span class="label declaration">'foo</span><span class="colon">:</span> <span class="keyword control">loop</span> <span class="brace">{</span>
         <span class="keyword control">break</span> <span class="label">'foo</span><span class="semicolon">;</span>