style(phpcs): Fix white space and test method naming coding standards (drupal-graphql#1061)

Author: Dylan203

examples/graphql_example/src/Plugin/GraphQL/DataProducer/QueryArticles.php

@@ -105,4 +105,5 @@ public function resolve($offset, $limit, RefinableCacheableDependencyInterface $
 
     return new QueryConnection($query);
   }
-}
+
+}

examples/graphql_example/src/Wrappers/QueryConnection.php

@@ -45,4 +45,5 @@ public function items() {
       return $callback();
     });
   }
-}
+
+}

graphql.libraries.yml

@@ -32,4 +32,4 @@ persisted_queries:
   dependencies:
     - core/drupal
     - core/jquery
-    - core/jquery.once
+    - core/jquery.once

phpcs.xml.dist

@@ -55,10 +55,6 @@
     <exclude name="Drupal.Commenting.VariableComment.IncorrectVarType"/>
     <exclude name="Drupal.Commenting.VariableComment.Missing"/>
     <exclude name="Drupal.ControlStructures.ControlSignature.NewlineAfterCloseBrace"/>
-    <exclude name="Drupal.Files.EndFileNewline.NoneFound"/>
-    <exclude name="Drupal.Files.EndFileNewline.TooMany"/>
-    <exclude name="Drupal.Files.LineLength.TooLong"/>
-    <exclude name="Drupal.NamingConventions.ValidFunctionName.ScopeNotCamelCaps"/>
   </rule>
 
 </ruleset>

src/Cache/Context/StaticCacheContext.php

@@ -28,4 +28,4 @@ public function getCacheableMetadata($parameter = NULL) {
     return new CacheableMetadata();
   }
 
-}
+}

src/Config/LanguageConfigOverride.php

@@ -84,4 +84,4 @@ public function getCacheableMetadata($name) {
     return new CacheableMetadata();
   }
 
-}
+}

src/Event/OperationEvent.php

@@ -57,4 +57,4 @@ public function getContext() {
     return $this->context;
   }
 
-}
+}

src/EventSubscriber/CurrentLanguageResetTrait.php

@@ -52,4 +52,4 @@ protected function resetLanguageContext() {
     }
   }
 
-}
+}

src/GraphQL/Buffers/BufferBase.php

@@ -145,4 +145,5 @@ protected function resolveBuffer(\SplObjectStorage $buffer) {
   protected function resolveBufferArray(array $buffer) {
     throw new \LogicException('Method not implemented.');
   }
-}
+
+}

src/GraphQL/Execution/Executor.php

@@ -267,7 +267,8 @@ protected function cachePrefix() {
     // TODO: Should we try to sort these recursively?
     $variables = $this->variables ?: [];
     ksort($variables);
-    // TODO: Should we submit a pull request to also pass the extensions in the executor?
+    // TODO: Should we submit a pull request to also pass the extensions in the
+    // executor?
     $extensions = $this->context->getOperation()->extensions ?: [];
     ksort($extensions);
 

src/GraphQL/ResolverRegistryInterface.php

@@ -63,4 +63,4 @@ public function addTypeResolver($abstract, callable $resolver);
    */
   public function getTypeResolver($type);
 
-}
+}

src/GraphQL/Utility/DeferredUtility.php

@@ -116,4 +116,4 @@ public static function containsDeferred(array $values) {
     return FALSE;
   }
 
-}
+}

src/GraphQL/Utility/DocumentSerializer.php

@@ -32,4 +32,5 @@ public static function sanitizeRecursive(array $item) {
 
     return $item;
   }
-}
+
+}

src/GraphqlServiceProvider.php

@@ -20,4 +20,4 @@ public function alter(ContainerBuilder $container) {
     }
   }
 
-}
+}

src/Plugin/GraphQL/DataProducer/DataProducerPluginCachingTrait.php

@@ -26,4 +26,4 @@ public function edgeCachePrefix() {
    */
   abstract public function getContextValues();
 
-}
+}

src/Plugin/GraphQL/DataProducer/Entity/EntityLoad.php

