OZG-6223 OZG-7401 Timeout for async requests

Default is 30 seconds, here set to 5 minutes (value in milliseconds)

requested review from @felix

resolved all threads

added 2 commits

• 7645c890 - OZG-6223 OZG-7381 Test for getFileSize
• 625bd98b - OZG-6223 OZG-7381 Refactor ZipDownloadService

Compare with previous version

added 1 commit

• 0cf0f44d - OZG-6223 OZG-7381 Fix test

Compare with previous version

Hier fehlt der Test auf den Rückgabewert

Ich glaube, dass wir jetzt immer auf die Lizenz achten müssen.

Ich glaube, dass wir jetzt immer auf die Lizenz achten müssen.

Ich finde es besser statische mocks von fremden Klassen zu vermeiden. Ich hätte Tests in der Art erwartet:

			private static final byte[] BYTES = LoremIpsum.getInstance().getParagraphs(1, 2).getBytes();

			@Test
			@SneakyThrows
			void shouldReturnFileSize() {
				var file = TempFileUtils.createTmpFile().toFile();
				try (var outputStream = new FileOutputStream(file)) {
					outputStream.write(BYTES);
				}

				var size = new OzgZipFile(file).getSize();

				assertThat(size).isEqualTo(BYTES.length);
			}

			@Test
			void shouldThrowTechnicalException() {
				var file = new OzgZipFile(Path.of("path").toFile());

				assertThrows(TechnicalException.class, file::getSize);
			}

Hier würde auch einfach zipFile.length() gehen

Sonarqube meckert hier ("Test further the exception raised by this assertThatThrownBy call."). Man könnte noch // NOSONAR hinzufügen.

Hier fehlt noch eine Leerzeile

Static mock würde ich hier versuchen zu vermeiden

		@Test
		@SneakyThrows
		void shouldDeleteFile() {
			var file = TempFileUtils.createTmpFile().toFile();

			new OzgZipFile(file).delete();

			assertThat(file).doesNotExist();
		}

Statt static mock, vielleicht sowas wie:

		@Test
		void shouldNotThrowException() {
			var file = Path.of("path").toFile();

			var zipFile = new OzgZipFile(file);

			assertThatNoException().isThrownBy(ozgZipFile::delete);
		}

closed