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

Adding Testcase Description field #318

Merged
merged 8 commits into from
May 31, 2018
Merged

Conversation

sidhujasminder
Copy link
Contributor

What issues does this PR fix or reference?

Fix #309

Previous Behavior

No description Field in testcase definition file

New Behavior

Now the testcases can be defined in the following manner.

test_show_interfaces:
  - rpc: get-interface-information
  - kwargs:
      interface-name: lo0
  - iterate:
      xpath: physical-interface
      id: name
      tests:
         - no-diff: oper-status
           info: "Passed"
           err: "Failed"

test_show_interfaces_1:
  - rpc: get-interface-information
  - kwargs:
      interface-name: lo0
  - iterate:
      xpath: physical-interface
      id: name
      tests:
         - no-diff: oper-status
           info: "Passed"
           err: "Failed"
  - description: "This Test is for comparing the diff between the oper-status node in rpc get-interface-information."

**************************** Device: 10.204.49.97 ****************************
Tests Included: test_show_interfaces_1
Description: This Test is for comparing the diff between the oper-status node in rpc get-interface-information.
RPC is get-interface-information
Failed
FAIL | All "oper-status" is not same in pre and post snapshot [ 0 matched / 1 failed ]
Tests Included: test_show_interfaces
RPC is get-interface-information
Failed
FAIL | All "oper-status" is not same in pre and post snapshot [ 0 matched / 1 failed ]
------------------------------- Final Result!! -------------------------------
test_show_interfaces_1 : Failed
test_show_interfaces : Failed
Total No of tests passed: 0
Total No of tests failed: 2
Overall Tests failed!!!

Tests written?

Yes, will be given in next commit

@coveralls
Copy link

Pull Request Test Coverage Report for Build 258

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 46 unchanged lines in 2 files lost coverage.
  • Overall coverage decreased (-0.1%) to 92.525%

Files with Coverage Reduction New Missed Lines %
/home/travis/build/Juniper/jsnapy/lib/jnpr/jsnapy/jsnapy.py 1 94.62%
/home/travis/build/Juniper/jsnapy/lib/jnpr/jsnapy/check.py 45 85.56%
Totals Coverage Status
Change from base Build 257: -0.1%
Covered Lines: 2624
Relevant Lines: 2836

💛 - Coveralls

1 similar comment
@coveralls
Copy link

Pull Request Test Coverage Report for Build 258

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 46 unchanged lines in 2 files lost coverage.
  • Overall coverage decreased (-0.1%) to 92.525%

Files with Coverage Reduction New Missed Lines %
/home/travis/build/Juniper/jsnapy/lib/jnpr/jsnapy/jsnapy.py 1 94.62%
/home/travis/build/Juniper/jsnapy/lib/jnpr/jsnapy/check.py 45 85.56%
Totals Coverage Status
Change from base Build 257: -0.1%
Covered Lines: 2624
Relevant Lines: 2836

💛 - Coveralls

@coveralls
Copy link

coveralls commented May 31, 2018

Pull Request Test Coverage Report for Build 259

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 42 unchanged lines in 2 files lost coverage.
  • Overall coverage increased (+0.02%) to 92.666%

Files with Coverage Reduction New Missed Lines %
/home/travis/build/Juniper/jsnapy/lib/jnpr/jsnapy/jsnapy.py 1 94.62%
/home/travis/build/Juniper/jsnapy/lib/jnpr/jsnapy/check.py 41 86.63%
Totals Coverage Status
Change from base Build 257: 0.02%
Covered Lines: 2628
Relevant Lines: 2836

💛 - Coveralls

@vnitinv vnitinv merged commit 4eca5c8 into Juniper:master May 31, 2018
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.

Include description field for tests
3 participants