spreewald 4.6.2 → 4.6.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a8b36c0e7f58e2fdc53690d9bed5690077215e16f874cf953d04e3bc96eb6ad5
-  data.tar.gz: 5be73a131b725ed532f318f8dc373f2758b29620ad11a4e83154ba068b754a5f
+  metadata.gz: f21efaeee956e332f09bf91655335cfc24bcf5f5a46403c11ea6fa4d2f390b25
+  data.tar.gz: '09cae0432a1190dbfc515456705b6977ca4320c688e46ee5e1307a8ca07f9f69'
 SHA512:
-  metadata.gz: c0ccd9c02ae206858b788f96d8a5a79144046e64cc8164e577fd81c4b02e1eccb94fd241295ddb29827374115c83d072b24e2818d3aea0c42685ec907dd874e7
-  data.tar.gz: a8ecc4aff210a281a7e077ea6e7c9200ab402ce5e2171db3e9c4dee251adad36f834227f5d70c27ace7f697df1dc71b40556e5ef6fbed2b7e9e3bc2eb1805d3f
+  metadata.gz: 295f1e8f0ea1b1b544d622005e020b8ba168eb27612077d55a58613162eb4ee65c0e32ba287ddef7f0fc58408817f48c3093e9b6c8ff94603076de3878ceccfc
+  data.tar.gz: d7ab22c6640d44c85c5b18f7c9e1028194eadd284d872aedf8ed56c64423a2e0409e8e545b2fbacf782d89ef219292797a5dc7a583494dcd5ce54c7bb7790399

data/CHANGELOG.md

@@ -3,6 +3,9 @@ All notable changes to this project will be documented in this file.
 
 This project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
