Skip to content

Tests failing on live#19

Open
arianakataokatesting wants to merge 1 commit intomainfrom
tests/add-errors-phpunit
Open

Tests failing on live#19
arianakataokatesting wants to merge 1 commit intomainfrom
tests/add-errors-phpunit

Conversation

@arianakataokatesting
Copy link
Collaborator

No description provided.

Copy link
Owner

@ariskataoka ariskataoka left a comment

Choose a reason for hiding this comment

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

phpcs scanning turned up:

🚫 7 errors

⚠️ 2 warnings


hashes-api-scanning skipped

@@ -0,0 +1,17 @@
<?php

echo $_POST[ 'phpcs should catch this problem' ];
Copy link
Owner

Choose a reason for hiding this comment

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

⚠️ Warning: Detected usage of a non-sanitized input variable: $_POST['phpcs should catch this problem'] (WordPress.Security.ValidatedSanitizedInput.InputNotSanitized).

@@ -0,0 +1,17 @@
<?php

echo $_POST[ 'phpcs should catch this problem' ];
Copy link
Owner

Choose a reason for hiding this comment

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

🚫 Error: All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$_POST['phpcs should catch this problem']' (WordPress.Security.EscapeOutput.OutputNotEscaped).

@@ -0,0 +1,17 @@
<?php

echo $_POST[ 'phpcs should catch this problem' ];
Copy link
Owner

Choose a reason for hiding this comment

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

🚫 Error: Processing form data without nonce verification (WordPress.Security.NonceVerification.Missing).

@@ -0,0 +1,17 @@
<?php

echo $_POST[ 'phpcs should catch this problem' ];
Copy link
Owner

Choose a reason for hiding this comment

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

🚫 Error: Detected usage of a possibly undefined superglobal array index: $_POST['phpcs should catch this problem']. Use isset() or empty() to check the index exists before using it (WordPress.Security.ValidatedSanitizedInput.InputNotValidated).




echo 'linter will catch this fatal;
Copy link
Owner

Choose a reason for hiding this comment

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

🚫 Error: PHP syntax error: syntax error, unexpected ''linter will catch this fatal;' (T_ENCAPSED_AND_WHITESPACE) (Generic.PHP.Syntax.PHPSyntax).

@@ -0,0 +1,8 @@
<?php

echo $_POST[ 'phpcs should catch this problem' ];
Copy link
Owner

Choose a reason for hiding this comment

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

⚠️ Warning: Detected usage of a non-sanitized input variable: $_POST['phpcs should catch this problem'] (WordPress.Security.ValidatedSanitizedInput.InputNotSanitized).

@@ -0,0 +1,8 @@
<?php

echo $_POST[ 'phpcs should catch this problem' ];
Copy link
Owner

Choose a reason for hiding this comment

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

🚫 Error: All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$_POST['phpcs should catch this problem']' (WordPress.Security.EscapeOutput.OutputNotEscaped).

@@ -0,0 +1,8 @@
<?php

echo $_POST[ 'phpcs should catch this problem' ];
Copy link
Owner

Choose a reason for hiding this comment

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

🚫 Error: Processing form data without nonce verification (WordPress.Security.NonceVerification.Missing).

@@ -0,0 +1,8 @@
<?php

echo $_POST[ 'phpcs should catch this problem' ];
Copy link
Owner

Choose a reason for hiding this comment

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

🚫 Error: Detected usage of a possibly undefined superglobal array index: $_POST['phpcs should catch this problem']. Use isset() or empty() to check the index exists before using it (WordPress.Security.ValidatedSanitizedInput.InputNotValidated).

@ariskataoka ariskataoka force-pushed the tests/add-errors-phpunit branch from 955008c to 44a73c0 Compare September 14, 2021 09:18
@ariskataoka ariskataoka force-pushed the tests/add-errors-phpunit branch from 4b49f6c to 1d23f43 Compare September 14, 2021 09:24
Copy link
Owner

@ariskataoka ariskataoka left a comment

Choose a reason for hiding this comment

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

phpcs scanning turned up:

🚫 7 errors

⚠️ 2 warnings


hashes-api-scanning skipped

@@ -0,0 +1,17 @@
<?php

echo $_POST[ 'phpcs should catch this problem' ];
Copy link
Owner

Choose a reason for hiding this comment

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

⚠️ Warning: Detected usage of a non-sanitized input variable: $_POST['phpcs should catch this problem'] (WordPress.Security.ValidatedSanitizedInput.InputNotSanitized).

@@ -0,0 +1,17 @@
<?php

