Bladeren bron

Merge pull request #6 from Miksus/fix/test_pd_xlsx_attachment

Fix/test pd xlsx attachment
Mikael Koli 4 jaren geleden
bovenliggende
commit
ba860bbcee
4 gewijzigde bestanden met toevoegingen van 7 en 3 verwijderingen
  1. 2 1
      redmail/test/email/test_attachments.py
  2. 1 0
      requirements.txt
  3. 2 1
      requirements/build.txt
  4. 2 1
      requirements/coverage.txt

+ 2 - 1
redmail/test/email/test_attachments.py

@@ -135,6 +135,7 @@ def test_dict_dataframe_invalid():
 def test_dict_dataframe_excel_no_error():
     pytest.importorskip("pandas")
     pytest.importorskip("openpyxl")
+    import pandas as pd
 
     sender = EmailSender(host=None, port=1234)
     msg = sender.get_message(
@@ -146,7 +147,7 @@ def test_dict_dataframe_excel_no_error():
     payload = msg.get_payload(0)
     filename = payload.get_filename()
     data = payload.get_payload()
-    assert filename == 'myfile.html'
+    assert filename == 'myfile.xlsx'
     # Excels are harder to verify
 
 def test_dict_pil_no_error():

+ 1 - 0
requirements.txt

@@ -5,3 +5,4 @@ jinja2
 pandas
 matplotlib
 Pillow
+openpyxl

+ 2 - 1
requirements/build.txt

@@ -6,4 +6,5 @@ jinja2
 # Optional
 pandas
 matplotlib
-Pillow
+Pillow
+openpyxl

+ 2 - 1
requirements/coverage.txt

@@ -8,4 +8,5 @@ jinja2
 # Optional
 pandas
 matplotlib
-Pillow
+Pillow
+openpyxl