Merge pull request #1065 from alixander/move-match-delta
fix move id delta
This commit is contained in:
Alexander Wang
GitHub
commit
abfaf3fbdc
4 changed files with 379 additions and 2 deletions
|
|
@ -1886,7 +1886,7 @@ func MoveIDDeltas(g *d2graph.Graph, key, newKey string) (deltas map[string]strin
|
||||||
}
|
}
|
||||||
|
|
||||||
if _, ok := g.Root.HasChild(d2graph.Key(hoistedMK.Key)); ok || conflictsWithNewID {
|
if _, ok := g.Root.HasChild(d2graph.Key(hoistedMK.Key)); ok || conflictsWithNewID {
|
||||||
newKey, _, err := generateUniqueKey(g, hoistedAbsID, nil, newIDs)
|
newKey, _, err := generateUniqueKey(g, hoistedAbsID, obj, newIDs)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return nil, err
|
return nil, err
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -3372,6 +3372,25 @@ b: {
|
||||||
exp: `b: {
|
exp: `b: {
|
||||||
a
|
a
|
||||||
}
|
}
|
||||||
|
`,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
name: "container_conflicts_generated",
|
||||||
|
text: `Square 2: "" {
|
||||||
|
Square: ""
|
||||||
|
}
|
||||||
|
Square: ""
|
||||||
|
Square 3
|
||||||
|
`,
|
||||||
|
key: `Square 2`,
|
||||||
|
newKey: `Square 3.Square 2`,
|
||||||
|
|
||||||
|
exp: `Square 2: ""
|
||||||
|
|
||||||
|
Square: ""
|
||||||
|
Square 3: {
|
||||||
|
Square 2: ""
|
||||||
|
}
|
||||||
`,
|
`,
|
||||||
},
|
},
|
||||||
}
|
}
|
||||||
|
|
@ -4939,6 +4958,7 @@ Text
|
||||||
Text 2
|
Text 2
|
||||||
`,
|
`,
|
||||||
},
|
},
|
||||||
|
|
||||||
{
|
{
|
||||||
name: "drop_value",
|
name: "drop_value",
|
||||||
text: `a.b.c: "c label"
|
text: `a.b.c: "c label"
|
||||||
|
|
@ -5278,6 +5298,22 @@ x.a -> x.b
|
||||||
"x.(a -> b)[0]": "(a 2 -> b)[0]",
|
"x.(a -> b)[0]": "(a 2 -> b)[0]",
|
||||||
"x.a": "a 2",
|
"x.a": "a 2",
|
||||||
"x.b": "b"
|
"x.b": "b"
|
||||||
|
}`,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
name: "container_conflicts_generated",
|
||||||
|
text: `Square 2: "" {
|
||||||
|
Square: ""
|
||||||
|
}
|
||||||
|
Square: ""
|
||||||
|
Square 3
|
||||||
|
`,
|
||||||
|
key: `Square 2`,
|
||||||
|
newKey: `Square 3.Square 2`,
|
||||||
|
|
||||||
|
exp: `{
|
||||||
|
"Square 2": "Square 3.Square 2",
|
||||||
|
"Square 2.Square": "Square 2"
|
||||||
}`,
|
}`,
|
||||||
},
|
},
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -14,7 +14,7 @@ func GetParentID(g *d2graph.Graph, absID string) (string, error) {
|
||||||
}
|
}
|
||||||
obj, ok := g.Root.HasChild(d2graph.Key(mk.Key))
|
obj, ok := g.Root.HasChild(d2graph.Key(mk.Key))
|
||||||
if !ok {
|
if !ok {
|
||||||
return "", fmt.Errorf("%v parent not found", absID)
|
return "", fmt.Errorf("%v not found", absID)
|
||||||
}
|
}
|
||||||
|
|
||||||
return obj.Parent.AbsID(), nil
|
return obj.Parent.AbsID(), nil
|
||||||
|
|
|
||||||
341
testdata/d2oracle/TestMove/container_conflicts_generated.exp.json
generated
vendored
Normal file
341
testdata/d2oracle/TestMove/container_conflicts_generated.exp.json
generated
vendored
Normal file
|
|
@ -0,0 +1,341 @@
|
||||||
|
{
|
||||||
|
"graph": {
|
||||||
|
"name": "",
|
||||||
|
"isFolderOnly": false,
|
||||||
|
"ast": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestMove/container_conflicts_generated.d2,0:0:0-6:0:54",
|
||||||
|
"nodes": [
|
||||||
|
{
|
||||||
|
"map_key": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestMove/container_conflicts_generated.d2,0:0:0-0:12:12",
|
||||||
|
"key": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestMove/container_conflicts_generated.d2,0:0:0-0:8:8",
|
||||||
|
"path": [
|
||||||
|
{
|
||||||
|
"unquoted_string": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestMove/container_conflicts_generated.d2,0:0:0-0:8:8",
|
||||||
|
"value": [
|
||||||
|
{
|
||||||
|
"string": "Square 2",
|
||||||
|
"raw_string": "Square 2"
|
||||||
|
}
|
||||||
|
]
|
||||||
|
}
|
||||||
|
}
|
||||||
|
]
|
||||||
|
},
|
||||||
|
"primary": {},
|
||||||
|
"value": {
|
||||||
|
"double_quoted_string": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestMove/container_conflicts_generated.d2,0:10:10-0:12:12",
|
||||||
|
"value": null
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
},
|
||||||
|
{
|
||||||
|
"map_key": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestMove/container_conflicts_generated.d2,2:0:14-2:10:24",
|
||||||
|
"key": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestMove/container_conflicts_generated.d2,2:0:14-2:6:20",
|
||||||
|
"path": [
|
||||||
|
{
|
||||||
|
"unquoted_string": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestMove/container_conflicts_generated.d2,2:0:14-2:6:20",
|
||||||
|
"value": [
|
||||||
|
{
|
||||||
|
"string": "Square",
|
||||||
|
"raw_string": "Square"
|
||||||
|
}
|
||||||
|
]
|
||||||
|
}
|
||||||
|
}
|
||||||
|
]
|
||||||
|
},
|
||||||
|
"primary": {},
|
||||||
|
"value": {
|
||||||
|
"double_quoted_string": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestMove/container_conflicts_generated.d2,2:8:22-2:10:24",
|
||||||
|
"value": null
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
},
|
||||||
|
{
|
||||||
|
"map_key": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestMove/container_conflicts_generated.d2,3:0:25-5:1:53",
|
||||||
|
"key": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestMove/container_conflicts_generated.d2,3:0:25-3:8:33",
|
||||||
|
"path": [
|
||||||
|
{
|
||||||
|
"unquoted_string": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestMove/container_conflicts_generated.d2,3:0:25-3:8:33",
|
||||||
|
"value": [
|
||||||
|
{
|
||||||
|
"string": "Square 3",
|
||||||
|
"raw_string": "Square 3"
|
||||||
|
}
|
||||||
|
]
|
||||||
|
}
|
||||||
|
}
|
||||||
|
]
|
||||||
|
},
|
||||||
|
"primary": {},
|
||||||
|
"value": {
|
||||||
|
"map": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestMove/container_conflicts_generated.d2,3:10:35-5:0:52",
|
||||||
|
"nodes": [
|
||||||
|
{
|
||||||
|
"map_key": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestMove/container_conflicts_generated.d2,4:2:39-4:14:51",
|
||||||
|
"key": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestMove/container_conflicts_generated.d2,4:2:39-4:10:47",
|
||||||
|
"path": [
|
||||||
|
{
|
||||||
|
"unquoted_string": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestMove/container_conflicts_generated.d2,4:2:39-4:10:47",
|
||||||
|
"value": [
|
||||||
|
{
|
||||||
|
"string": "Square 2",
|
||||||
|
"raw_string": "Square 2"
|
||||||
|
}
|
||||||
|
]
|
||||||
|
}
|
||||||
|
}
|
||||||
|
]
|
||||||
|
},
|
||||||
|
"primary": {},
|
||||||
|
"value": {
|
||||||
|
"double_quoted_string": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestMove/container_conflicts_generated.d2,4:12:49-4:14:51",
|
||||||
|
"value": null
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
]
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
]
|
||||||
|
},
|
||||||
|
"root": {
|
||||||
|
"id": "",
|
||||||
|
"id_val": "",
|
||||||
|
"label_dimensions": {
|
||||||
|
"width": 0,
|
||||||
|
"height": 0
|
||||||
|
},
|
||||||
|
"attributes": {
|
||||||
|
"label": {
|
||||||
|
"value": ""
|
||||||
|
},
|
||||||
|
"style": {},
|
||||||
|
"near_key": null,
|
||||||
|
"shape": {
|
||||||
|
"value": ""
|
||||||
|
},
|
||||||
|
"direction": {
|
||||||
|
"value": ""
|
||||||
|
},
|
||||||
|
"constraint": {
|
||||||
|
"value": ""
|
||||||
|
}
|
||||||
|
},
|
||||||
|
"zIndex": 0
|
||||||
|
},
|
||||||
|
"edges": null,
|
||||||
|
"objects": [
|
||||||
|
{
|
||||||
|
"id": "Square 2",
|
||||||
|
"id_val": "Square 2",
|
||||||
|
"label_dimensions": {
|
||||||
|
"width": 0,
|
||||||
|
"height": 0
|
||||||
|
},
|
||||||
|
"references": [
|
||||||
|
{
|
||||||
|
"key": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestMove/container_conflicts_generated.d2,0:0:0-0:8:8",
|
||||||
|
"path": [
|
||||||
|
{
|
||||||
|
"unquoted_string": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestMove/container_conflicts_generated.d2,0:0:0-0:8:8",
|
||||||
|
"value": [
|
||||||
|
{
|
||||||
|
"string": "Square 2",
|
||||||
|
"raw_string": "Square 2"
|
||||||
|
}
|
||||||
|
]
|
||||||
|
}
|
||||||
|
}
|
||||||
|
]
|
||||||
|
},
|
||||||
|
"key_path_index": 0,
|
||||||
|
"map_key_edge_index": -1
|
||||||
|
}
|
||||||
|
],
|
||||||
|
"attributes": {
|
||||||
|
"label": {
|
||||||
|
"value": ""
|
||||||
|
},
|
||||||
|
"style": {},
|
||||||
|
"near_key": null,
|
||||||
|
"shape": {
|
||||||
|
"value": "rectangle"
|
||||||
|
},
|
||||||
|
"direction": {
|
||||||
|
"value": ""
|
||||||
|
},
|
||||||
|
"constraint": {
|
||||||
|
"value": ""
|
||||||
|
}
|
||||||
|
},
|
||||||
|
"zIndex": 0
|
||||||
|
},
|
||||||
|
{
|
||||||
|
"id": "Square",
|
||||||
|
"id_val": "Square",
|
||||||
|
"label_dimensions": {
|
||||||
|
"width": 0,
|
||||||
|
"height": 0
|
||||||
|
},
|
||||||
|
"references": [
|
||||||
|
{
|
||||||
|
"key": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestMove/container_conflicts_generated.d2,2:0:14-2:6:20",
|
||||||
|
"path": [
|
||||||
|
{
|
||||||
|
"unquoted_string": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestMove/container_conflicts_generated.d2,2:0:14-2:6:20",
|
||||||
|
"value": [
|
||||||
|
{
|
||||||
|
"string": "Square",
|
||||||
|
"raw_string": "Square"
|
||||||
|
}
|
||||||
|
]
|
||||||
|
}
|
||||||
|
}
|
||||||
|
]
|
||||||
|
},
|
||||||
|
"key_path_index": 0,
|
||||||
|
"map_key_edge_index": -1
|
||||||
|
}
|
||||||
|
],
|
||||||
|
"attributes": {
|
||||||
|
"label": {
|
||||||
|
"value": ""
|
||||||
|
},
|
||||||
|
"style": {},
|
||||||
|
"near_key": null,
|
||||||
|
"shape": {
|
||||||
|
"value": "rectangle"
|
||||||
|
},
|
||||||
|
"direction": {
|
||||||
|
"value": ""
|
||||||
|
},
|
||||||
|
"constraint": {
|
||||||
|
"value": ""
|
||||||
|
}
|
||||||
|
},
|
||||||
|
"zIndex": 0
|
||||||
|
},
|
||||||
|
{
|
||||||
|
"id": "Square 3",
|
||||||
|
"id_val": "Square 3",
|
||||||
|
"label_dimensions": {
|
||||||
|
"width": 0,
|
||||||
|
"height": 0
|
||||||
|
},
|
||||||
|
"references": [
|
||||||
|
{
|
||||||
|
"key": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestMove/container_conflicts_generated.d2,3:0:25-3:8:33",
|
||||||
|
"path": [
|
||||||
|
{
|
||||||
|
"unquoted_string": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestMove/container_conflicts_generated.d2,3:0:25-3:8:33",
|
||||||
|
"value": [
|
||||||
|
{
|
||||||
|
"string": "Square 3",
|
||||||
|
"raw_string": "Square 3"
|
||||||
|
}
|
||||||
|
]
|
||||||
|
}
|
||||||
|
}
|
||||||
|
]
|
||||||
|
},
|
||||||
|
"key_path_index": 0,
|
||||||
|
"map_key_edge_index": -1
|
||||||
|
}
|
||||||
|
],
|
||||||
|
"attributes": {
|
||||||
|
"label": {
|
||||||
|
"value": "Square 3"
|
||||||
|
},
|
||||||
|
"style": {},
|
||||||
|
"near_key": null,
|
||||||
|
"shape": {
|
||||||
|
"value": "rectangle"
|
||||||
|
},
|
||||||
|
"direction": {
|
||||||
|
"value": ""
|
||||||
|
},
|
||||||
|
"constraint": {
|
||||||
|
"value": ""
|
||||||
|
}
|
||||||
|
},
|
||||||
|
"zIndex": 0
|
||||||
|
},
|
||||||
|
{
|
||||||
|
"id": "Square 2",
|
||||||
|
"id_val": "Square 2",
|
||||||
|
"label_dimensions": {
|
||||||
|
"width": 0,
|
||||||
|
"height": 0
|
||||||
|
},
|
||||||
|
"references": [
|
||||||
|
{
|
||||||
|
"key": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestMove/container_conflicts_generated.d2,4:2:39-4:10:47",
|
||||||
|
"path": [
|
||||||
|
{
|
||||||
|
"unquoted_string": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestMove/container_conflicts_generated.d2,4:2:39-4:10:47",
|
||||||
|
"value": [
|
||||||
|
{
|
||||||
|
"string": "Square 2",
|
||||||
|
"raw_string": "Square 2"
|
||||||
|
}
|
||||||
|
]
|
||||||
|
}
|
||||||
|
}
|
||||||
|
]
|
||||||
|
},
|
||||||
|
"key_path_index": 0,
|
||||||
|
"map_key_edge_index": -1
|
||||||
|
}
|
||||||
|
],
|
||||||
|
"attributes": {
|
||||||
|
"label": {
|
||||||
|
"value": ""
|
||||||
|
},
|
||||||
|
"style": {},
|
||||||
|
"near_key": null,
|
||||||
|
"shape": {
|
||||||
|
"value": "rectangle"
|
||||||
|
},
|
||||||
|
"direction": {
|
||||||
|
"value": ""
|
||||||
|
},
|
||||||
|
"constraint": {
|
||||||
|
"value": ""
|
||||||
|
}
|
||||||
|
},
|
||||||
|
"zIndex": 0
|
||||||
|
}
|
||||||
|
]
|
||||||
|
},
|
||||||
|
"err": "<nil>"
|
||||||
|
}
|
||||||
Loading…
Reference in a new issue