Add basic CLI

Use symbols in notifications

Add tests

Fix test_defaults

Add test_notify

Try to use apt installed site-packages

Say yes to pew!

Mention dependencies explicitly

Add dbus to travis

Skip if no dbus

Simply use unittest instead of nose

Correct package name

Author: ashwinvis

.travis.yml

@@ -6,12 +6,18 @@ python:
   - "3.4"
   - "3.5"
 
+addons:
+  apt:
+    packages:
+      - python3-notify2
+      - python-notify2
+      - dbus
+
 install:
   - python setup.py install
-  - pip install nose
   - pip install coveralls
 script:
-  - nosetests  --with-coverage --cover-package=forex_python
+  - coverage run --source=forex_python -m unittest discover
 
 after_success:
   coveralls

forex_python/__main__.py

@@ -0,0 +1,84 @@
+"""Command line utilty for forex-python
+=======================================
+
+Inspired by another package: anshulc95/exch
+"""
+from __future__ import print_function
+
+import argparse
+import os
+import sys
+
+from . import converter
+
+
+parser = argparse.ArgumentParser(
+    formatter_class=argparse.ArgumentDefaultsHelpFormatter
+)
+parser.add_argument(
+    "-b", "--base", default="USD", help="Currency you are converting from."
+)
+parser.add_argument(
+    "-d", "--dest", default="INR", help="Currency you are converting to."
+)
+parser.add_argument(
+    "-a", "--amount", default=1.0, type=float, help="Amount to convert."
+)
+parser.add_argument(
+    "-n", "--notify", action="store_true", help="Display desktop alerts."
+)
+
+
+def symbol(currency_code):
+    sym = converter.get_symbol(currency_code)
+    if sym is not None:
+        return sym
+    else:
+        return currency_code
+
+
+def conversion_result(amount, base, converted_amount, dest, use_symbols=False):
+    if use_symbols:
+        return "{} {} = {} {}".format(
+            symbol(base), amount, symbol(dest), converted_amount
+        )
+    else:
+        return "{} {} = {} {}".format(amount, base, converted_amount, dest)
+
+
+
+def notify_posix(args):
+    try:
+        import notify2
+    except ImportError:
+        print("Requires Linux or macOS with notify2 and dbus package.")
+        raise
+    notify2.init("forex-python")
+    notification = conversion_result(
+        1.0, args.base, converter.get_rate(args.base, args.dest), args.dest,
+        True
+    )
+    n = notify2.Notification(
+        "forex-python", notification, "notification-message-im"  # Icon name
+    )
+    n.show()
+
+
+def run(args=None, output=sys.stdout):
+    args = parser.parse_args(args)
+    if args.notify:
+        if os.name == "posix":
+            notify_posix(args)
+        else:
+            raise ValueError(
+                "The option [-n] is available only for POSIX operating systems")
+    else:
+        print(
+            conversion_result(
+                args.amount,
+                args.base,
+                converter.convert(args.base, args.dest, args.amount),
+                args.dest,
+            ),
+            file=output
+        )

setup.py

@@ -1,4 +1,3 @@
-import io
 import os
 from setuptools import setup, find_packages
 
@@ -19,6 +18,17 @@
 
 """
 
+install_requires=[
+    'requests',
+    'simplejson',
+]
+
+if os.name == 'posix':
+    install_requires += [
+        'notify2',
+        'dbus-python'
+    ]
+
 setup(
     name='forex-python',
     version=VERSION,
@@ -29,10 +39,7 @@
     long_description=long_description_text,
     packages=find_packages(exclude=['tests', 'tests.*']),
     include_package_data=True,
-    install_requires=[
-        'requests',
-        'simplejson',
-    ],
+    install_requires=install_requires,
     classifiers=[
         'Intended Audience :: Developers',
         'Operating System :: OS Independent',
@@ -45,4 +52,8 @@
         'Programming Language :: Python :: 3.5',
         'Topic :: Software Development :: Internationalization',
     ],
+    entry_points={
+        'console_scripts':
+        ['forex-python=forex_python.__main__:run']
+    },
 )

tests/test_main.py

@@ -0,0 +1,29 @@
+import unittest
+import os
+from forex_python.__main__ import run, symbol
+try:
+    import dbus
+except ImportError:
+    dbus = False
+
+
+class TestCLI(unittest.TestCase):
+    """Test forex-python command-line interface."""
+
+    def test_defaults(self):
+        with open(os.devnull, "w") as null:
+            run([], output=null)
+
+    @unittest.skipIf(not dbus, "dbus is not available")
+    def test_notify(self):
+        run(["--notify"])
+
+    def test_options(self):
+        with open(os.devnull, "w") as null:
+            run(["-b", "GBP", "-d", "EUR", "-a", "121"], null)
+
+    def test_symbol(self):
+        assert symbol("EUR") == u"\u20ac"
+
+if __name__ == "__main__":
+    unittest.main()