Could someone point me in the right direction for creating a test-case? I would need a (mock) driver that supports foreign-keys

@rharink check https://github.com/doctrine/doctrine2/blob/master/tests/Doctrine/Tests/ORM/Functional/Ticket for examples of test cases.

I'd suggest a scenario that runs on SQLite only and:

1. creates a table
2. instantiates the DatabaseDriver
3. verifies the schema information coming from DatabaseDriver

@Ocramius Thanks for the reply,

The SqlLite driver doesn't support foreign keys. The DatabaseDriver class will not resolve foreign keys, thus the method getFieldNameForColumn is never called in the testcase.

returns false for Sqllite

$this->_em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints())

True, would need to design this with MySQL or equivalent

Should I use a DbalFunctionalTestCase or is there something else I can use to setup a MySQL test?
Thanks for the help!

Sorry should read more before asking ;) I found documentation on how to run tests on mysql.

@Ocramius Checks have passed, should / can I do anything else?

@Ocramius can I do something else? maybe add a unit test in the databaseDriverTest.php? Just let me know

Ok, I've merged upstream.

I rebased, I fixed the build in #7731, this means the tests might now pass.

Wow good catch! Well at least the build is green now 😅

@rharink please look into this 🙏

Good one! Ill look in to it.

@ostrolucky nice catch indeed, thanks!

@rharink since we're aiming at releasing the last patch of the 2.6.x series and release 2.7.0 I'm moving this to 2.7.1 so we don't put a lot of pressure on your shoulders 😄

Thanks for putting effort into this and I hope we can solve it together in the next patch 👍

I've improved the unit-test. it didn't pass for previous behaviour as the key gh7684IdentityTest in the associationMapping would not have been present. Or am i interpreting 'previous behaviour' wrong 😅

Please kindly squash your commits together. If you don't, we'll try to remember to do it for you but it's best if you save us this trouble.

How to do that?

1. git rebase -i origin/2.6, assuming origin is a git remote that points to this repository, and not your fork. If you're not sure what your remotes are, run git remote -vvv, there should be your fork and the holy/reference/base/origin/whatever-you-call-it repository.
2. A window will show up with many lines, replace pick with fixup on every line but the first one
3. Close your editor, git should do its magic, and you should end up with one commit
4. Use git push --force to overwrite what you already push. Don't forget the --force option otherwise git will try to merge both things together.

@rharink @ostrolucky awesome, thanks for being that quick. Please follow @greg0ire's instructions and let's ship this!

@lcobucci @ostrolucky done, lets go!

@rharink 🚢