Merge remote-tracking branch 'upstream/master' into grandparent_test

.github/workflows/reviewdog.yml

@@ -8,6 +8,25 @@ permissions:
   checks: write
 
 jobs:
+  rubocop:
+    name: Rubocop
+    runs-on: ubuntu-latest
+    steps:
+      - name: Check out code
+        uses: actions/checkout@v4
+
+      - name: Set up Ruby and run bundle install
+        uses: ruby/setup-ruby@v1
+        with:
+          bundler-cache: true
+
+      - name: rubocop
+        uses: reviewdog/action-rubocop@e70b014b8062c447d6b515ee0209f834ea93e696
+        with:
+          use_bundler: true
+          reporter: github-pr-check
+          skip_install: true
+
   erb-lint:
     name: ERB Lint runner
     runs-on: ubuntu-latest

.hound.yml

@@ -6,5 +6,4 @@ jshint:
   ignore_file: .jshintignore
 
 rubocop:
-  version: 1.22.1
-  config_file: .rubocop.yml
+  enabled: false

features/collections/collection_anonymity.feature

@@ -60,6 +60,8 @@ Feature: Collection
       And all emails have been delivered
     When I am logged in as "first_user"
       And I post the work "First Snippet" to the collection "Hidden Treasury" as a gift for "third_user"
+      # Delay before posting to make sure first work is clearly older
+      And it is currently 1 second from now
       And I post the work "Second Snippet" to the collection "Hidden Treasury" as a gift for "fourth_user"
       And subscription notifications are sent
     Then 0 emails should be delivered
@@ -160,13 +162,17 @@ Feature: Collection
       And the user "third_user" allows gifts
       And I am logged in as "first_user"
       And I post the work "First Snippet" to the collection "Anonymous Hugs" as a gift for "third_user"
+      # Delay before posting to make sure first work is clearly older
+      And it is currently 1 second from now
       And I post the work "Second Snippet" to the collection "Anonymous Hugs" as a gift for "not a user"
     When subscription notifications are sent
     Then "second_user" should not be emailed
     When I am logged in as "moderator"
       And I view the approved collection items page for "Anonymous Hugs"
       # items listed in date order so checking the second will reveal the older work
       And I uncheck the 2nd checkbox with id matching "collection_items_\d+_anonymous"
+      # Delay before submitting to make sure the cache is expired
+      And it is currently 1 second from now
       And I submit
     Then the author of "First Snippet" should be publicly visible
     When subscription notifications are sent
@@ -388,6 +394,8 @@ Feature: Collection
     When I am logged in as the owner of "Anonymous Collection"
       And I go to "Anonymous Collection" collection edit page
       And I follow "Delete Collection"
+      # Delay before deleting to make sure the cache is expired
+      And it is currently 1 second from now
       And I press "Yes, Delete Collection"
       And I go to creator's works page
     Then I should see "Secret Work"
@@ -418,6 +426,8 @@ Feature: Collection
     When I am logged in as the owner of "Anonymous Collection"
       And I view the approved collection items page for "Anonymous Collection"
       And I check "Remove"
+      # Delay before submitting to make sure the cache is expired
+      And it is currently 1 second from now
       And I submit
       And I go to creator's works page
     Then I should see "Secret Work"
@@ -451,8 +461,9 @@ Feature: Collection
 
     When I edit the work "Secret Work"
       And I fill in "Collections" with "Holidays,Fluffy"
+      # Delay before posting to make sure the cache is expired
+      And it is currently 1 second from now
       And I press "Post"
-      And all indexing jobs have been run
       And I go to my works page
     Then I should see "Secret Work"
 

features/other_a/orphan_work.feature

@@ -24,6 +24,8 @@ Feature: Orphan work
     When I follow "Edit"
     Then I should see "Edit Work"
       And I should see "Orphan Work"
+      # Delay before orphaning to make sure the cache is expired
+      And it is currently 1 second from now
     When I follow "Orphan Work"
     Then I should see "Read More About The Orphaning Process"
     When I choose "Take my pseud off as well"
@@ -46,6 +48,8 @@ Feature: Orphan work
     When I follow "Edit"
     Then I should see "Edit Work"
       And I should see "Orphan Work"
+      # Delay before orphaning to make sure the cache is expired
+      And it is currently 1 second from now
     When I follow "Orphan Work"
     Then I should see "Read More About The Orphaning Process"
     When I choose "Leave a copy of my pseud on"
@@ -131,6 +135,8 @@ Feature: Orphan work
     And I should see "Glorious"
     And I should see "Excellent"
     And I should not see "Lovely"
+    # Delay before orphaning to make sure the cache is expired
+    And it is currently 1 second from now
   When I follow "Orphan Works Instead"
   Then I should see "Orphaning a work removes it from your account and re-attaches it to the specially created orphan_account."
   When I press "Yes, I'm sure"

features/other_b/series.feature

@@ -207,6 +207,8 @@ Feature: Create and Edit Series
     Then I should see "Work was successfully updated."
       And "moon" should be a creator of the series "Ponies"
       And "son" should be a creator on the series "Ponies"
+      # Delay to make sure the cache is expired
+      And it is currently 1 second from now
     When I follow "Remove Me As Co-Creator"
     Then I should see "You have been removed as a creator from the series and its works."
       And "moon" should not be the creator of the series "Ponies"