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

Adds tests to increase coverage #45

Merged
merged 4 commits into from
Aug 6, 2016
Merged

Adds tests to increase coverage #45

merged 4 commits into from
Aug 6, 2016

Conversation

sopankhosla
Copy link
Member

No description provided.

@sopankhosla sopankhosla self-assigned this Aug 2, 2016
@sopankhosla sopankhosla mentioned this pull request Aug 2, 2016
Closed
@sopankhosla sopankhosla changed the title Adds tests to increase coverage (INCOMPLETE) Adds tests to increase coverage Aug 2, 2016
@coveralls
Copy link

coveralls commented Aug 2, 2016 •
edited
Loading

Coverage Status

Coverage increased (+13.9%) to 46.759% when pulling 5e09846 on sopu:test-suite into dbdaf72 on loklak:master.

@sopankhosla sopankhosla force-pushed the test-suite branch 2 times, most recently from 4a67855 to 38a75a5 Compare August 2, 2016 07:03
@coveralls
Copy link

coveralls commented Aug 2, 2016 •
edited
Loading

Coverage Status

Coverage increased (+12.04%) to 44.907% when pulling 38a75a5 on sopu:test-suite into dbdaf72 on loklak:master.

@coveralls
Copy link

coveralls commented Aug 2, 2016 •
edited
Loading

Coverage Status

Coverage increased (+31.02%) to 63.889% when pulling f4bb730 on sopu:test-suite into dbdaf72 on loklak:master.

@coveralls
Copy link

coveralls commented Aug 2, 2016 •
edited
Loading

Coverage Status

Coverage increased (+31.02%) to 63.889% when pulling 44966dc on sopu:test-suite into dbdaf72 on loklak:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+31.02%) to 63.889% when pulling b620a11 on sopu:test-suite into dbdaf72 on loklak:master.

1 similar comment
@coveralls
Copy link

coveralls commented Aug 2, 2016 •
edited
Loading

Coverage Status

Coverage increased (+31.02%) to 63.889% when pulling b620a11 on sopu:test-suite into dbdaf72 on loklak:master.

@coveralls
Copy link

coveralls commented Aug 3, 2016 •
edited
Loading

Coverage Status

Coverage increased (+36.5%) to 65.06% when pulling 5547e77 on sopu:test-suite into 8d2ba8f on loklak:master.

@coveralls
Copy link

coveralls commented Aug 3, 2016 •
edited
Loading

Coverage Status

Coverage increased (+33.7%) to 62.249% when pulling 42d2588 on sopu:test-suite into 8d2ba8f on loklak:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+33.7%) to 62.249% when pulling ec6db0f on sopu:test-suite into 8d2ba8f on loklak:master.

@coveralls
Copy link

coveralls commented Aug 3, 2016 •
edited
Loading

Coverage Status

Coverage increased (+33.7%) to 62.249% when pulling a0d1542 on sopu:test-suite into 8d2ba8f on loklak:master.

@mariobehling
Copy link
Member

@sopu Travis is failing. Please check.

@sopankhosla
Adds tests for Suggest API
937583e 
Fixes syntax errors in suggest tests

Adds count field

Removes unnecessary code
@sopankhosla
Adds tests for map and markdown APIs
26c2f9e 
Fixes testing condition
@sopankhosla
Fixes erroneous query for rendering topology of user
aabc982
@sopankhosla
Adds tests for Push and Susi API functions
bad5da8 
Sends php array to push function in tests

Fixes indentation issues for PHP
@coveralls
Copy link

coveralls commented Aug 4, 2016 •
edited
Loading

Coverage Status

Coverage increased (+32.9%) to 61.446% when pulling bad5da8 on sopu:test-suite into 8d2ba8f on loklak:master.

@sopankhosla
Copy link
Member Author

@mariobehling it is due to cURL time-out of 10 sec. Loklak server did not respond for 10sec, so these errors occured.

@sopankhosla sopankhosla changed the title (INCOMPLETE) Adds tests to increase coverage Adds tests to increase coverage Aug 4, 2016
@daminisatya daminisatya merged commit 4e33932 into loklak:master Aug 6, 2016
@daminisatya
Copy link
Member

@mariobehling These are timed out errors, and no issue with the code.
selection_274

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

@sopankhosla sopankhosla

Labels
review+
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@sopankhosla @coveralls @mariobehling @daminisatya