Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Magic folder integration tests #322

Merged

Conversation

meejah
Copy link
Contributor

@meejah meejah commented Aug 29, 2016

This ports all the magic-folders integration-style tests (from check_magicfolder_smoke.py) to a proper event-runner, etc. Uses py.test fixtures to build up the introducer, storage servers and alice + bob clients. It also runs a flog_tool gather instance to grab all logs from all the running services and dumps them at the end of the run.

@coveralls
Copy link
Collaborator

coveralls commented Aug 29, 2016

Coverage Status

Coverage decreased (-0.01%) to 92.184% when pulling 7c3ea8c on meejah:magic-folder-integration-tests-pytest.2 into 2d358f2 on tahoe-lafs:master.

@coveralls
Copy link
Collaborator

coveralls commented Aug 29, 2016

Coverage Status

Coverage decreased (-0.01%) to 92.184% when pulling c141d20 on meejah:magic-folder-integration-tests-pytest.2 into 2d358f2 on tahoe-lafs:master.

try:
mkdir(join(temp_dir, 'magic-alice'))
except OSError:
pass
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Put this before creating the process.

@meejah
Copy link
Contributor Author

meejah commented Aug 30, 2016

Note to self: try on windows-vm to see if the appveyor failure is "real" or not.

@warner
Copy link
Member

warner commented Aug 30, 2016

I've kicked appveyor to re-build that one, maybe it was something transient.

@meejah
Copy link
Contributor Author

meejah commented Aug 30, 2016

Okay, I've addressed the comments and this is ready to go. Note that two of the tests will fail until the PR for 2814 is landed (if @warner agrees I can squash before merge if you want, perhaps after 2814 is landed so I can rebase it as well).

@coveralls
Copy link
Collaborator

coveralls commented Aug 31, 2016

Coverage Status

Coverage increased (+0.004%) to 92.203% when pulling c1f0bc5 on meejah:magic-folder-integration-tests-pytest.2 into 2d358f2 on tahoe-lafs:master.

@coveralls
Copy link
Collaborator

coveralls commented Aug 31, 2016

Coverage Status

Coverage decreased (-0.007%) to 92.191% when pulling e05c06b on meejah:magic-folder-integration-tests-pytest.2 into 2d358f2 on tahoe-lafs:master.

@warner
Copy link
Member

warner commented Aug 31, 2016

looks reasonable to me, squash at will

This introduces a py.test-based integration suite (currently just
containing magic-folder end-to-end tests). Also adds a tox environment
("integration") to run them.

The test setup is:

 - a "flogtool gather" instance
 - an Introducer
 - five Storage nodes
 - Alice and Bob client nodes
 - Alice and Bob have paired magic-folders
@meejah meejah force-pushed the magic-folder-integration-tests-pytest.2 branch from e05c06b to 72f17af Compare August 31, 2016 02:50
@meejah
Copy link
Contributor Author

meejah commented Aug 31, 2016

Okay, I rebased to master and just squashed everything into one commit.

@coveralls
Copy link
Collaborator

coveralls commented Aug 31, 2016

Coverage Status

Coverage decreased (-0.003%) to 92.208% when pulling 72f17af on meejah:magic-folder-integration-tests-pytest.2 into 0670144 on tahoe-lafs:master.

@warner warner merged commit 72f17af into tahoe-lafs:master Aug 31, 2016
@meejah meejah deleted the magic-folder-integration-tests-pytest.2 branch September 5, 2016 03:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants