Hi Iwamoto-San,
Thank you very much for your advise!
On 2017年07月04日 12:01, IWAMOTO Toshihiro wrote:
At Mon, 3 Jul 2017 10:40:39 +0900,
Iwase Yusuke wrote:
Hi Fujita-San,
On 2017年07月02日 20:15, FUJITA Tomonori wrote:
I did a clean revert because I want to avoid experiments just before
the new release.
The current size is 1MB, right? Is it the problem?
No, the current size has no problem.
I just aimed to avoid the redundant files (e.g., .pcap, .json) which are
not needed for Ryu users and whose size will become larger if we add new
tests to cover non-tested modules.
How about keeping *.py files and just moving packet_data and jsons out
of ryu/tests?
In that case, some unit tests should be failed even if those tests are
included in Ryu module.
Mostly, that has no problem though.
FYI, as the current installed files size, "ryu/tests" modules amount to
19 MB of "ryu" module (29MB).
Most of these files are not widely used by Ryu users...
Or, regenerating json files with separators=(',', ':') and indent=None
gives 7.5MB smaller json files.
$ du -h -d 1 ryu
772K ryu/app
2.8M ryu/ofproto
16K ryu/contrib
3.2M ryu/lib
52K ryu/base
224K ryu/controller
2.4M ryu/services
128K ryu/topology
19M ryu/tests
88K ryu/cmd
32K ryu/__pycache__
29M ryu
Thanks,
Iwase
On Fri, 30 Jun 2017 13:29:43 +0900
IWASE Yusuke <iwase.yusu...@gmail.com> wrote:
This patch puts some test files back which seperated by the following
commit.
[Ryu-devel] [PATCH] tests: Separate test files from Ryu module
Signed-off-by: IWASE Yusuke <iwase.yusu...@gmail.com>
---
{tests/integrated/common => ryu/tests/integrated}/__init__.py | 0
ryu/tests/integrated/common/__init__.py | 0
{tests => ryu/tests}/integrated/common/docker_base.py | 0
{tests => ryu/tests}/integrated/common/quagga.py | 0
{tests => ryu/tests}/integrated/common/ryubgp.py | 0
tests/integrated/bgp/base.py | 6 +++---
tests/integrated/bgp/base_ip6.py | 6 +++---
tests/integrated/bgp/test_basic.py | 3 ++-
tests/integrated/bgp/test_ip6_basic.py | 3 ++-
9 files changed, 10 insertions(+), 8 deletions(-)
rename {tests/integrated/common => ryu/tests/integrated}/__init__.py (100%)
create mode 100644 ryu/tests/integrated/common/__init__.py
rename {tests => ryu/tests}/integrated/common/docker_base.py (100%)
rename {tests => ryu/tests}/integrated/common/quagga.py (100%)
rename {tests => ryu/tests}/integrated/common/ryubgp.py (100%)
diff --git a/tests/integrated/common/__init__.py
b/ryu/tests/integrated/__init__.py
similarity index 100%
rename from tests/integrated/common/__init__.py
rename to ryu/tests/integrated/__init__.py
diff --git a/ryu/tests/integrated/common/__init__.py
b/ryu/tests/integrated/common/__init__.py
new file mode 100644
index 0000000..e69de29
diff --git a/tests/integrated/common/docker_base.py
b/ryu/tests/integrated/common/docker_base.py
similarity index 100%
rename from tests/integrated/common/docker_base.py
rename to ryu/tests/integrated/common/docker_base.py
diff --git a/tests/integrated/common/quagga.py
b/ryu/tests/integrated/common/quagga.py
similarity index 100%
rename from tests/integrated/common/quagga.py
rename to ryu/tests/integrated/common/quagga.py
diff --git a/tests/integrated/common/ryubgp.py
b/ryu/tests/integrated/common/ryubgp.py
similarity index 100%
rename from tests/integrated/common/ryubgp.py
rename to ryu/tests/integrated/common/ryubgp.py
diff --git a/tests/integrated/bgp/base.py b/tests/integrated/bgp/base.py
index 9d05707..26fa396 100644
--- a/tests/integrated/bgp/base.py
+++ b/tests/integrated/bgp/base.py
@@ -20,9 +20,9 @@ import logging
import sys
import unittest
-from tests.integrated.common import docker_base as ctn_base
-from tests.integrated.common import ryubgp
-from tests.integrated.common import quagga
+from ryu.tests.integrated.common import docker_base as ctn_base
+from ryu.tests.integrated.common import ryubgp
+from ryu.tests.integrated.common import quagga
LOG = logging.getLogger(__name__)
diff --git a/tests/integrated/bgp/base_ip6.py b/tests/integrated/bgp/base_ip6.py
index 8d3ef41..be26faf 100644
--- a/tests/integrated/bgp/base_ip6.py
+++ b/tests/integrated/bgp/base_ip6.py
@@ -20,9 +20,9 @@ import logging
import sys
import unittest
-from tests.integrated.common import docker_base as ctn_base
-from tests.integrated.common import ryubgp
-from tests.integrated.common import quagga
+from ryu.tests.integrated.common import docker_base as ctn_base
+from ryu.tests.integrated.common import ryubgp
+from ryu.tests.integrated.common import quagga
LOG = logging.getLogger(__name__)
diff --git a/tests/integrated/bgp/test_basic.py
b/tests/integrated/bgp/test_basic.py
index 9ef430c..5817d44 100644
--- a/tests/integrated/bgp/test_basic.py
+++ b/tests/integrated/bgp/test_basic.py
@@ -18,7 +18,8 @@ from __future__ import absolute_import
import time
-from tests.integrated.common import docker_base as ctn_base
+from ryu.tests.integrated.common import docker_base as ctn_base
+
from . import base
diff --git a/tests/integrated/bgp/test_ip6_basic.py
b/tests/integrated/bgp/test_ip6_basic.py
index a92e4ed..40461a5 100644
--- a/tests/integrated/bgp/test_ip6_basic.py
+++ b/tests/integrated/bgp/test_ip6_basic.py
@@ -18,7 +18,8 @@ from __future__ import absolute_import
import time
-from tests.integrated.common import docker_base as ctn_base
+from ryu.tests.integrated.common import docker_base as ctn_base
+
from . import base_ip6 as base
--
2.7.4
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Ryu-devel mailing list
Ryu-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ryu-devel
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Ryu-devel mailing list
Ryu-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ryu-devel
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Ryu-devel mailing list
Ryu-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ryu-devel
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Ryu-devel mailing list
Ryu-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ryu-devel
