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 tests #222

Merged
merged 4 commits into from
Mar 27, 2020
Merged

Added tests #222

merged 4 commits into from
Mar 27, 2020

Conversation

HarsheetKakar
Copy link
Contributor

@HarsheetKakar HarsheetKakar commented Mar 27, 2020

References to other Issues or PRs or Relevant literature

Fixes #19

Brief description of what is fixed or changed

Added tests for mary node, binomial tree node and added test_m_ary_trees.py

Other comments

Added str function for mary node else it would take it as a TreeNode and raise error because there are no left and right attributes

@codecov
Copy link

codecov bot commented Mar 27, 2020

Codecov Report

Merging #222 into master will increase coverage by 1.390%.
The diff coverage is 100.000%.

@@              Coverage Diff              @@
##            master      #222       +/-   ##
=============================================
+ Coverage   97.123%   98.513%   +1.390%     
=============================================
  Files           24        24               
  Lines         2016      2018        +2     
=============================================
+ Hits          1958      1988       +30     
+ Misses          58        30       -28     
Impacted Files Coverage Δ
pydatastructs/utils/misc_util.py 100.000% <100.000%> (+9.259%) ⬆️
pydatastructs/trees/m_ary_trees.py 90.625% <0.000%> (+56.250%) ⬆️

Impacted file tree graph

@HarsheetKakar HarsheetKakar changed the title Added tests for misc_utils.py Added tests Mar 27, 2020
@czgdp1807 czgdp1807 merged commit 571dcd1 into codezonediitj:master Mar 27, 2020
@HarsheetKakar HarsheetKakar deleted the test branch March 27, 2020 14:25
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.

Track coverage
3 participants