Skip to content

Test Case For Block Info Handler and Config Functions#18

Open
sandeeplandt wants to merge 5 commits intolntdev:sandeepfrom
sandeeplandt:test_block_info
Open

Test Case For Block Info Handler and Config Functions#18
sandeeplandt wants to merge 5 commits intolntdev:sandeepfrom
sandeeplandt:test_block_info

Conversation

@sandeeplandt
Copy link
Copy Markdown
Collaborator

Test case added for block_info/handler.py and block_info/block_info.py

@sandeeplandt sandeeplandt requested a review from askmish April 13, 2018 05:30
Copy link
Copy Markdown
Collaborator

@askmish askmish left a comment

Choose a reason for hiding this comment

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

Please correct spelling and grammar in commit messages and address the review comments.

expose:
- 4004
command: nose2-3
command: nose2-3 --coverage-report term-missing
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This should probably not be checked in. Do you need to modify this?

def test_validate_hex_ve(self):
string_hex = validate_hex("test", 128)
self.assertEqual(string_hex, False)

No newline at end of file
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please remove the trailing blank lines at end of file.

-c /project/sawtooth-core/families/block_info/nose2.cfg
-v
-s /project/sawtooth-core/families/block_info/tests
test_tp_block_info
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

why was this removed.

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.

2 participants