[Test] Integration tests

[hrishikesh-dahiya]

Description

• Integrations tests for generate.py, keyword_clustering.py, data_cleanup.py
• Added params to generate.py and add the documentation as well
• Fixes [Test] Integration Tests for lib/input/ #155 and [Test] Integration Tests for lib/analysis/author/ #152

Motivation

• To increase the code coverage

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 not work as expected)

Checks

• Squash your commits down to one, or a few, discreet changesets before submitting a pull request
• Read the contributing guide
• PR should be motivated, i.e. what does it fix, why, and if relevant how
• If possible, include an example in the description, that could help all readers (including project members) to get a better picture of the change
• Avoid other runtime dependencies unless absolutely necessary
• Rebase your commit history so that each commit is meaningful and states the intention behind the change
• Ensure that your pull request follows the style guidelines
• At least one commit should mention Fixes #152 and #155 if relevant

Notify reviewers

@prasadtalasila

[codecov-io]

Codecov Report

Merging #158 into development will increase coverage by 5.15%.
The diff coverage is 100%.

@@               Coverage Diff               @@
##           development     #158      +/-   ##
===============================================
+ Coverage        14.23%   19.38%   +5.15%     
===============================================
  Files               42       45       +3     
  Lines             2958     2992      +34     
===============================================
+ Hits               421      580     +159     
+ Misses            2537     2412     -125

Impacted Files	Coverage Δ
...ion_test/lib/input/mbox/test_keyword_clustering.py	100% <100%> (ø)
...st/integration_test/lib/input/test_data_cleanup.py	100% <100%> (ø)
lib/analysis/author/graph/generate.py	100% <100%> (+100%)	⬆️
...on_test/lib/analysis/author/graph/test_generate.py	100% <100%> (ø)
lib/input/mbox/keyword_clustering.py	21.92% <0%> (+21.92%)	⬆️
lib/input/data_cleanup.py	68.88% <0%> (+68.88%)	⬆️
... and 1 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e4030bc...40ad6f6. Read the comment docs.

[hrishikesh-dahiya]

@prasadtalasila any solution to codeclimate issue for now? Because we need all 5 param for testing it. I think refactoring work can be done later.

[prasadtalasila]

please use positional arguments here.

[hrishikesh-dahiya]

get_lone_author_threads() is a function of read.py, which was to be refactored afterward.

[prasadtalasila]

short test data files are preferable to long ones. If you can reduce the number of entries in this file and still have the test coverage, that would be a good thing.

[hrishikesh-dahiya]

Data input size reduced for test_data_cleanup.py.
nx.is_isomorphic(G1, G2) used in test_generate.py for asserting two graphs.
nx.read_pajek(filename) used to read a .net file to a graph.
Previous tests' output files added for reference.
Travis CI Build Passing, same codeclimate issue(to be refactored later).

[prasadtalasila]

@hrishikesh-dahiya one minor change in the sphinx docs

[prasadtalasila]

please move this description to sphinx docs of ignore_lat variable.