Skip to content

language: fix system tests#1739

Merged
callmehiphop merged 1 commit intogoogleapis:masterfrom
stephenplusplus:spp--language-system-tests-silent-failures
Oct 21, 2016
Merged

language: fix system tests#1739
callmehiphop merged 1 commit intogoogleapis:masterfrom
stephenplusplus:spp--language-system-tests-silent-failures

Conversation

@stephenplusplus
Copy link
Contributor

One of the Language system tests had a failure. Fixed that. But also, the terminal wasn't doing any reporting of the error-- it just exited the mocha process quietly. Wrapping the tests in a try/catch corrected this.

@stephenplusplus stephenplusplus added api: language Issues related to the Cloud Natural Language API API. testing labels Oct 21, 2016
@callmehiphop callmehiphop merged commit fe8e303 into googleapis:master Oct 21, 2016
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling a26d43d on stephenplusplus:spp--language-system-tests-silent-failures into 88990ac on GoogleCloudPlatform:master.

sofisl pushed a commit that referenced this pull request Oct 11, 2022
sofisl pushed a commit that referenced this pull request Oct 13, 2022
sofisl pushed a commit that referenced this pull request Feb 24, 2026
Co-authored-by: Denver Coneybeare <dconeybe@google.com>
Co-authored-by: Tom Andersen <tom-andersen@users.noreply.github.com>
sofisl pushed a commit that referenced this pull request Feb 26, 2026
* fix: cleanup different types of session pools

* fix: revert some changes to prevent backward compatibility breakage

* fix: revert some changes to prevent backward compatibility breakage

* fix: deprecated getReadSession and getWriteSession methods

* fix: add `session-available` event

* fix: addressing code review feedbacks

* fix: minor documentation updates
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: language Issues related to the Cloud Natural Language API API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants