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

Bring test coverage to 100% #870

Merged

Conversation

stephenplusplus
Copy link
Contributor

... I think. Running npm cover doesn't work on my end.

But, I went over all of these < 100% files: https://coveralls.io/builds/3590802 and covered up the holes.

Along the way, I also caught some places we were inconsistent with our setMetadata method. The signature is supposed to be:

object.setMetadata({}, function(err, apiResponse) {});

But some places, it was:

object.setMetadata({}, function(err, metadata, apiResponse) {});

Coincidentally, apiResponse and metadata would have been the same thing-- the apiResponse is what becomes the metadata. Now, however, it's only function(err, apiResponse) {} so that if an error occurs, the handler still gets the benefit of receiving the API response.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 16, 2015
@@ -281,6 +279,18 @@ describe('BigQuery/Table', function() {
table.createWriteStream(fileType).emit('writing');
});

it('should format a schema', function(done) {
var expectedSchema = Table.createSchemaFromString_(SCHEMA_STRING);

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

return (tableOverrides[tableMethod] || tableCached[tableMethod])
.apply(this, arguments);
};
});

This comment was marked as spam.

This comment was marked as spam.

it('should add success redirect', function(done) {
file.getSignedPolicy({
expires: Date.now() + 5,
successRedirect: 'redirect'

This comment was marked as spam.

This comment was marked as spam.

tests: cover bigquery/table leaks
tests: cover datastore/transaction leaks
tests: cover datastore/dataset leaks
tests: cover storage/file leaks
tests: cover storage/bucket leaks
tests: cover search/index leaks
tests: cover storage/acl leaks
tests: cover dns/zone leaks
tests: cover datastore/entity leaks
tests: cover pubsub/subscription leaks
callmehiphop added a commit that referenced this pull request Sep 17, 2015
@callmehiphop callmehiphop merged commit 1716bae into googleapis:master Sep 17, 2015
@callmehiphop
Copy link
Contributor

Now we have to work to keep it at 100% forever!

@stephenplusplus
Copy link
Contributor Author

screen shot 2015-09-17 at 1 45 03 pm

Woo!

sofisl pushed a commit that referenced this pull request Nov 11, 2022
* docs: clarified some LRO types

PiperOrigin-RevId: 394302922

Source-Link: googleapis/googleapis@cd16abb

Source-Link: googleapis/googleapis-gen@64782e6

* 🦉 Updates from OwlBot

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* 🦉 Updates from OwlBot

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Co-authored-by: Jeffrey Rennie <[email protected]>
sofisl pushed a commit that referenced this pull request Jan 17, 2023
sofisl pushed a commit that referenced this pull request Jan 24, 2023
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Co-authored-by: Summer Ji <[email protected]>
sofisl pushed a commit that referenced this pull request Jan 25, 2023
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Co-authored-by: Summer Ji <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement. core
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants