Skip to content
This repository was archived by the owner on Aug 10, 2022. It is now read-only.

Commit dfd3403

Browse files
committed
Make compatible with Python 2 namespace package
1 parent 352135f commit dfd3403

File tree

12 files changed

+20
-55
lines changed

12 files changed

+20
-55
lines changed

eyes_core/applitools/__init__.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
__path__ = __import__('pkgutil').extend_path(__path__, __name__) # type: ignore

eyes_core/applitools/core/logger.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,6 @@
1010
import functools
1111
import typing as tp
1212

13-
from applitools.core.utils.compat import ABC
14-
1513

1614
_DEFAULT_EYES_LOGGER_NAME = 'eyes'
1715
_DEFAULT_EYES_FORMATTER = logging.Formatter('%(asctime)s [%(levelname)s] %(name)s: %(message)s')
@@ -22,7 +20,7 @@
2220
__all__ = ('StdoutLogger', 'FileLogger', 'NullLogger')
2321

2422

25-
class _Logger(ABC):
23+
class _Logger(object):
2624
"""
2725
Simple logger. Supports only info and debug.
2826
"""

eyes_core/applitools/core/positioning/position_provider.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33

44
import attr
55

6-
from applitools.core import Point
6+
from applitools.core.geometry import Point
77
from applitools.core.utils.compat import ABC
88

99
if tp.TYPE_CHECKING:
@@ -63,7 +63,8 @@ class InvalidPositionProvider(PositionProvider):
6363
def get_current_position(self):
6464
raise NotImplementedError("This class does not implement methods!")
6565

66-
def set_position(self, location: Point):
66+
def set_position(self, location):
67+
# type: (Point) -> None
6768
raise NotImplementedError("This class does not implement methods!")
6869

6970
def get_entire_size(self):

eyes_core/applitools/core/positioning/region_provider.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,11 +2,11 @@
22

33
import attr
44

5-
from applitools.core import Region
5+
from applitools.core.geometry import Region
66
from applitools.core.utils import ABC
77

88
if tp.TYPE_CHECKING:
9-
from applitools.core import EyesScreenshot
9+
from applitools.core.capture import EyesScreenshot
1010

1111
__all__ = ('RegionProvider', 'NullRegionProvider', 'NULL_REGION_INSTANCE')
1212

eyes_core/applitools/core/utils/general_utils.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,14 @@
11
"""
22
General purpose utilities.
33
"""
4-
from __future__ import absolute_import
5-
64
import itertools
75
import json
86
import time
97
import types
108
import typing as tp
119
from datetime import timedelta, tzinfo
1210

13-
from applitools.core import logger
11+
from .. import logger
1412
from .compat import urlparse
1513

1614
if tp.TYPE_CHECKING:

eyes_core/setup.py

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,8 @@
1-
import os
21
import re
32
import codecs
43
from os import path
54

6-
from setuptools import setup
7-
8-
try:
9-
from setuptools import find_namespace_packages
10-
except ImportError:
11-
raise ImportError("Please update your version of setuptools: pip install -U setuptools")
5+
from setuptools import setup, find_packages
126

137
here = path.abspath(path.dirname(__file__))
148

@@ -32,7 +26,7 @@ def get_version(package_name):
3226
setup(
3327
name='eyes_core',
3428
version=get_version('core'),
35-
packages=find_namespace_packages(include=['applitools.*'], exclude=('tests',)),
29+
packages=find_packages(include=['applitools.*'], exclude=('tests',)),
3630
url='http://www.applitools.com',
3731
license='Apache License, Version 2.0',
3832
author='Applitools Team',

eyes_images/applitools/__init__.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
__path__ = __import__('pkgutil').extend_path(__path__, __name__) # type: ignore

eyes_images/applitools/images/target.py

Lines changed: 4 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@
33
import typing as tp
44

55
from PIL import Image
6-
from multimethod import multidispatch
76

87
from applitools.core import RegionProvider
98
from applitools.core.utils import image_utils
@@ -126,28 +125,14 @@ def timeout(self, timeout):
126125

127126
def image(self, image_or_path):
128127
# type: (tp.Union[Image.Image, tp.Text]) -> Target
129-
self._image = _image_dispatch(image_or_path)
128+
if isinstance(image_or_path, Image.Image):
129+
self._image = image_or_path
130+
elif isinstance(image_or_path, str) or isinstance(image_or_path, tp.Text):
131+
self._image = image_utils.image_from_file(image_or_path)
130132
return self
131133

132134
def region(self, image_or_path, rect):
133135
# type: (tp.Union[Image.Image, tp.Text], Region) -> Target
134136
target = self.image(image_or_path)
135137
target._target_region = rect
136138
return target
137-
138-
139-
@multidispatch
140-
def _image_dispatch(image):
141-
raise TypeError('Not supported type.')
142-
143-
144-
@_image_dispatch.register(Image.Image) # type: ignore
145-
def __image_dispatch(image):
146-
return image
147-
148-
149-
@_image_dispatch.register(tp.Text) # type: ignore
150-
@_image_dispatch.register(str)
151-
def __image_dispatch(image_path):
152-
image = image_utils.image_from_file(image_path)
153-
return image

eyes_images/setup.py

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,8 @@
1-
import os
21
import re
32
import codecs
43
from os import path
54

6-
from setuptools import setup
7-
8-
try:
9-
from setuptools import find_namespace_packages
10-
except ImportError:
11-
raise ImportError("Please update your version of setuptools: pip install -U setuptools")
5+
from setuptools import setup, find_packages
126

137
here = path.abspath(path.dirname(__file__))
148

@@ -32,7 +26,7 @@ def get_version(package_name):
3226
setup(
3327
name='eyes_images',
3428
version=get_version('images'),
35-
packages=find_namespace_packages(include=['applitools.*'], exclude=('tests',)),
29+
packages=find_packages(include=['applitools.*'], exclude=('tests',)),
3630
url='http://www.applitools.com',
3731
license='Apache License, Version 2.0',
3832
author='Applitools Team',
@@ -55,7 +49,6 @@ def get_version(package_name):
5549
install_requires=[
5650
'eyes-core=={}'.format(get_version('images')),
5751
'Pillow>=5.0.0',
58-
'multimethod>=0.7.1',
5952
'typing>=3.5.2; python_version<="3.4"',
6053
],
6154
package_data={
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
__path__ = __import__('pkgutil').extend_path(__path__, __name__) # type: ignore

0 commit comments

Comments
 (0)