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.

Sign up for GitHub

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

Jump to bottom

Add tests for cleanse #5499

Merged
merged 14 commits into from
Sep 25, 2017
Merged

Add tests for cleanse #5499

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
72b78b9
Add 2 basic tests for cleanse (admin only and admin pass with output)
pwalczysko Sep 8, 2017
2b9f5a3
Merge remote-tracking branch 'origin/develop' into cleanse-test
pwalczysko Sep 8, 2017
d90d2f4
flake8 fix
pwalczysko Sep 8, 2017
e9e4a37
Replace double slashes, use omero.managed.dir to compare
pwalczysko Sep 12, 2017
e30ab4c
Compare the whole string, also the start
pwalczysko Sep 12, 2017
4d69e5d
Add test for renaming the original file and subsequent run of cleanse
pwalczysko Sep 14, 2017
22a5fa5
Delete the original file after renaming, then run cleanse and check
pwalczysko Sep 15, 2017
e401afe
Add checks for logfile, improve readability, clean code
pwalczysko Sep 18, 2017
7aaa2b0
Add helper method make_path, define data_dir and mrepo_dir in setup
pwalczysko Sep 18, 2017
9f79786
Add Restricted Admins test for cleanse
pwalczysko Sep 19, 2017
af4269d
Add comments to privileges parameter of new_user method
pwalczysko Sep 20, 2017
b0ca085
Make changes cf. Josh's comments
pwalczysko Sep 22, 2017
08afcb5
Extend CLITest class to run the test as Full Admin, not root
pwalczysko Sep 23, 2017
a0dfa53
Update doc string to use () rather than []
joshmoore Sep 25, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
2 changes: 1 addition & 1 deletion components/tools/OmeroPy/src/omero/testlib/__init__.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -538,7 +538,7 @@ def new_user(cls, group=None, perms=None,
:system: If user is to be a system group member
:privileges: If system group member is to have privileges
privileges=None gives all privileges (full admin)
privileges=[] gives no privileges
privileges=() gives no privileges
Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, that is how it should be, consistent. I forgot about this one, thank you @joshmoore

"""

if not cls.root:
Expand Down