Skip to content

Commit

Permalink
[hotfix] Fix ProcessMemoryUtilsTestBase#testExceptionShouldContainReq…
Browse files Browse the repository at this point in the history
…uiredConfigOptions to run also for TM
  • Loading branch information
azagrebin committed Apr 27, 2020
1 parent 4385fcc commit d4d8f92
Showing 1 changed file with 1 addition and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,6 @@
import org.apache.flink.configuration.IllegalConfigurationException;
import org.apache.flink.configuration.MemorySize;
import org.apache.flink.core.testutils.CommonTestUtils;
import org.apache.flink.runtime.jobmanager.JobManagerProcessUtils;
import org.apache.flink.util.TestLogger;

import org.junit.After;
Expand Down Expand Up @@ -118,7 +117,7 @@ public void testConfigTotalProcessMemorySize() {
@Test
public void testExceptionShouldContainRequiredConfigOptions() {
try {
JobManagerProcessUtils.processSpecFromConfig(new Configuration());
processSpecFromConfig(new Configuration());
} catch (IllegalConfigurationException e) {
options.getRequiredFineGrainedOptions().forEach(option -> assertThat(e.getMessage(), containsString(option.key())));
assertThat(e.getMessage(), containsString(options.getTotalFlinkMemoryOption().key()));
Expand Down

0 comments on commit d4d8f92

Please sign in to comment.