diff --git a/scripts/notify_backup.rb b/scripts/notify_backup.rb index da95126..f493b96 100644 --- a/scripts/notify_backup.rb +++ b/scripts/notify_backup.rb @@ -69,7 +69,9 @@ class PayloadBuilder machine: @options[:machine], destination: @options[:destination], status: @options[:status], - bytes_transferred: ByteFormatter.format(@options[:bytes]), + # Raw integer: the backup_runs datatable column is type number; + # human formatting belongs in the reporter, not the payload. + bytes_transferred: @options[:bytes], snapshot_id: @options[:snapshot], log_path: @options[:log_path], notes: @options[:notes] diff --git a/scripts/notify_backup_test.rb b/scripts/notify_backup_test.rb index 30e340b..3559ff9 100644 --- a/scripts/notify_backup_test.rb +++ b/scripts/notify_backup_test.rb @@ -43,9 +43,9 @@ class PayloadBuilderTest < Minitest::Test assert_match(/\A\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}Z\z/, payload[:timestamp]) end - def test_formats_bytes_to_human_readable + def test_sends_raw_integer_bytes payload = PayloadBuilder.new(base_options.merge(bytes: 1_073_741_824)).build - assert_equal '1.00 GB', payload[:bytes_transferred] + assert_equal 1_073_741_824, payload[:bytes_transferred] end def test_omits_nil_optional_fields @@ -64,7 +64,7 @@ class PayloadBuilderTest < Minitest::Test notes: 'snapshot abc12345 saved' ) payload = PayloadBuilder.new(opts).build - assert_equal '1.00 MB', payload[:bytes_transferred] + assert_equal 1_048_576, payload[:bytes_transferred] assert_equal 'abc12345', payload[:snapshot_id] assert_equal '/home/jared/.local/log/restic/2026-05-07.log', payload[:log_path] assert_equal 'snapshot abc12345 saved', payload[:notes]