From bade09cf14c25941e1cb5896c25a50c08e5a8b20 Mon Sep 17 00:00:00 2001 From: heartsucker Date: Fri, 26 Apr 2019 19:49:32 +0200 Subject: [PATCH 1/2] enforce python3 syntax --- .travis.yml | 2 +- newsfragments/3030.other | 1 + py3_valid_syntax.txt | 197 --------------------------------------- 3 files changed, 2 insertions(+), 198 deletions(-) create mode 100644 newsfragments/3030.other delete mode 100644 py3_valid_syntax.txt diff --git a/.travis.yml b/.travis.yml index 6180b1bc0..1fdcac4f9 100644 --- a/.travis.yml +++ b/.travis.yml @@ -25,7 +25,7 @@ script: - | set -eo pipefail if [ "${T}" = "py35" ]; then - cat py3_valid_syntax.txt | xargs python3 -m compileall -f + python3 -m compileall -f . else tox -e ${T} fi diff --git a/newsfragments/3030.other b/newsfragments/3030.other new file mode 100644 index 000000000..7c7b1f0b0 --- /dev/null +++ b/newsfragments/3030.other @@ -0,0 +1 @@ +Updated CI to enforce Python3 syntax for entire repo. diff --git a/py3_valid_syntax.txt b/py3_valid_syntax.txt deleted file mode 100644 index 857915f3a..000000000 --- a/py3_valid_syntax.txt +++ /dev/null @@ -1,197 +0,0 @@ -docs/ -integration/ -misc/ -release-tools/ -setup.py -src/allmydata/_auto_deps.py -src/allmydata/blacklist.py -src/allmydata/check_results.py -src/allmydata/client.py -src/allmydata/codec.py -src/allmydata/control.py -src/allmydata/deep_stats.py -src/allmydata/dirnode.py -src/allmydata/frontends/ -src/allmydata/hashtree.py -src/allmydata/history.py -src/allmydata/immutable/checker.py -src/allmydata/immutable/downloader/ -src/allmydata/immutable/encode.py -src/allmydata/immutable/filenode.py -src/allmydata/immutable/__init__.py -src/allmydata/immutable/layout.py -src/allmydata/immutable/literal.py -src/allmydata/immutable/offloaded.py -src/allmydata/immutable/repairer.py -src/allmydata/immutable/upload.py -src/allmydata/__init__.py -src/allmydata/interfaces.py -src/allmydata/introducer/ -src/allmydata/magicfolderdb.py -src/allmydata/__main__.py -src/allmydata/monitor.py -src/allmydata/mutable/ -src/allmydata/nodemaker.py -src/allmydata/node.py -src/allmydata/scripts/admin.py -src/allmydata/scripts/backupdb.py -src/allmydata/scripts/cli.py -src/allmydata/scripts/common_http.py -src/allmydata/scripts/common.py -src/allmydata/scripts/create_node.py -src/allmydata/scripts/debug.py -src/allmydata/scripts/default_nodedir.py -src/allmydata/scripts/__init__.py -src/allmydata/scripts/magic_folder_cli.py -src/allmydata/scripts/slow_operation.py -src/allmydata/scripts/stats_gatherer.py -src/allmydata/scripts/tahoe_add_alias.py -src/allmydata/scripts/tahoe_backup.py -src/allmydata/scripts/tahoe_check.py -src/allmydata/scripts/tahoe_cp.py -src/allmydata/scripts/tahoe_daemonize.py -src/allmydata/scripts/tahoe_get.py -src/allmydata/scripts/tahoe_invite.py -src/allmydata/scripts/tahoe_ls.py -src/allmydata/scripts/tahoe_manifest.py -src/allmydata/scripts/tahoe_mkdir.py -src/allmydata/scripts/tahoe_mv.py -src/allmydata/scripts/tahoe_put.py -src/allmydata/scripts/tahoe_restart.py -src/allmydata/scripts/tahoe_run.py -src/allmydata/scripts/tahoe_start.py -src/allmydata/scripts/tahoe_status.py -src/allmydata/scripts/tahoe_stop.py -src/allmydata/scripts/tahoe_unlink.py -src/allmydata/scripts/tahoe_webopen.py -src/allmydata/stats.py -src/allmydata/storage/ -src/allmydata/test/bench_dirnode.py -src/allmydata/test/check_grid.py -src/allmydata/test/check_load.py -src/allmydata/test/check_memory.py -src/allmydata/test/check_speed.py -src/allmydata/test/cli/common.py -src/allmydata/test/cli/__init__.py -src/allmydata/test/cli/test_alias.py -src/allmydata/test/cli/test_backup.py -src/allmydata/test/cli/test_check.py -src/allmydata/test/cli/test_cli.py -src/allmydata/test/cli/test_create_alias.py -src/allmydata/test/cli/test_create.py -src/allmydata/test/cli/test_daemonize.py -src/allmydata/test/cli/test_invite.py -src/allmydata/test/cli/test_list.py -src/allmydata/test/cli/test_magic_folder.py -src/allmydata/test/cli/test_mv.py -src/allmydata/test/cli/test_put.py -src/allmydata/test/cli/test_start.py -src/allmydata/test/cli/test_status.py -src/allmydata/test/common.py -src/allmydata/test/common_util.py -src/allmydata/test/common_web.py -src/allmydata/test/eliotutil.py -src/allmydata/test/__init__.py -src/allmydata/test/mutable/ -src/allmydata/test/no_network.py -src/allmydata/test/plugins/ -src/allmydata/test/status.py -src/allmydata/test/test_auth.py -src/allmydata/test/test_backupdb.py -src/allmydata/test/test_checker.py -src/allmydata/test/test_client.py -src/allmydata/test/test_codec.py -src/allmydata/test/test_configutil.py -src/allmydata/test/test_connections.py -src/allmydata/test/test_crawler.py -src/allmydata/test/test_deepcheck.py -src/allmydata/test/test_dirnode.py -src/allmydata/test/test_download.py -src/allmydata/test/test_eliotutil.py -src/allmydata/test/test_encode.py -src/allmydata/test/test_encodingutil.py -src/allmydata/test/test_filenode.py -src/allmydata/test/test_ftp.py -src/allmydata/test/test_happiness.py -src/allmydata/test/test_hashtree.py -src/allmydata/test/test_helper.py -src/allmydata/test/test_hung_server.py -src/allmydata/test/test_i2p_provider.py -src/allmydata/test/test_immutable.py -src/allmydata/test/test_import.py -src/allmydata/test/test_inotify.py -src/allmydata/test/test_introducer.py -src/allmydata/test/test_iputil.py -src/allmydata/test/test_json_metadata.py -src/allmydata/test/test_magic_folder.py -src/allmydata/test/test_magicpath.py -src/allmydata/test/test_multi_introducers.py -src/allmydata/test/test_netstring.py -src/allmydata/test/test_node.py -src/allmydata/test/test_no_network.py -src/allmydata/test/test_observer.py -src/allmydata/test/test_repairer.py -src/allmydata/test/test_runner.py -src/allmydata/test/test_sftp.py -src/allmydata/test/test_stats.py -src/allmydata/test/test_storage_client.py -src/allmydata/test/test_storage.py -src/allmydata/test/test_system.py -src/allmydata/test/test_tor_provider.py -src/allmydata/test/test_upload.py -src/allmydata/test/test_uri.py -src/allmydata/test/test_util.py -src/allmydata/test/test_version.py -src/allmydata/test/_twisted_9607.py -src/allmydata/test/web/ -src/allmydata/unknown.py -src/allmydata/uri.py -src/allmydata/util/abbreviate.py -src/allmydata/util/assertutil.py -src/allmydata/util/base32.py -src/allmydata/util/base62.py -src/allmydata/util/cachedir.py -src/allmydata/util/configutil.py -src/allmydata/util/connection_status.py -src/allmydata/util/consumer.py -src/allmydata/util/dbutil.py -src/allmydata/util/deferredutil.py -src/allmydata/util/dictutil.py -src/allmydata/util/eliotutil.py -src/allmydata/util/fake_inotify.py -src/allmydata/util/fileutil.py -src/allmydata/util/hashutil.py -src/allmydata/util/i2p_provider.py -src/allmydata/util/idlib.py -src/allmydata/util/__init__.py -src/allmydata/util/iputil.py -src/allmydata/util/keyutil.py -src/allmydata/util/limiter.py -src/allmydata/util/log.py -src/allmydata/util/mathutil.py -src/allmydata/util/namespace.py -src/allmydata/util/netstring.py -src/allmydata/util/nummedobj.py -src/allmydata/util/observer.py -src/allmydata/util/pipeline.py -src/allmydata/util/pkgresutil.py -src/allmydata/util/pollmixin.py -src/allmydata/util/progress.py -src/allmydata/util/repeatable_random.py -src/allmydata/util/rrefutil.py -src/allmydata/util/sibpath.py -src/allmydata/util/spans.py -src/allmydata/util/statistics.py -src/allmydata/util/time_format.py -src/allmydata/util/tor_provider.py -src/allmydata/util/verlib.py -src/allmydata/util/yamlutil.py -src/allmydata/_version.py -src/allmydata/watchdog/ -src/allmydata/web/ -src/allmydata/windows/__init__.py -src/allmydata/windows/inotify.py -src/allmydata/windows/registry.py -src/allmydata/windows/tahoesvc.py -static/ -ws_client.py From bc3d48ef30477229ec6c3ae25a0543420584878c Mon Sep 17 00:00:00 2001 From: meejah Date: Wed, 8 May 2019 14:52:19 -0600 Subject: [PATCH 2/2] get rid of skipIf usage --- src/allmydata/test/cli/test_backup.py | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/src/allmydata/test/cli/test_backup.py b/src/allmydata/test/cli/test_backup.py index 65ad89891..e05e4600e 100644 --- a/src/allmydata/test/cli/test_backup.py +++ b/src/allmydata/test/cli/test_backup.py @@ -18,9 +18,6 @@ from .common import ( CLITestMixin, parse_options, ) -from ..common import ( - skipIf, -) timeout = 480 # deep_check takes 360s on Zandr's linksys box, others take > 240s @@ -447,11 +444,13 @@ class Backup(GridTestMixin, CLITestMixin, StallMixin, unittest.TestCase): return self._ignore_something_test(u"Symlink", make_symlink) - @skipIf(getattr(os, "mkfifo", None) is None, _unsupported("FIFOs")) def test_ignore_fifo(self): """ A FIFO encountered in the backed-up directory is skipped with a warning. """ + if getattr(os, "mkfifo", None) is None: + raise unittest.SkipTest(_unsupported("FIFOs")) + def make_fifo(path): # Create the thing to ignore os.makedirs(os.path.dirname(path))