Fields cannot be used as the row style on style plugins that support it

Changing status for tests.

+++ b/core/modules/views/lib/Drupal/views/Tests/Plugin/StyleTest.phpundefined
@@ -50,45 +50,46 @@ protected function setUp() {
+    $view->style_plugin->setUsesRowPlugin(FALSE);
+    $this->assertTrue($view->style_plugin instanceof StyleTestPlugin, 'Make sure the right style plugin class is loaded.');

Should we also check that $view->rowPlugin is now not an instance of RowTest?

Beside from this, this looks perfect.

Thank you!

+++ b/core/modules/views/lib/Drupal/views/Tests/Plugin/StyleTest.phpundefined
@@ -50,45 +51,47 @@ protected function setUp() {
-    $this->assertTrue(strpos($output, $random_text) !== FALSE, 'Take sure that the rendering of the style plugin appears in the output of the view.');

I know it's already there, but two cases of 'Take sure' in lines that are changed.

I'm sorry for all the "take" sure all over the place.

+++ b/core/modules/views/tests/modules/views_test_data/lib/Drupal/views_test_data/Plugin/views/style/StyleTest.phpundefined
@@ -34,6 +34,13 @@ class StyleTest extends StylePluginBase {
+   * Does the style plugin allows to use style plugins.

This should be something like... Can the style plugin use row plugins.

+1

Committed 62ed405 and pushed to 8.x. Thanks!

diff --git a/core/git b/core/git
new file mode 100644

What's this file for?

In case it's not there for a reason, here's the patch to remove it.

Would be cool to understand what added the file in the first place.

Looks like it got added in #8. assume it was just an accident on the command line. things happen and its hard to see in the patch. RTBC for sure. Maybe the first 0 line patch? ;)

Whoops should have caught that on review... my bad.

Committed f36f480 and pushed to 8.x. Thanks!