Skip to content

Commit bf904ea

Browse files
michaelnebelmichaelnebel
committed
C#: Update partial tests and expected output.
1 parent 6ef03b3 commit bf904ea

14 files changed

Lines changed: 183 additions & 164 deletions

‎csharp/ql/test/library-tests/dispatch/CallGraph.expected‎

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -270,9 +270,10 @@
270270
| ViableCallable.cs:679:17:679:20 | Run3 | ViableCallable.cs:637:21:637:21 | M |
271271
| ViableCallable.cs:679:17:679:20 | Run3 | ViableCallable.cs:646:21:646:21 | M |
272272
| ViableCallable.cs:679:17:679:20 | Run3 | ViableCallable.cs:648:21:648:21 | M |
273-
| ViableCallable.cs:709:17:709:20 | Run1 | ViableCallable.cs:703:42:703:44 | get_Property |
274-
| ViableCallable.cs:709:17:709:20 | Run1 | ViableCallable.cs:703:63:703:65 | set_Property |
275-
| ViableCallable.cs:709:17:709:20 | Run1 | ViableCallable.cs:705:49:705:51 | get_Item |
276-
| ViableCallable.cs:709:17:709:20 | Run1 | ViableCallable.cs:705:70:705:72 | set_Item |
277-
| ViableCallable.cs:709:17:709:20 | Run1 | ViableCallable.cs:706:51:706:53 | add_Event |
278-
| ViableCallable.cs:709:17:709:20 | Run1 | ViableCallable.cs:706:59:706:64 | remove_Event |
273+
| ViableCallable.cs:711:17:711:20 | Run1 | ViableCallable.cs:695:24:695:31 | Partial1 |
274+
| ViableCallable.cs:711:17:711:20 | Run1 | ViableCallable.cs:699:43:699:47 | add_Event |
275+
| ViableCallable.cs:711:17:711:20 | Run1 | ViableCallable.cs:699:43:699:47 | remove_Event |
276+
| ViableCallable.cs:711:17:711:20 | Run1 | ViableCallable.cs:705:42:705:44 | get_Property |
277+
| ViableCallable.cs:711:17:711:20 | Run1 | ViableCallable.cs:705:63:705:65 | set_Property |
278+
| ViableCallable.cs:711:17:711:20 | Run1 | ViableCallable.cs:707:49:707:51 | get_Item |
279+
| ViableCallable.cs:711:17:711:20 | Run1 | ViableCallable.cs:707:70:707:72 | set_Item |

‎csharp/ql/test/library-tests/dispatch/GetADynamicTarget.expected‎

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -518,9 +518,10 @@
518518
| ViableCallable.cs:683:9:683:16 | call to method M | C22+TestOverloadResolution2<System.Int32>.M(Int32[]) |
519519
| ViableCallable.cs:687:9:687:16 | call to method M | C22+TestOverloadResolution1<System.Int32>.M(List<int>) |
520520
| ViableCallable.cs:687:9:687:16 | call to method M | C22+TestOverloadResolution2<System.Int32>.M(List<int>) |
521-
| ViableCallable.cs:714:9:714:18 | access to property Property | C23+Partial1.set_Property(object) |
522-
| ViableCallable.cs:717:13:717:22 | access to property Property | C23+Partial1.get_Property() |
523-
| ViableCallable.cs:720:9:720:12 | access to indexer | C23+Partial1.set_Item(int, object) |
524-
| ViableCallable.cs:723:13:723:16 | access to indexer | C23+Partial1.get_Item(int) |
525-
| ViableCallable.cs:726:9:726:15 | access to event Event | C23+Partial1.add_Event(EventHandler) |
526-
| ViableCallable.cs:729:9:729:15 | access to event Event | C23+Partial1.remove_Event(EventHandler) |
521+
| ViableCallable.cs:716:9:716:18 | access to property Property | C23+Partial1.set_Property(object) |
522+
| ViableCallable.cs:719:13:719:22 | access to property Property | C23+Partial1.get_Property() |
523+
| ViableCallable.cs:722:9:722:12 | access to indexer | C23+Partial1.set_Item(int, object) |
524+
| ViableCallable.cs:725:13:725:16 | access to indexer | C23+Partial1.get_Item(int) |
525+
| ViableCallable.cs:728:9:728:15 | access to event Event | C23+Partial1.add_Event(EventHandler) |
526+
| ViableCallable.cs:731:9:731:15 | access to event Event | C23+Partial1.remove_Event(EventHandler) |
527+
| ViableCallable.cs:734:18:734:43 | object creation of type Partial1 | C23+Partial1.Partial1(object) |

‎csharp/ql/test/library-tests/dispatch/ViableCallable.cs‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -692,6 +692,7 @@ public class C23
692692
{
693693
public partial class Partial1
694694
{
695+
public partial Partial1(object obj);
695696
public partial object Property { get; set; }
696697

697698
public partial object this[int index] { get; set; }
@@ -700,6 +701,7 @@ public partial class Partial1
700701

701702
public partial class Partial1
702703
{
704+
public partial Partial1(object obj) { }
703705
public partial object Property { get { return null; } set { } }
704706

705707
public partial object this[int index] { get { return null; } set { } }
@@ -727,5 +729,8 @@ public void Run1(Partial1 p)
727729

728730
// Viable callable: Partial1.remove_Event
729731
p.Event -= (sender, e) => { };
732+
733+
// Viable callable: Partial1.Partial1(object)
734+
var p0 = new Partial1(new object());
730735
}
731736
}

‎csharp/ql/test/library-tests/partial/MethodIsPartial.expected‎

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
1-
| Partial.cs:6:18:6:42 | PartialMethodWithoutBody1 | true |
2-
| Partial.cs:7:17:7:23 | Method2 | false |
3-
| Partial.cs:18:18:18:39 | PartialMethodWithBody1 | true |
4-
| Partial.cs:19:17:19:23 | Method3 | false |
5-
| Partial.cs:41:18:41:42 | PartialMethodWithoutBody2 | true |
6-
| Partial.cs:42:17:42:23 | Method4 | false |
7-
| Partial.cs:47:17:47:23 | Method5 | false |
1+
| Partial.cs:8:18:8:42 | PartialMethodWithoutBody1 | true |
2+
| Partial.cs:9:17:9:23 | Method2 | false |
3+
| Partial.cs:22:18:22:39 | PartialMethodWithBody1 | true |
4+
| Partial.cs:23:17:23:23 | Method3 | false |
5+
| Partial.cs:45:18:45:42 | PartialMethodWithoutBody2 | true |
6+
| Partial.cs:46:17:46:23 | Method4 | false |
7+
| Partial.cs:52:17:52:23 | Method5 | false |

‎csharp/ql/test/library-tests/partial/Partial.cs‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,8 @@
22

33
partial class TwoPartClass
44
{
5+
// Declaring declaration.
6+
public partial TwoPartClass(object obj);
57
partial void PartialMethodWithBody1();
68
partial void PartialMethodWithoutBody1();
79
public void Method2() { }
@@ -15,6 +17,8 @@ public void Method2() { }
1517

1618
partial class TwoPartClass
1719
{
20+
// Implementation declaration.
21+
public partial TwoPartClass(object obj) { }
1822
partial void PartialMethodWithBody1() { }
1923
public void Method3() { }
2024
private object _backingField;
@@ -44,6 +48,7 @@ public void Method4() { }
4448

4549
class NonPartialClass
4650
{
51+
public NonPartialClass(object obj) { }
4752
public void Method5() { }
4853
public object Property { get; set; }
4954
public object this[int index]

‎csharp/ql/test/library-tests/partial/Partial1.expected‎

Lines changed: 15 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,18 @@
11
| Partial.cs:3:15:3:26 | TwoPartClass |
2-
| Partial.cs:6:18:6:42 | PartialMethodWithoutBody1 |
3-
| Partial.cs:16:15:16:26 | TwoPartClass |
4-
| Partial.cs:18:18:18:39 | PartialMethodWithBody1 |
5-
| Partial.cs:22:27:22:42 | PartialProperty1 |
6-
| Partial.cs:24:9:24:11 | get_PartialProperty1 |
7-
| Partial.cs:25:9:25:11 | set_PartialProperty1 |
8-
| Partial.cs:29:27:29:30 | Item |
9-
| Partial.cs:31:9:31:11 | get_Item |
10-
| Partial.cs:32:9:32:11 | set_Item |
11-
| Partial.cs:36:39:36:51 | PartialEvent1 |
12-
| Partial.cs:36:55:36:57 | add_PartialEvent1 |
13-
| Partial.cs:36:63:36:68 | remove_PartialEvent1 |
14-
| Partial.cs:39:15:39:33 | OnePartPartialClass |
15-
| Partial.cs:41:18:41:42 | PartialMethodWithoutBody2 |
2+
| Partial.cs:6:20:6:31 | TwoPartClass |
3+
| Partial.cs:8:18:8:42 | PartialMethodWithoutBody1 |
4+
| Partial.cs:18:15:18:26 | TwoPartClass |
5+
| Partial.cs:22:18:22:39 | PartialMethodWithBody1 |
6+
| Partial.cs:26:27:26:42 | PartialProperty1 |
7+
| Partial.cs:28:9:28:11 | get_PartialProperty1 |
8+
| Partial.cs:29:9:29:11 | set_PartialProperty1 |
9+
| Partial.cs:33:27:33:30 | Item |
10+
| Partial.cs:35:9:35:11 | get_Item |
11+
| Partial.cs:36:9:36:11 | set_Item |
12+
| Partial.cs:40:39:40:51 | PartialEvent1 |
13+
| Partial.cs:40:55:40:57 | add_PartialEvent1 |
14+
| Partial.cs:40:63:40:68 | remove_PartialEvent1 |
15+
| Partial.cs:43:15:43:33 | OnePartPartialClass |
16+
| Partial.cs:45:18:45:42 | PartialMethodWithoutBody2 |
1617
| PartialMultipleFiles1.cs:1:22:1:41 | PartialMultipleFiles |
1718
| PartialMultipleFiles2.cs:1:22:1:41 | PartialMultipleFiles |

‎csharp/ql/test/library-tests/partial/Partial2.expected‎

Lines changed: 11 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,13 @@
1-
| Partial.cs:3:15:3:26 | TwoPartClass | Partial.cs:3:15:3:26 | <object initializer> |
2-
| Partial.cs:3:15:3:26 | TwoPartClass | Partial.cs:6:18:6:42 | PartialMethodWithoutBody1 |
3-
| Partial.cs:3:15:3:26 | TwoPartClass | Partial.cs:7:17:7:23 | Method2 |
4-
| Partial.cs:3:15:3:26 | TwoPartClass | Partial.cs:18:18:18:39 | PartialMethodWithBody1 |
5-
| Partial.cs:3:15:3:26 | TwoPartClass | Partial.cs:19:17:19:23 | Method3 |
6-
| Partial.cs:16:15:16:26 | TwoPartClass | Partial.cs:3:15:3:26 | <object initializer> |
7-
| Partial.cs:16:15:16:26 | TwoPartClass | Partial.cs:6:18:6:42 | PartialMethodWithoutBody1 |
8-
| Partial.cs:16:15:16:26 | TwoPartClass | Partial.cs:7:17:7:23 | Method2 |
9-
| Partial.cs:16:15:16:26 | TwoPartClass | Partial.cs:18:18:18:39 | PartialMethodWithBody1 |
10-
| Partial.cs:16:15:16:26 | TwoPartClass | Partial.cs:19:17:19:23 | Method3 |
11-
| Partial.cs:39:15:39:33 | OnePartPartialClass | Partial.cs:39:15:39:33 | <object initializer> |
12-
| Partial.cs:39:15:39:33 | OnePartPartialClass | Partial.cs:41:18:41:42 | PartialMethodWithoutBody2 |
13-
| Partial.cs:39:15:39:33 | OnePartPartialClass | Partial.cs:42:17:42:23 | Method4 |
1+
| Partial.cs:3:15:3:26 | TwoPartClass | Partial.cs:8:18:8:42 | PartialMethodWithoutBody1 |
2+
| Partial.cs:3:15:3:26 | TwoPartClass | Partial.cs:9:17:9:23 | Method2 |
3+
| Partial.cs:3:15:3:26 | TwoPartClass | Partial.cs:22:18:22:39 | PartialMethodWithBody1 |
4+
| Partial.cs:3:15:3:26 | TwoPartClass | Partial.cs:23:17:23:23 | Method3 |
5+
| Partial.cs:18:15:18:26 | TwoPartClass | Partial.cs:8:18:8:42 | PartialMethodWithoutBody1 |
6+
| Partial.cs:18:15:18:26 | TwoPartClass | Partial.cs:9:17:9:23 | Method2 |
7+
| Partial.cs:18:15:18:26 | TwoPartClass | Partial.cs:22:18:22:39 | PartialMethodWithBody1 |
8+
| Partial.cs:18:15:18:26 | TwoPartClass | Partial.cs:23:17:23:23 | Method3 |
9+
| Partial.cs:43:15:43:33 | OnePartPartialClass | Partial.cs:43:15:43:33 | <object initializer> |
10+
| Partial.cs:43:15:43:33 | OnePartPartialClass | Partial.cs:45:18:45:42 | PartialMethodWithoutBody2 |
11+
| Partial.cs:43:15:43:33 | OnePartPartialClass | Partial.cs:46:17:46:23 | Method4 |
1412
| PartialMultipleFiles1.cs:1:22:1:41 | PartialMultipleFiles | PartialMultipleFiles1.cs:1:22:1:41 | <object initializer> |
1513
| PartialMultipleFiles2.cs:1:22:1:41 | PartialMultipleFiles | PartialMultipleFiles1.cs:1:22:1:41 | <object initializer> |

‎csharp/ql/test/library-tests/partial/PartialAccessors.expected‎

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
1-
| Partial.cs:24:9:24:11 | get_PartialProperty1 | true |
2-
| Partial.cs:25:9:25:11 | set_PartialProperty1 | true |
3-
| Partial.cs:31:9:31:11 | get_Item | true |
4-
| Partial.cs:32:9:32:11 | set_Item | true |
5-
| Partial.cs:36:55:36:57 | add_PartialEvent1 | true |
6-
| Partial.cs:36:63:36:68 | remove_PartialEvent1 | true |
7-
| Partial.cs:48:30:48:32 | get_Property | false |
8-
| Partial.cs:48:35:48:37 | set_Property | false |
9-
| Partial.cs:51:9:51:11 | get_Item | false |
10-
| Partial.cs:52:9:52:11 | set_Item | false |
11-
| Partial.cs:54:31:54:35 | add_Event | false |
12-
| Partial.cs:54:31:54:35 | remove_Event | false |
1+
| Partial.cs:28:9:28:11 | get_PartialProperty1 | true |
2+
| Partial.cs:29:9:29:11 | set_PartialProperty1 | true |
3+
| Partial.cs:35:9:35:11 | get_Item | true |
4+
| Partial.cs:36:9:36:11 | set_Item | true |
5+
| Partial.cs:40:55:40:57 | add_PartialEvent1 | true |
6+
| Partial.cs:40:63:40:68 | remove_PartialEvent1 | true |
7+
| Partial.cs:53:30:53:32 | get_Property | false |
8+
| Partial.cs:53:35:53:37 | set_Property | false |
9+
| Partial.cs:56:9:56:11 | get_Item | false |
10+
| Partial.cs:57:9:57:11 | set_Item | false |
11+
| Partial.cs:59:31:59:35 | add_Event | false |
12+
| Partial.cs:59:31:59:35 | remove_Event | false |

‎csharp/ql/test/library-tests/partial/PartialConstructors.expected‎

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
| Partial.cs:3:15:3:26 | TwoPartClass | Partial.cs:3:15:3:26 | {...} |
2-
| Partial.cs:39:15:39:33 | OnePartPartialClass | Partial.cs:39:15:39:33 | {...} |
3-
| Partial.cs:45:7:45:21 | NonPartialClass | Partial.cs:45:7:45:21 | {...} |
1+
| Partial.cs:43:15:43:33 | OnePartPartialClass | Partial.cs:43:15:43:33 | {...} |
2+
| Partial.cs:51:12:51:26 | NonPartialClass | Partial.cs:51:40:51:42 | {...} |
43
| PartialMultipleFiles1.cs:1:22:1:41 | PartialMultipleFiles | PartialMultipleFiles1.cs:1:22:1:41 | {...} |

‎csharp/ql/test/library-tests/partial/PartialEvents.expected‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,2 @@
1-
| Partial.cs:36:39:36:51 | PartialEvent1 | true |
2-
| Partial.cs:54:31:54:35 | Event | false |
1+
| Partial.cs:40:39:40:51 | PartialEvent1 | true |
2+
| Partial.cs:59:31:59:35 | Event | false |

0 commit comments

Comments
 (0)