Feeling Lucky
?
?
[#17031] - correct test schema - change `TEXT` or `TIMESTAMP` to `DATETIME`
- Fixed in Code Base
- 15 Aug 2017
- Medium
- Build: staging
- # 17031
- Diff
- photodude:patch-6
- Success hound No violations found. Woof! Details
- Success continuous-integration/drone/pr the build was successful Details
- Success continuous-integration/travis-ci/pr The Travis CI build passed Details
- Success continuous-integration/appveyor/pr AppVeyor build succeeded Details
- Error continuous-integration/jenkins/pr-merge This commit cannot be built Details
User tests: Successful: Unsuccessful:
Pull Request for Issue test schema use wrong types.
Summary of Changes
change some fields from TEXT or TIMESTAMP to DATETIME
Testing Instructions
code review / tests pass
Expected result
tests pass and have correct types
Actual result
some fields used TEXT or TIMESTAMP when they should have been using DATETIME
Documentation Changes Required
None, this is a testing setup fix
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Unit Tests |
| Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-08-15 11:06:57 |
| Closed_By | ⇒ | wilsonge |
wilsonge
- comment
- 15 Aug 2017
Given this is only doing tests and it's consistent with what we already have in the installation sql files i'm good with this
@mbabker @wilsonge, bump for review and merge consideration