postmark 1.17.0 → 1.18.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3f7b40fc4752d18a1e0c14357f256540b7324cf5eb5e919075f8970db47c88f2
-  data.tar.gz: 7169e072affbaced002b978a6b206723f6b494aa520dba8587bf01784e160139
+  metadata.gz: 3483ee7403059d95be73f52b76f383260f74e5799d9e7bff9da45ebfeb354668
+  data.tar.gz: a2c1e8e611a41862c3e65ae7da38957c2441b87675f7033c3fb5154389276196
 SHA512:
-  metadata.gz: 5ab871fbf075102964bae2a5abb888e85514c1cd1e7d6b4a060bfeaf048ec021d0ea40ac389af785757e47b5fce9525bc2b926e7af63385cb0f2ee05d56d7890
-  data.tar.gz: 67d33ea5395c6c36d03a7081d8cf56d7517feb6877540aac7e5e612b57bd065ffc7932367ee175ea8999f125d2ead1b1377cfe4f5858f6ffcad5dfa0ce89e01f
+  metadata.gz: 0c8eddecfa0c578503694fb5768987bbba25a67c55d5039c34f5cbf743bbdbc0ecd42b04c4613862acb13c94a003bc4fad0d7e6b29a24391dbc790ebd04da558
+  data.tar.gz: 97e52b184eb0e4256639bf17821019f945a86770dfdc0f5f51527726aa28ef2f009de36993d203dcaf1f4cc726edc7fa279e092e156c92b155d348e599cbbcff

data/.travis.yml

@@ -5,9 +5,9 @@ env:
     # POSTMARK_ACCOUNT_API_KEY=?
     - secure: "lau1m+ws6T5LBiRXwd+Q0b20Wjh62W/nHnCmzOSZc2+V97XTldl3MCJdXNV7J6X7O9CzFabcAG8FSbGGwPRiyhY/qhxkYSc/9pWE8RUjyCjbw5q2RzZQelMKizyKU92eOjd6jQw/wDSZcww1OceW/pgdRjqEo21Pk6XH2yzLAd8="
     # POSTMARK_CI_RECIPIENT=?
-    - secure: "V5GeJn1rch72qcAz/YI9tSEnZ2Zb4utWzq9ntr9gojRVFGpocgIbULmeslTKyIN7s5kF23qnhsGmPA9Olwz1ZqV9zhdWcsm4PUhUU/n3tVAnLWpUDjeneD5P+IOO6S+bCzVZX+Gk5/agBt2bQeAbVe+lVh5RZQJc3lsEvkVv7QM="
+    - secure: "YnccT2MM+hIfwxl1WtcTk48q+D9XY7p6FTHkeJbUoaT9PFXQqXyUCQpwk5wYAwzZbVWG/CgH4PitWyRbWBP6kAqYKlsnAkQip0FNjs0DbK0SH9YvcFFSDZsTAl9yfwSxlgnBZHGZ49LO3yISMPUOdNn1QmaF4Ndpl4hiycH7UA8="
     # POSTMARK_CI_SENDER=?
-    - secure: "qP1h4CajeKaVL7cUF3G9cKCm+KBjcTjTKRC3uOVN+X5dgQZFAuMQ9oJ3k1868AFMEJ2qqqfLCm177SlDXxqSisR7MEcCDIchiE+gUGH0YqZKLEYONyl00ts3/A6iMkQCODEB+kzLZAlByziO3CDZ6bHeLb6f7UJ4ExCSCQMYIyc="
+    - secure: "Cz/KfiXkd90GQr+47LunlZnal8clrRpKmZUd9K161RYd5bfkL+P6SnEmhMprwdzomUBqIcqleR9HjmECey2V/Zd9V8vtBaLVZ5CoeMTkz3Tu6SllfmpymkcuJKf7GGizs97gmhPMlJd9iMukL6eqW4/1C9HW0cLZFB/jlzqFHpE="
 language: ruby
 rvm:
   - 2.0.0

data/lib/postmark/message_extensions/mail.rb

@@ -143,10 +143,8 @@ module Mail
       [].tap do |headers|
         self.header.fields.each do |field|
           key, value = field.name, field.value
-          next if bogus_headers.include? key.downcase
-          name = key.split(/-/).map { |i| i.capitalize }.join('-')
-
-          headers << { "Name" => name, "Value" => value }
+          next if reserved_headers.include? key.downcase
+          headers << { "Name" => key, "Value" => value }
         end
       end
     end
@@ -173,7 +171,7 @@ module Mail
       end
     end
 
