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

Run PHPCS on tests #1192

Merged
merged 2 commits into from
Jul 26, 2018
Merged

Run PHPCS on tests #1192

merged 2 commits into from
Jul 26, 2018

Conversation

jdpedrie
Copy link
Contributor

This change updates the PHPCS configuration to enforce coding standards on all test files (unit, snippet and system tests). It continues to exclude generated code.

It also includes a sweep of test cleanup to conform to standards and modernizes some very old test code. This change shouldn't include changes to actual client code.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jul 24, 2018
Copy link
Contributor

@tmatsuo tmatsuo left a comment

Choose a reason for hiding this comment

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

Just some minor nits

@@ -0,0 +1,56 @@
<?php
/**
* Copyright 2018 Google Inc.

This comment was marked as spam.

This comment was marked as spam.

@@ -0,0 +1,30 @@
<?php
/**
* Copyright 2018 Google Inc.

This comment was marked as spam.

Copy link
Contributor

@tmatsuo tmatsuo left a comment

Choose a reason for hiding this comment

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

Thanks! It's great!

Copy link
Contributor

@dwsupplee dwsupplee left a comment

Choose a reason for hiding this comment

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

Outside of Takashi's note, LGTM

@jdpedrie jdpedrie merged commit 9e06a29 into googleapis:master Jul 26, 2018
@jdpedrie jdpedrie deleted the phpcs-tests branch July 26, 2018 14:01
@jdpedrie jdpedrie mentioned this pull request Aug 8, 2018
jdpedrie added a commit that referenced this pull request Aug 8, 2018
## Google Cloud PHP v0.73.0

### google/cloud-bigquery v1.4.1

* Added documentation for BigQuery job time filter. (#1183)
* Updated tests to conform to linting requirements. (#1192)

### google/cloud-bigquerydatatransfer v0.7.0

* Updated BigQueryDataTransfer generated client. (#1184)

### google/cloud-core v1.22.0

* Updated Symfony Lock Adapter to provide better exception messaging and context. (#1204)
* Added support for custom headers on resumable uploaders. (#1111)
* Updated tests to conform to linting requirements. (#1192)

### google/cloud-datastore v1.5.5

* Updated tests to conform to linting requirements. (#1192)

### google/cloud-debugger v0.12.3

* Updated tests to conform to linting requirements. (#1192)

### google/cloud-error-reporting v0.11.3

* Updated tests to conform to linting requirements. (#1192)

### google/cloud-firestore v0.12.1

* Updated tests to conform to linting requirements. (#1192)

### google/cloud-language v0.15.3

* Updated tests to conform to linting requirements. (#1192)

### google/cloud-logging v1.12.6

* Updated tests to conform to linting requirements. (#1192)

### google/cloud-pubsub v1.4.2

* Updated tests to conform to linting requirements. (#1192)

### google/cloud-spanner v1.6.3

* Fixed a bug which prevented field values from being set to `null`. (#1201)
* Updated tests to conform to linting requirements. (#1192)

### google/cloud-speech v0.16.0

* **BREAKING CHANGE** Removed deprecated v1beta1 client for Cloud Speech and update generated clients. (#1196)
* Updated tests to conform to linting requirements. (#1192)

### google/cloud-storage v1.7.0

* Improve Storage Signed URL support. Added support for CORS headers and improved validation of inputs. (#1111)
* Updated tests to conform to linting requirements. (#1192)

### google/cloud-text-to-speech v0.1.2

* Updated tests to conform to linting requirements. (#1192)

### google/cloud-trace v0.10.3

* Updated tests to conform to linting requirements. (#1192)

### google/cloud-translate v1.2.4

* Updated tests to conform to linting requirements. (#1192)

### google/cloud-vision v0.15.2

* Updated tests to conform to linting requirements. (#1192)
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants