Merge pull request #1310 from alixander/fix-delete-generated
Fix delete generated edge case
This commit is contained in:
Alexander Wang
GitHub
commit
3bacc345e0
3 changed files with 376 additions and 1 deletions
|
|
@ -915,7 +915,27 @@ func renameConflictsToParent(g *d2graph.Graph, key *d2ast.KeyPath) (*d2graph.Gra
|
||||||
hoistedAbsKey.Path = append(hoistedAbsKey.Path, ref.Key.Path[:ref.KeyPathIndex]...)
|
hoistedAbsKey.Path = append(hoistedAbsKey.Path, ref.Key.Path[:ref.KeyPathIndex]...)
|
||||||
hoistedAbsKey.Path = append(hoistedAbsKey.Path, absKey.Path[len(absKey.Path)-1])
|
hoistedAbsKey.Path = append(hoistedAbsKey.Path, absKey.Path[len(absKey.Path)-1])
|
||||||
|
|
||||||
uniqueKeyStr, _, err := generateUniqueKey(g, strings.Join(d2graph.Key(hoistedAbsKey), "."), ignored, newIDs)
|
// Can't generate a key that'd conflict with sibling
|
||||||
|
var siblingHoistedIDs []string
|
||||||
|
for _, otherAbsKey := range absKeys {
|
||||||
|
if absKey == otherAbsKey {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
ida := d2graph.Key(otherAbsKey)
|
||||||
|
absKeyStr := strings.Join(ida, ".")
|
||||||
|
if _, ok := dedupedRenames[absKeyStr]; ok {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
hoistedAbsKey, err := d2parser.ParseKey(ref.ScopeObj.AbsID())
|
||||||
|
if err != nil {
|
||||||
|
hoistedAbsKey = &d2ast.KeyPath{}
|
||||||
|
}
|
||||||
|
hoistedAbsKey.Path = append(hoistedAbsKey.Path, ref.Key.Path[:ref.KeyPathIndex]...)
|
||||||
|
hoistedAbsKey.Path = append(hoistedAbsKey.Path, otherAbsKey.Path[len(otherAbsKey.Path)-1])
|
||||||
|
siblingHoistedIDs = append(siblingHoistedIDs, strings.Join(d2graph.Key(hoistedAbsKey), "."))
|
||||||
|
}
|
||||||
|
|
||||||
|
uniqueKeyStr, _, err := generateUniqueKey(g, strings.Join(d2graph.Key(hoistedAbsKey), "."), ignored, append(newIDs, siblingHoistedIDs...))
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return nil, err
|
return nil, err
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -5816,7 +5816,25 @@ Text
|
||||||
Text 2
|
Text 2
|
||||||
`,
|
`,
|
||||||
},
|
},
|
||||||
|
{
|
||||||
|
name: "conflicts_generated_3",
|
||||||
|
text: `x: {
|
||||||
|
Square 2
|
||||||
|
Square 3
|
||||||
|
}
|
||||||
|
|
||||||
|
Square 2
|
||||||
|
Square
|
||||||
|
`,
|
||||||
|
key: `x`,
|
||||||
|
|
||||||
|
exp: `Square 4
|
||||||
|
Square 3
|
||||||
|
|
||||||
|
Square 2
|
||||||
|
Square
|
||||||
|
`,
|
||||||
|
},
|
||||||
{
|
{
|
||||||
name: "drop_value",
|
name: "drop_value",
|
||||||
text: `a.b.c: "c label"
|
text: `a.b.c: "c label"
|
||||||
|
|
@ -6874,6 +6892,23 @@ Text 2
|
||||||
|
|
||||||
exp: `{
|
exp: `{
|
||||||
"Text.Text 2": "Text"
|
"Text.Text 2": "Text"
|
||||||
|
}`,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
name: "conflicts_generated_3",
|
||||||
|
text: `x: {
|
||||||
|
Square 2
|
||||||
|
Square 3
|
||||||
|
}
|
||||||
|
|
||||||
|
Square 2
|
||||||
|
Square
|
||||||
|
`,
|
||||||
|
key: `x`,
|
||||||
|
|
||||||
|
exp: `{
|
||||||
|
"x.Square 2": "Square 4",
|
||||||
|
"x.Square 3": "Square 3"
|
||||||
}`,
|
}`,
|
||||||
},
|
},
|
||||||
}
|
}
|
||||||
|
|
|
||||||
320
testdata/d2oracle/TestDelete/conflicts_generated_3.exp.json
generated
vendored
Normal file
320
testdata/d2oracle/TestDelete/conflicts_generated_3.exp.json
generated
vendored
Normal file
|
|
@ -0,0 +1,320 @@
|
||||||
|
{
|
||||||
|
"graph": {
|
||||||
|
"name": "",
|
||||||
|
"isFolderOnly": false,
|
||||||
|
"ast": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestDelete/conflicts_generated_3.d2,0:0:0-5:0:35",
|
||||||
|
"nodes": [
|
||||||
|
{
|
||||||
|
"map_key": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestDelete/conflicts_generated_3.d2,0:0:0-0:8:8",
|
||||||
|
"key": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestDelete/conflicts_generated_3.d2,0:0:0-0:8:8",
|
||||||
|
"path": [
|
||||||
|
{
|
||||||
|
"unquoted_string": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestDelete/conflicts_generated_3.d2,0:0:0-0:8:8",
|
||||||
|
"value": [
|
||||||
|
{
|
||||||
|
"string": "Square 4",
|
||||||
|
"raw_string": "Square 4"
|
||||||
|
}
|
||||||
|
]
|
||||||
|
}
|
||||||
|
}
|
||||||
|
]
|
||||||
|
},
|
||||||
|
"primary": {},
|
||||||
|
"value": {}
|
||||||
|
}
|
||||||
|
},
|
||||||
|
{
|
||||||
|
"map_key": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestDelete/conflicts_generated_3.d2,1:0:9-1:8:17",
|
||||||
|
"key": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestDelete/conflicts_generated_3.d2,1:0:9-1:8:17",
|
||||||
|
"path": [
|
||||||
|
{
|
||||||
|
"unquoted_string": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestDelete/conflicts_generated_3.d2,1:0:9-1:8:17",
|
||||||
|
"value": [
|
||||||
|
{
|
||||||
|
"string": "Square 3",
|
||||||
|
"raw_string": "Square 3"
|
||||||
|
}
|
||||||
|
]
|
||||||
|
}
|
||||||
|
}
|
||||||
|
]
|
||||||
|
},
|
||||||
|
"primary": {},
|
||||||
|
"value": {}
|
||||||
|
}
|
||||||
|
},
|
||||||
|
{
|
||||||
|
"map_key": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestDelete/conflicts_generated_3.d2,3:0:19-3:8:27",
|
||||||
|
"key": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestDelete/conflicts_generated_3.d2,3:0:19-3:8:27",
|
||||||
|
"path": [
|
||||||
|
{
|
||||||
|
"unquoted_string": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestDelete/conflicts_generated_3.d2,3:0:19-3:8:27",
|
||||||
|
"value": [
|
||||||
|
{
|
||||||
|
"string": "Square 2",
|
||||||
|
"raw_string": "Square 2"
|
||||||
|
}
|
||||||
|
]
|
||||||
|
}
|
||||||
|
}
|
||||||
|
]
|
||||||
|
},
|
||||||
|
"primary": {},
|
||||||
|
"value": {}
|
||||||
|
}
|
||||||
|
},
|
||||||
|
{
|
||||||
|
"map_key": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestDelete/conflicts_generated_3.d2,4:0:28-4:6:34",
|
||||||
|
"key": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestDelete/conflicts_generated_3.d2,4:0:28-4:6:34",
|
||||||
|
"path": [
|
||||||
|
{
|
||||||
|
"unquoted_string": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestDelete/conflicts_generated_3.d2,4:0:28-4:6:34",
|
||||||
|
"value": [
|
||||||
|
{
|
||||||
|
"string": "Square",
|
||||||
|
"raw_string": "Square"
|
||||||
|
}
|
||||||
|
]
|
||||||
|
}
|
||||||
|
}
|
||||||
|
]
|
||||||
|
},
|
||||||
|
"primary": {},
|
||||||
|
"value": {}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
]
|
||||||
|
},
|
||||||
|
"root": {
|
||||||
|
"id": "",
|
||||||
|
"id_val": "",
|
||||||
|
"attributes": {
|
||||||
|
"label": {
|
||||||
|
"value": ""
|
||||||
|
},
|
||||||
|
"labelDimensions": {
|
||||||
|
"width": 0,
|
||||||
|
"height": 0
|
||||||
|
},
|
||||||
|
"style": {},
|
||||||
|
"near_key": null,
|
||||||
|
"shape": {
|
||||||
|
"value": ""
|
||||||
|
},
|
||||||
|
"direction": {
|
||||||
|
"value": ""
|
||||||
|
},
|
||||||
|
"constraint": {
|
||||||
|
"value": ""
|
||||||
|
}
|
||||||
|
},
|
||||||
|
"zIndex": 0
|
||||||
|
},
|
||||||
|
"edges": null,
|
||||||
|
"objects": [
|
||||||
|
{
|
||||||
|
"id": "Square 4",
|
||||||
|
"id_val": "Square 4",
|
||||||
|
"references": [
|
||||||
|
{
|
||||||
|
"key": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestDelete/conflicts_generated_3.d2,0:0:0-0:8:8",
|
||||||
|
"path": [
|
||||||
|
{
|
||||||
|
"unquoted_string": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestDelete/conflicts_generated_3.d2,0:0:0-0:8:8",
|
||||||
|
"value": [
|
||||||
|
{
|
||||||
|
"string": "Square 4",
|
||||||
|
"raw_string": "Square 4"
|
||||||
|
}
|
||||||
|
]
|
||||||
|
}
|
||||||
|
}
|
||||||
|
]
|
||||||
|
},
|
||||||
|
"key_path_index": 0,
|
||||||
|
"map_key_edge_index": -1
|
||||||
|
}
|
||||||
|
],
|
||||||
|
"attributes": {
|
||||||
|
"label": {
|
||||||
|
"value": "Square 4"
|
||||||
|
},
|
||||||
|
"labelDimensions": {
|
||||||
|
"width": 0,
|
||||||
|
"height": 0
|
||||||
|
},
|
||||||
|
"style": {},
|
||||||
|
"near_key": null,
|
||||||
|
"shape": {
|
||||||
|
"value": "rectangle"
|
||||||
|
},
|
||||||
|
"direction": {
|
||||||
|
"value": ""
|
||||||
|
},
|
||||||
|
"constraint": {
|
||||||
|
"value": ""
|
||||||
|
}
|
||||||
|
},
|
||||||
|
"zIndex": 0
|
||||||
|
},
|
||||||
|
{
|
||||||
|
"id": "Square 3",
|
||||||
|
"id_val": "Square 3",
|
||||||
|
"references": [
|
||||||
|
{
|
||||||
|
"key": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestDelete/conflicts_generated_3.d2,1:0:9-1:8:17",
|
||||||
|
"path": [
|
||||||
|
{
|
||||||
|
"unquoted_string": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestDelete/conflicts_generated_3.d2,1:0:9-1:8:17",
|
||||||
|
"value": [
|
||||||
|
{
|
||||||
|
"string": "Square 3",
|
||||||
|
"raw_string": "Square 3"
|
||||||
|
}
|
||||||
|
]
|
||||||
|
}
|
||||||
|
}
|
||||||
|
]
|
||||||
|
},
|
||||||
|
"key_path_index": 0,
|
||||||
|
"map_key_edge_index": -1
|
||||||
|
}
|
||||||
|
],
|
||||||
|
"attributes": {
|
||||||
|
"label": {
|
||||||
|
"value": "Square 3"
|
||||||
|
},
|
||||||
|
"labelDimensions": {
|
||||||
|
"width": 0,
|
||||||
|
"height": 0
|
||||||
|
},
|
||||||
|
"style": {},
|
||||||
|
"near_key": null,
|
||||||
|
"shape": {
|
||||||
|
"value": "rectangle"
|
||||||
|
},
|
||||||
|
"direction": {
|
||||||
|
"value": ""
|
||||||
|
},
|
||||||
|
"constraint": {
|
||||||
|
"value": ""
|
||||||
|
}
|
||||||
|
},
|
||||||
|
"zIndex": 0
|
||||||
|
},
|
||||||
|
{
|
||||||
|
"id": "Square 2",
|
||||||
|
"id_val": "Square 2",
|
||||||
|
"references": [
|
||||||
|
{
|
||||||
|
"key": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestDelete/conflicts_generated_3.d2,3:0:19-3:8:27",
|
||||||
|
"path": [
|
||||||
|
{
|
||||||
|
"unquoted_string": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestDelete/conflicts_generated_3.d2,3:0:19-3:8:27",
|
||||||
|
"value": [
|
||||||
|
{
|
||||||
|
"string": "Square 2",
|
||||||
|
"raw_string": "Square 2"
|
||||||
|
}
|
||||||
|
]
|
||||||
|
}
|
||||||
|
}
|
||||||
|
]
|
||||||
|
},
|
||||||
|
"key_path_index": 0,
|
||||||
|
"map_key_edge_index": -1
|
||||||
|
}
|
||||||
|
],
|
||||||
|
"attributes": {
|
||||||
|
"label": {
|
||||||
|
"value": "Square 2"
|
||||||
|
},
|
||||||
|
"labelDimensions": {
|
||||||
|
"width": 0,
|
||||||
|
"height": 0
|
||||||
|
},
|
||||||
|
"style": {},
|
||||||
|
"near_key": null,
|
||||||
|
"shape": {
|
||||||
|
"value": "rectangle"
|
||||||
|
},
|
||||||
|
"direction": {
|
||||||
|
"value": ""
|
||||||
|
},
|
||||||
|
"constraint": {
|
||||||
|
"value": ""
|
||||||
|
}
|
||||||
|
},
|
||||||
|
"zIndex": 0
|
||||||
|
},
|
||||||
|
{
|
||||||
|
"id": "Square",
|
||||||
|
"id_val": "Square",
|
||||||
|
"references": [
|
||||||
|
{
|
||||||
|
"key": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestDelete/conflicts_generated_3.d2,4:0:28-4:6:34",
|
||||||
|
"path": [
|
||||||
|
{
|
||||||
|
"unquoted_string": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestDelete/conflicts_generated_3.d2,4:0:28-4:6:34",
|
||||||
|
"value": [
|
||||||
|
{
|
||||||
|
"string": "Square",
|
||||||
|
"raw_string": "Square"
|
||||||
|
}
|
||||||
|
]
|
||||||
|
}
|
||||||
|
}
|
||||||
|
]
|
||||||
|
},
|
||||||
|
"key_path_index": 0,
|
||||||
|
"map_key_edge_index": -1
|
||||||
|
}
|
||||||
|
],
|
||||||
|
"attributes": {
|
||||||
|
"label": {
|
||||||
|
"value": "Square"
|
||||||
|
},
|
||||||
|
"labelDimensions": {
|
||||||
|
"width": 0,
|
||||||
|
"height": 0
|
||||||
|
},
|
||||||
|
"style": {},
|
||||||
|
"near_key": null,
|
||||||
|
"shape": {
|
||||||
|
"value": "rectangle"
|
||||||
|
},
|
||||||
|
"direction": {
|
||||||
|
"value": ""
|
||||||
|
},
|
||||||
|
"constraint": {
|
||||||
|
"value": ""
|
||||||
|
}
|
||||||
|
},
|
||||||
|
"zIndex": 0
|
||||||
|
}
|
||||||
|
]
|
||||||
|
},
|
||||||
|
"err": "<nil>"
|
||||||
|
}
|
||||||
Loading…
Reference in a new issue