+## 4.6.3
+- Fix email steps not showing the attachments 
+
 ## 4.6.2
 - Fix uninitialized constant `XPath::HTML` when using `When I fill in "..." with "..." inside any "..."` ([#210](https://github.com/makandra/spreewald/issues/210))
 

data/Gemfile.ruby266.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    spreewald (4.6.2)
+    spreewald (4.6.3)
       capybara
       cucumber
       cucumber_priority (>= 0.3.0)

data/Gemfile.ruby320.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    spreewald (4.6.2)
+    spreewald (4.6.3)
       capybara
       cucumber
       cucumber_priority (>= 0.3.0)

data/lib/spreewald_support/mail_finder.rb

@@ -24,7 +24,6 @@ class MailFinder
         body = [header, body].join("\n\n")
       end
 
-      filename_method = Rails::VERSION::MAJOR < 3 ? 'original_filename' : 'filename'
       matching_header = ActionMailer::Base.deliveries.select do |mail|
         [ conditions[:to].nil? || mail.to.include?(resolve_email conditions[:to]),
           conditions[:cc].nil? || mail.cc && mail.cc.include?(resolve_email conditions[:cc]),
@@ -32,7 +31,7 @@ class MailFinder
           conditions[:from].nil? || mail.from.include?(resolve_email conditions[:from]),
           conditions[:reply_to].nil? || mail.reply_to.include?(resolve_email conditions[:reply_to]),
           conditions[:subject].nil? || mail.subject.include?(conditions[:subject]),
-          conditions[:attachments].nil? || conditions[:attachments].split(/\s*,\s*/).sort == Array(mail.attachments).collect(&:"#{filename_method}").sort
+          conditions[:attachments].nil? || conditions[:attachments].split(/\s*,\s*/).sort == attachment_filenames(mail)
         ].all?
       end
 
@@ -64,6 +63,9 @@ class MailFinder
       header << "CC: #{mail.cc.join(', ')}\n" if mail.cc
       header << "BCC: #{mail.bcc.join(', ')}\n" if mail.bcc
       header << "Subject: #{mail.subject}\n"
+      header << "Attachments: #{attachment_filenames(mail).join(', ')}\n" if mail.attachments.any?
+
+      header
     end
 
     def email_text_body(mail, type = '')
@@ -94,6 +96,12 @@ class MailFinder
       Regexp.new(expected)
     end
 
+    private
+
+    def attachment_filenames(mail)
+      Array(mail.attachments).collect(&:filename).sort
+    end
+
   end
 end
 

data/lib/spreewald_support/version.rb

@@ -1,3 +1,3 @@
 module Spreewald
-  VERSION = '4.6.2'
+  VERSION = '4.6.3'
 end

data/spec/steps/show_me_the_mails_spec.rb

@@ -93,4 +93,26 @@ describe Spreewald::Steps::ShowMeTheMails do
       expect { step.run }.to output(expected_output).to_stdout
     end
   end
+
+  context "with attachments" do
+    it 'includes the filenames of the attachments' do
+      mail = Mail.new do
+        attachments['attached_file.pdf'] = File.open("tests/shared/public/fixture_files/attachment.pdf") {}
+        attachments['other_attached_file.pdf'] = File.open("tests/shared/public/fixture_files/attachment.pdf") {}
+      end
+
+      expected_output = <<~TXT
+        E-Mail #0
+        --------------------------------------------------------------------------------
+        From: 
+        Subject: 
+        Attachments: attached_file.pdf, other_attached_file.pdf
+        --------------------------------------------------------------------------------
+
+      TXT
+      step = Spreewald::Steps::ShowMeTheMails.new([mail], true)
+      expect { step.run }.to output(expected_output).to_stdout
+    end
+  end
+
 end

data/tests/rails-7_capybara-3/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: ../..
   specs:
-    spreewald (4.6.2)
+    spreewald (4.6.3)
       capybara
       cucumber
       cucumber_priority (>= 0.3.0)

data/tests/shared/app/controllers/emails_controller.rb

@@ -67,14 +67,7 @@ class EmailsController < ApplicationController
   private
 
   def deliver(method_name)
-    case
-    when Rails.version.to_i >= 5
-      SpreewaldMailer.send(method_name).deliver
-    when Rails.version.to_i >= 3
-      Mailer.public_send(method_name).deliver
-    else
-      Mailer.public_send("deliver_#{method_name}")
-    end
+    SpreewaldMailer.send(method_name).deliver
   end
 
 end

data/tests/shared/app/models/mailer.rb

@@ -7,113 +7,60 @@ class Mailer < ActionMailer::Base
   FROM = "from@example.com"
   SUBJECT = "SUBJECT"
 
-  if Rails.version >= "3"
-
-    def email
-      attachments['attached_file.pdf'] = File.open("#{Rails.root}/public/fixture_files/attachment.pdf", "w") {}
-      mail(
-        :from => FROM,
-        :reply_to => REPLY_TO,
-        :to => TO,
-        :cc => CC,
-        :bcc => BCC,
-        :subject => SUBJECT
-      )
-    end
-
-    def email_crlf
-      email
-    end
-
-    def email_with_umlauts
-      email
-    end
-
-    def html_email_with_links
-      email
-    end
-
-    def text_email_with_links
-      email
-    end
-
-    def html_email_with_linebreaks
-      email
-    end
-
-    def html_email_with_specific_line
-      email
-    end
-
-    def text_email_with_specific_line
-      email
-    end
-
-    def html_email_for_successful_test_without_header
-      email
-    end
-
-    def text_email_for_successful_test_without_header
-      email
-    end
-
-    def html_email_for_failed_test_without_header
-      email
-    end
-
-    def text_email_for_failed_test_without_header
-      email
-    end
-
-  else
+  def email
+    attachments['attached_file.pdf'] = File.open("#{Rails.root}/public/fixture_files/attachment.pdf") {}
+    mail(
+      :from => FROM,
+      :reply_to => REPLY_TO,
+      :to => TO,
+      :cc => CC,
+      :bcc => BCC,
+      :subject => SUBJECT
+    )
+  end
 
-    def email
-      attachments['attached_file.pdf'] = File.open("#{Rails.root}/public/fixture_files/attachment.pdf", "w") {}
-      recipients TO
-      reply_to REPLY_TO
-      from FROM
-      cc CC
-      bcc BCC
-      subject SUBJECT
-      body BODY
-    end
+  def email_crlf
+    email
+  end
 
-    def email_crlf
-      email
-    end
+  def email_with_umlauts
+    email
+  end
 
-    def html_email_with_links
-      email
-    end
+  def html_email_with_links
+    email
+  end
 
-    def text_email_with_links
-      email
-    end
+  def text_email_with_links
+    email
+  end
 
-    def html_email_with_specific_line
-      email
-    end
+  def html_email_with_linebreaks
+    email
+  end
 
-    def text_email_with_specific_line
-      email
-    end
+  def html_email_with_specific_line
+    email
+  end
 
-    def html_email_for_successful_test_without_header
-      email
-    end
+  def text_email_with_specific_line
+    email
+  end
 
-    def text_email_for_successful_test_without_header
-      email
-    end
+  def html_email_for_successful_test_without_header
+    email
+  end
 
-    def html_email_for_failed_test_without_header
-      email
-    end
+  def text_email_for_successful_test_without_header
+    email
+  end
 
-    def text_email_for_failed_test_without_header
-      email
-    end
+  def html_email_for_failed_test_without_header
+    email
+  end
 
+  def text_email_for_failed_test_without_header
+    email
   end
 
 end

data/tests/shared/features/shared/email_steps.feature

@@ -297,6 +297,24 @@ Feature: Test Spreewald's email steps
         '''
       """
 
+    Then the following multiline step should succeed:
+      """
+      Then an email should have been sent with:
+        '''
+        From: from@example.com
+        Attachments: attached_file.pdf
+        '''
+      """
+
+    Then the following multiline step should fail:
+      """
+      Then an email should have been sent with:
+        '''
+        From: from@example.com
+        Attachments: not_attached_file.pdf
+        '''
+      """
+
     When I clear my emails
       And I go to "/emails/send_crlf_email"
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: spreewald
 version: !ruby/object:Gem::Version
-  version: 4.6.2
+  version: 4.6.3
 platform: ruby
 authors:
 - Tobias Kraze
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-06-07 00:00:00.000000000 Z
+date: 2024-11-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: cucumber