Skip to content

SiteParserServiceImpl: add unit tests for constructor #983

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.

Sign up for GitHub

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

Jump to bottom
Open
0pdd opened this issue Dec 15, 2018 · 0 comments
Open

SiteParserServiceImpl: add unit tests for constructor #983

0pdd opened this issue Dec 15, 2018 · 0 comments
Labels
area/unit tests techdebt

Comments

@0pdd
Copy link

0pdd commented Dec 15, 2018

The puzzle 975-c9aec551 from #975 has to be resolved:

mystamps/src/main/java/ru/mystamps/web/feature/series/importing/extractor/JsoupSiteParser.java

Line 68 in 5f36177

// @todo #975 SiteParserServiceImpl: add unit tests for constructor

The puzzle was created by Slava Semushin on 15-Dec-18.

If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.
@0pdd 0pdd added the techdebt label Dec 15, 2018
0pdd referenced this issue Dec 15, 2018
@php-coder
refactor: update code to get site parsers from database.
5f36177 
BREAKING CHANGE: site parsers from application.properties are now ignored. It's supposed that they
have been migrated by running application with the code from previous commit. Otherwise,
administrator can migrate them manually.

Addressed to #975
@0pdd 0pdd mentioned this issue Dec 15, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/unit tests techdebt
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@php-coder @0pdd