echo $_POST[ 'phpcs should catch this problem' ];
Copy link
Owner

Choose a reason for hiding this comment

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

🚫 Error: All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$_POST['phpcs should catch this problem']' (WordPress.Security.EscapeOutput.OutputNotEscaped).

@@ -0,0 +1,17 @@
<?php

echo $_POST[ 'phpcs should catch this problem' ];
Copy link
Owner

Choose a reason for hiding this comment

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

🚫 Error: Processing form data without nonce verification (WordPress.Security.NonceVerification.Missing).

@@ -0,0 +1,17 @@
<?php

echo $_POST[ 'phpcs should catch this problem' ];
Copy link
Owner

Choose a reason for hiding this comment

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

🚫 Error: Detected usage of a possibly undefined superglobal array index: $_POST['phpcs should catch this problem']. Use isset() or empty() to check the index exists before using it (WordPress.Security.ValidatedSanitizedInput.InputNotValidated).




echo 'linter will catch this fatal;
Copy link
Owner

Choose a reason for hiding this comment

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

🚫 Error: PHP syntax error: syntax error, unexpected ''linter will catch this fatal;' (T_ENCAPSED_AND_WHITESPACE) (Generic.PHP.Syntax.PHPSyntax).

@@ -0,0 +1,8 @@
<?php

echo $_POST[ 'phpcs should catch this problem' ];
Copy link
Owner

Choose a reason for hiding this comment

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

⚠️ Warning: Detected usage of a non-sanitized input variable: $_POST['phpcs should catch this problem'] (WordPress.Security.ValidatedSanitizedInput.InputNotSanitized).

@@ -0,0 +1,8 @@
<?php

echo $_POST[ 'phpcs should catch this problem' ];
Copy link
Owner

Choose a reason for hiding this comment

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

🚫 Error: All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '$_POST['phpcs should catch this problem']' (WordPress.Security.EscapeOutput.OutputNotEscaped).

@@ -0,0 +1,8 @@
<?php

echo $_POST[ 'phpcs should catch this problem' ];
Copy link
Owner

Choose a reason for hiding this comment

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

🚫 Error: Processing form data without nonce verification (WordPress.Security.NonceVerification.Missing).

@@ -0,0 +1,8 @@
<?php

echo $_POST[ 'phpcs should catch this problem' ];
Copy link
Owner

Choose a reason for hiding this comment

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

🚫 Error: Detected usage of a possibly undefined superglobal array index: $_POST['phpcs should catch this problem']. Use isset() or empty() to check the index exists before using it (WordPress.Security.ValidatedSanitizedInput.InputNotValidated).

Copy link
Owner

@ariskataoka ariskataoka left a comment

Choose a reason for hiding this comment

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

hashes-api-scanning skipped


skipped-files

Maximum number of lines exceeded (7):

  • GoogleAtom.php
  • MySuccessClass.php
  • MySuccessClass2.php
  • tests1/myfile1.php
  • tests1/myfile2.php

Note that the above file(s) were not analyzed due to their length.

Copy link
Owner

@ariskataoka ariskataoka left a comment

Choose a reason for hiding this comment

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

hashes-api-scanning skipped


skipped-files

Maximum number of lines exceeded (7):

  • GoogleAtom.php
  • MySuccessClass.php
  • MySuccessClass2.php
  • tests1/myfile1.php
  • tests1/myfile2.php

Note that the above file(s) were not analyzed due to their length.

Copy link
Owner

@ariskataoka ariskataoka left a comment

Choose a reason for hiding this comment

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

hashes-api-scanning skipped


skipped-files

Maximum number of lines exceeded (7):

  • GoogleAtom.php
  • MySuccessClass.php
  • MySuccessClass2.php
  • tests1/myfile1.php
  • tests1/myfile2.php

Note that the above file(s) were not analyzed due to their length.

Copy link
Owner

@ariskataoka ariskataoka left a comment

Choose a reason for hiding this comment

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

hashes-api-scanning skipped


skipped-files

Maximum number of lines exceeded (7):

Note that the above file(s) were not analyzed due to their length.

@ariskataoka
Copy link
Owner

PHP Syntax Errors Found

Scan performed on the code at commit 1d23f43 (view code).


Error: syntax error, unexpected end of file, expecting ; or ,

https://github.com/ariskataoka/vipgoci-tests/blob/1d23f431bdcc5fd69b9ff1c7511d654aa5ddebc7/src/SyntaxError.php#L4

Copy link
Owner

@ariskataoka ariskataoka left a comment

Choose a reason for hiding this comment

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

hashes-api-scanning skipped

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