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

getTestCases is returning Suite not array of TestCases? #254

Closed
terotests opened this issue Oct 26, 2018 · 5 comments · Fixed by #248
Closed

getTestCases is returning Suite not array of TestCases? #254

terotests opened this issue Oct 26, 2018 · 5 comments · Fixed by #248
Assignees

Comments

@terotests
Copy link

terotests commented Oct 26, 2018

Environment

Node version: 8
Npm version: 6.2.0
OS and version: Mac OS 10.13.6
azure-devops-node-api version: 6.6.2

Issue Description

getTestCases is returning object (which apperas to be a Suite) instead of array of TestCases

Expected behaviour

getTestCases should return array of SuiteTestCase

Actual behaviour

Returns Object corresponding to the Suite, for example

{ id: 553,
   name: '...',
   url: 'https://dev.azure.com/project/projectname/_apis/test/Plans/552/Suites/553',

Steps to reproduce

  • Using URL https://dev.azure.com/projectname
  • Using Token Based Authentication
  1. Create a Test Plan, for example with ID 552
  2. Create a Test Suite, for example with ID 553
  3. Create some test cases to the suite...
  4. Call console.log(await connection.getTestApi().getTestCases( projectname 552, 553 ))
  5. See what gets returned

Logs

@terotests
Copy link
Author

And, if you have anything you need to ask, or any workaround you want to suggest, please do not hesitate to ask - I follow this thread every day and can respond in 1-2 office days at least.

@damccorm damccorm self-assigned this Oct 29, 2018
@damccorm
Copy link

I'm looking into this, it looks like its caused by the same underlying issue as #198 and it should be fixed by #248

@luker2
Copy link

luker2 commented Nov 26, 2018

Seems like a dupe of #50 which was never actually fixed

@damccorm
Copy link

@luker2 are you still having issues with version 7.0.0 of this library?

@luker2
Copy link

luker2 commented Nov 26, 2018

Nope I'm good. I just wanted to reference #50 here since it seemed to be the same issue (#50 was closed long ago, but don't think it was ever truly fixed). I haven't had time to upgrade/test this, but I noticed action: "TestCases" was finally added to TestApi.ts, which has been my local fix for a long time.

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 a pull request may close this issue.

3 participants