From 4160c186e13a106a3eb4ed8835d9f2f867124c24 Mon Sep 17 00:00:00 2001 From: Fedor Isakov Date: Wed, 23 Oct 2019 15:39:17 +0200 Subject: [PATCH] Introduce dependency to Config in typechecking options and service. Avoid relying on static getInstance() method to access Config. Config is an application-level "component" initialized from an application plugin. All usages on application/project-level should declare explicit dependency on it. Tests can either create a new instance or access the shared one. --- .../jetbrains.mps.coderules.plugin@tests.mps | 104 +- ...ains.mps.coderules.typechecking.plugin.mps | 854 +++++++++-------- ...ins.mps.coderules.typechecking.service.mps | 885 +++++++++++++----- .../jetbrains.mps.coderules/models/plugin.mps | 512 ++++++---- 4 files changed, 1473 insertions(+), 882 deletions(-) diff --git a/coderules/solutions/jetbrains.mps.coderules.test/models/jetbrains.mps.coderules.plugin@tests.mps b/coderules/solutions/jetbrains.mps.coderules.test/models/jetbrains.mps.coderules.plugin@tests.mps index 86bad666..d2065681 100644 --- a/coderules/solutions/jetbrains.mps.coderules.test/models/jetbrains.mps.coderules.plugin@tests.mps +++ b/coderules/solutions/jetbrains.mps.coderules.test/models/jetbrains.mps.coderules.plugin@tests.mps @@ -14,6 +14,7 @@ + @@ -32,6 +33,10 @@ + + + + @@ -64,6 +69,12 @@ + + + + + + @@ -72,6 +83,7 @@ + @@ -111,6 +123,18 @@ + + + + + + + + + + + + @@ -120,9 +144,8 @@ - - - + + @@ -170,9 +193,8 @@ - - - + + @@ -184,9 +206,8 @@ - - - + + @@ -198,9 +219,8 @@ - - - + + @@ -219,9 +239,8 @@ - - - + + @@ -266,9 +285,8 @@ - - - + + @@ -281,9 +299,8 @@ - - - + + @@ -331,9 +348,8 @@ - - - + + @@ -345,9 +361,8 @@ - - - + + @@ -362,9 +377,8 @@ - - - + + @@ -383,9 +397,8 @@ - - - + + @@ -433,9 +446,8 @@ - - - + + @@ -448,9 +460,8 @@ - - - + + @@ -498,9 +509,8 @@ - - - + + @@ -518,9 +528,8 @@ - - - + + @@ -537,9 +546,8 @@ - - - + + diff --git a/coderules/solutions/jetbrains.mps.coderules.typechecking/models/jetbrains.mps.coderules.typechecking.plugin.mps b/coderules/solutions/jetbrains.mps.coderules.typechecking/models/jetbrains.mps.coderules.typechecking.plugin.mps index 9321d023..94f394dc 100644 --- a/coderules/solutions/jetbrains.mps.coderules.typechecking/models/jetbrains.mps.coderules.typechecking.plugin.mps +++ b/coderules/solutions/jetbrains.mps.coderules.typechecking/models/jetbrains.mps.coderules.typechecking.plugin.mps @@ -171,9 +171,7 @@ - - - + @@ -262,11 +260,6 @@ - - - - - @@ -402,23 +395,26 @@ - - - - - - - - + + + + + + - - - - - + + + + + + + + + + @@ -430,11 +426,14 @@ - - - - - + + + + + + + + @@ -854,14 +853,18 @@ - - - - - + + + + + + + + + @@ -872,14 +875,18 @@ - - - - - + + + + + + + + + @@ -890,14 +897,18 @@ - - - - - + + + + + + + + + @@ -1180,51 +1191,6 @@ - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - @@ -1251,9 +1217,6 @@ - - - @@ -1305,6 +1268,7 @@ + @@ -1323,19 +1287,6 @@ - - - - - - - - - - - - - @@ -1343,28 +1294,8 @@ - - - - - - - - - - - - - - - - - - - - @@ -1382,11 +1313,11 @@ - + - + @@ -1404,57 +1335,57 @@ - - + + - - - - + + + + - - + + - - - - + + + + - + - + - + - + - + - + - + - + - + - + @@ -1509,7 +1440,7 @@ - + @@ -1536,7 +1467,7 @@ - + @@ -1566,11 +1497,11 @@ - + - + @@ -1588,63 +1519,63 @@ - - + + - - - - + + + + - - + + - - - - + + + + - + - + - + - + - + - + - + - + - + - + - + - + @@ -1709,7 +1640,7 @@ - + @@ -1736,7 +1667,7 @@ - + @@ -1768,11 +1699,11 @@ - + - + @@ -1790,63 +1721,63 @@ - - + + - - - - + + + + - - + + - - - - + + + + - + - + - + - + - + - + - + - + - + - + - + - + @@ -1911,7 +1842,7 @@ - + @@ -1938,7 +1869,7 @@ - + @@ -2077,11 +2008,11 @@ - + - + @@ -2132,52 +2063,52 @@ - - - + + + - + - - - - - + + + + + - + - + - + - + - - + + - + - + - + - + - + @@ -2422,141 +2353,6 @@ - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - @@ -2564,12 +2360,139 @@ - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + @@ -2580,10 +2503,17 @@ + + + + + + + - + @@ -2600,14 +2530,83 @@ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + @@ -2638,6 +2637,12 @@ + + + + + + @@ -2742,10 +2747,33 @@ + + + + + + + + + + + + + + + + + + + + + + + @@ -4103,6 +4131,12 @@ + + + + + + @@ -4121,6 +4155,9 @@ + + + @@ -4320,14 +4357,19 @@ + + + + + + - - - + + @@ -4462,9 +4504,8 @@ - - - + + @@ -4479,6 +4520,12 @@ + + + + + + @@ -4498,6 +4545,19 @@ + + + + + + + + + + + + + @@ -4505,6 +4565,12 @@ + + + + + + @@ -4515,6 +4581,9 @@ + + + @@ -4528,6 +4597,9 @@ + + + @@ -4546,6 +4618,12 @@ + + + + + + @@ -4606,9 +4684,8 @@ - - - + + @@ -4753,9 +4830,8 @@ - - - + + @@ -4770,17 +4846,29 @@ + + + + + + - + + + + + + + @@ -4798,6 +4886,10 @@ + + + + diff --git a/coderules/solutions/jetbrains.mps.coderules.typechecking/models/jetbrains.mps.coderules.typechecking.service.mps b/coderules/solutions/jetbrains.mps.coderules.typechecking/models/jetbrains.mps.coderules.typechecking.service.mps index deecc1a5..9a1b42be 100644 --- a/coderules/solutions/jetbrains.mps.coderules.typechecking/models/jetbrains.mps.coderules.typechecking.service.mps +++ b/coderules/solutions/jetbrains.mps.coderules.typechecking/models/jetbrains.mps.coderules.typechecking.service.mps @@ -91,6 +91,9 @@ + + + @@ -4787,6 +4790,592 @@ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + @@ -4927,16 +5516,15 @@ - - - - - - - - - - + + + + + + + + + @@ -4960,75 +5548,22 @@ - - - - - + + + + - - - - + + + - - - - - - - - - - - - - + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - @@ -5047,75 +5582,22 @@ - - - - - + + + + - - - - + + + - - - - - - - - - - - - - + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - @@ -5129,16 +5611,15 @@ - - - - - - - - - - + + + + + + + + + @@ -5159,11 +5640,14 @@ - - - - - + + + + + + + + @@ -7943,114 +8427,7 @@ - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - diff --git a/coderules/solutions/jetbrains.mps.coderules/models/plugin.mps b/coderules/solutions/jetbrains.mps.coderules/models/plugin.mps index 9567571a..3bfef711 100644 --- a/coderules/solutions/jetbrains.mps.coderules/models/plugin.mps +++ b/coderules/solutions/jetbrains.mps.coderules/models/plugin.mps @@ -274,9 +274,6 @@ - - - @@ -378,6 +375,7 @@ + @@ -392,6 +390,7 @@ + @@ -7030,8 +7029,9 @@ - - + + + @@ -7050,8 +7050,36 @@ + + - + + + + + + + + + + + + + + + + + + + + + + + + + + + @@ -7065,202 +7093,37 @@ - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + - - - - - + + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + - + @@ -7294,17 +7157,9 @@ - - + + - - - - - - - - @@ -7421,8 +7276,205 @@ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + @@ -8078,6 +8130,44 @@ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + @@ -8216,5 +8306,29 @@ + + + + + + + + + + + + + + + + + + + + + + + +