@@ -144,8 +144,9 @@ public function resolve($type, $id, $language, ?array $bundles, ?bool $access, ?
 
     return new Deferred(function () use ($type, $id, $language, $bundles, $resolver, $context, $access, $accessUser, $accessOperation) {
       if (!$entity = $resolver()) {
-        // If there is no entity with this id, add the list cache tags so that the
-        // cache entry is purged whenever a new entity of this type is saved.
+        // If there is no entity with this id, add the list cache tags so that
+        // the cache entry is purged whenever a new entity of this type is
+        // saved.
         $type = $this->entityTypeManager->getDefinition($type);
         /** @var \Drupal\Core\Entity\EntityTypeInterface $type */
         $tags = $type->getListCacheTags();

src/Plugin/GraphQL/DataProducer/Entity/EntityLoadByUuid.php

@@ -144,8 +144,9 @@ public function resolve($type, $uuid, $language, $bundles, ?bool $access, ?Accou
 
     return new Deferred(function () use ($type, $language, $bundles, $resolver, $context, $access, $accessUser, $accessOperation) {
       if (!$entity = $resolver()) {
-        // If there is no entity with this id, add the list cache tags so that the
-        // cache entry is purged whenever a new entity of this type is saved.
+        // If there is no entity with this id, add the list cache tags so that
+        // the cache entry is purged whenever a new entity of this type is
+        // saved.
         $type = $this->entityTypeManager->getDefinition($type);
         /** @var \Drupal\Core\Entity\EntityTypeInterface $type */
         $tags = $type->getListCacheTags();

src/Plugin/GraphQL/DataProducer/Field/EntityReferenceLayoutRevisions.php

@@ -123,7 +123,7 @@ public function __construct(
   }
 
   /**
-   * Resolves entity reference layout revisions for a given field of a given entity.
+   * Resolves entity reference layout revisions for a field of a given entity.
    *
    * May optionally respect the entity bundles and language.
    *

src/Plugin/GraphQL/DataProducer/Menu/MenuLink/MenuLinkAttribute.php

@@ -36,7 +36,8 @@ class MenuLinkAttribute extends DataProducerPluginBase {
    */
   public function resolve(MenuLinkInterface $link, $attribute) {
     $options = $link->getOptions();
-    // Certain attributes like class can be arrays. Check for that and implode them.
+    // Certain attributes like class can be arrays. Check for that and implode
+    // them.
     $attributeValue = NestedArray::getValue($options, ['attributes', $attribute]);
     if (is_array($attributeValue)) {
       return implode(" ", $attributeValue);

src/Plugin/LanguageNegotiation/OperationLanguageNegotiation.php

@@ -54,4 +54,5 @@ public function getLangcode(Request $request = NULL) {
   public static function setContext(ResolveContext $context = NULL) {
     static::$context = $context;
   }
-}
+
+}

src/SubRequestResponse.php

@@ -41,4 +41,4 @@ public function getResult() {
     return $this->result;
   }
 
-}
+}

tests/src/Kernel/DataProducer/Entity/Fields/Image/ImageDerivativeTest.php

@@ -77,7 +77,8 @@ public function testImageDerivative() {
     );
 
     // TODO: Add cache checks.
-    //    $this->assertContains('config:image.style.test_style', $metadata->getCacheTags());
+    //    $this->assertContains('config:image.style.test_style',
+    //    $metadata->getCacheTags());
     //    $this->assertContains('test_tag', $metadata->getCacheTags());
 
     // Test that we don't get the derivative if we don't have access to the
@@ -90,7 +91,8 @@ public function testImageDerivative() {
     $this->assertNull($result);
 
     // TODO: Add cache checks.
-    //    $this->assertContains('test_tag_forbidden', $metadata->getCacheTags());
+    //    $this->assertContains('test_tag_forbidden',
+    //    $metadata->getCacheTags());
   }
 
 }

tests/src/Kernel/DataProducer/Entity/Fields/Image/ImageUrlTest.php

@@ -61,7 +61,8 @@ public function testImageUrl() {
     $this->assertNull($result);
 
     // TODO: Add cache checks.
-    //    $this->assertContains('test_tag_forbidden', $metadata->getCacheTags());
+    //    $this->assertContains('test_tag_forbidden',
+    //    $metadata->getCacheTags());
   }
 
 }

tests/src/Kernel/DataProducer/XML/XMLAttributeTest.php

@@ -12,7 +12,7 @@ class XMLAttributeTest extends XMLTestBase {
   /**
    * @covers \Drupal\graphql\Plugin\GraphQL\DataProducer\XML\XMLAttribute::resolve
    */
-  public function testXMLAttribute() {
+  public function testXmlAttribute() {
     $document = $this->loadDocument();
     $xpath = new \DOMXPath($document->ownerDocument);
     $div = iterator_to_array($xpath->query('//div/div', $document));

tests/src/Kernel/DataProducer/XML/XMLContentTest.php

@@ -12,7 +12,7 @@ class XMLContentTest extends XMLTestBase {
   /**
    * @covers \Drupal\graphql\Plugin\GraphQL\DataProducer\XML\XMLContent::resolve
    */
-  public function testXMLContent() {
+  public function testXmlContent() {
     $document = $this->loadDocument();
     $xpath = new \DOMXPath($document->ownerDocument);
     $h1 = iterator_to_array($xpath->query('//div/h1', $document));

tests/src/Kernel/DataProducer/XML/XMLParseTest.php

@@ -12,7 +12,7 @@ class XMLParseTest extends XMLTestBase {
   /**
    * @covers \Drupal\graphql\Plugin\GraphQL\DataProducer\XML\XMLParse::resolve
    */
-  public function testXMLParse() {
+  public function testXmlParse() {
     $result = $this->executeDataProducer('xml_parse', [
       'input' => $this->getDocumentSource(),
     ]);

tests/src/Kernel/DataProducer/XML/XMLXpathTest.php

@@ -12,7 +12,7 @@ class XMLXpathTest extends XMLTestBase {
   /**
    * @covers \Drupal\graphql\Plugin\GraphQL\DataProducer\XML\XMLXpath::resolve
    */
-  public function testXMLXpath() {
+  public function testXmlXpath() {
     $document = $this->loadDocument();
 
     $result = $this->executeDataProducer('xml_xpath', [

tests/src/Kernel/ResolverBuilderTest.php

@@ -432,4 +432,3 @@ public function testDeferredDefaultValue() {
   }
 
 }
-