Skip to content

Conversation

@jrfnl
Copy link
Member

@jrfnl jrfnl commented Jul 4, 2020

Added tests to the following methods to verify that the new mixed type declaration is handled correctly (hint: it is).

  • BCFile::getMethodParameters()
  • FunctionDeclarations::getParameters() (via BCFile::getMethodParameters())
  • BCFile::getMethodProperties()
  • FunctionDeclarations::getProperties() (via BCFile::getMethodProperties())
  • BCFile::getMemberProperties()
  • Variables::getMemberProperties() (via BCFile::getMemberProperties())

@jrfnl jrfnl added this to the 1.0.0-alpha4 milestone Jul 4, 2020
@jrfnl jrfnl force-pushed the feature/php8-mixed-type-add-unit-tests branch 2 times, most recently from 3260633 to 2470e36 Compare July 5, 2020 18:10
jrfnl added 3 commits July 10, 2020 07:35
…PHP 8 "mixed" type

No changes needed to the actual method, the parameter type is already handled correctly.
…PHP 8 "mixed" return type

No changes needed to the actual method, the return type is already handled correctly.
… property type

No changes needed to the actual method, the type is already handled correctly.
@jrfnl jrfnl force-pushed the feature/php8-mixed-type-add-unit-tests branch from 2470e36 to cf89158 Compare July 10, 2020 05:37
@jrfnl jrfnl merged commit a07fd2e into develop Jul 10, 2020
@jrfnl jrfnl deleted the feature/php8-mixed-type-add-unit-tests branch July 10, 2020 05:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants