Skip to content

Commit

Permalink
Format code and fix checkstyle error
Browse files Browse the repository at this point in the history
  • Loading branch information
nmyphp committed Jul 17, 2020
1 parent b2ede44 commit a023fd5
Show file tree
Hide file tree
Showing 5 changed files with 17 additions and 13 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@
* Identify job with job class name.
*/
public class JobClassNameIdentificationStrategy implements JobIdentificationStrategy {

@Override
public String identify(final ElasticJob elasticJob, final String jobName) {
return elasticJob.getClass().getName();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,11 +23,12 @@
* Job identification strategy.
*/
public interface JobIdentificationStrategy {

/**
* Identify job.
*
* @param elasticJob job instance
* @param jobName job name
* @param jobName job name
* @return job identification
*/
String identify(ElasticJob elasticJob, String jobName);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,20 +17,22 @@

package org.apache.shardingsphere.elasticjob.lite.internal.setup;

import java.util.ServiceLoader;
import lombok.AccessLevel;
import lombok.NoArgsConstructor;

import java.util.ServiceLoader;

/**
* Job identification strategy factory.
*/
@NoArgsConstructor(access = AccessLevel.PRIVATE)
public class JobIdentificationStrategyFactory {

private static final JobIdentificationStrategy DEFAULT_STRATEGY = new JobClassNameIdentificationStrategy();

/**
* Get job identification strategy.
*
* @return job identification strategy
*/
public static JobIdentificationStrategy getStrategy() {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,12 +17,13 @@

package org.apache.shardingsphere.elasticjob.lite.internal.setup;

import static org.hamcrest.Matchers.instanceOf;
import static org.junit.Assert.assertThat;
import org.junit.Test;

public class JobIdentificationStrategyFactoryTest {
import static org.hamcrest.CoreMatchers.instanceOf;
import static org.junit.Assert.assertThat;

public class JobIdentificationStrategyFactoryTest {

@Test
public void assertGetDefaultStrategy() {
assertThat(JobIdentificationStrategyFactory.getStrategy(), instanceOf(JobClassNameIdentificationStrategy.class));
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,14 +17,14 @@

package org.apache.shardingsphere.elasticjob.lite.spring.setup;

import static org.hamcrest.Matchers.instanceOf;
import static org.junit.Assert.assertThat;

import org.apache.shardingsphere.elasticjob.lite.internal.setup.JobIdentificationStrategyFactory;
import org.junit.Test;

public class JobIdentificationStrategyFactoryTest {
import static org.hamcrest.CoreMatchers.instanceOf;
import static org.junit.Assert.assertThat;

public class JobIdentificationStrategyFactoryTest {

@Test
public void assertGetStrategy() {
assertThat(JobIdentificationStrategyFactory.getStrategy(), instanceOf(JobClassNameBySpringIdentificationStrategy.class));
Expand Down

0 comments on commit a023fd5

Please sign in to comment.