Skip to content
This repository has been archived by the owner on May 9, 2020. It is now read-only.

Scaffolding fixes #35

Merged
merged 8 commits into from
Jun 24, 2018
Merged

Conversation

qbikez
Copy link
Contributor

@qbikez qbikez commented Jun 23, 2018

Fixes and tests for DbContext scaffolding.

I see most of the changes are already included in the source, but I also added some tests for verifying generated context.

* use same namespace in FbDesignTimeServices  as everywhere else
* add FbOptions to serviceCollection in FbDesignTimeServices
* handle indices without fields
* handle tables without PK
* fix nullable columns (NULL switched with NOT NULL)
* better foreign key detection
@qbikez qbikez mentioned this pull request Jun 23, 2018
After changing Book<->Author relation to many-to-many, inserted items count changed.
@ralmsdeveloper
Copy link
Owner

There is something breaking the test, the Author Column was not found!

I'll see what's going on.

Repository owner deleted a comment from AppVeyorBot Jun 23, 2018
Repository owner deleted a comment from AppVeyorBot Jun 23, 2018
Repository owner deleted a comment from AppVeyorBot Jun 23, 2018
Repository owner deleted a comment from AppVeyorBot Jun 23, 2018
Repository owner deleted a comment from AppVeyorBot Jun 23, 2018
Repository owner deleted a comment from AppVeyorBot Jun 23, 2018
@ralmsdeveloper ralmsdeveloper self-assigned this Jun 23, 2018
xunit allows different test classes to run in parallel, which caused
file access errors on TestContext
@AppVeyorBot
Copy link

1 similar comment
@AppVeyorBot
Copy link

@qbikez
Copy link
Contributor Author

qbikez commented Jun 24, 2018

There were some file access issues with "EFCoreSample.fdb", I changed the new class to use a different file name.

@ralmsdeveloper ralmsdeveloper merged commit ddb1ba9 into ralmsdeveloper:dev Jun 24, 2018
@ralmsdeveloper
Copy link
Owner

@qbikez Thanks!

@qbikez qbikez deleted the scaffolding-fixes branch June 25, 2018 07:10
@qbikez qbikez restored the scaffolding-fixes branch September 1, 2019 06:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants