#14 Add beefy says, with test!
Closed 6 years ago by adamwill. Opened 6 years ago by adamwill.

file modified
+4
@@ -68,6 +68,10 @@ 

  # to re-use it

  COLLECTIONS = None

  

+ def beefy_says(text):

+     """Everything's better if Beefy says it."""

+     print("Beefy says: {0}".format(text))

+ 

  def date_check(value, fail_raise=True, out='obj'):

      """Checks whether a value is a date, and returns it if so. Valid

      dates are datetime.date instances or a string or int 'YYYYMMDD'.

file modified
+5
@@ -113,6 +113,11 @@ 

  @pytest.mark.usefixtures("clean_home")

  class TestHelpers:

      """Tests for the functions in helpers.py."""

+     def test_beefy_says(self):

+         """Test beefy_says."""

+         fedfind.helpers.beefy_says("Fedora is great!")

+         # hey, at least nothing crashed. we're good here!

+ 

      def test_date_check(self):

          """Tests for date_check."""

          invalid = 'notadate'

file modified
+1 -1
@@ -1,5 +1,5 @@ 

  [tox]

- envlist = py26,py27,py34,py35,py36,py37

+ envlist = py27,py34,py35,py36,py37

  skip_missing_interpreters=true

  [testenv]

  deps=py27,py34,py35,py36,py37: -r{toxinidir}/install.requires

What a glorious PR! Who could possibly object? (This is DEMO STUFF for a talk, do not merge).

2 new commits added

  • Add test for beefy_says
  • Add very important 'beefy_says' helper
6 years ago

1 new commit added

  • Disable Python 2.6 testing for demo purposes
6 years ago

Pull-Request has been closed by adamwill

6 years ago