nodejs-logging icon indicating copy to clipboard operation
nodejs-logging copied to clipboard

need to have test for request batching

Open alexander-fenster opened this issue 3 years ago • 2 comments

As reported in #835, we lost the batching functionality because of a bug in the generator. Unfortunately, the tests did not catch it. We need to cover this piece of functionality with tests (possibly mocking the part that makes an actual request with sinon, and checking how many times it is called).

alexander-fenster avatar Aug 11 '20 01:08 alexander-fenster

@alexander-fenster , is this feature related to this library or needs to be implemented in gapic layer?

losalex avatar Oct 14 '22 21:10 losalex

@losalex Logging library is the only library that makes use of the batching feature, so it could go either way, but it will be easier to implement it here in this library. Then it can be backported to the generator if needed.

alexander-fenster avatar Oct 15 '22 02:10 alexander-fenster