Как протестировать класс реализующий интерфейс Runnable?

Question

[thatmaniscool]

Как протестировать класс, где используется реализация интерфейса Runnable ?
В настоящий момент пытаюсь протестировать его в JUNIT 4, но тестирование не срабатывает, хотя код точно работает.

Comments

[сергей кузьмин]

вы код покажите то

[thatmaniscool]

сергей кузьмин, элементраный код.

@Test
    public void justTest() {
        Thread thread = new Thread(new Runnable() {
            @Override
            public void run() {
                System.out.println("HEHEHEHEHEHEHEH");
            }
        });

        thread.start();
    }

[сергей кузьмин]

thatmaniscool, пока из скопипастенного тестировать нечего да и никак невозможно - все классы конкретные

[thatmaniscool]

сергей кузьмин,
Есть интерфейс, который реализуется в каком - либо классе.

public interface LoaderFromNet {
    public void loadFromNetDone (final String object);
}

Есть вспомогательный интерфейс, который реализуется в рамках одного пакета.

public interface LoadingDataFromNet {
    public void loadingDataFromNet (final String object);
}

Есть базовый класс пакета, который реализует вспомогательный интерфейс и принимает в конструкторе интерфейс

/**
 * Base Net class for download data from internet
 */
public class Net implements LoadingDataFromNet {
    private final String log_Net = "log_Net";
    private LoaderFromNet loaderFromNet;
    private String url;

    /**
     * Public constructor sets main parameters for downloading json string
     *
     * @param loaderFromNet - Interface
     * @param url           - website's address
     */
    public Net(LoaderFromNet loaderFromNet, String url) {
        this.loaderFromNet = loaderFromNet;
        this.url = url;
        execute();
    }

    private void execute() {
        if (loaderFromNet == null)
            throw new NullPointerException("Loader interface is null!");
        if (url == null)
            throw new NullPointerException("URL is null!");

        new Loading(this, url);

    }

    @Override
    public void loadingDataFromNet(String object) {
        loaderFromNet.loadFromNetDone(object);
    }
}

И вспомогательный класс в рамках пакета.

public class Loading implements Runnable {
    private LoadingDataFromNet loadingDataFromNet;
    private String url;

    protected Loading(LoadingDataFromNet loadingDataFromNet, String url) {
        this.loadingDataFromNet = loadingDataFromNet;
        this.url = url;
        execute();
    }

    private void execute() {
        new Thread(this).start();
    }

    @Override
    public void run() {
        try {
            HttpsURLConnection httpsURLConnection = (HttpsURLConnection) new URL(url).openConnection();
            httpsURLConnection.setRequestMethod("GET");
            httpsURLConnection.connect();
            int responseCode = httpsURLConnection.getResponseCode();
            httpsURLConnection.disconnect();

            if (responseCode == 200){
                InputStream inputStream = new URL(url).openStream();
                InputStreamReader inputStreamReader = new InputStreamReader(inputStream, StandardCharsets.UTF_8);
                BufferedReader bufferedReader = new BufferedReader(inputStreamReader);

                int letter = -1;
                StringBuilder stringBuilder = new StringBuilder();

                while ((letter = bufferedReader.read()) != -1)
                    stringBuilder.append((char) letter);

                bufferedReader.close();
                inputStreamReader.close();
                inputStream.close();

                loadingDataFromNet.loadingDataFromNet(stringBuilder.toString());

            } else {
                loadingDataFromNet.loadingDataFromNet(null);
            }

        } catch (IOException e) {
            e.printStackTrace();
        }

    }
}

В данный момент суть следующая, просто скачать json строку. В будущем привести все в понятный вид и сделать многопоточность.
Вот эту многопоточном мне и нужно оттестировать. А он не тестируется даже с одним потоком.

[сергей кузьмин]

thatmaniscool, сами видите

new Thread(this) - опс класс конкретный - юнит тест это не делает
тоже самое дальше повсюду
new URL(url).openConnection() опс
new InputStreamReader(inputStream, StandardCharsets.UTF_8); опс

[thatmaniscool]

сергей кузьмин, так что получается? Мне нужно объявлять классы не анонимно?
Переписал и ничего не поменялось. При этом с интерфейсом Callable работает корректно, про проблема в том, что он работает медлее Runnable.
Может есть другие способы тестирования многопоточности?

/**
 * Base Net class for download data from internet
 */
public class Net implements LoadingDataFromNet {
    private final String log_Net = "log_Net";
    private LoaderFromNet loaderFromNet;
    private String url;
    private Loading loading;

    /**
     * Public constructor sets main parameters for downloading json string
     *
     * @param loaderFromNet - Interface
     * @param url           - website's address
     */
    public Net(LoaderFromNet loaderFromNet, String url) {
        this.loaderFromNet = loaderFromNet;
        this.url = url;
        execute();
    }

    private void execute() {
        if (loaderFromNet == null)
            throw new NullPointerException("Loader interface is null!");
        if (url == null)
            throw new NullPointerException("URL is null!");

       loading =  new Loading(this, url);

    }

    @Override
    public void loadingDataFromNet(String object) {
        loaderFromNet.loadFromNetDone(object);
    }
}

public class Loading implements Runnable {
    private LoadingDataFromNet loadingDataFromNet;
    private String url;

    protected Loading(LoadingDataFromNet loadingDataFromNet, String url) {
        this.loadingDataFromNet = loadingDataFromNet;
        this.url = url;
        execute();
    }

    private void execute() {
        new Thread(this).start();
    }

    @Override
    public void run() {
        try {
            URL mURL = new URL(url);
            HttpsURLConnection httpsURLConnection = (HttpsURLConnection) mURL.openConnection();
            httpsURLConnection.setRequestMethod("GET");
            httpsURLConnection.connect();
            int responseCode = httpsURLConnection.getResponseCode();
            httpsURLConnection.disconnect();

            if (responseCode == 200){
                InputStream inputStream = mURL.openStream();
                InputStreamReader inputStreamReader = new InputStreamReader(inputStream, StandardCharsets.UTF_8);
                BufferedReader bufferedReader = new BufferedReader(inputStreamReader);

                int letter = -1;
                StringBuilder stringBuilder = new StringBuilder();

                while ((letter = bufferedReader.read()) != -1)
                    stringBuilder.append((char) letter);

                bufferedReader.close();
                inputStreamReader.close();
                inputStream.close();

                loadingDataFromNet.loadingDataFromNet(stringBuilder.toString());

            } else {
                loadingDataFromNet.loadingDataFromNet(null);
            }

        } catch (IOException e) {
            e.printStackTrace();
        }

    }
}

[сергей кузьмин]

имелось в виду что тестирование хорошо работает с кодом котоый делает инверсию зависимостей и плохо от слова совсем не работает с кодом который сам свои зависимости создает
https://habr.com/ru/post/313796/

пожалуйста опишите еще раз чего вы хотите добиться ...

подозреваю что под тестировнием вы имеете в виде что то другое не то что все

[thatmaniscool]

сергей кузьмин, Спасибо, прочитал. Узнал много нового для себя. Так уже узнал, что многопоточность не тестируется в JUNIT и тестировать можно только один объект.
По поводу вашей ссылки, вот примерная диаграмма, как я все представляю и мне кажется, у меня реализовано именно с инверсией зависимостей.

[сергей кузьмин]

thatmaniscool,
нарисованная диаграмма наслендования не имеет ничего общего с инверсией зависимостей.
https://xufocoder.medium.com/a-quick-intro-to-depe...

про это написаны тонны статей...