Skip to content

Commit

Permalink
Merge pull request #439 from vidartf/chore-tests
Browse files Browse the repository at this point in the history
Chore cleanup + fixe tests/CI
  • Loading branch information
vidartf committed Nov 19, 2018
2 parents d44e3b5 + b669e85 commit ed222a2
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 8 deletions.
13 changes: 8 additions & 5 deletions nbdime/tests/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -280,7 +280,7 @@ def merge_validator(request, json_schema_merge):

class NBTestDataBase(object):
def __init__(self):
self.filespath = filespath()
self.filespath = os.path.join(testspath(), "files")
self.cache = {}
filenames = glob.glob(os.path.join(self.filespath, "*.ipynb"))
names = [os.path.basename(fn).replace(".ipynb", "") for fn in filenames]
Expand Down Expand Up @@ -390,14 +390,17 @@ def matching_nb_triplets(request):

_port = 62019


@fixture()
def unique_port():
def make_unique_port():
global _port
_port += 1
return _port


@fixture()
def unique_port():
return make_unique_port()


@fixture()
def reset_diff_targets():
try:
Expand Down Expand Up @@ -455,7 +458,7 @@ def _kill_nb_app():
env = os.environ.copy()
env.update({'JUPYTER_CONFIG_DIR': config_dir})

port = unique_port()
port = make_unique_port()
root_dir = str(tmpdir_factory.getbasetemp())

os.chdir(root_dir)
Expand Down
2 changes: 1 addition & 1 deletion packages/webapp/webpack.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ module.exports = {
],
},
resolve: {
// Add '.ts' and '.tsx' as resolvable extensions.
// Add '.ts' as resolvable extension.
extensions: [".webpack.js", ".web.js", ".ts", ".js"]
}

Expand Down
4 changes: 2 additions & 2 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -115,10 +115,10 @@

extras_require = setuptools_args['extras_require'] = {
'test': [
'pytest>=3.3',
'pytest>=3.6',
'pytest-cov',
'pytest-timeout',
'pytest-tornado5',
'pytest-tornado5>=2',
'jsonschema',
'mock',
'requests',
Expand Down

0 comments on commit ed222a2

Please sign in to comment.