Skip to content

Conversation

@mazimkhan
Copy link

Support for working with pipeline jobs.

@kinow
Copy link
Member

kinow commented Nov 4, 2016

Really great stuff @mazimkhan

I believe we are ready to merge it once I can confirm

i) the code from that testlink pull request has been released
ii) 1.9.15 is compatible with the Java API (it was broken the last I tried some weeks ago)
iii ) we have merged and released testlink-java-api

Test\ Plan\ Name=Nome do Plano de Testes
Platform\ Name=Nome da Plataforma
Build\ Name=Nome do Build
Build\ Custom\ Fields\ Values=Construir valores personalizados Campos
Copy link
Member

Choose a reason for hiding this comment

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

That'd probably be something like "Campos customizados do Build" (Build as a noun rather than the verb construir). Ditto for Spanish. Will fix after merging.

* Calculates the new value of this wrapped test case execution status,
* given a number of custom fields.
* @param numberOfCustomFields
* @param keyCustomFieldName
Copy link
Member

Choose a reason for hiding this comment

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

Nice catch!!! :D

<groupId>br.eti.kinoshita</groupId>
<artifactId>testlink-java-api</artifactId>
<version>1.9.14-0</version>
<version>1.9.14-1-SNAPSHOT</version>
Copy link
Member

Choose a reason for hiding this comment

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

What was the reason for using 1.9.14-1-SNAPSHOT? Which version of TestLink did you test your code against, please?

Copy link
Member

Choose a reason for hiding this comment

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

Never mind the reason for using the snapshot. Probably you were using your local version with the buildCustomFields.

protected String buildName;
/**
* The Build custom fields.
*/
Copy link
Member

Choose a reason for hiding this comment

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

Once we merge that other pull request from testlink-java-api, we can merge this PR too.

@kinow
Copy link
Member

kinow commented Feb 7, 2019

I think there was an old review in GitHub, so I just pressed "submit review". Some conflicting files too.... sorry.

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