TiagoMSSantos/MobileRT

View on GitHub
app/src/androidTest/java/puscas/mobilertapp/AbstractTest.java

Summary

Maintainability
C
1 day
Test Coverage

File AbstractTest.java has 369 lines of code (exceeds 250 allowed). Consider refactoring.
Open

package puscas.mobilertapp;

import android.Manifest;
import android.app.Activity;
import android.app.ActivityManager;
Severity: Minor
Found in app/src/androidTest/java/puscas/mobilertapp/AbstractTest.java - About 4 hrs to fix

    Method executeShellCommand has a Cognitive Complexity of 12 (exceeds 5 allowed). Consider refactoring.
    Open

        private static void executeShellCommand(final String shellCommand) {
            try {
                if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.LOLLIPOP) {
                    final ParcelFileDescriptor parcelFileDescriptor = InstrumentationRegistry.getInstrumentation().getUiAutomation().executeShellCommand(shellCommand);
                    parcelFileDescriptor.checkError();
    Severity: Minor
    Found in app/src/androidTest/java/puscas/mobilertapp/AbstractTest.java - About 1 hr to fix

    Cognitive Complexity

    Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

    A method's cognitive complexity is based on a few simple rules:

    • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
    • Code is considered more complex for each "break in the linear flow of the code"
    • Code is considered more complex when "flow breaking structures are nested"

    Further reading

    Method executeShellCommand has 33 lines of code (exceeds 25 allowed). Consider refactoring.
    Open

        private static void executeShellCommand(final String shellCommand) {
            try {
                if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.LOLLIPOP) {
                    final ParcelFileDescriptor parcelFileDescriptor = InstrumentationRegistry.getInstrumentation().getUiAutomation().executeShellCommand(shellCommand);
                    parcelFileDescriptor.checkError();
    Severity: Minor
    Found in app/src/androidTest/java/puscas/mobilertapp/AbstractTest.java - About 1 hr to fix

      Method setUp has 31 lines of code (exceeds 25 allowed). Consider refactoring.
      Open

          @Before
          @CallSuper
          public void setUp() {
              final String methodName = Thread.currentThread().getStackTrace()[2].getMethodName();
              logger.info(methodName + ": " + this.testName.getMethodName());
      Severity: Minor
      Found in app/src/androidTest/java/puscas/mobilertapp/AbstractTest.java - About 1 hr to fix

        Method setUpAll has 26 lines of code (exceeds 25 allowed). Consider refactoring.
        Open

            @BeforeClass
            @CallSuper
            public static void setUpAll() {
                final String methodName = Thread.currentThread().getStackTrace()[2].getMethodName();
                logger.info(methodName);
        Severity: Minor
        Found in app/src/androidTest/java/puscas/mobilertapp/AbstractTest.java - About 1 hr to fix

          Method grantPermissions has 26 lines of code (exceeds 25 allowed). Consider refactoring.
          Open

              private static void grantPermissions() {
                  InstrumentationRegistry.getInstrumentation().waitForIdleSync();
                  Espresso.onIdle();
                  logger.info("Granting permissions to the MainActivity to be able to read files from an external storage.");
                  if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.R) {
          Severity: Minor
          Found in app/src/androidTest/java/puscas/mobilertapp/AbstractTest.java - About 1 hr to fix

            Method assertRenderScene has 7 arguments (exceeds 4 allowed). Consider refactoring.
            Open

                protected void assertRenderScene(final Scene scene,
                                                 final Shader shader,
                                                 final Accelerator accelerator,
                                                 final int spp,
                                                 final int spl,
            Severity: Major
            Found in app/src/androidTest/java/puscas/mobilertapp/AbstractTest.java - About 50 mins to fix

              Method isActivityRunning has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.
              Open

                  @SuppressWarnings({"deprecation"})
                  private static boolean isActivityRunning(@NonNull final Activity activity) {
                      // Note that 'Activity#isDestroyed' only exists on Android API 17+.
                      // More info: https://developer.android.com/reference/android/app/Activity#isDestroyed()
                      if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.JELLY_BEAN_MR1) {
              Severity: Minor
              Found in app/src/androidTest/java/puscas/mobilertapp/AbstractTest.java - About 45 mins to fix

              Cognitive Complexity

              Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

              A method's cognitive complexity is based on a few simple rules:

              • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
              • Code is considered more complex for each "break in the linear flow of the code"
              • Code is considered more complex when "flow breaking structures are nested"

              Further reading

              Define and throw a dedicated exception instead of using a generic one.
              Open

                          throw new RuntimeException(ex);

              Using such generic exceptions as Error, RuntimeException, Throwable, and Exception prevents calling methods from handling true, system-generated exceptions differently than application-generated errors.

              Noncompliant Code Example

              public void foo(String bar) throws Throwable {  // Noncompliant
                throw new RuntimeException("My Message");     // Noncompliant
              }
              

              Compliant Solution

              public void foo(String bar) {
                throw new MyOwnRuntimeException("My Message");
              }
              

              Exceptions

              Generic exceptions in the signatures of overriding methods are ignored, because overriding method has to follow signature of the throw declaration in the superclass. The issue will be raised on superclass declaration of the method (or won't be raised at all if superclass is not part of the analysis).

              @Override
              public void myMethod() throws Exception {...}
              

              Generic exceptions are also ignored in the signatures of methods that make calls to methods that throw generic exceptions.

              public void myOtherMethod throws Exception {
                doTheThing();  // this method throws Exception
              }
              

              See

              Define and throw a dedicated exception instead of using a generic one.
              Open

                          throw new RuntimeException("Permission '" + permission + "' NOT granted to the app: " + context.getPackageName());

              Using such generic exceptions as Error, RuntimeException, Throwable, and Exception prevents calling methods from handling true, system-generated exceptions differently than application-generated errors.

              Noncompliant Code Example

              public void foo(String bar) throws Throwable {  // Noncompliant
                throw new RuntimeException("My Message");     // Noncompliant
              }
              

              Compliant Solution

              public void foo(String bar) {
                throw new MyOwnRuntimeException("My Message");
              }
              

              Exceptions

              Generic exceptions in the signatures of overriding methods are ignored, because overriding method has to follow signature of the throw declaration in the superclass. The issue will be raised on superclass declaration of the method (or won't be raised at all if superclass is not part of the analysis).

              @Override
              public void myMethod() throws Exception {...}
              

              Generic exceptions are also ignored in the signatures of methods that make calls to methods that throw generic exceptions.

              public void myOtherMethod throws Exception {
                doTheThing();  // this method throws Exception
              }
              

              See

              Define and throw a dedicated exception instead of using a generic one.
              Open

                          throw new RuntimeException(ex);

              Using such generic exceptions as Error, RuntimeException, Throwable, and Exception prevents calling methods from handling true, system-generated exceptions differently than application-generated errors.

              Noncompliant Code Example

              public void foo(String bar) throws Throwable {  // Noncompliant
                throw new RuntimeException("My Message");     // Noncompliant
              }
              

              Compliant Solution

              public void foo(String bar) {
                throw new MyOwnRuntimeException("My Message");
              }
              

              Exceptions

              Generic exceptions in the signatures of overriding methods are ignored, because overriding method has to follow signature of the throw declaration in the superclass. The issue will be raised on superclass declaration of the method (or won't be raised at all if superclass is not part of the analysis).

              @Override
              public void myMethod() throws Exception {...}
              

              Generic exceptions are also ignored in the signatures of methods that make calls to methods that throw generic exceptions.

              public void myOtherMethod throws Exception {
                doTheThing();  // this method throws Exception
              }
              

              See

              Make the enclosing method "static" or remove this set.
              Open

                              oneTestFailed = true;

              Correctly updating a static field from a non-static method is tricky to get right and could easily lead to bugs if there are multiple class instances and/or multiple threads in play. Ideally, static fields are only updated from synchronized static methods.

              This rule raises an issue each time a static field is updated from a non-static method.

              Noncompliant Code Example

              public class MyClass {
              
                private static int count = 0;
              
                public void doSomething() {
                  //...
                  count++;  // Noncompliant
                }
              }
              

              Remove this unused private "dismissANRSystemDialog" method.
              Open

                  private static void dismissANRSystemDialog() {

              private methods that are never executed are dead code: unnecessary, inoperative code that should be removed. Cleaning out dead code decreases the size of the maintained codebase, making it easier to understand the program and preventing bugs from being introduced.

              Note that this rule does not take reflection into account, which means that issues will be raised on private methods that are only accessed using the reflection API.

              Noncompliant Code Example

              public class Foo implements Serializable
              {
                private Foo(){}     //Compliant, private empty constructor intentionally used to prevent any direct instantiation of a class.
                public static void doSomething(){
                  Foo foo = new Foo();
                  ...
                }
                private void unusedPrivateMethod(){...}
                private void writeObject(ObjectOutputStream s){...}  //Compliant, relates to the java serialization mechanism
                private void readObject(ObjectInputStream in){...}  //Compliant, relates to the java serialization mechanism
              }
              

              Compliant Solution

              public class Foo implements Serializable
              {
                private Foo(){}     //Compliant, private empty constructor intentionally used to prevent any direct instantiation of a class.
                public static void doSomething(){
                  Foo foo = new Foo();
                  ...
                }
              
                private void writeObject(ObjectOutputStream s){...}  //Compliant, relates to the java serialization mechanism
              
                private void readObject(ObjectInputStream in){...}  //Compliant, relates to the java serialization mechanism
              }
              

              Exceptions

              This rule doesn't raise any issue on annotated methods.

              Use the built-in formatting to construct this argument.
              Open

                      logger.info(methodName + " finished");

              Passing message arguments that require further evaluation into a Guava com.google.common.base.Preconditions check can result in a performance penalty. That's because whether or not they're needed, each argument must be resolved before the method is actually called.

              Similarly, passing concatenated strings into a logging method can also incur a needless performance hit because the concatenation will be performed every time the method is called, whether or not the log level is low enough to show the message.

              Instead, you should structure your code to pass static or pre-computed values into Preconditions conditions check and logging calls.

              Specifically, the built-in string formatting should be used instead of string concatenation, and if the message is the result of a method call, then Preconditions should be skipped altogether, and the relevant exception should be conditionally thrown instead.

              Noncompliant Code Example

              logger.log(Level.DEBUG, "Something went wrong: " + message);  // Noncompliant; string concatenation performed even when log level too high to show DEBUG messages
              
              logger.fine("An exception occurred with message: " + message); // Noncompliant
              
              LOG.error("Unable to open file " + csvPath, e);  // Noncompliant
              
              Preconditions.checkState(a > 0, "Arg must be positive, but got " + a);  // Noncompliant. String concatenation performed even when a > 0
              
              Preconditions.checkState(condition, formatMessage());  // Noncompliant. formatMessage() invoked regardless of condition
              
              Preconditions.checkState(condition, "message: %s", formatMessage());  // Noncompliant
              

              Compliant Solution

              logger.log(Level.SEVERE, "Something went wrong: {0} ", message);  // String formatting only applied if needed
              
              logger.fine("An exception occurred with message: {}", message);  // SLF4J, Log4j
              
              logger.log(Level.SEVERE, () -> "Something went wrong: " + message); // since Java 8, we can use Supplier , which will be evaluated lazily
              
              LOG.error("Unable to open file {0}", csvPath, e);
              
              if (LOG.isDebugEnabled() {
                LOG.debug("Unable to open file " + csvPath, e);  // this is compliant, because it will not evaluate if log level is above debug.
              }
              
              Preconditions.checkState(arg > 0, "Arg must be positive, but got %d", a);  // String formatting only applied if needed
              
              if (!condition) {
                throw new IllegalStateException(formatMessage());  // formatMessage() only invoked conditionally
              }
              
              if (!condition) {
                throw new IllegalStateException("message: " + formatMessage());
              }
              

              Exceptions

              catch blocks are ignored, because the performance penalty is unimportant on exceptional paths (catch block should not be a part of standard program flow). Getters are ignored as well as methods called on annotations which can be considered as getters. This rule accounts for explicit test-level testing with SLF4J methods isXXXEnabled and ignores the bodies of such if statements.

              Use the built-in formatting to construct this argument.
              Open

                              logger.info(methodName + ": Finishing the Activity.");

              Passing message arguments that require further evaluation into a Guava com.google.common.base.Preconditions check can result in a performance penalty. That's because whether or not they're needed, each argument must be resolved before the method is actually called.

              Similarly, passing concatenated strings into a logging method can also incur a needless performance hit because the concatenation will be performed every time the method is called, whether or not the log level is low enough to show the message.

              Instead, you should structure your code to pass static or pre-computed values into Preconditions conditions check and logging calls.

              Specifically, the built-in string formatting should be used instead of string concatenation, and if the message is the result of a method call, then Preconditions should be skipped altogether, and the relevant exception should be conditionally thrown instead.

              Noncompliant Code Example

              logger.log(Level.DEBUG, "Something went wrong: " + message);  // Noncompliant; string concatenation performed even when log level too high to show DEBUG messages
              
              logger.fine("An exception occurred with message: " + message); // Noncompliant
              
              LOG.error("Unable to open file " + csvPath, e);  // Noncompliant
              
              Preconditions.checkState(a > 0, "Arg must be positive, but got " + a);  // Noncompliant. String concatenation performed even when a > 0
              
              Preconditions.checkState(condition, formatMessage());  // Noncompliant. formatMessage() invoked regardless of condition
              
              Preconditions.checkState(condition, "message: %s", formatMessage());  // Noncompliant
              

              Compliant Solution

              logger.log(Level.SEVERE, "Something went wrong: {0} ", message);  // String formatting only applied if needed
              
              logger.fine("An exception occurred with message: {}", message);  // SLF4J, Log4j
              
              logger.log(Level.SEVERE, () -> "Something went wrong: " + message); // since Java 8, we can use Supplier , which will be evaluated lazily
              
              LOG.error("Unable to open file {0}", csvPath, e);
              
              if (LOG.isDebugEnabled() {
                LOG.debug("Unable to open file " + csvPath, e);  // this is compliant, because it will not evaluate if log level is above debug.
              }
              
              Preconditions.checkState(arg > 0, "Arg must be positive, but got %d", a);  // String formatting only applied if needed
              
              if (!condition) {
                throw new IllegalStateException(formatMessage());  // formatMessage() only invoked conditionally
              }
              
              if (!condition) {
                throw new IllegalStateException("message: " + formatMessage());
              }
              

              Exceptions

              catch blocks are ignored, because the performance penalty is unimportant on exceptional paths (catch block should not be a part of standard program flow). Getters are ignored as well as methods called on annotations which can be considered as getters. This rule accounts for explicit test-level testing with SLF4J methods isXXXEnabled and ignores the bodies of such if statements.

              Use the built-in formatting to construct this argument.
              Open

                          logger.info(methodName + ": Will wait for the Activity triggered by the test to finish. Max timeout in secs: " + timeToWaitSecs);

              Passing message arguments that require further evaluation into a Guava com.google.common.base.Preconditions check can result in a performance penalty. That's because whether or not they're needed, each argument must be resolved before the method is actually called.

              Similarly, passing concatenated strings into a logging method can also incur a needless performance hit because the concatenation will be performed every time the method is called, whether or not the log level is low enough to show the message.

              Instead, you should structure your code to pass static or pre-computed values into Preconditions conditions check and logging calls.

              Specifically, the built-in string formatting should be used instead of string concatenation, and if the message is the result of a method call, then Preconditions should be skipped altogether, and the relevant exception should be conditionally thrown instead.

              Noncompliant Code Example

              logger.log(Level.DEBUG, "Something went wrong: " + message);  // Noncompliant; string concatenation performed even when log level too high to show DEBUG messages
              
              logger.fine("An exception occurred with message: " + message); // Noncompliant
              
              LOG.error("Unable to open file " + csvPath, e);  // Noncompliant
              
              Preconditions.checkState(a > 0, "Arg must be positive, but got " + a);  // Noncompliant. String concatenation performed even when a > 0
              
              Preconditions.checkState(condition, formatMessage());  // Noncompliant. formatMessage() invoked regardless of condition
              
              Preconditions.checkState(condition, "message: %s", formatMessage());  // Noncompliant
              

              Compliant Solution

              logger.log(Level.SEVERE, "Something went wrong: {0} ", message);  // String formatting only applied if needed
              
              logger.fine("An exception occurred with message: {}", message);  // SLF4J, Log4j
              
              logger.log(Level.SEVERE, () -> "Something went wrong: " + message); // since Java 8, we can use Supplier , which will be evaluated lazily
              
              LOG.error("Unable to open file {0}", csvPath, e);
              
              if (LOG.isDebugEnabled() {
                LOG.debug("Unable to open file " + csvPath, e);  // this is compliant, because it will not evaluate if log level is above debug.
              }
              
              Preconditions.checkState(arg > 0, "Arg must be positive, but got %d", a);  // String formatting only applied if needed
              
              if (!condition) {
                throw new IllegalStateException(formatMessage());  // formatMessage() only invoked conditionally
              }
              
              if (!condition) {
                throw new IllegalStateException("message: " + formatMessage());
              }
              

              Exceptions

              catch blocks are ignored, because the performance penalty is unimportant on exceptional paths (catch block should not be a part of standard program flow). Getters are ignored as well as methods called on annotations which can be considered as getters. This rule accounts for explicit test-level testing with SLF4J methods isXXXEnabled and ignores the bodies of such if statements.

              Wrong lexicographical order for 'java.io.File' import. Should be before 'org.junit.runner.Description'.
              Open

              import java.io.File;

              Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 128).
              Open

                          logger.info(methodName + ": Activity finished: " + !isActivityRunning(activity) + " (" + currentTimeSecs + "secs)");

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 115).
              Open

                      logger.info("Granting permissions to the MainActivity to be able to read files from an external storage.");

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 109).
              Open

                                  throw new FailureException("Command '" + shellCommand + "' failed with: " + outputError);

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Wrong lexicographical order for 'java.util.Collections' import. Should be before 'org.junit.runner.Description'.
              Open

              import java.util.Collections;

              Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              'private' modifier out of order with the JLS suggestions.
              Open

                  final private Deque<Runnable> closeActions = new ArrayDeque<>();

              Checks that the order of modifiers conforms to the suggestions inthe JavaLanguage specification, § 8.1.1, 8.3.1, 8.4.3 and9.4. The correct order is:

              1. public
              2. protected
              3. private
              4. abstract
              5. default
              6. static
              7. final
              8. transient
              9. volatile
              10. synchronized
              11. native
              12. strictfp

              In additional, modifiers are checked to ensure all annotations aredeclared before all other modifiers.

              Rationale: Code is easier to read if everybody follows a standard.

              ATTENTION: We skiptype annotations from validation.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 103).
              Open

                      UtilsPickerT.changePickerValue(ConstantsUI.PICKER_SHADER, R.id.pickerShader, shader.ordinal());

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 105).
              Open

                          UtilsContextT.waitUntil(this.testName.getMethodName(), activity, Constants.STOP, State.BUSY);

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line continuation have incorrect indentation level, expected level should be 4.
              Open

                   * whether the expected mocked {@link Intent} used by this method was really received by the

              Checks the indentation of the continuation lines in block tags.That is whether thecontinued description of at clauses should be indented or not. If the text is not properlyindented it throws a violation. A continuation line is when the description starts/spanspast the line with the tag. Default indentation required is at least 4, but this can bechanged with the help of properties below.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 101).
              Open

                          Intents.intended(IntentMatchers.filterEquals(expectedIntent), VerificationModes.times(1))

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Wrong lexicographical order for 'java.util.Objects' import. Should be before 'org.junit.runner.Description'.
              Open

              import java.util.Objects;

              Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Wrong lexicographical order for 'java.util.concurrent.TimeUnit' import. Should be before 'org.junit.runner.Description'.
              Open

              import java.util.concurrent.TimeUnit;

              Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 130).
              Open

                              Matchers.anyOf(IntentMatchers.hasAction(Intent.ACTION_GET_CONTENT), IntentMatchers.hasAction(Intent.ACTION_MAIN)),

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 141).
              Open

                          logger.info(methodName + ": Will wait for the Activity triggered by the test to finish. Max timeout in secs: " + timeToWaitSecs);

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 132).
              Open

                   * @param showRenderWhenPressingButton Whether to show the {@link Constants#RENDER} text in the render button after pressing it.

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line continuation have incorrect indentation level, expected level should be 4.
              Open

                   * tested application. This is done to avoid duplicated code.

              Checks the indentation of the continuation lines in block tags.That is whether thecontinued description of at clauses should be indented or not. If the text is not properlyindented it throws a violation. A continuation line is when the description starts/spanspast the line with the tag. Default indentation required is at least 4, but this can bechanged with the help of properties below.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 105).
              Open

                                  final String errorMessage = testName.getMethodName() + ": " + exception.getMessage();

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 108).
              Open

                      protected void skipped(final AssumptionViolatedException exception, final Description description) {

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 126).
              Open

                          logger.warning(this.testName.getMethodName() + ": The MainActivity didn't start as expected. Forcing a restart.");

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 128).
              Open

                      logger.info(this.testName.getMethodName() + ": " + methodName + " validating '" + intentsToVerify.size() + "' Intents");

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 109).
              Open

                  private static void waitForPermission(@NonNull final Context context, @NonNull final String permission) {

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 107).
              Open

                          logger.info("Permission '" + permission + "' granted to the app: " + context.getPackageName());

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 114).
              Open

                          final Uri firstFile = Uri.fromFile(new File(storagePath + ConstantsUI.FILE_SEPARATOR + filesPath[0]));

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Extra separation in import group before 'com.google.common.base.Preconditions'
              Open

              import com.google.common.base.Preconditions;

              Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Wrong lexicographical order for 'java.util.concurrent.TimeoutException' import. Should be before 'org.junit.runner.Description'.
              Open

              import java.util.concurrent.TimeoutException;

              Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 133).
              Open

                              InstrumentationRegistry.getInstrumentation().getContext().getPackageName(), Manifest.permission.READ_EXTERNAL_STORAGE

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 165).
              Open

                          final ClipData clipData = new ClipData(new ClipDescription("Scene", new String[]{"*" + ConstantsUI.FILE_SEPARATOR + "*"}), new ClipData.Item(firstFile));

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 105).
              Open

                      // Temporarily store the assertion that verifies if the application received the expected Intent.

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 114).
              Open

                              final BufferedReader reader = new BufferedReader(new InputStreamReader(process.getInputStream()));

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Extra separation in import group before 'java.io.BufferedReader'
              Open

              import java.io.BufferedReader;

              Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              <p> tag should be placed immediately before the first word, with no space after.</p>
              Open

                   * <p>

              Checks the Javadoc paragraph.

              Checks that:

              • There is one blank line between each of two paragraphs.
              • Each paragraph but the first has <p> immediately before the first word, withno space after.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 157).
              Open

                      final Intent expectedIntent = MainActivity.createIntentToLoadFiles(InstrumentationRegistry.getInstrumentation().getTargetContext().getPackageName());

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 112).
              Open

                   * Click on device to dismiss any "Application Not Responding" (ANR) system dialog that might have appeared.

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Wrong lexicographical order for 'java.io.InputStreamReader' import. Should be before 'org.junit.runner.Description'.
              Open

              import java.io.InputStreamReader;

              Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 143).
              Open

                              if (taskRunning.baseActivity != null && Objects.equals(activity.getPackageName(), taskRunning.baseActivity.getPackageName())) {

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 106).
              Open

                          InstrumentationRegistry.getInstrumentation().getUiAutomation().adoptShellPermissionIdentity();

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 138).
              Open

                          waitForPermission(InstrumentationRegistry.getInstrumentation().getTargetContext(), Manifest.permission.READ_EXTERNAL_STORAGE);

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 126).
              Open

                          throw new RuntimeException("Permission '" + permission + "' NOT granted to the app: " + context.getPackageName());

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 121).
              Open

                      final int numCores = UtilsContext.getNumOfCores(InstrumentationRegistry.getInstrumentation().getTargetContext());

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 103).
              Open

                      UtilsPickerT.changePickerValue(ConstantsUI.PICKER_SAMPLES_PIXEL, R.id.pickerSamplesPixel, spp);

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 115).
              Open

                          : UtilsContext.getInternalStoragePath(InstrumentationRegistry.getInstrumentation().getTargetContext());

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 114).
              Open

                          resultIntent.setData(Uri.fromFile(new File(storagePath + ConstantsUI.FILE_SEPARATOR + filesPath[0])));

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Wrong lexicographical order for 'java.nio.file.FileSystem' import. Should be before 'org.junit.runner.Description'.
              Open

              import java.nio.file.FileSystem;

              Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Extra separation in import group before 'puscas.mobilertapp.constants.Accelerator'
              Open

              import puscas.mobilertapp.constants.Accelerator;

              Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 140).
              Open

                          waitForPermission(InstrumentationRegistry.getInstrumentation().getTargetContext(), Manifest.permission.MANAGE_EXTERNAL_STORAGE);

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 141).
              Open

                      while (ContextCompat.checkSelfPermission(context, permission) != PackageManager.PERMISSION_GRANTED && currentTimeMs < timeToWaitMs) {

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line continuation have incorrect indentation level, expected level should be 4.
              Open

                   * call it in the {@link #tearDown()} method after every test. This {@link Runnable} verifies

              Checks the indentation of the continuation lines in block tags.That is whether thecontinued description of at clauses should be indented or not. If the text is not properlyindented it throws a violation. A continuation line is when the description starts/spanspast the line with the tag. Default indentation required is at least 4, but this can bechanged with the help of properties below.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 163).
              Open

                              final ParcelFileDescriptor parcelFileDescriptor = InstrumentationRegistry.getInstrumentation().getUiAutomation().executeShellCommand(shellCommand);

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              <p> tag should be preceded with an empty line.</p>
              Open

                   * <p>

              Checks the Javadoc paragraph.

              Checks that:

              • There is one blank line between each of two paragraphs.
              • Each paragraph but the first has <p> immediately before the first word, withno space after.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 101).
              Open

                      mainActivityActivityTestRule.getScenario().onActivity(newActivity -> activity = newActivity);

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 115).
              Open

                          final ActivityScenario<MainActivity> newActivityScenario = ActivityScenario.launch(MainActivity.class);

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 161).
              Open

                          // Necessary for the tests and MobileRT to be able to read any file from SD Card on Android 11+, by having the permission: 'MANAGE_EXTERNAL_STORAGE'.

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 132).
              Open

                          waitForPermission(InstrumentationRegistry.getInstrumentation().getContext(), Manifest.permission.READ_EXTERNAL_STORAGE);

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 155).
              Open

                      final Intent resultIntent = MainActivity.createIntentToLoadFiles(InstrumentationRegistry.getInstrumentation().getTargetContext().getPackageName());

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Wrong lexicographical order for 'java.io.BufferedReader' import. Should be before 'org.junit.runner.Description'.
              Open

              import java.io.BufferedReader;

              Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 167).
              Open

                          InstrumentationRegistry.getInstrumentation().getUiAutomation().executeShellCommand("pm grant puscas.mobilertapp android.permission.READ_EXTERNAL_STORAGE");

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 102).
              Open

                   * @param expectedSameValues           Whether the {@link Bitmap} should have have only one color.

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 137).
              Open

                              clipData.addItem(new ClipData.Item(Uri.fromFile(new File(storagePath + ConstantsUI.FILE_SEPARATOR + filesPath[index]))));

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 123).
              Open

                      final Instrumentation.ActivityResult result = new Instrumentation.ActivityResult(Activity.RESULT_OK, resultIntent);

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Wrong lexicographical order for 'java.util.logging.Logger' import. Should be before 'org.junit.runner.Description'.
              Open

              import java.util.logging.Logger;

              Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 122).
              Open

                          final List<ActivityManager.RunningTaskInfo> tasksRunning = activityManager.getRunningTasks(Integer.MAX_VALUE);

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 129).
              Open

                          logger.info("Waiting for the permission '" + permission + "' to be granted to the app: " + context.getPackageName());

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Abbreviation in name 'dismissANRSystemDialog' must contain no more than '2' consecutive capital letters.
              Open

                  private static void dismissANRSystemDialog() {

              Validates abbreviations (consecutive capital letters) length in identifier name,it also allows to enforce camel case naming. Please read more atGoogle Style Guideto get to know how to avoid long abbreviations in names.

              allowedAbbreviationLength specifies how many consecutive capital letters areallowed in the identifier.A value of 3 indicates that up to 4 consecutive capital letters are allowed,one after the other, before a violation is printed. The identifier 'MyTEST' would beallowed, but 'MyTESTS' would not be.A value of 0 indicates that only 1 consecutive capital letter is allowed. Thisis what should be used to enforce strict camel casing. The identifier 'MyTest' wouldbe allowed, but 'MyTEst' would not be.

              ignoreFinal, ignoreStatic, and ignoreStaticFinalcontrol whether variables with the respective modifiers are to be ignored.Note that a variable that is both static and final will always be considered underignoreStaticFinal only, regardless of the values of ignoreFinaland ignoreStatic. So for example if ignoreStatic is true butignoreStaticFinal is false, then static final variables will not be ignored.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Extra separation in import group before 'org.hamcrest.Matchers'
              Open

              import org.hamcrest.Matchers;

              Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Wrong lexicographical order for 'java.util.Deque' import. Should be before 'org.junit.runner.Description'.
              Open

              import java.util.Deque;

              Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Wrong lexicographical order for 'java.util.List' import. Should be before 'org.junit.runner.Description'.
              Open

              import java.util.List;

              Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 106).
              Open

                      if (ContextCompat.checkSelfPermission(context, permission) == PackageManager.PERMISSION_GRANTED) {

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 118).
              Open

                      UtilsPickerT.changePickerValue(ConstantsUI.PICKER_ACCELERATOR, R.id.pickerAccelerator, accelerator.ordinal());

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 119).
              Open

                      UtilsContextT.waitUntil(this.testName.getMethodName(), activity, Constants.RENDER, State.IDLE, State.FINISHED);

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Extra separation in import group before 'androidx.annotation.CallSuper'
              Open

              import androidx.annotation.CallSuper;

              Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Wrong lexicographical order for 'java.util.ArrayDeque' import. Should be before 'org.junit.runner.Description'.
              Open

              import java.util.ArrayDeque;

              Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 172).
              Open

                          InstrumentationRegistry.getInstrumentation().getUiAutomation().executeShellCommand("pm grant puscas.mobilertapp.test android.permission.READ_EXTERNAL_STORAGE");

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 134).
              Open

                          waitForPermission(InstrumentationRegistry.getInstrumentation().getContext(), Manifest.permission.MANAGE_EXTERNAL_STORAGE);

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Wrong lexicographical order for 'java.io.IOException' import. Should be before 'org.junit.runner.Description'.
              Open

              import java.io.IOException;

              Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 152).
              Open

                          Matchers.allOf(IntentMatchers.hasCategories(Collections.singleton(Intent.CATEGORY_LAUNCHER)), IntentMatchers.hasAction(Intent.ACTION_MAIN)),

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 122).
              Open

                          final ActivityManager activityManager = (ActivityManager) activity.getSystemService(Context.ACTIVITY_SERVICE);

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 105).
              Open

                          ? UtilsContext.getSdCardPath(InstrumentationRegistry.getInstrumentation().getTargetContext())

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 137).
              Open

                  public static final ActivityScenarioRule<MainActivity> mainActivityActivityTestRule = new ActivityScenarioRule<>(MainActivity.class);

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 139).
              Open

                              InstrumentationRegistry.getInstrumentation().getTargetContext().getPackageName(), Manifest.permission.READ_EXTERNAL_STORAGE

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 103).
              Open

                      UtilsPickerT.changePickerValue(ConstantsUI.PICKER_SAMPLES_LIGHT, R.id.pickerSamplesLight, spl);

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 123).
              Open

                                  final BufferedReader errorReader = new BufferedReader(new InputStreamReader(process.getErrorStream()));

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 117).
              Open

                          logger.info(this.testName.getMethodName() + ": Resetting Intents that were missing from previous test.");

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              Line is longer than 100 characters (found 107).
              Open

                  protected void mockFileManagerReply(final boolean externalSdcard, @NonNull final String... filesPath) {

              Checks for long lines.

              Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

              This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

              There are no issues that match your filters.

              Category
              Status