Don't generate huge floats in test yet

We don't deal with exponent notation.

Author: camelpunch

lib/mudbrick/parser.ex

@@ -14,6 +14,10 @@ defmodule Mudbrick.Parser do
     TJ
   }
 
+  defmodule Error do
+    defexception [:message]
+  end
+
   @doc """
   Parse Mudbrick-generated `iodata` into a `Mudbrick.Document`.
 
@@ -27,50 +31,53 @@ defmodule Mudbrick.Parser do
   """
   @spec parse(iodata()) :: Mudbrick.Document.t()
   def parse(iodata) do
-    {:ok, parsed_items, _rest, %{}, _, _} =
-      iodata
-      |> IO.iodata_to_binary()
-      |> pdf()
-
-    items = Enum.flat_map(parsed_items, &to_mudbrick/1)
-
-    page_refs = page_refs(items)
-    font_files = Enum.filter(items, &font?/1)
-    image_files = Enum.filter(items, &image?/1)
-    fonts = decompressed_resources_option(font_files, "F")
-    images = decompressed_resources_option(image_files, "I")
-    compress? = Enum.any?(items, &match?(%{value: %{compress: true}}, &1))
-
-    opts =
-      [compress: compress?] ++
-        case Enum.find(items, &metadata?/1) do
-          nil ->
-            []
-
-          metadata ->
-            xml =
-              if metadata.value.compress do
-                Mudbrick.decompress(metadata.value.data)
-              else
-                metadata.value.data
-              end
-
-            metadata(xml)
+    input = IO.iodata_to_binary(iodata)
+
+    case pdf(input) do
+      {:error, msg, rest, %{}, _things, _bytes} ->
+        raise Error, "#{msg}\n#{rest}"
+
+      {:ok, parsed_items, _rest, %{}, _, _} ->
+        items = Enum.flat_map(parsed_items, &to_mudbrick/1)
+
+        page_refs = page_refs(items)
+        font_files = Enum.filter(items, &font?/1)
+        image_files = Enum.filter(items, &image?/1)
+        fonts = decompressed_resources_option(font_files, "F")
+        images = decompressed_resources_option(image_files, "I")
+        compress? = Enum.any?(items, &match?(%{value: %{compress: true}}, &1))
+
+        opts =
+          [compress: compress?] ++
+            case Enum.find(items, &metadata?/1) do
+              nil ->
+                []
+
+              metadata ->
+                xml =
+                  if metadata.value.compress do
+                    Mudbrick.decompress(metadata.value.data)
+                  else
+                    metadata.value.data
+                  end
+
+                metadata(xml)
+            end
+
+        opts = if map_size(fonts) > 0, do: Keyword.put(opts, :fonts, fonts), else: opts
+        opts = if map_size(images) > 0, do: Keyword.put(opts, :images, images), else: opts
+
+        for page <- all(items, page_refs), reduce: Mudbrick.new(opts) do
+          acc ->
+            [_, _, w, h] = page.value[:MediaBox]
+
+            Mudbrick.page(acc, size: {w, h})
+            |> Mudbrick.ContentStream.update_operations(fn ops ->
+              operations(items, page) ++ ops
+            end)
         end
-
-    opts = if map_size(fonts) > 0, do: Keyword.put(opts, :fonts, fonts), else: opts
-    opts = if map_size(images) > 0, do: Keyword.put(opts, :images, images), else: opts
-
-    for page <- all(items, page_refs), reduce: Mudbrick.new(opts) do
-      acc ->
-        [_, _, w, h] = page.value[:MediaBox]
-
-        Mudbrick.page(acc, size: {w, h})
-        |> Mudbrick.ContentStream.update_operations(fn ops ->
-          operations(items, page) ++ ops
-        end)
+        |> Mudbrick.Document.finish()
     end
-    |> Mudbrick.Document.finish()
   end
 
   @doc """

test/mudbrick/parser/roundtrip_test.exs

@@ -11,11 +11,8 @@ defmodule Mudbrick.ParseRoundtripTest do
     check all document_options <- document_options(),
               pages_options <- list_of(page_options()),
               images_options <- list_of(image_options()),
-              max_runs: 20 do
-      doc =
-        document_options
-        |> new()
-
+              max_runs: 75 do
+      doc = new(document_options)
       images = Keyword.get(document_options, :images, %{})
 
       input =

test/test_helper.exs

@@ -184,11 +184,11 @@ defmodule Mudbrick.TestHelper do
   end
 
   def coords do
-    {float(), float()}
+    {float(max: 999), float(max: 999)}
   end
 
   def non_negative_coords do
-    {float(min: 0), float(min: 0)}
+    {float(min: 0, max: 999), float(min: 0, max: 999)}
   end
 
   def scale do