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.

Sign up for GitHub

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

Jump to bottom

Enable ext/pdo/tests/pdo_022.phpt test #30

Open
Girgias opened this issue Jan 18, 2021 · 2 comments
Open

Enable ext/pdo/tests/pdo_022.phpt test #30

Girgias opened this issue Jan 18, 2021 · 2 comments
Labels
testing

Comments

@Girgias
Copy link
Member

Girgias commented Jan 18, 2021

This PDO test is mark as XFAIL as "Feature not implemented no test makes sense" but is actually implemented...

However as it tests the PDOStatement::getColumnMeta() method it might make more sense that each individual driver has their own corresponding test than a common PDO test.
@cmb69
Copy link
Member

cmb69 commented Jan 18, 2021

But see https://bugs.php.net/bug.php?id=46508#1246268547.

@Girgias
Copy link
Member Author

Girgias commented Jan 18, 2021

But see https://bugs.php.net/bug.php?id=46508#1246268547.

Well... maybe that's something which should be done, or because it's been already 12 years the current behaviour is the current spec.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
testing
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@cmb69 @Girgias