 a90c45b7e9
			
		
	
	
		a90c45b7e9
		
	
	
	
	
		
			
			Close #304. Notes: - This patch was made on top of Pleroma develop, so I created a separate cachex worker for request signature actions, instead of Akkoma's instance cache. If that is a merge blocker, I can attempt to move logic around for that. - Regarding the `has_request_signatures: true -> false` state transition: I think that is a higher level thing (resetting instance state on new instance actor key) which is separate from the changes relevant to this one. Co-authored-by: Luna <git@l4.pm> Reviewed-on: https://akkoma.dev/AkkomaGang/akkoma/pulls/312 Co-authored-by: @luna@f.l4.pm <akkoma@l4.pm> Co-committed-by: @luna@f.l4.pm <akkoma@l4.pm>
		
			
				
	
	
		
			142 lines
		
	
	
	
		
			4.6 KiB
		
	
	
	
		
			Elixir
		
	
	
	
	
	
			
		
		
	
	
			142 lines
		
	
	
	
		
			4.6 KiB
		
	
	
	
		
			Elixir
		
	
	
	
	
	
| # Pleroma: A lightweight social networking server
 | |
| # Copyright © 2017-2021 Pleroma Authors <https://pleroma.social/>
 | |
| # SPDX-License-Identifier: AGPL-3.0-only
 | |
| 
 | |
| defmodule Pleroma.InstancesTest do
 | |
|   alias Pleroma.Instances
 | |
|   alias Pleroma.Instances.Instance
 | |
| 
 | |
|   use Pleroma.DataCase
 | |
| 
 | |
|   setup_all do: clear_config([:instance, :federation_reachability_timeout_days], 1)
 | |
| 
 | |
|   describe "reachable?/1" do
 | |
|     test "returns `true` for host / url with unknown reachability status" do
 | |
|       assert Instances.reachable?("unknown.site")
 | |
|       assert Instances.reachable?("http://unknown.site")
 | |
|     end
 | |
| 
 | |
|     test "returns `false` for host / url marked unreachable for at least `reachability_datetime_threshold()`" do
 | |
|       host = "consistently-unreachable.name"
 | |
|       Instances.set_consistently_unreachable(host)
 | |
| 
 | |
|       refute Instances.reachable?(host)
 | |
|       refute Instances.reachable?("http://#{host}/path")
 | |
|     end
 | |
| 
 | |
|     test "returns `true` for host / url marked unreachable for less than `reachability_datetime_threshold()`" do
 | |
|       url = "http://eventually-unreachable.name/path"
 | |
| 
 | |
|       Instances.set_unreachable(url)
 | |
| 
 | |
|       assert Instances.reachable?(url)
 | |
|       assert Instances.reachable?(URI.parse(url).host)
 | |
|     end
 | |
| 
 | |
|     test "raises FunctionClauseError exception on non-binary input" do
 | |
|       assert_raise FunctionClauseError, fn -> Instances.reachable?(nil) end
 | |
|       assert_raise FunctionClauseError, fn -> Instances.reachable?(1) end
 | |
|     end
 | |
|   end
 | |
| 
 | |
|   describe "filter_reachable/1" do
 | |
|     setup do
 | |
|       host = "consistently-unreachable.name"
 | |
|       url1 = "http://eventually-unreachable.com/path"
 | |
|       url2 = "http://domain.com/path"
 | |
| 
 | |
|       Instances.set_consistently_unreachable(host)
 | |
|       Instances.set_unreachable(url1)
 | |
| 
 | |
|       result = Instances.filter_reachable([host, url1, url2, nil])
 | |
|       %{result: result, url1: url1, url2: url2}
 | |
|     end
 | |
| 
 | |
|     test "returns a map with keys containing 'not marked consistently unreachable' elements of supplied list",
 | |
|          %{result: result, url1: url1, url2: url2} do
 | |
|       assert is_map(result)
 | |
|       assert Enum.sort([url1, url2]) == result |> Map.keys() |> Enum.sort()
 | |
|     end
 | |
| 
 | |
|     test "returns a map with `unreachable_since` values for keys",
 | |
|          %{result: result, url1: url1, url2: url2} do
 | |
|       assert is_map(result)
 | |
|       assert %NaiveDateTime{} = result[url1]
 | |
|       assert is_nil(result[url2])
 | |
|     end
 | |
| 
 | |
|     test "returns an empty map for empty list or list containing no hosts / url" do
 | |
|       assert %{} == Instances.filter_reachable([])
 | |
|       assert %{} == Instances.filter_reachable([nil])
 | |
|     end
 | |
|   end
 | |
| 
 | |
|   describe "set_reachable/1" do
 | |
|     test "sets unreachable url or host reachable" do
 | |
|       host = "domain.com"
 | |
|       Instances.set_consistently_unreachable(host)
 | |
|       refute Instances.reachable?(host)
 | |
| 
 | |
|       Instances.set_reachable(host)
 | |
|       assert Instances.reachable?(host)
 | |
|     end
 | |
| 
 | |
|     test "keeps reachable url or host reachable" do
 | |
|       url = "https://site.name?q="
 | |
|       assert Instances.reachable?(url)
 | |
| 
 | |
|       Instances.set_reachable(url)
 | |
|       assert Instances.reachable?(url)
 | |
|     end
 | |
| 
 | |
|     test "returns error status on non-binary input" do
 | |
|       assert {:error, _} = Instances.set_reachable(nil)
 | |
|       assert {:error, _} = Instances.set_reachable(1)
 | |
|     end
 | |
|   end
 | |
| 
 | |
|   # Note: implementation-specific (e.g. Instance) details of set_unreachable/1
 | |
|   # should be tested in implementation-specific tests
 | |
|   describe "set_unreachable/1" do
 | |
|     test "returns error status on non-binary input" do
 | |
|       assert {:error, _} = Instances.set_unreachable(nil)
 | |
|       assert {:error, _} = Instances.set_unreachable(1)
 | |
|     end
 | |
|   end
 | |
| 
 | |
|   describe "set_consistently_unreachable/1" do
 | |
|     test "sets reachable url or host unreachable" do
 | |
|       url = "http://domain.com?q="
 | |
|       assert Instances.reachable?(url)
 | |
| 
 | |
|       Instances.set_consistently_unreachable(url)
 | |
|       refute Instances.reachable?(url)
 | |
|     end
 | |
| 
 | |
|     test "keeps unreachable url or host unreachable" do
 | |
|       host = "site.name"
 | |
|       Instances.set_consistently_unreachable(host)
 | |
|       refute Instances.reachable?(host)
 | |
| 
 | |
|       Instances.set_consistently_unreachable(host)
 | |
|       refute Instances.reachable?(host)
 | |
|     end
 | |
|   end
 | |
| 
 | |
|   describe "set_request_signatures/1" do
 | |
|     test "sets instance has request signatures" do
 | |
|       host = "domain.com"
 | |
| 
 | |
|       {:ok, instance} = Instances.set_request_signatures(host)
 | |
|       assert instance.has_request_signatures
 | |
| 
 | |
|       {:ok, cached_instance} = Instance.get_cached_by_url(host)
 | |
|       assert cached_instance.has_request_signatures
 | |
|     end
 | |
| 
 | |
|     test "returns error status on non-binary input" do
 | |
|       assert {:error, _} = Instances.set_request_signatures(nil)
 | |
|       assert {:error, _} = Instances.set_request_signatures(1)
 | |
|     end
 | |
|   end
 | |
| end
 |