postmark 1.16.0 → 1.17.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 42ed88f1a2900b1aab8bfddc4f0dd0a77015bdaa221bb3d964aea4f4f2192802
-  data.tar.gz: e0881eb66ae061731ecf5ce16335da4feb29e44af8b7c57d3705d3e6c49d9a96
+  metadata.gz: 3f7b40fc4752d18a1e0c14357f256540b7324cf5eb5e919075f8970db47c88f2
+  data.tar.gz: 7169e072affbaced002b978a6b206723f6b494aa520dba8587bf01784e160139
 SHA512:
-  metadata.gz: 327637a52ec906d22bd92036eae1ea68dd1d0ea9e8c38909297855fba5cacfc21a22384ff3801961218191306ff23fc95f2eb24d48f3a7e656c28532e652fd8d
-  data.tar.gz: d82c2d97f23125a664405197d275b1f2485cdaa4b506e0598faf8e11b0820bd32b9bde4613d54a2db5abc094d4a5f6520b382be75afb8b8787cc94b50403b25a
+  metadata.gz: 5ab871fbf075102964bae2a5abb888e85514c1cd1e7d6b4a060bfeaf048ec021d0ea40ac389af785757e47b5fce9525bc2b926e7af63385cb0f2ee05d56d7890
+  data.tar.gz: 67d33ea5395c6c36d03a7081d8cf56d7517feb6877540aac7e5e612b57bd065ffc7932367ee175ea8999f125d2ead1b1377cfe4f5858f6ffcad5dfa0ce89e01f

data/.travis.yml

@@ -7,7 +7,7 @@ env:
     # POSTMARK_CI_RECIPIENT=?
     - secure: "V5GeJn1rch72qcAz/YI9tSEnZ2Zb4utWzq9ntr9gojRVFGpocgIbULmeslTKyIN7s5kF23qnhsGmPA9Olwz1ZqV9zhdWcsm4PUhUU/n3tVAnLWpUDjeneD5P+IOO6S+bCzVZX+Gk5/agBt2bQeAbVe+lVh5RZQJc3lsEvkVv7QM="
     # POSTMARK_CI_SENDER=?
-    - secure: "BjCKS4oNN1n2pQx31tA9kmfcim4BkAsO2Rg1clezEu1kEbsGzn6sjoVVeXZSHhfkZ0Gk6qW8y1fQm0gnyMQyugh8wlz8UfIS7m5CIR7ZwrWroToisbF6sHPVtoBuY8+hvtB89PXyoKKv9TBjnLEPTZFKnxP9vcYsbi45Sm06MOg="
+    - secure: "qP1h4CajeKaVL7cUF3G9cKCm+KBjcTjTKRC3uOVN+X5dgQZFAuMQ9oJ3k1868AFMEJ2qqqfLCm177SlDXxqSisR7MEcCDIchiE+gUGH0YqZKLEYONyl00ts3/A6iMkQCODEB+kzLZAlByziO3CDZ6bHeLb6f7UJ4ExCSCQMYIyc="
 language: ruby
 rvm:
   - 2.0.0
@@ -17,10 +17,12 @@ rvm:
 matrix:
   include:
     - rvm: 1.8.7
+      dist: precise
       gemfile: gemfiles/Gemfile.legacy
     - rvm: 1.9.3
       gemfile: gemfiles/Gemfile.legacy
     - rvm: jruby-19mode
-      jdk: oraclejdk8 
+      jdk: oraclejdk8
+      dist: trusty
       gemfile: gemfiles/Gemfile.legacy
 script: bundle exec rake spec

data/CHANGELOG.rdoc

@@ -1,5 +1,9 @@
 = Changelog
 
+== 1.17.0
+
+* Update sent email message properly and not altering it's Message-ID with Postmark unique message id.
+
 == 1.16.0
 
 * Added support for template pushes.

data/VERSION

@@ -1 +1 @@
-1.16.0
+1.17.0

data/lib/postmark/api_client.rb

@@ -323,7 +323,7 @@ module Postmark
 
     def update_message(message, response)
       response ||= {}
-      message['Message-ID'] = response['MessageID']
+      message['X-PM-Message-Id'] = response['MessageID']
       message.delivered = response['ErrorCode'] && response['ErrorCode'].zero?
       message.postmark_response = response
     end

data/lib/postmark/version.rb

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

data/spec/integration/api_client_hashes_spec.rb

@@ -1,12 +1,9 @@
 require 'spec_helper'
 
 describe "Sending messages as Ruby hashes with Postmark::ApiClient" do
+  let(:message_id_format) { /<.+@.+>/ }
   let(:postmark_message_id_format) { /\w{8}\-\w{4}-\w{4}-\w{4}-\w{12}/ }
