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

Added pytest for Mock Network Driver #50

Closed
wants to merge 144 commits into from
Closed

Added pytest for Mock Network Driver #50

wants to merge 144 commits into from

Conversation

Hugh472
Copy link
Member

@Hugh472 Hugh472 commented Apr 14, 2017

I wrote a pytest based on Sarthak's Mock Network driver, and redesigned it to be consistent with Dan's pytest format in order to develop a standard for our code base.

sarthaksharmalive and others added 30 commits March 9, 2017 11:58
adding test-quads-local.sh from course repo
committing testing.txt for git workflow demo
added testing.txt for git workflow
saacg and others added 20 commits April 11, 2017 17:52
added .DS_Store to gitignore
Created a pytest for mock interface driver and running into issues with init, so pushing to get remote to get more eyes on it.
This is the PyTest that can be used to test the MockInventory Driver for Demo4. Pushing up for better testing from others.
PyTest for MockInventoryDriver for Demo4
Dan and I further discussed Pytesting and fixture use, and I
edited my pytest to look more like his.  In addition there was an
issue with the yaml config file, it didn't update when I merged class
to my repository for some reason, so I fixed that to make it consistent
 with the code base
@Hugh472 Hugh472 requested a review from saacg April 14, 2017 19:25
saacg and others added 3 commits April 14, 2017 16:41
…InventoryDriver.py to reflect upstream changes to lib/libquads.py in update_clouds.py

modified update_clouds function in QuadsNativeInventoryDriver to match upstream/master lib/libquads.py

updated MockInventoryDriver.py to reflect upstream changes to lib/libquads.py update_cloud
updated update_cloud method in QuadsNativeInventoryDriver.py and Mock…
@portante portante self-assigned this Apr 19, 2017
@portante portante self-requested a review April 19, 2017 14:39
@portante portante changed the base branch from demo-5-branch to master April 21, 2017 15:40
@portante
Copy link
Member

I merged this commit manually for Kyle, since he is traveling. His authorship is maintained in the merged commit, e874800.

Closing this PR. We can still refer back to closed PRs to see what was done.

@portante portante closed this Apr 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants