PostgreSQL: Fix views\Tests\Handler\FieldGroupRowsWebTest (again)

Well, in that case I'd always suggest to add the most minimal order, which is NID almost ever ...

Passes SQLite on my local environment.

+++ b/core/modules/views/tests/modules/views_test_config/test_views/views.view.test_ungroup_rows.yml
@@ -126,6 +126,18 @@ display:
+        delta:
+          id: delta
+          table: node__field_views_testing_group_rows
+          field: delta
+          relationship: none
+          group_type: group
+          admin_label: ''
+          order: ASC
+          exposed: false
+          expose:
+            label: ''
+          plugin_id: standard
   page_1:

It seems to be for me that the easiest way to fix that would be to replace the created one with sorting by field_views_testing_group_?

I can confirm that this patch indeed fixes the failing test in Drupal\views\Tests\Handler\FieldGroupRowsWebTest. I've also reviewed the patch manually, and while I don't pretend to understand everything, it looks quite straightforward. Marking as RTBC.

Before the patch:
Drupal\views\Tests\Handler\FieldGroupRowsWebTest 9 passes 1 fails

After the patch:
Drupal\views\Tests\Handler\FieldGroupRowsWebTest 10 passes

Doesn't look like #4 has been addressed.

I wasn't able to generate an interdiff with this patch. Since this changed the sort, we need to run through tests again.

I was running views test group thru drupalci, but drupalci just stopped halfway. The two tests passed though.

Drupal\views\Tests\Handler\FieldGroupRowsTest 6 passes
Drupal\views\Tests\Handler\FieldGroupRowsWebTest 10 passes

I have reviewed the code in the patch again, as well as tested it again using the new DrupalCI infrastructure.

Before the patch:
Drupal\views\Tests\Handler\FieldGroupRowsWebTest 9 passes 1 fails

After the patch:
Drupal\views\Tests\Handler\FieldGroupRowsWebTest 10 passes

Additionally, it causes no other regressions in the views test suite. As such, I'm setting this back to RTBC.

Committed and pushed to 8.0.x. Thanks!