Pure Elixir G.711 implementation

[sgfn]

No description provided.

[Rados13]

All these tests look pretty the same. Maybe we could think about using the table-driven pattern here (it is mostly used in Golang)? (Some resources about table-driven tests 1, 2.)

[sgfn]

Dunno if that's what you had in mind, but I've done something to alleviate the code duplication

[Rados13]

I thought more of things like this:

  test_cases = [
    {"adds timestamps",@stream_format, %Parser{overwrite_pts?: true}, 0}
    {"adds timestamps with offset",@stream_format, %Parser{overwrite_pts?: true, pts_offset: 10}, 10},
    {"can have `RemoteStream` as input",%RemoteStream{}, %Parser{stream_format: @stream_format}, 0},
  ]

  for {test_name, source_stream_format,parser, offset} <- test_cases do 
    test "parser #{test_name}" do 
      ...
    end
  end

But I think that for start this is good enough.

[Rados13]

I thought more of things like this:

  test_cases = [
    {"adds timestamps",@stream_format, %Parser{overwrite_pts?: true}, 0}
    {"adds timestamps with offset",@stream_format, %Parser{overwrite_pts?: true, pts_offset: 10}, 10},
    {"can have `RemoteStream` as input",%RemoteStream{}, %Parser{stream_format: @stream_format}, 0},
  ]

  for {test_name, source_stream_format,parser, offset} <- test_cases do 
    test "parser #{test_name}" do 
      ...
    end
  end

But I think that for start this is good enough.