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

add unittest TestAppBase, including setup the app and add the admin user for further test #26

Closed
wants to merge 1 commit into from

Conversation

VertexC
Copy link
Contributor

@VertexC VertexC commented Jun 24, 2019

(pipot-env) root@bowenc-MS-7A59:/usr/src/pipot/server/tests# python TestNestedMenus.py
.....
----------------------------------------------------------------------
Ran 5 tests in 0.067s

OK
(pipot-env) root@bowenc-MS-7A59:/usr/src/pipot/server/tests# python TestAppBase.py
...
----------------------------------------------------------------------
Ran 3 tests in 0.073s

OK

@pep8speaks
Copy link

pep8speaks commented Jun 24, 2019

Hello @VertexC, Thank you for updating the PR!

Line 10:1: E402 module level import not at top of file
Line 11:1: E402 module level import not at top of file
Line 12:1: E402 module level import not at top of file
Line 13:1: E402 module level import not at top of file
Line 14:1: E402 module level import not at top of file
Line 15:1: E402 module level import not at top of file
Line 16:1: E402 module level import not at top of file

Line 9:1: E402 module level import not at top of file

Line 10:1: E402 module level import not at top of file
Line 11:1: E402 module level import not at top of file
Line 12:1: E402 module level import not at top of file
Line 13:1: E402 module level import not at top of file
Line 14:1: E402 module level import not at top of file
Line 15:1: E402 module level import not at top of file
Line 16:1: E402 module level import not at top of file

Line 9:1: E402 module level import not at top of file
Line 35:21: E128 continuation line under-indented for visual indent
Line 36:21: E128 continuation line under-indented for visual indent
Line 37:21: E128 continuation line under-indented for visual indent
Line 85:21: E128 continuation line under-indented for visual indent
Line 86:21: E128 continuation line under-indented for visual indent
Line 87:21: E128 continuation line under-indented for visual indent
Line 113:13: E128 continuation line under-indented for visual indent
Line 114:13: E128 continuation line under-indented for visual indent
Line 115:13: E128 continuation line under-indented for visual indent
Line 116:13: E128 continuation line under-indented for visual indent
Line 135:13: E128 continuation line under-indented for visual indent
Line 136:13: E128 continuation line under-indented for visual indent
Line 137:13: E128 continuation line under-indented for visual indent
Line 138:13: E128 continuation line under-indented for visual indent
Line 157:13: E128 continuation line under-indented for visual indent
Line 158:13: E128 continuation line under-indented for visual indent
Line 159:13: E128 continuation line under-indented for visual indent
Line 160:13: E128 continuation line under-indented for visual indent

Comment last updated at 2019-06-26 06:11:11 UTC

@VertexC VertexC force-pushed the unittest branch 3 times, most recently from 25ead8a to c4f3c32 Compare June 24, 2019 06:56
@canihavesomecoffee
Copy link
Member

@pep8speaks suggest diff

@pep8speaks
Copy link

Here you go with the gist !

You can ask me to create a PR against this branch with those fixes. Simply comment @pep8speaks pep8ify.

@canihavesomecoffee @VertexC

Copy link
Member

@canihavesomecoffee canihavesomecoffee left a comment

Choose a reason for hiding this comment

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

Can you add a .travis.yml file so I can enable Travis?

@VertexC
Copy link
Contributor Author

VertexC commented Jun 26, 2019

Open a new PR for the feature. #27

@VertexC VertexC closed this Jun 26, 2019
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.

3 participants