Merge pull request imas#453 from imas/fix/do-not-deliver-private-status-to-hashtag-stream

visibilityがprivateであるstatusが誤ってタグTLのストリームに配信されていたバグを修正

Author: takayamaki

.circleci/config.yml

@@ -68,7 +68,9 @@ jobs:
           cache-version: v1
           pkg-manager: yarn
       - run:
-          command: ./bin/rails assets:precompile
+          command: |
+            export NODE_OPTIONS=--openssl-legacy-provider
+            ./bin/rails assets:precompile
           name: Precompile assets
       - persist_to_workspace:
           paths:

app/services/fan_out_on_write_service.rb

@@ -41,10 +41,13 @@ def fan_out_to_local_recipients!
     notify_about_update! if update?
 
     case @status.visibility.to_sym
-    when :public, :private, :unlisted
+    when :public, :unlisted
       deliver_to_all_followers!
       deliver_to_lists!
       deliver_to_hashtag_streams_authorized_channel!
+    when :private
+      deliver_to_all_followers!
+      deliver_to_lists!
     when :limited
       deliver_to_mentioned_followers!
     else

spec/imastodon/services/fan_out_on_write_service_spec.rb

@@ -88,6 +88,20 @@
       it_behaves_like 'tagTLのauthorizedチャンネルに配信される'
     end
 
+    context 'private' do
+      let(:visibility) { :private }
+      it_behaves_like 'LTLに配信されない'
+      it_behaves_like 'tagTLのunauthorizedチャンネルに配信されない'
+      it_behaves_like 'tagTLのauthorizedチャンネルに配信されない'
+    end
+
+    context 'direct' do
+      let(:visibility) { :direct }
+      it_behaves_like 'LTLに配信されない'
+      it_behaves_like 'tagTLのunauthorizedチャンネルに配信されない'
+      it_behaves_like 'tagTLのauthorizedチャンネルに配信されない'
+    end
+
     context 'メンションが含まれる投稿' do
       let(:body) { '@tachibana ヘイヘーイ。 そこのカノジョー、一緒にブラデリカしないかーい？ #hoge #fuga #nyowa' }
 
@@ -104,6 +118,20 @@
         it_behaves_like 'tagTLのunauthorizedチャンネルに配信されない'
         it_behaves_like 'tagTLのauthorizedチャンネルに配信される'
       end
+
+      context 'private' do
+        let(:visibility) { :private }
+        it_behaves_like 'LTLに配信されない'
+        it_behaves_like 'tagTLのunauthorizedチャンネルに配信されない'
+        it_behaves_like 'tagTLのauthorizedチャンネルに配信されない'
+      end
+  
+      context 'direct' do
+        let(:visibility) { :direct }
+        it_behaves_like 'LTLに配信されない'
+        it_behaves_like 'tagTLのunauthorizedチャンネルに配信されない'
+        it_behaves_like 'tagTLのauthorizedチャンネルに配信されない'
+      end
     end
 
     context '自分以外の投稿へのin_reply_toが設定されている投稿' do
@@ -122,6 +150,20 @@
         it_behaves_like 'tagTLのunauthorizedチャンネルに配信されない'
         it_behaves_like 'tagTLのauthorizedチャンネルに配信されない'
       end
+
+      context 'private' do
+        let(:visibility) { :private }
+        it_behaves_like 'LTLに配信されない'
+        it_behaves_like 'tagTLのunauthorizedチャンネルに配信されない'
+        it_behaves_like 'tagTLのauthorizedチャンネルに配信されない'
+      end
+  
+      context 'direct' do
+        let(:visibility) { :direct }
+        it_behaves_like 'LTLに配信されない'
+        it_behaves_like 'tagTLのunauthorizedチャンネルに配信されない'
+        it_behaves_like 'tagTLのauthorizedチャンネルに配信されない'
+      end
     end
 
     context '自分の投稿へのin_reply_toが設定されている投稿' do
@@ -140,6 +182,20 @@
         it_behaves_like 'tagTLのunauthorizedチャンネルに配信されない'
         it_behaves_like 'tagTLのauthorizedチャンネルに配信される'
       end
+
+      context 'private' do
+        let(:visibility) { :private }
+        it_behaves_like 'LTLに配信されない'
+        it_behaves_like 'tagTLのunauthorizedチャンネルに配信されない'
+        it_behaves_like 'tagTLのauthorizedチャンネルに配信されない'
+      end
+  
+      context 'direct' do
+        let(:visibility) { :direct }
+        it_behaves_like 'LTLに配信されない'
+        it_behaves_like 'tagTLのunauthorizedチャンネルに配信されない'
+        it_behaves_like 'tagTLのauthorizedチャンネルに配信されない'
+      end
     end
   end
 end