-
Notifications
You must be signed in to change notification settings - Fork 2
Conversation
526aa26
to
7f1e478
Compare
|
||
private static final Logger LOGGER = LoggerFactory.getLogger(CatalogUpdater.class); | ||
|
||
public void updtateRawCatalog(Map<String, String> schemaIdsToRawJsonSchemaCache, Path basePath, Collection<Path> paths) { |
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.
the method arguments can be final
return ofNullable(schemaIdsToRawJsonSchemaCache.get(schemaId)); | ||
} | ||
|
||
public void updateCatalogSchemaCache(final Path basePath, final Collection<Path> paths) { |
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.
There needs to be a test for this method
public class CatalogUpdaterTest { | ||
|
||
/*@Mock | ||
private Logger logger;*/ |
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.
Can this be removed?
@InjectMocks | ||
private RawCatalog rawCatalog; | ||
|
||
@InjectMocks private CatalogUpdater catalogUpdater; |
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.
private CatalogUpdater catalogUpdater; could be on a separate line
schemaIdsToRawJsonSchemaCache.put("http://justice.gov.uk/standards/address.json", "json schema" ); | ||
|
||
rawCatalog.initialize(); | ||
Collection<Path> paths = new ArrayList<>(); |
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.
paths can be final
schemaIdsToRawJsonSchemaCache.put("http://justice.gov.uk/standards/address.json", "json schema" ); | ||
|
||
rawCatalog.initialize(); | ||
Collection<Path> paths = new ArrayList<>(); |
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.
paths can be final
schemaIdsToRawJsonSchemaCache.put("http://justice.gov.uk/standards/address.json", "json schema"); | ||
|
||
rawCatalog.initialize(); | ||
Collection<Path> paths = new ArrayList<>(); |
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.
paths can be final
67e6970
to
7d77967
Compare
7d77967
to
48d2ae4
Compare
48d2ae4
to
429bb8b
Compare
No description provided.