New-DbaDbTable - Drop table if creation fails and change DefaultEx to DefaultExpression

[andreasjordan]

Type of Change

• Bug fix (non-breaking change, fixes # )
• New feature (non-breaking change, adds functionality, fixes # )
• Breaking change (effects multiple commands or functionality, fixes # )
• Ran manual Pester test and has passed (.\tests\manual.pester.ps1)
• Adding code coverage to existing functionality
• Pester test is included
• If new file reference added for test, has is been added to github.com/dataplat/appveyor-lab ?
• Unit test is included
• Documentation
• Build system

[andreasjordan]

Don't know why the unrelated test fails.

[potatoqualitee]

Rerunning all incomplete

[potatoqualitee]

oh hey andreas, tests for new-dbadbtable are disabled on Appveyor

# do not run on appveyor
    "appveyor_disabled" = @(

found that by looking for what could possibly be impacting that other command that's failing

[potatoqualitee]

Seems it fails too often -- perhaps create a test on gh-actions.ps1

[potatoqualitee]

you can also remove that test from pester.groups.ps1 to actually make it run, to see if tests actually pass

[andreasjordan]

Thanks for the hint - now it works, tests pass.
Is there anything else that should be included in this PR? What about "DefaultString"?

[potatoqualitee]

I like that too, if you wish

[potatoqualitee]

this is awesome! I love it, thank you 🥳

[drstonephd]

Wow, you folks are fast. I like the DefaultExpression and DefaultString. DBATools makes a DBAs life easier. Thanks!