Skip to content

Commit 585fb33

Browse files
committed
test(rules/renderer): Add tests for rendererer method declarations
1 parent a2b73b4 commit 585fb33

File tree

5 files changed

+53
-28
lines changed

5 files changed

+53
-28
lines changed
Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
sap.ui.define(["sap/ui/core/Control"], function (Control) {
2+
var myControl = Control.extend("mycomp.myControl", {
3+
metadata: {},
4+
renderer: function(oRm, oControl) {
5+
}
6+
});
7+
return myControl;
8+
});
Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
sap.ui.define(["sap/ui/core/Control"], function (Control) {
2+
var myControl = Control.extend("mycomp.myControl", {
3+
metadata: {},
4+
renderer(oRm, oControl) {
5+
}
6+
});
7+
return myControl;
8+
});

test/fixtures/linter/rules/renderer/ControlRendererDeclaration_negative.js

Lines changed: 0 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -19,24 +19,6 @@ sap.ui.define([
1919
renderer: undefined
2020
});
2121

22-
const NegativeExample4 = Control.extend("sap.ui.demo.linter.controls.NegativeExample4", {
23-
metadata: {},
24-
renderer: function(oRm, oControl) {
25-
}
26-
});
27-
28-
const NegativeExample5 = Control.extend("sap.ui.demo.linter.controls.NegativeExample5", {
29-
metadata: {},
30-
renderer: (oRm, oControl) => {
31-
}
32-
});
33-
34-
const NegativeExample6 = Control.extend("sap.ui.demo.linter.controls.NegativeExample6", {
35-
metadata: {},
36-
renderer(oRm, oControl) {
37-
}
38-
});
39-
4022
const NegativeExample7 = Button.extend("sap.ui.demo.linter.controls.NegativeExample7", {
4123
metadata: {},
4224
renderer: {

test/lib/linter/rules/snapshots/renderer.ts.md

Lines changed: 37 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -63,6 +63,42 @@ Generated by [AVA](https://avajs.dev).
6363
],
6464
warningCount: 0,
6565
},
66+
{
67+
coverageInfo: [],
68+
errorCount: 1,
69+
fatalErrorCount: 0,
70+
filePath: '13Control.js',
71+
messages: [
72+
{
73+
column: 3,
74+
line: 4,
75+
message: 'Use of deprecated renderer detected. Define explicitly the {apiVersion: 2} parameter in the renderer object',
76+
messageDetails: 'Renderer Object (https://ui5.sap.com/#/topic/c9ab34570cc14ea5ab72a6d1a4a03e3f)',
77+
ruleId: 'no-deprecated-api',
78+
severity: 2,
79+
ui5TypeInfo: undefined,
80+
},
81+
],
82+
warningCount: 0,
83+
},
84+
{
85+
coverageInfo: [],
86+
errorCount: 1,
87+
fatalErrorCount: 0,
88+
filePath: '14Control.js',
89+
messages: [
90+
{
91+
column: 3,
92+
line: 4,
93+
message: 'Use of deprecated renderer detected. Define explicitly the {apiVersion: 2} parameter in the renderer object',
94+
messageDetails: 'Renderer Object (https://ui5.sap.com/#/topic/c9ab34570cc14ea5ab72a6d1a4a03e3f)',
95+
ruleId: 'no-deprecated-api',
96+
severity: 2,
97+
ui5TypeInfo: undefined,
98+
},
99+
],
100+
warningCount: 0,
101+
},
66102
{
67103
coverageInfo: [
68104
{
@@ -473,7 +509,7 @@ Generated by [AVA](https://avajs.dev).
473509
},
474510
{
475511
coverageInfo: [],
476-
errorCount: 2,
512+
errorCount: 1,
477513
fatalErrorCount: 0,
478514
filePath: 'ControlRendererDeclaration_negative.js',
479515
messages: [
@@ -494,15 +530,6 @@ Generated by [AVA](https://avajs.dev).
494530
},
495531
},
496532
},
497-
{
498-
column: 13,
499-
line: 30,
500-
message: 'Use of deprecated renderer detected. Define explicitly the {apiVersion: 2} parameter in the renderer object',
501-
messageDetails: 'Renderer Object (https://ui5.sap.com/#/topic/c9ab34570cc14ea5ab72a6d1a4a03e3f)',
502-
ruleId: 'no-deprecated-api',
503-
severity: 2,
504-
ui5TypeInfo: undefined,
505-
},
506533
],
507534
warningCount: 0,
508535
},
117 Bytes
Binary file not shown.

0 commit comments

Comments
 (0)