-    def bogus_headers
+    def reserved_headers
       %q[
         return-path  x-pm-rcpt
         from         reply-to

data/lib/postmark/version.rb

@@ -1,3 +1,3 @@
 module Postmark
-  VERSION = '1.17.0'
+  VERSION = '1.18.0'
 end

data/spec/integration/account_api_client_spec.rb

@@ -8,12 +8,9 @@ describe 'Account API client usage' do
   let(:unique_token) { rand(36**32).to_s(36) }
   let(:unique_from_email) { ENV['POSTMARK_CI_SENDER'].gsub(/(\+.+)?@/, "+#{unique_token}@") }
 
-  it 'can be used to manage senders' do
-    new_sender = nil
-
+  it 'manage senders' do
     # create & count
-    new_sender = subject.create_sender(:name => 'Integration Test',
-                                       :from_email => unique_from_email)
+    new_sender = subject.create_sender(:name => 'Integration Test', :from_email => unique_from_email)
     expect(subject.get_senders_count).to be > 0
 
     # get
@@ -47,8 +44,7 @@ describe 'Account API client usage' do
     expect { subject.delete_sender(new_sender[:id]) }.not_to raise_error
   end
 
-  it 'can be used to manage domains' do
-    new_domain = nil
+  it 'manage domains' do
     domain_name = "#{unique_token}-gem-integration.test"
     return_path = "return.#{domain_name}"
     updated_return_path = "updated-return.#{domain_name}"
@@ -85,9 +81,7 @@ describe 'Account API client usage' do
     expect { subject.delete_domain(new_domain[:id]) }.not_to raise_error
   end
 
-  it 'can be used to manage servers' do
-    new_server = nil
-
+  it 'manage servers' do
     # create & count
     new_server = subject.create_server(:name => "server-#{unique_token}",
                                        :color => 'red')
@@ -111,5 +105,4 @@ describe 'Account API client usage' do
     # delete
     expect { subject.delete_server(new_server[:id]) }.not_to raise_error
   end
-
 end

data/spec/integration/mail_delivery_method_spec.rb

@@ -17,11 +17,7 @@ describe "Sending Mail::Messages with delivery_method Mail::Postmark" do
     end
   }
 
