Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Make tests #5

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Make tests #5

wants to merge 4 commits into from

Conversation

Robar3
Copy link

@Robar3 Robar3 commented Dec 24, 2019

Make 8 test.

@Autowired
BLZServicePortType blzServicePortType;

@Override
public DetailsType getBank(String blz) {
return blzServicePortType.getBank(blz);
}
// @Override
Copy link
Owner

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);
Copy link
Owner

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>
Copy link
Owner

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
Copy link
Owner

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 {
Copy link
Owner

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;
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Зачем этот импорт?

@Robar3 Robar3 requested a review from DarkWeird December 24, 2019 11:53
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants