Fix credo

Author: mickel8

.credo.exs

@@ -104,7 +104,7 @@
           {Credo.Check.Readability.ParenthesesInCondition, []},
           {Credo.Check.Readability.ParenthesesOnZeroArityDefs, []},
           {Credo.Check.Readability.PipeIntoAnonymousFunctions, []},
-          {Credo.Check.Readability.PredicateFunctionNames, [exit_status: 0]},
+          {Credo.Check.Readability.PredicateFunctionNames, []},
           {Credo.Check.Readability.PreferImplicitTry, []},
           {Credo.Check.Readability.RedundantBlankLines, []},
           {Credo.Check.Readability.Semicolons, []},

lib/ex_webrtc/peer_connection.ex

@@ -1008,7 +1008,7 @@ defmodule ExWebRTC.PeerConnection do
 
     # This is not defined in the W3C but see https://github.com/w3c/webrtc-pc/issues/2927
     tr =
-      if SDPUtils.is_rejected(mline),
+      if SDPUtils.rejected?(mline),
         do: RTPTransceiver.stop(tr, on_track_ended(owner, tr.receiver.track.id)),
         else: tr
 
@@ -1028,7 +1028,7 @@ defmodule ExWebRTC.PeerConnection do
     {:mid, mid} = ExSDP.get_attribute(mline, :mid)
 
     direction =
-      if SDPUtils.is_rejected(mline),
+      if SDPUtils.rejected?(mline),
         do: :inactive,
         else: SDPUtils.get_media_direction(mline) |> reverse_direction()
 
@@ -1044,7 +1044,7 @@ defmodule ExWebRTC.PeerConnection do
     tr = if sdp_type == :answer, do: %RTPTransceiver{tr | current_direction: direction}, else: tr
 
     tr =
-      if SDPUtils.is_rejected(mline),
+      if SDPUtils.rejected?(mline),
         do: RTPTransceiver.stop(tr, on_track_ended(owner, tr.receiver.track.id)),
         else: tr
 
@@ -1160,7 +1160,7 @@ defmodule ExWebRTC.PeerConnection do
     do: state
 
   defp update_negotiation_needed(state) do
-    negotiation_needed = is_negotiation_needed(state.transceivers, state)
+    negotiation_needed = negotiation_needed?(state.transceivers, state)
 
     cond do
       negotiation_needed == true and state.negotiation_needed == true ->
@@ -1182,11 +1182,11 @@ defmodule ExWebRTC.PeerConnection do
   # We don't support MSIDs and stopping transceivers so 
   # we only check 5.2 and 5.3 from 4.7.3#check-if-negotiation-is-needed
   # https://www.w3.org/TR/webrtc/#dfn-check-if-negotiation-is-needed
-  defp is_negotiation_needed([], _), do: false
+  defp negotiation_needed?([], _), do: false
 
-  defp is_negotiation_needed([tr | _transceivers], _state) when tr.mid == nil, do: true
+  defp negotiation_needed?([tr | _transceivers], _state) when tr.mid == nil, do: true
 
-  defp is_negotiation_needed([tr | transceivers], state) do
+  defp negotiation_needed?([tr | transceivers], state) do
     {local_desc_type, local_desc} = state.current_local_desc
     {_, remote_desc} = state.current_remote_desc
 
@@ -1211,7 +1211,7 @@ defmodule ExWebRTC.PeerConnection do
         true
 
       true ->
-        is_negotiation_needed(transceivers, state)
+        negotiation_needed?(transceivers, state)
     end
   end
 

lib/ex_webrtc/peer_connection/configuration.ex

@@ -157,20 +157,20 @@ defmodule ExWebRTC.PeerConnection.Configuration do
   end
 
   @doc false
-  @spec is_supported_codec(t(), RTPCodecParameters.t()) :: boolean()
-  def is_supported_codec(config, codec) do
+  @spec supported_codec?(t(), RTPCodecParameters.t()) :: boolean()
+  def supported_codec?(config, codec) do
     # This function doesn't check if rtcp-fb is supported.
-    # Instead, `is_supported_rtcp_fb` has to be used to filter out
+    # Instead, `supported_rtcp_fb?` has to be used to filter out
     # rtcp-fb that are not supported.
     Enum.any?(config.audio_codecs ++ config.video_codecs, fn supported_codec ->
       %{supported_codec | rtcp_fbs: codec.rtcp_fbs} == codec
     end)
   end
 
   @doc false
-  @spec is_supported_rtp_hdr_extension(t(), Extmap.t(), :audio | :video) ::
+  @spec supported_rtp_hdr_extension?(t(), Extmap.t(), :audio | :video) ::
           boolean()
-  def is_supported_rtp_hdr_extension(config, rtp_hdr_extension, media_type) do
+  def supported_rtp_hdr_extension?(config, rtp_hdr_extension, media_type) do
     supported_uris =
       case media_type do
         :audio -> Map.keys(config.audio_rtp_hdr_exts)
@@ -181,8 +181,8 @@ defmodule ExWebRTC.PeerConnection.Configuration do
   end
 
   @doc false
-  @spec is_supported_rtcp_fb(t(), RTCPFeedback.t()) :: boolean()
-  def is_supported_rtcp_fb(_config, _rtcp_fb), do: false
+  @spec supported_rtcp_fb?(t(), RTCPFeedback.t()) :: boolean()
+  def supported_rtcp_fb?(_config, _rtcp_fb), do: false
 
   @doc false
   @spec update(t(), ExSDP.t()) :: t()

lib/ex_webrtc/rtp_transceiver.ex

@@ -239,11 +239,11 @@ defmodule ExWebRTC.RTPTransceiver do
   defp get_codecs(mline, config) do
     mline
     |> SDPUtils.get_rtp_codec_parameters()
-    |> Stream.filter(&Configuration.is_supported_codec(config, &1))
+    |> Stream.filter(&Configuration.supported_codec?(config, &1))
     |> Enum.map(fn codec ->
       rtcp_fbs =
         Enum.filter(codec.rtcp_fbs, fn rtcp_fb ->
-          Configuration.is_supported_rtcp_fb(config, rtcp_fb)
+          Configuration.supported_rtcp_fb?(config, rtcp_fb)
         end)
 
       %RTPCodecParameters{codec | rtcp_fbs: rtcp_fbs}
@@ -253,6 +253,6 @@ defmodule ExWebRTC.RTPTransceiver do
   defp get_rtp_hdr_extensions(mline, config) do
     mline
     |> ExSDP.get_attributes(ExSDP.Attribute.Extmap)
-    |> Enum.filter(&Configuration.is_supported_rtp_hdr_extension(config, &1, mline.type))
+    |> Enum.filter(&Configuration.supported_rtp_hdr_extension?(config, &1, mline.type))
   end
 end

lib/ex_webrtc/sdp_utils.ex

@@ -268,9 +268,9 @@ defmodule ExWebRTC.SDPUtils do
     |> Enum.find_index(fn {mline, idx} -> mline.port == 0 and idx not in indices end)
   end
 
-  @spec is_rejected(ExSDP.Media.t()) :: boolean()
-  def is_rejected(%ExSDP.Media{port: 0}), do: true
-  def is_rejected(%ExSDP.Media{}), do: false
+  @spec rejected?(ExSDP.Media.t()) :: boolean()
+  def rejected?(%ExSDP.Media{port: 0}), do: true
+  def rejected?(%ExSDP.Media{}), do: false
 
   defp do_get_ice_credentials(sdp_or_mline) do
     ice_ufrag =

test/ex_webrtc/sdp_utils_test.exs

@@ -3,11 +3,11 @@ defmodule ExWebRTC.SDPUtilsTest do
 
   alias ExWebRTC.SDPUtils
 
-  test "is_rejected/1" do
+  test "rejected?/1" do
     mline = ExSDP.Media.new(:audio, 0, "UDP/TLS/RTP/SAVPF", [8])
-    assert true == SDPUtils.is_rejected(mline)
+    assert true == SDPUtils.rejected?(mline)
 
     mline = ExSDP.Media.new(:audio, 9, "UDP/TLS/RTP/SAVPF", [8])
-    assert false == SDPUtils.is_rejected(mline)
+    assert false == SDPUtils.rejected?(mline)
   end
 end