-  let(:tagged_message) {
-    message.tap do |m|
-      m.tag "postmark-gem"
-    end
-  }
+  let(:tagged_message) { message.tap { |m| m.tag "postmark-gem" } }
 
   let(:message_with_no_body) {
     Mail.new do
@@ -54,34 +50,36 @@ describe "Sending Mail::Messages with delivery_method Mail::Postmark" do
   end
 
   it 'updates a message object with X-PM-Message-Id' do
-    expect { message.deliver }.
-        to change{message['X-PM-Message-Id'].to_s}.to(postmark_message_id_format)
+    expect { message.deliver }.to change{message['X-PM-Message-Id'].to_s}.to(postmark_message_id_format)
   end
 
   it 'updates a message object with full postmark response' do
-    expect { message.deliver }.
-        to change{message.postmark_response}.from(nil)
+    expect { message.deliver }.to change{message.postmark_response}.from(nil)
   end
 
   it 'delivers a tagged message' do
-    expect { tagged_message.deliver }.
-        to change{message.delivered?}.to(true)
+    expect { tagged_message.deliver }.to change{message.delivered?}.to(true)
   end
 
   it 'delivers a message with attachment' do
-    expect { message_with_attachment.deliver }.
-        to change{message_with_attachment.delivered?}.to(true)
+    expect { message_with_attachment.deliver }.to change{message_with_attachment.delivered?}.to(true)
   end
 
-  it 'fails to deliver a message without body' do
-    expect { message_with_no_body.deliver! }.
-        to raise_error(Postmark::InvalidMessageError)
-    message_with_no_body.should_not be_delivered
-  end
+  context 'fails to deliver a message' do
+    it ' without body - raise error' do
+      expect { message_with_no_body.deliver! }.to raise_error(Postmark::InvalidMessageError)
+    end
 
-  it 'fails to deliver a message with invalid To address' do
-    expect { message_with_invalid_to.deliver! }.
-        to raise_error(Postmark::InvalidMessageError)
-    message_with_invalid_to.should_not be_delivered
+    it 'without body - do not deliver' do
+      expect(message_with_no_body).not_to be_delivered
+    end
+
+    it 'with invalid To address - raise error' do
+      expect { message_with_invalid_to.deliver! }.to raise_error(Postmark::InvalidMessageError)
+    end
+
+    it 'with invalid To address - do not deliver' do
+      expect(message_with_invalid_to).not_to be_delivered
+    end
   end
 end

data/spec/unit/postmark/error_spec.rb

@@ -6,7 +6,7 @@ end
 
 describe(Postmark::HttpClientError) do
   it { is_expected.to be_a(Postmark::Error) }
-  specify { expect(subject.retry?).to be true }
+  it { expect(subject.retry?).to be true }
 end
 
 describe(Postmark::HttpServerError) do
@@ -85,7 +85,7 @@ describe(Postmark::ApiInputError) do
 
       shared_examples_for 'api input error' do
         its(:status_code) { is_expected. to eq 422 }
-        specify { expect(subject.retry?).to be false }
+        it { expect(subject.retry?).to be false }
         it { is_expected.to be_a(Postmark::ApiInputError) }
         it { is_expected.to be_a(Postmark::HttpServerError) }
       end
@@ -117,7 +117,7 @@ end
 
 describe(Postmark::TimeoutError) do
   it { is_expected.to be_a(Postmark::Error) }
-  specify { expect(subject.retry?).to be true }
+  it { expect(subject.retry?).to be true }
 end
 
 describe(Postmark::UnknownMessageType) do

data/spec/unit/postmark/message_extensions/mail_spec.rb

@@ -33,25 +33,6 @@ describe Mail::Message do
     end
   end
 
-  let(:mail_message_with_bogus_headers) do
-    mail_message.header['Return-Path'] = 'bounce@wildbit.com'
-    mail_message.header['From'] = 'info@wildbit.com'
-    mail_message.header['Sender'] = 'info@wildbit.com'
-    mail_message.header['Received'] = 'from mta.pstmrk.it ([72.14.252.155]:54907)'
-    mail_message.header['Date'] = 'January 25, 2013 3:30:58 PM PDT'
-    mail_message.header['Content-Type'] = 'application/json'
-    mail_message.header['To'] = 'lenard@bigbangtheory.com'
-    mail_message.header['Cc'] = 'sheldon@bigbangtheory.com'
-    mail_message.header['Bcc'] = 'penny@bigbangtheory.com'
-    mail_message.header['Subject'] = 'You want not to use a bogus header'
-    mail_message.header['Tag'] = 'bogus-tag'
-    mail_message.header['Attachment'] = 'anydatahere'
-    mail_message.header['Allowed-Header'] = 'value'
-    mail_message.header['TRACK-OPENS'] = 'true'
-    mail_message.header['TRACK-LINKS'] = 'HtmlOnly'
-    mail_message
-  end
-
   describe '#tag' do
 
     it 'value set on tag=' do
@@ -237,11 +218,42 @@ describe Mail::Message do
   end
 
   describe "#export_headers" do
-    let(:headers) { mail_message_with_bogus_headers.export_headers }
-    let(:header_names) { headers.map { |h| h['Name'] } }
+    let(:mail_message_with_reserved_headers) do
+      mail_message.header['Return-Path'] = 'bounce@wildbit.com'
+      mail_message.header['From'] = 'info@wildbit.com'
+      mail_message.header['Sender'] = 'info@wildbit.com'
+      mail_message.header['Received'] = 'from mta.pstmrk.it ([72.14.252.155]:54907)'
+      mail_message.header['Date'] = 'January 25, 2013 3:30:58 PM PDT'
+      mail_message.header['Content-Type'] = 'application/json'
+      mail_message.header['To'] = 'lenard@bigbangtheory.com'
+      mail_message.header['Cc'] = 'sheldon@bigbangtheory.com'
+      mail_message.header['Bcc'] = 'penny@bigbangtheory.com'
+      mail_message.header['Subject'] = 'You want not to use a bogus header'
+      mail_message.header['Tag'] = 'bogus-tag'
+      mail_message.header['Attachment'] = 'anydatahere'
+      mail_message.header['Allowed-Header'] = 'value'
+      mail_message.header['TRACK-OPENS'] = 'true'
+      mail_message.header['TRACK-LINKS'] = 'HtmlOnly'
+      mail_message
+    end
+
+
+    it 'only allowed headers' do
+      headers = mail_message_with_reserved_headers.export_headers
+      header_names = headers.map { |h| h['Name'] }
 
-    specify { expect(header_names).to include('Allowed-Header') }
-    specify { expect(header_names.count).to eq 1 }
+      aggregate_failures do
+        expect(header_names).to include('Allowed-Header')
+        expect(header_names.count).to eq 1
+      end
+    end
+
+    it 'custom header character case preserved' do
+      custom_header = {"Name"=>"custom-Header", "Value"=>"cUsTomHeaderValue"}
+      mail_message.header[custom_header['Name']] = custom_header['Value']
+
+      expect(mail_message.export_headers.first).to match(custom_header)
+    end
   end
 
   describe "#to_postmark_hash" do
@@ -249,8 +261,8 @@ describe Mail::Message do
   end
 
   describe '#templated?' do
-    specify { expect(mail_message).to_not be_templated }
-    specify { expect(templated_message).to be_templated }
+    it { expect(mail_message).to_not be_templated }
+    it { expect(templated_message).to be_templated }
   end
 
   describe '#prerender' do
@@ -317,7 +329,7 @@ describe Mail::Message do
       context 'and using a non-Postmark delivery method' do
         let(:delivery_method) { Mail::SMTP }
 
-        specify { expect { rendering }.to raise_error(Postmark::MailAdapterError) }
+        it { expect { rendering }.to raise_error(Postmark::MailAdapterError) }
       end
 
       context 'and using a Postmark delivery method' do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: postmark
 version: !ruby/object:Gem::Version
-  version: 1.17.0
+  version: 1.18.0
 platform: ruby
 authors:
 - Petyo Ivanov
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-08-19 00:00:00.000000000 Z
+date: 2019-09-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: json