-  let(:api_client) {
-    Postmark::ApiClient.new('POSTMARK_API_TEST',
-                            :http_open_timeout => 15,
-                            :http_read_timeout => 15)
-  }
+  let(:api_client) { Postmark::ApiClient.new('POSTMARK_API_TEST', :http_open_timeout => 15, :http_read_timeout => 15) }
 
   let(:message) {
     {
@@ -49,33 +46,29 @@ describe "Sending messages as Ruby hashes with Postmark::ApiClient" do
     end
 
     it 'updates a message object with Message-ID' do
-      api_client.deliver(message)[:message_id].
-          should =~ postmark_message_id_format
+      expect(api_client.deliver(message)[:message_id]).to be =~ postmark_message_id_format
     end
 
     it 'returns full Postmark response' do
-      api_client.deliver(message).should be_a Hash
+      expect(api_client.deliver(message)).to be_a Hash
     end
 
     it 'delivers a message with attachment' do
-      api_client.deliver(message_with_attachment).
-          should have_key(:message_id)
+      expect(api_client.deliver(message_with_attachment)).to have_key(:message_id)
     end
 
     it 'fails to deliver a message without body' do
-      expect { api_client.deliver(message_with_no_body) }.
-          to raise_error(Postmark::InvalidMessageError)
+      expect { api_client.deliver(message_with_no_body) }.to raise_error(Postmark::InvalidMessageError)
     end
 
     it 'fails to deliver a message with invalid To address' do
-      expect { api_client.deliver(message_with_invalid_to) }.
-          to raise_error(Postmark::InvalidMessageError)
+      expect { api_client.deliver(message_with_invalid_to) }.to raise_error(Postmark::InvalidMessageError)
     end
   end
 
   context "in batches" do
     it 'returns as many responses as many messages were sent' do
-      api_client.deliver_in_batches(valid_messages).count.should == valid_messages.count
+      expect(api_client.deliver_in_batches(valid_messages).count).to eq valid_messages.count
     end
 
     context "given custom max_batch_size" do
@@ -84,7 +77,7 @@ describe "Sending messages as Ruby hashes with Postmark::ApiClient" do
       end
 
       it 'returns as many responses as many messages were sent' do
-        api_client.deliver_in_batches(valid_messages).count.should == valid_messages.count
+        expect(api_client.deliver_in_batches(valid_messages).count).to eq valid_messages.count
       end
     end
 
@@ -95,11 +88,7 @@ describe "Sending messages as Ruby hashes with Postmark::ApiClient" do
 
     it "doesn't deliver a batch of invalid Mail::Message objects" do
       response = api_client.deliver_in_batches(invalid_messages)
-
       expect(response).to satisfy { |r| r.all? { |mr| !!mr[:error_code] } }
     end
   end
-
-
-
 end

data/spec/integration/api_client_messages_spec.rb

@@ -2,11 +2,7 @@ require 'spec_helper'
 
 describe "Sending Mail::Messages with Postmark::ApiClient" do
   let(:postmark_message_id_format) { /\w{8}\-\w{4}-\w{4}-\w{4}-\w{12}/ }
-  let(:api_client) {
-    Postmark::ApiClient.new('POSTMARK_API_TEST',
-                            :http_open_timeout => 15,
-                            :http_read_timeout => 15)
-  }
+  let(:api_client) { Postmark::ApiClient.new('POSTMARK_API_TEST', :http_open_timeout => 15, :http_read_timeout => 15) }
 
   let(:message) {
     Mail.new do
@@ -25,11 +21,7 @@ describe "Sending Mail::Messages with Postmark::ApiClient" do
     end
   }
 
-  let(:message_with_attachment) {
-    message.tap do |msg|
-      msg.attachments["test.gif"] = File.read(empty_gif_path)
-    end
-  }
+  let(:message_with_attachment) { message.tap { |msg| msg.attachments["test.gif"] = File.read(empty_gif_path) } }
 
   let(:message_with_invalid_to) {
     Mail.new do
@@ -52,57 +44,48 @@ describe "Sending Mail::Messages with Postmark::ApiClient" do
 
   context "message by message" do
     it 'delivers a plain text message' do
-      api_client.deliver_message(message).should have_key(:message_id)
+      expect(api_client.deliver_message(message)).to have_key(:message_id)
     end
 
     it 'updates a message object with Message-ID' do
-      api_client.deliver_message(message)[:message_id].
-          should =~ postmark_message_id_format
+      expect(api_client.deliver_message(message)[:message_id]).to be =~ postmark_message_id_format
     end
 
     it 'returns full Postmark response' do
-      api_client.deliver_message(message).should be_a Hash
+      expect(api_client.deliver_message(message)).to be_a Hash
     end
 
     it 'delivers a message with attachment' do
-      api_client.deliver_message(message_with_attachment).
-          should have_key(:message_id)
+      expect(api_client.deliver_message(message_with_attachment)).to have_key(:message_id)
     end
 
     it 'fails to deliver a message without body' do
-      expect { api_client.deliver_message(message_with_no_body) }.
-          to raise_error(Postmark::InvalidMessageError)
+      expect { api_client.deliver_message(message_with_no_body) }.to raise_error(Postmark::InvalidMessageError)
     end
 
     it 'fails to deliver a message with invalid To address' do
-      expect { api_client.deliver_message(message_with_invalid_to) }.
-          to raise_error(Postmark::InvalidMessageError)
+      expect { api_client.deliver_message(message_with_invalid_to) }.to raise_error(Postmark::InvalidMessageError)
     end
   end
 
   context "in batches" do
     it 'delivers a batch of valid Mail::Message objects' do
       expect { api_client.deliver_messages(valid_messages) }.
-          to change{valid_messages.all? { |m| m.delivered? }}.
-             to true
+          to change{valid_messages.all? { |m| m.delivered? }}.to true
     end
 
-    it 'updates delivered messages with Message-IDs' do
+    it 'updates delivered messages with X-PM-Message-Ids' do
       api_client.deliver_messages(valid_messages)
-
-      expect(valid_messages.all? { |m| m.message_id =~ postmark_message_id_format }).
-          to be true
+      expect(valid_messages.all? { |m| m['X-PM-Message-Id'].to_s =~ postmark_message_id_format }).to be true
     end
 
     it 'updates delivered messages with related Postmark responses' do
       api_client.deliver_messages(valid_messages)
-
-      expect(valid_messages.all? { |m| m.postmark_response["To"] == m.to[0] }).
-          to be true
+      expect(valid_messages.all? { |m| m.postmark_response["To"] == m.to[0] }).to be true
     end
 
     it 'returns as many responses as many messages were sent' do
-      api_client.deliver_messages(valid_messages).count.should == valid_messages.count
+      expect(api_client.deliver_messages(valid_messages).count).to eq valid_messages.count
     end
 
     context "given custom max_batch_size" do
@@ -112,28 +95,26 @@ describe "Sending Mail::Messages with Postmark::ApiClient" do
 
       it 'updates delivered messages with related Postmark responses' do
         api_client.deliver_messages(valid_messages)
-
-        expect(valid_messages.all? { |m| m.postmark_response["To"] == m.to[0] }).
-            to be true
+        expect(valid_messages.all? { |m| m.postmark_response["To"] == m.to[0] }).to be true
       end
 
       it 'returns as many responses as many messages were sent' do
-        api_client.deliver_messages(valid_messages).count.should == valid_messages.count
+        expect(api_client.deliver_messages(valid_messages).count).to eq valid_messages.count
       end
     end
 
     it 'partially delivers a batch of partially valid Mail::Message objects' do
       expect { api_client.deliver_messages(partially_valid_messages) }.
-          to change{partially_valid_messages.select { |m| m.delivered? }.count}.
-             to 2
+          to change{partially_valid_messages.select { |m| m.delivered? }.count}.to 2
     end
 
     it "doesn't deliver a batch of invalid Mail::Message objects" do
-      expect { api_client.deliver_messages(invalid_messages) }.
-          to change{invalid_messages.all? { |m| m.delivered? == false }}.
-             to true
+      aggregate_failures do
+        expect { api_client.deliver_messages(invalid_messages) }.
+            to change{invalid_messages.all? { |m| m.delivered? == false }}.to true
 
-      invalid_messages.should satisfy { |ms| ms.all? { |m| !!m.postmark_response }}
+        expect(invalid_messages).to satisfy { |ms| ms.all? { |m| !!m.postmark_response }}
+      end
     end
   end
 end

data/spec/integration/api_client_resources_spec.rb

@@ -8,7 +8,7 @@ describe 'Accessing server resources using the API' do
   let(:recipient) { ENV['POSTMARK_CI_RECIPIENT'] }
   let(:message) {
     {
-      :from => "tema+ci@wildbit.com",
+      :from => ENV['POSTMARK_CI_SENDER'],
       :to => recipient,
       :subject => "Mail::Message object",
       :text_body => "Lorem ipsum dolor sit amet, consectetur adipisicing elit, " \

data/spec/integration/mail_delivery_method_spec.rb

@@ -53,9 +53,9 @@ describe "Sending Mail::Messages with delivery_method Mail::Postmark" do
     expect { message.deliver }.to change{message.delivered?}.to(true)
   end
 
-  it 'updates a message object with Message-ID' do
+  it 'updates a message object with X-PM-Message-Id' do
     expect { message.deliver }.
-        to change{message['Message-ID'].to_s}.to(postmark_message_id_format)
+        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

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: postmark
 version: !ruby/object:Gem::Version
-  version: 1.16.0
+  version: 1.17.0
 platform: ruby
 authors:
 - Petyo Ivanov
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-01-23 00:00:00.000000000 Z
+date: 2019-08-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: json
@@ -146,8 +146,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: 1.3.7
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.8
+rubygems_version: 3.0.1
 signing_key: 
 specification_version: 4
 summary: Official Postmark API wrapper.