Skip to content

Conversation

@jycor
Copy link
Contributor

@jycor jycor commented Oct 10, 2022

tests for: #4420
companion pr: dolthub/go-mysql-server#1320

Copy link
Member

@zachmu zachmu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test in the wrong place but actual test looks good

if !types.IsFormat_DOLT(types.Format_Default) {
t.Skip("not fixing unique index on keyless tables for old format")
}
enginetest.TestScript(t, newDoltHarness(t), queries.ScriptTest{
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't put this here. Add it to somewhere in ddl_queries.go and call it from TestDoltDdlScripts

@jycor jycor merged commit f86c6fa into main Oct 12, 2022
@Hydrocharged Hydrocharged deleted the james/unique branch October 13, 2022 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants