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

Tests for JOIN needs fixing #200

Closed
TheTechsTech opened this issue Feb 5, 2021 · 0 comments · Fixed by #201
Closed

Tests for JOIN needs fixing #200

TheTechsTech opened this issue Feb 5, 2021 · 0 comments · Fixed by #201
Labels
error-bug Issues causing application termination. warning-bug Issues not causing scripts/routines to die/fail.

Comments

@TheTechsTech
Copy link
Contributor

@dpDesignz I been working on getting library updated for PHP 8, and discovered some errors show only up under it.

The tests for the Joins has issues that exists regardless of the PHP version.

In:

public function testJoins()

public function testJoins()

public function testJoins()

public function testJoins()

The second tables created is not receiving any inserted data, it's going to first table only still, and should have failed, no test for real insertion check.

@TheTechsTech TheTechsTech added warning-bug Issues not causing scripts/routines to die/fail. error-bug Issues causing application termination. labels Feb 5, 2021
dpDesignz added a commit to dpDesignz/ezsql that referenced this issue Feb 5, 2021
Resolves ezSQL#200 and also fixed phpunit error `expectExceptionMessageRegExp() is deprecated in PHPUnit 8 and will be removed in PHPUnit 9. Use expectExceptionMessageMatches() instead.`
@dpDesignz dpDesignz mentioned this issue Feb 5, 2021
TheTechsTech added a commit that referenced this issue Feb 8, 2021
…red in doing so

- bug fixes, prior to PHP 8, `**@**` silenced errors which this library used to return `**false**` instead, that is not no longer possible with PHP 8
- Linux and Windows CI tests move to GitHub Actions
- General code style fixes
- merged bug fix #199
- fixed tests in issue #200, and corrections for PR #201
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
error-bug Issues causing application termination. warning-bug Issues not causing scripts/routines to die/fail.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant