Skip to content

Conversation

DanielEScherzer
Copy link
Member

@DanielEScherzer DanielEScherzer commented Oct 13, 2024

Move more low-hanging fruit, creating new directories for the tests for

  • comparisons
  • dynamic calls
  • error messages
  • error_reporting()
  • exceptions
  • foreach()
  • garbage collection
  • group use statements
  • heredoc and nowdoc
  • goto jumps
  • late static binding
  • magic methods
  • namespaces
  • numeric literal separators
  • objects
  • settype()
  • cleaning of temporary values
  • unset()

Additionally, move some tests into the existing sub directory for list() tests.

Also fix some test numbers in the names of the goto tests.

Work towards GH-15631

@DanielEScherzer
Copy link
Member Author

CC @Girgias from the last patch

Move more low-hanging fruit, creating new directories for the tests for

* comparisons
* dynamic calls
* error messages
* `error_reporting()`
* exceptions
* `foreach()`
* garbage collection
* group `use` statements
* heredoc and nowdoc
* `goto` jumps
* late static binding
* magic methods
* namespaces
* numeric literal separators
* objects
* oss-fuzz cases
* `settype()`
* cleaning of temporary values
* `unset()`

Additionally, move some tests into the existing sub directory for `list()`
tests.

Also fix some test numbers in the names of the `goto` tests.

Work towards phpGH-15631
@jorgsowa
Copy link
Contributor

oss-fuzz cases are a mix of different features. IMO they should be allocated to the corresponding feature directory rather than a separate one.

@DanielEScherzer
Copy link
Member Author

oss-fuzz cases are a mix of different features. IMO they should be allocated to the corresponding feature directory rather than a separate one.

I reverted that part, I can review those individually later

Copy link
Member

@Girgias Girgias left a comment

Choose a reason for hiding this comment

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

Thank you!

@Girgias Girgias merged commit 275f63e into php:master Oct 14, 2024
9 of 10 checks passed
@dstogov
Copy link
Member

dstogov commented Oct 14, 2024

I don't think this moving was a good idea (git log now doesn't show the previous history).

@Girgias
Copy link
Member

Girgias commented Oct 14, 2024

@dstogov using git log --follow shows the previous history.

@DanielEScherzer DanielEScherzer deleted the test-org-2 branch October 14, 2024 19:52
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.

4 participants