WIP. Sigh, I no longer remember what I was trying to do here.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

Merge pull request #184 from vu3rdd/2477.refactor-all-the-things.0

test for get_pathinfo()

test_get_pathinfo: testdir is unnecessary

Use basedir itself, instead of creating another directory inside basedir.

test_get_pathinfo: rename the identifier d to testdir

d, by convention, is used for the Twisted Deferred objects.

add more tests for get_pathinfo()

When operating on a file, test if the "file" is not a directory
or a symlink. Likewise for directory and symlink.

Add test for get_pathinfo()

Use get_pathinfo instead of separate os.path.{is*,exists} calls.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

Extend get_pathinfo to add size, ctime and mtime.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

Remove an unnecessary precondition and an incorrect comment.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

Add fileutil.get_pathinfo function.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

Extract get_metadata and get_filenode to methods of Uploader.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

Processing symlinks and special files should not be an error.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

Refactor uploader by inlining _add_file into _process._maybe_upload.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

Refactor uploader by inlining _add_file into _process._maybe_upload.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

Refactor uploader by extracting _process_child to a top-level method.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

Swap two checks to clarify which is failing.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

fix code that adds version number of deleted file to magic-folder db

enable version checking tests - currently a righteous fail

make various changes - work in progress

add version checking to alice bob test

Update magic-folder db after download

Add final download to alice bob test

- that is to say after alice deletes the file she rewrites it
and this causes bob to download this latest version

Remove old unused append to deque function from downloader

WIP to unify queue processing between uploader and downloader.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

Repair tests.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

Move uploader's _append_to_deque and _turn_deque to QueueMixin.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

Make sure that counter names and log messages include "uploader" or "downloader".

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

Trivial fix.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

remove debugging print statements

Fix alice bob test

Teach downloader to stop when told to stop

Add logic to only add object to download queue when appropriate

wip

Try to fix some tests - WIP.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

Rename and refactor path variables to be more consistent.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

WIP - fix some bugs

Refactor MagicFolder into Uploader and Downloader classes (WIP). refs ticket:2477

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

Simplify and document the type of MagicFolder._download_scan_batch.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

Remove redundant call to abspath_expanduser_unicode.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

Reduce code duplication and make sure errors from delegate commands are output.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

Change INVITE_SEPARATOR to '+'.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

Rename MagicFolderCLITestMixin to MagicFolderTestMixin.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

Fix tests.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

Add checks (that currently fail) for version in local db.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

Use relative paths in magic folder db.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

WIP

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

Fix some pyflakes errors and warnings.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

Repair backupdb.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

fix bug in test_errors and add test

- test that we report the correct error when
a write cap is passed as a collective dircap

add kw arg to _get_count for tests

make test_magic_folder use non-ascii chars in nickname

Make alice bob test use non-ascii chars in nicknames

add comments and blank lines to make setup_alice_and_bob more readable

rename to do_invite and do_join for tests

rename to do_create_magic_folder for tests

revert previous additional blank line

wrap long import line

wrap long line and spell magic folder without hyphen

Spell magic folder without the hyphen

wrap long line

replace bare asserts with usage error message

get rid of another diminish hack and force alias args to end with ':'

get rid of terrible diminish cap hack

minor corrections

slightly better error message

change spelling module variable INVITE_SEPARATOR

don't double increment version

start with version 0

simplify _db_file_is_uploaded

better handle download failures

remove superfluous plus sign

fix remote scan procedure

replace calls to filter with list comprehension

simplify _should_download helper method

rename ignore count to upload ignore count

remove superfluous constructor which the parent class already has

Make backupdb schema evolution more visible

alice bob test alice rewrite file

unlink old version of file from remote tahoe upload dirnode

for local fs scan compare local version with remote

Fix test_errors to check for exception message associated with
an invalid collective dircap.

Remove workaround for collective_dircap and repair some tests.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

Eliminate near-duplicate code in CLITestMixin.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

Fix test_create_magic_folder_service (was test_create_magic_folder).

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

Add missing Deferred creation.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

fix references to notifier

various pyflakes fixes

Add ready method to MockMagicFolder to partially fix a test.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

Fixes to Magic Folder client initialization.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>

remove some superfluous print statements

Fix Alice Bob mock test

fix test_persistence

fix some tests

Fix version deletion propagation and test

Add version to magic folder db schema

- also update handling file addition and deletion events
to utilize local and remote version numbers in the uploader...

^ untested so far althought he basic Alice + Bob test continues to pass

Add rough naive downloader + remote scan

- makes the basic naive Alice + Bob unit test pass
-  `_should_download` is currently a stub function and
should be implemented
- handling of local+remote file versions is currently faked...
and should be implemented

Rename deque to upload deque

here we also rename the pending deque set
to pending upload deque..

add partial remote scan implementation

Add magic-folder download callback

Add test for Bob download callback