Skip to content

Commit f0fdbc5

Browse files
authored
fix MockBuilder (#26)
1 parent 80307cc commit f0fdbc5

File tree

4 files changed

+22
-10
lines changed

4 files changed

+22
-10
lines changed

compiler/src/main/java/io/jbock/simple/processor/writing/MockBuilder.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -131,6 +131,8 @@ private List<MethodSpec> getMethods() {
131131
FieldSpec auxField = FieldSpec.builder(TypeName.BOOLEAN, namedBinding.auxName(), PRIVATE).build();
132132
method.addStatement("this.$N = $L", auxField, true);
133133
}
134+
method.addStatement("return this");
135+
method.returns(getClassName());
134136
methods.add(method.build());
135137
}
136138
return methods;

compiler/src/test/java/io/jbock/simple/processor/BuilderTest.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -205,8 +205,9 @@ void builderParameter() {
205205
" return new Builder_Impl(this);",
206206
" }",
207207
"",
208-
" public void a(TestClass.A a) {",
208+
" public MockBuilder a(TestClass.A a) {",
209209
" this.a = a;",
210+
" return this;",
210211
" }",
211212
" }",
212213
"}");

compiler/src/test/java/io/jbock/simple/processor/PrimitiveTest.java

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -108,13 +108,15 @@ void providesPrimitive() {
108108
" return new Factory_Impl(this);",
109109
" }",
110110
"",
111-
" public void b(int b) {",
111+
" public MockBuilder b(int b) {",
112112
" this.b = b;",
113113
" this.b_isSet = true;",
114+
" return this;",
114115
" }",
115116
"",
116-
" public void a(TestClass.A a) {",
117+
" public MockBuilder a(TestClass.A a) {",
117118
" this.a = a;",
119+
" return this;",
118120
" }",
119121
" }",
120122
"}");

compiler/src/test/java/io/jbock/simple/processor/ProcessorComponentTest.java

Lines changed: 14 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -81,14 +81,17 @@ void staticMethodBindings() {
8181
" TestClass.A a = this.a != null ? this.a : new TestClass.A(b);",
8282
" return new TestClass_AComponent_Impl(a);",
8383
" }",
84-
" public void c(TestClass.C c) {",
84+
" public MockBuilder c(TestClass.C c) {",
8585
" this.c = c;",
86+
" return this;",
8687
" }",
87-
" public void b(TestClass.B b) {",
88+
" public MockBuilder b(TestClass.B b) {",
8889
" this.b = b;",
90+
" return this;",
8991
" }",
90-
" public void a(TestClass.A a) {",
92+
" public MockBuilder a(TestClass.A a) {",
9193
" this.a = a;",
94+
" return this;",
9295
" }",
9396
" }",
9497
"}");
@@ -172,20 +175,24 @@ void dependencyDiamond() {
172175
" return new TestClass_AComponent_Impl(a);",
173176
" }",
174177
"",
175-
" void e(TestClass.E e) {",
178+
" MockBuilder e(TestClass.E e) {",
176179
" this.e = e;",
180+
" return this;",
177181
" }",
178182
"",
179-
" void c(TestClass.C c) {",
183+
" MockBuilder c(TestClass.C c) {",
180184
" this.c = c;",
185+
" return this;",
181186
" }",
182187
"",
183-
" void b(TestClass.B b) {",
188+
" MockBuilder b(TestClass.B b) {",
184189
" this.b = b;",
190+
" return this;",
185191
" }",
186192
"",
187-
" void a(TestClass.A a) {",
193+
" MockBuilder a(TestClass.A a) {",
188194
" this.a = a;",
195+
" return this;",
189196
" }",
190197
" }",
191198
"}");

0 commit comments

Comments
 (0)