Set S5918 scope to ALL (#1174)

Author: guillaume-dequenne

python-checks/src/main/java/org/sonar/python/checks/tests/ImplicitlySkippedTestCheck.java

@@ -101,6 +101,11 @@ public void initialize(Context context) {
     });
   }
 
+  @Override
+  public CheckScope scope() {
+    return CheckScope.ALL;
+  }
+
   private static ReturnStatement getReturnStatementFromFirstIfStatement(FunctionDef functionDef) {
     // check first non assignment statement is an if with a return
     for (Statement statement : functionDef.body().statements()) {

python-checks/src/test/java/org/sonar/python/checks/tests/ImplicitlySkippedTestCheckTest.java

@@ -20,8 +20,11 @@
 package org.sonar.python.checks.tests;
 
 import org.junit.Test;
+import org.sonar.plugins.python.api.PythonCheck;
 import org.sonar.python.checks.utils.PythonCheckVerifier;
 
+import static org.assertj.core.api.Assertions.assertThat;
+
 public class ImplicitlySkippedTestCheckTest {
 
   @Test
@@ -38,4 +41,10 @@ public void testWithWrapper() {
   public void testAnotherLibrary() {
     PythonCheckVerifier.verify("src/test/resources/checks/tests/implicitlySkippedTestAnotherLibrary.py", new ImplicitlySkippedTestCheck());
   }
+
+  @Test
+  public void test_scope() {
+    assertThat(new ImplicitlySkippedTestCheck().scope()).isEqualTo(PythonCheck.CheckScope.ALL);
+  }
+
 }