Merge pull request #136 from splitio/development

Development

Author: mredolatti

.github/pull_request_template.md

@@ -8,7 +8,8 @@
 * Bullet 2
 
 ## How to test new changes?
-*
+* python setup.py test to test everythin
+* pytest <SPECIFIC_FILE> to test a file in particular (requires pytest, pytest-cov & pytest-mock to be installed)
 
 ## Extra Notes
 * Bullet 1

.travis.yml

@@ -2,15 +2,14 @@ language: python
 
 python:
   - "2.7"
+  - "3.6"
 
 services:
   - redis-server
 
 install:
-  - pip install --upgrade setuptools
-  - pip install redis>=2.6
-  - pip install jsonpickle>=0.9.3
-  - pip install uwsgi>=2.0.0
+  - pip install -U setuptools pip
+  - pip install -e .[cpphash,redis,uwsgi]
 
 script:
-  - python setup.py nosetests
+  - python setup.py test

CHANGES.txt

@@ -1,3 +1,11 @@
+8.0.0 (Apr 24, 2019)
+ - Full SDK Refactor/rewrite.
+ - New block until ready behaviour.
+ - Support for async destroy.
+ - Dynamic configs.
+ - Impressions not flushing on destroy bugfix.
+ - Removed unnecessary dependencies.
+ - Test suite rewritten.
 7.0.1 (Mar 8, 2019)
  - Updated Splits refreshing rate.
  - Replaced exception log level to error level.

setup.cfg

@@ -8,17 +8,18 @@ description-file = README.md
 max-line-length=100
 exclude=tests/*
 
-[nosetests]
-verbosity=1
-detailed-errors=1
-with-coverage=1
-cover-package=splitio
-debug=nose.loader
+[aliases]
+test=pytest
+
+[tool:pytest]
+ignore_glob=./splitio/_OLD/*
+addopts = --cov=splitio --verbose
+python_classes=*Tests
 
 [build_sphinx]
 source-dir = doc/source
 build-dir  = doc/build
 all_files  = 1
 
 [upload_sphinx]
-upload-dir = doc/build/html
+upload-dir = doc/build/html

setup.py

@@ -1,13 +1,14 @@
+"""Setup module."""
 #!/usr/bin/env python
 
-from setuptools import setup
+from setuptools import setup, find_packages
 from os import path
 from sys import version_info
 
-tests_require = ['flake8', 'nose', 'coverage']
+tests_require = ['flake8', 'pytest', 'pytest-mock', 'coverage', 'pytest-cov']
 install_requires = [
-    'arrow>=0.7.0',
     'requests>=2.9.1',
+    'pyyaml>=5.1',
     'future>=0.15.2',
     'docopt>=0.6.2',
 ]
@@ -33,11 +34,11 @@
       tests_require=tests_require,
       extras_require={
           'test': tests_require,
-          'redis': ['redis>=2.10.5', 'jsonpickle>=0.9.3'],
-          'uwsgi': ['uwsgi>=2.0.0', 'jsonpickle>=0.9.3'],
+          'redis': ['redis>=2.10.5'],
+          'uwsgi': ['uwsgi>=2.0.0'],
           'cpphash': ['mmh3cffi>=0.1.4']
       },
-      setup_requires=['nose'],
+      setup_requires=['pytest-runner'],
       classifiers=[
           'Development Status :: 3 - Alpha',
           'Environment :: Console',
@@ -47,4 +48,4 @@
           'Programming Language :: Python :: 3',
           'Topic :: Software Development :: Libraries'
       ],
-      packages=['splitio', 'splitio.update_scripts', 'splitio.bin', 'splitio.hashfns'])
+      packages=find_packages())

splitio/__init__.py

@@ -1,31 +1,6 @@
 from __future__ import absolute_import, division, print_function, \
     unicode_literals
 
-from .factories import get_factory # noqa
-from .key import Key # noqa
-from .version import __version__  # noqa
-
-__all__ = ('api', 'brokers', 'cache', 'clients', 'matchers', 'segments',
-           'settings', 'splits', 'splitters', 'transformers', 'treatments',
-           'version', 'factories', 'manager')
-
-
-# Functions defined to maintain compatibility with previous sdk versions.
-# ======================================================================
-#
-# This functions are not supposed to be used directly, factory method should be
-# called instead, but since they were previously exposed, they're re-added here
-# as helper function so that if someone was using we don't break their code.
-
-def get_client(apikey, **kwargs):
-    from .clients import Client
-    from .brokers import get_self_refreshing_broker
-    broker = get_self_refreshing_broker(apikey, **kwargs)
-    return Client(broker)
-
-
-def get_redis_client(apikey, **kwargs):
-    from .clients import Client
-    from .brokers import get_redis_broker
-    broker = get_redis_broker(apikey, **kwargs)
-    return Client(broker)
+from splitio.client.factory import get_factory
+from splitio.client.key import Key
+from splitio.version import __version__