-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make tests #5
base: master
Are you sure you want to change the base?
Make tests #5
Conversation
@Autowired | ||
BLZServicePortType blzServicePortType; | ||
|
||
@Override | ||
public DetailsType getBank(String blz) { | ||
return blzServicePortType.getBank(blz); | ||
} | ||
// @Override |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Закомментированый код - мертвый код. Удалить
@@ -6,4 +6,5 @@ | |||
public interface BlzCodeService { | |||
|
|||
DetailsType getBank(String blz); | |||
// DetailsType getCalc(String calk); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Закоментированный код - мертвый код
@@ -48,6 +48,13 @@ | |||
<artifactId>sqlite-jdbc</artifactId> | |||
<version>3.8.11.2</version> | |||
</dependency> | |||
<!-- https://mvnrepository.com/artifact/com.sun.xml.ws/jaxws-rt --> | |||
<dependency> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Это работает?
public class CalcExampleTest extends SpringCalcTest{ | ||
@Autowired | ||
CalculatorSoap calculatorSoap; | ||
@Test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Проблемы с отступами для аннотаций. в IDEA ctrl+alt+L
import org.springframework.test.context.ContextConfiguration; | ||
import org.springframework.test.context.testng.AbstractTestNGSpringContextTests; | ||
@ContextConfiguration(locations ={"classpath:spring-context.xml"} ) | ||
public class SpringCalcTest extends AbstractTestNGSpringContextTests { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
не зачем создавать лишний "Base" класс
@@ -5,6 +5,7 @@ | |||
import me.darkweird.testing.service.BlzCodeService; | |||
import org.springframework.beans.factory.annotation.Autowired; | |||
import org.springframework.stereotype.Service; | |||
import ru.Robar3.CalculatorSoap; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Зачем этот импорт?
Make 8 test.