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

Teach Alice to upload file for Alice+Bob test

note: this is not yet working for the mock test; the real
test passes.

teach setup_alice_and_bob to return results in deferred

- however it is terrible that i was unable to set local vars in the callbacks
so i had to use object attributes instead.

Add cleanup to Alice+Bob test + bug fix

bug fix == don't clobber the alice bob magicfolder instances

fix the rest of the unit tests

Fix magic-folder cli tests

Add helper to setup Alice and Bob magic-folders

this will help us write unit tests with multiple writers
to the magic-folder...

Add test to list remote collective dir

here you can see that i've avoided tampering with many of our
unit tests by making collective_dirnode conditional.
instead perhaps our unit tests should fail if collective_cap
is blank.

Remove/clean code duplication for CLI unit tests

Fix magic-folder Persistence test

here we teach this test to create invite join
a magic-folder using the CLI... and then we go
on to fix various bugs.

Teach magic-folder tests to use CLI for creating magic folder config

minor corrections to use of deferred from last commit

Teach magic-folder cli tests to test join results

- test if the client's dmd's readonly cap is present in the collective
readonly cap

Change return values to constants either 0 or -1

Test return vals and fix join return val

Test that join adds tahoe config options

Clean up join tahoe config append

Add complete usage functionality to create

Append magic-folder config options to tahoe.cfg

Implement most of the join command

Finish writing the invite implementation

Fix options passed to mkdir from magic invite

- here we also clean up the magic folder cli unit tests

Sleepy time WIP almost got invite's mkdir working...

Derive our own node-url, fixes magic-folder create test

Add rough create command - test FAILS still however

Add basic test for magic-folder create command

Add boiler plate for magic-folder subcommand

here we also:
- add the synopsis for the create command
- add the argument parser for the create command

Use Daira's fix to unblock CLI progress

WIP

Change behavior upon finding missing files

- Fix test test_magic_folder
- Do not throw exception for missing files...
- Check for entry in magic folder db; only write tahoe metadata
deleted=true if magic folder db entry for file exists.

Add check_file_db_exists to backupdb api

- this backupdb utility function checks to if a file's entry
exists in the db but does not stat the file on disk or otherwise
talk to the filesystem

handle local file deletion events

Rename drop-upload to Magic Folder. fixes ticket:2405

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

Remove a debug print.

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

Remove unused code.

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

Fix test errors.

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