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 user tests for UserManager.py #867

Closed
wants to merge 331 commits into from
Closed

Add user tests for UserManager.py #867

wants to merge 331 commits into from

Conversation

carpecodeum
Copy link

Description

added user tests till for adding regular and normal user updating and deleting them

#851

Motivation and Context

added tests to enhance testing of existing helpers

How Has This Been Tested?

Screenshot 2020-01-22 at 1 07 16 PM

Screenshots (In case of UI changes):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

vivonk and others added 30 commits September 15, 2018 04:51
@vivonk
Merge branch 'develop' into refactor-apis-delcaration
2db2841
@sidhantnagpal
Improve badges for the repository
d740ed2
@sidhantnagpal
Standardize GitHub template files
d7db574
@vivonk
changing to uppercase
41b4181
@vivonk
Update token_utils.py
65377ed
@vivonk
Update initializer.py
64c2b06
@vivonk
Update Download.py
a5849e2
@vivonk
Update User.py
bde2126
@agentmilindu
Merge pull request scorelab#475 from vivonk/refactor-apis-delcaration
2569f91 
refactored api end point declarations
@aakashbansal
Add Download input field returns to untouched state when download is …
a55d19d 
…added
@aakashbansal
Add files via upload
621fd77
@aakashbansal
Delete ui-patch-1.gif
6e3b216
@vivonk
Merge pull request scorelab#485 from aakashbansal/ui-patch-1
42df3c0 
Download input field now goes to default state when a download link is added (fixes scorelab#484)
@aakashbansal
Merge pull request scorelab#1 from scorelab/develop
405e554 
Just pulling in the latest changes from main repo
@iammosespaulr
Merge branch 'master' into patch-6
281800d
ui: username on navbar
753f40d
readme update
3acaa93
@aakashbansal
Duplicate links removed when adding/deleting downloads
0b53d45
@gaurav-diginix
Add Figlet & -h option
5322a68
@aakashbansal
HTTP Status Code checked before locally deleting the download
637d419
@agentmilindu
Merge branch 'develop' into 456-file-sharing-system
7406b69
@agentmilindu
Merge pull request scorelab#479 from vivonk/456-file-sharing-system
5dcfefd 
File sharing system with multi threading
@championpaddler
Fix -505 Add Cd Bassa Command
6736f3d
@agentmilindu
Merge branch 'develop' into develop
865b7b2
@agentmilindu
Merge pull request scorelab#480 from sidhantnagpal/develop
8b69a86 
Improve the badges and templates for the repository
ui buid passing
b0a1b7b
added package-lock
4fb8b4c
@agentmilindu
Merge branch 'build' of https://github.com/VibhorCodecianGupta/Bassa …
fb1eec1 
…into VibhorCodecianGupta-build
@Asutosh989 @agentmilindu
Add pylint to travis CI
6cce590
@agentmilindu
Update .travis.yml
5df678f
vivonk and others added 27 commits August 8, 2019 02:37
@vivonk
Merge pull request scorelab#821 from kmehant/documentation-bassa
668dd3c 
Updated documentation for Bassa
@vivonk
Merge pull request scorelab#782 from kmehant/bassaWeb-devDockerfile
2c5724d 
Updated and working version of development dockerfile for Bassa UI
@kmehant
Remove comment
9095e4b
@vivonk
Merge pull request scorelab#797 from kmehant/prod-UI-dockerfile
30029c8 
Implemented a working production UI dockerfile for Bassa
@JaDogg
Merge pull request scorelab#754 from kmehant/improved-travis-build
871eb4d 
Improved Travis CI builds for Bassa
@vivonk
Merge branch 'develop' into demon-36-minio_bassa
e97bae9
@vivonk
Merge pull request scorelab#353 from padamchopra/database-python
cbf3b62 
Python script to setup db
@vivonk
Merge pull request scorelab#789 from kmehant/dockerfile-aria2c
b758966 
Implemented a Dockerfile for Aria2c RPC server
@demon-36
Update MinioHandler.py
f62da9a
@JaDogg
Merge pull request scorelab#793 from kmehant/docker-compose-up
5933843 
Getting Bassa ready for container based deployment using docker-compose
@kmehant @vivonk
Update docker labels, production docker-compose and simple fixes. (sc…
4ca33fd 
…orelab#824)

* Fix docker build error for Web and update docs

* Update docker labels and add required args in docker-compose

* Update production docker-compose configuration

* revert passing args in docker-compose config
@singhrohan62
made changes according to Chathu's comments and added 404 Not Found R…
54df487 
…oute component
@demon-36
fixed codacy error
43a29ca
@demon-36
fixed codacy errors
6fea789
@JaDogg
Merge pull request scorelab#808 from demon-36/demon-36-minio_bassa
a2175c8 
Integrating Bassa with Storage Server (Minio)
@singhrohan62
made changes according to Bhathiya's comments
3f7355f
@singhrohan62
fixed codacy issues by removing console statements
e42bb69
@agentmilindu
Merge pull request scorelab#817 from singhrohan62/setup-router-redux
12e45f5 
Setup redux along with redux saga for handling asynchronous requests along with the data
@JaDogg
Merge pull request scorelab#794 from scorelab/demon-36-UI_fixes
348a21b 
Fixed UI issues of Dashboard, table and admin section
@kmehant
Make documentation concise
00ce530
@agentmilindu
Merge pull request scorelab#831 from kmehant/documentation-patch
1039cdd 
Make documentation concise
@selfisekai
fix builds (scorelab#832)
28e041c
@kmehant
Merge pull request scorelab#835 from selfisekai/fix-builds
58d2e3c 
fix docker build by changing MySQL client version.
@kmehant @vivonk
Reference contribution guildlines wiki (scorelab#838)
d104d5b 
Updates in README references
@carpecodeum
initial commit
74069dd
@carpecodeum
initial commit
af5f9d3
@carpecodeum
add user tests for usermanager.py
5ccc38e
JaDogg
JaDogg requested changes Jan 22, 2020
View reviewed changes
Copy link
Collaborator

@JaDogg JaDogg left a comment

Choose a reason for hiding this comment

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

Do the merge request to develop not master

@carpecodeum
Copy link
Author

@JaDogg okay ill do that

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JaDogg JaDogg JaDogg requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.