Fix bug that breaks live hover to tunneled cf apps

Also add a regression test that would have caught this issue.
This commit is contained in:
Kris De Volder
2019-07-11 15:05:49 -07:00
parent 19f64ce225
commit 7a6e311147
2 changed files with 28 additions and 1 deletions

View File

@@ -0,0 +1,27 @@
/*******************************************************************************
* Copyright (c) 2018, 2019 Pivotal, Inc.
* All rights reserved. This program and the accompanying materials
* are made available under the terms of the Eclipse Public License v1.0
* which accompanies this distribution, and is available at
* https://www.eclipse.org/legal/epl-v10.html
*
* Contributors:
* Pivotal, Inc. - initial API and implementation
*******************************************************************************/
package org.springframework.ide.vscode.commons.boot.app.cli;
import static org.junit.Assert.assertNotNull;
import static org.junit.Assert.assertTrue;
import org.junit.Test;
public class RemoteSpringBootAppTest {
@Test
public void canCreateInstance() throws Exception {
SpringBootApp instance = RemoteSpringBootApp.create("jmx:blah", "whatever.cfapps.io", "8888", "https", true);
assertNotNull(instance);
assertTrue(instance instanceof RemoteSpringBootApp);
}
}

View File

@@ -198,7 +198,7 @@ public class MemoizingProxy {
.defineField(F_CACHE, Cache.class, Opcodes.ACC_PRIVATE)
.method(CACHABLE_METHODS).intercept(MethodDelegation.to(SuperMethodInterceptor.class))
.defineConstructor(Visibility.PUBLIC).withParameters(argTypes).intercept(
MethodCall.invoke(klass.getConstructor(argTypes)).withAllArguments()
MethodCall.invoke(klass.getDeclaredConstructor(argTypes)).withAllArguments()
.andThen(METHOD_DELEGATION.to(ClassConstructorInterceptor.class))
);