auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>protected void prepareConfigFile() { <START> String catUrl = CATALOG_URL.get(); <END> File configFile = new File("repo.json"); try { ConfigFileFields configCatalogEntries = jsonMapper.readValue(catUrl, ConfigFileFields.class); FileOutputStream fos = new FileOutputStream(configFile.getAbsoluteFile()); jsonMapper.writeValue(fos, configCatalogEntries); } catch (IOException e) { e.printStackTrace(); } }</code><technical_language>this support comma separated value? needs if not</technical_language>

Target:
protected void prepareConfigFile() throws IOException { File configFile = new File("repo.json"); String catUrl = CATALOG_URL.get(); ConfigFileFields configCatalogEntries = new ConfigFileFields(); FileOutputStream fos; if(catUrl.startsWith("{")) { configCatalogEntries = jsonMapper.readValue(catUrl, ConfigFileFields.class); fos = new FileOutputStream(configFile.getAbsoluteFile()); jsonMapper.writeValue(fos, configCatalogEntries); } else { String []catalogs = catUrl.split(","); List<CatalogEntry> catalogEntryList = new ArrayList<>(); for (String catalog: catalogs) { CatalogEntry entry = new CatalogEntry(); String[]splitted = catalog.split("="); entry.setName(splitted[0]); entry.setRepoURL(splitted[1]); entry.setBranch("master"); catalogEntryList.add(entry); } configCatalogEntries.setCatalogs(catalogEntryList); fos = new FileOutputStream(configFile.getAbsoluteFile()); jsonMapper.writeValue(fos, configCatalogEntries); } }
Source:
<code>public void close() { if (holder != null) { try { if (invalidateOnClose) { log.trace("Invalidating CAS [{}]@[{}]({})", username, documentName, documentId); <START> exclusiveAccessPool.invalidateObject(key, holder); <END> exclusiveAccessHolders.remove(holder); } else { log.trace("Returning briefly borrowed CAS [{}]@[{}]({})", username, documentName, documentId); exclusiveAccessPool.returnObject(key, holder); } } catch (Exception e) { log.error("Unable to return CAS to exclusive access pool", e); } } }</code><technical_language>do need call exclusiveAccessPool.invalidateObject(key, holder); if is called here? <LINK_0></technical_language>

Target:
public void close() { if (holder != null) { log.trace("Returning briefly borrowed CAS [{}]@[{}]({})", username, documentName, documentId); exclusiveAccessPool.returnObject(key, holder); } }
Source:
<code>public Collection<ByteArrayWrapper> spop( ByteArrayWrapper key, int popCount) { RedisSet redisSet = get(key); Collection<ByteArrayWrapper> byteArrayWrappers = redisSet.spop(key, popCount); <START> if (get(key).scard(key) == 0) { <END> region.remove(key); } else { region.put(key, get(key)); } return byteArrayWrappers; }</code><technical_language>change "get(key)" "redisSet"</technical_language>

Target:
public Collection<ByteArrayWrapper> spop( ByteArrayWrapper key, int popCount) { return getRedisSet(key).spop(region, key, popCount); }
Source:
<code><START> @Override protected void executeQueryCommand() { <END> if (getParameters().getStorageType() != null) { getQueryReturnValue().setReturnValue( WipeAfterDeleteUtils.getDefaultWipeAfterDeleteFlag(getParameters().getStorageType())); } }</code><technical_language>style - move '@Override' annotation a separate line</technical_language>

Target:
protected void executeQueryCommand() { if (getParameters().getStorageType() != null) { getQueryReturnValue().setReturnValue( WipeAfterDeleteUtils.getDefaultWipeAfterDeleteFlag(getParameters().getStorageType())); } }
Source:
<code>private static void extraTarGzSource(File folder, URL url) throws IOException { HttpURLConnection connection = (HttpURLConnection) url.openConnection(); connection.connect(); try (TarArchiveInputStream inputStream = new TarArchiveInputStream(new GzipCompressorInputStream(connection.getInputStream()))) { TarArchiveEntry entry; while ((entry = inputStream.getNextTarEntry()) != null) { if (entry.isDirectory()) { continue; } File file = new File(folder, entry.getName()); File parent = file.getParentFile(); if (!parent.exists()) { parent.mkdirs(); } try (OutputStream outputStream = new FileOutputStream(file)) { IOUtils.copy(inputStream, outputStream); } } } <START> connection.disconnect(); <END> }</code><technical_language>in finally</technical_language>

Target:
private static void extraTarGzSource(File folder, URL url) throws IOException { HttpURLConnection connection = (HttpURLConnection) url.openConnection(); connection.connect(); try (TarArchiveInputStream inputStream = new TarArchiveInputStream(new GzipCompressorInputStream(connection.getInputStream()))) { TarArchiveEntry entry; while ((entry = inputStream.getNextTarEntry()) != null) { if (entry.isDirectory()) { continue; } File file = new File(folder, entry.getName()); File parent = file.getParentFile(); if (parent.exists() || parent.mkdirs()) { try (OutputStream outputStream = new FileOutputStream(file)) { IOUtils.copy(inputStream, outputStream); } } else { throw new IllegalStateException("Cannot create directory: " + parent.getAbsolutePath()); } } } finally { connection.disconnect(); } }
Source:
<code>public void shouldThrowIllegalArgumentExceptionIfRetentionIsNegative() { <START> Materialized.as("valid-name").withRetention(Duration.of(-1, ChronoUnit.DAYS)); <END> }</code><technical_language>While existing test written this way, try move this pattern expected annotation. Instead, assertThrows verify exception error message</technical_language>

Target:
public void shouldThrowIllegalArgumentExceptionIfRetentionIsNegative() { final IllegalArgumentException e = assertThrows(IllegalArgumentException.class, () -> Materialized.as("valid-name").withRetention(Duration.of(-1, ChronoUnit.DAYS))); assertEquals(e.getMessage(), "Retention must not be negative."); }
Source:
<code>public static String consumeNextRecordedEvent() { try { return concurrentStack.removeFirst(); } catch (java.util.NoSuchElementException nse) { <START> throw new IllegalStateException( "Attempted to consume an event, while no events were recorded. Check if the Byteman ruless actually triggered?" ); <END> } }</code><technical_language>I new message is misleading. I it: "Attempted consume event, while events left. If is call this method, check if Byteman rules triggered?" fact is error if consumed events</technical_language>

Target:
public static String consumeNextRecordedEvent() { try { return concurrentStack.removeFirst(); } catch (java.util.NoSuchElementException nse) { throw new IllegalStateException( "Attempted to consume an event, while no events are left on the stack. If it is the first call to this method, check if the Byteman rules are actually being triggered?" ); } }
Source:
<code>private void toTelemetryTable(TelemetryRow telemetryRow) { final TableWriter.Row row = writer.newRow(); row.putDate(0, telemetryRow.ts); <START> row.putStr(1, id); <END> row.putShort(2, telemetryRow.event); row.append(); }</code><technical_language>storing ID telemetry row string is efficient. ID stored in separate table cross-joined time series data out</technical_language>

Target:
private void toTelemetryTable(TelemetryRow telemetryRow) { final TableWriter.Row row = writer.newRow(); row.putDate(0, telemetryRow.ts); row.putShort(1, telemetryRow.event); row.append(); }
Source:
<code>public final void gauge(final String metricName, final JSONObject builddata, final String key, final HashMap<String,String> extraTags) { String builddataKey = nullSafeGetString(builddata, key); logger.fine(String.format("Sending metric '%s' with value %s", metricName, builddataKey)); JSONArray points = new JSONArray(); JSONArray point = new JSONArray(); long currentTime = System.currentTimeMillis() / DatadogBuildListener.THOUSAND_LONG; point.add(currentTime); point.add(builddata.get(key)); points.add(point); JSONObject metric = new JSONObject(); metric.put("metric", metricName); metric.put("points", points); metric.put("type", "gauge"); metric.put("host", builddata.get("hostname")); metric.put("tags", assembleTags(builddata, extraTags)); <START> <END> JSONArray series = new JSONArray(); series.add(metric); JSONObject payload = new JSONObject(); payload.put("series", series); logger.fine(String.format("Resulting payload: %s", payload.toString() )); try { post(payload, DatadogBuildListener.METRIC); } catch (Exception e) { logger.severe(e.toString()); } }</code><technical_language>tab</technical_language>

Target:
public final void gauge(final String metricName, final JSONObject builddata, final String key, final HashMap<String,String> extraTags) { String builddataKey = DatadogUtilities.nullSafeGetString(builddata, key); logger.fine(String.format("Sending metric '%s' with value %s", metricName, builddataKey)); JSONArray points = new JSONArray(); JSONArray point = new JSONArray(); long currentTime = System.currentTimeMillis() / DatadogBuildListener.THOUSAND_LONG; point.add(currentTime); point.add(builddata.get(key)); points.add(point); JSONObject metric = new JSONObject(); metric.put("metric", metricName); metric.put("points", points); metric.put("type", "gauge"); metric.put("host", builddata.get("hostname")); metric.put("tags", DatadogUtilities.assembleTags(builddata, extraTags)); JSONArray series = new JSONArray(); series.add(metric); JSONObject payload = new JSONObject(); payload.put("series", series); logger.fine(String.format("Resulting payload: %s", payload.toString() )); try { DatadogHttpRequests.post(payload, METRIC); } catch (Exception e) { logger.severe(e.toString()); } }
Source:
<code>public DockerConnectorProvider(Map<String, DockerConnector> connectors, @Named("che.docker.connector") String property) { if (connectors.containsKey(property)) { this.connector = connectors.get(property); } else { LOG.warn("Property 'che.docker_connector.provider' did not match any bound implementation of DockerConnector. Using default."); LOG.warn("\t Value of che.docker_connector.provider: {}", property); LOG.warn("\t Bound implementations: {}", connectors.toString()); <START> this.connector = connectors.get("default"); <END> } }</code><technical_language>do prefer setting default implementation instead of che.properties</technical_language>

Target:
public DockerConnectorProvider(Map<String, DockerConnector> connectors, @Named("che.docker.connector") String property) { if (connectors.containsKey(property)) { this.connector = connectors.get(property); } else { LOG.warn("Property 'che.docker.connector' did not match any bound implementation of DockerConnector. Using default."); LOG.warn("\t Value of 'che.docker.connector': {}", property); LOG.warn("\t Bound implementations: {}", connectors.toString()); this.connector = connectors.get("default"); } }
Source:
<code>public static QuotaModeType fromValue(String value) { try { return valueOf(value.toUpperCase()); <START> } catch (IllegalArgumentException e) { <END> return null; } }</code><technical_language>Send this exception log</technical_language>

Target:
public static QuotaModeType fromValue(String value) { try { return valueOf(value.toUpperCase()); } catch (IllegalArgumentException e) { log.error("missing value: " + value, e); return null; } }
Source:
<code>protected List<Map<String, Object>> read(final List<String> names, final String key, final String value) { logger.debug("Reading key / value: / {}", key, value); final List<Map<String, Object>> listAttributes = new ArrayList<>(); final TypeReference<HashMap<String,Object>> typeRef = new TypeReference<HashMap<String,Object>>() {}; if (key.equals(COUCH_ID)) { try { final InputStream oldDocStream = couchDbConnector.getAsStream(value); final Map<String, Object> res = objectMapper.readValue(oldDocStream, typeRef); final Map<String, Object> newAttributes = new HashMap<>(); for (final Map.Entry<String, Object> entry : res.entrySet()) { final String name = entry.getKey(); if (names == null || names.contains(name)) { newAttributes.put(name, entry.getValue()); } } listAttributes.add(newAttributes); <START> } catch (DocumentNotFoundException e) { <END> } catch (IOException e) { logger.error("", e); } } else { final ViewQuery query = new ViewQuery() .designDocId("_design/pac4j") .viewName("by_"+key) .key(value); final ViewResult result = couchDbConnector.queryView(query); for (ViewResult.Row row : result.getRows()) { final String stringValue = row.getValue(); Map<String, Object> res = null; try { res = objectMapper.readValue(stringValue, typeRef); final Map<String, Object> newAttributes = new HashMap<>(); for (final Map.Entry<String, Object> entry : res.entrySet()) { final String name = entry.getKey(); if (names == null || names.contains(name)) { newAttributes.put(name, entry.getValue()); } } listAttributes.add(newAttributes); } catch (IOException e) { logger.error("", e); } } } logger.debug("Found: {}", listAttributes); return listAttributes; }</code><technical_language>deserve a DEBUG log here</technical_language>

Target:
protected List<Map<String, Object>> read(final List<String> names, final String key, final String value) { logger.debug("Reading key / value: / {}", key, value); final List<Map<String, Object>> listAttributes = new ArrayList<>(); final TypeReference<HashMap<String,Object>> typeRef = new TypeReference<HashMap<String,Object>>() {}; if (key.equals(COUCH_ID)) { try { final InputStream oldDocStream = couchDbConnector.getAsStream(value); final Map<String, Object> res = objectMapper.readValue(oldDocStream, typeRef); listAttributes.add(populateAttributes(res, names)); } catch (DocumentNotFoundException e) { logger.debug("Document id not found", value); } catch (IOException e) { logger.error("Unexpected IO CouchDB Exception", e); } } else { final ViewQuery query = new ViewQuery() .designDocId("_design/pac4j") .viewName("by_"+key) .key(value); final ViewResult result = couchDbConnector.queryView(query); for (ViewResult.Row row : result.getRows()) { final String stringValue = row.getValue(); Map<String, Object> res = null; try { res = objectMapper.readValue(stringValue, typeRef); listAttributes.add(populateAttributes(res, names)); } catch (IOException e) { logger.error("Unexpected IO CouchDB Exception", e); } } } logger.debug("Found: {}", listAttributes); return listAttributes; }
Source:
<code>public void setUp() throws IOException { queryRunner = new MemoryLocalQueryRunner(ImmutableMap.of(SystemSessionProperties.DISTRIBUTED_JOIN, "false")); queryRunner.installPlugin(new GeoPlugin()); List<String> polygonValues = new ArrayList<>(); Path path = Paths.get(BenchmarkSpatialJoin.class.getClassLoader().getResource("us-states.tsv").getPath()); for (String line : Files.readAllLines(path)) { String[] parts = line.split("\t"); String name = parts[0]; String wkt = parts[1]; polygonValues.add(format("('%s', '%s')", name, wkt)); } queryRunner.execute(format("CREATE TABLE memory.default.polygons AS SELECT * FROM (VALUES %s) as t (name, wkt)", Joiner.on(",").join(polygonValues))); <START> queryRunner.execute(format("CREATE TABLE memory.default.points AS SELECT 'p' || cast(elem AS VARCHAR) as name, xMin + (xMax - xMin) * random() as longitude, yMin + (yMax - yMin) * random() as latitude " + <END> "FROM (SELECT -124 AS xMin, -65 AS xMax, 27 AS yMin, 49 AS yMax) CROSS JOIN UNNEST(sequence(1, %s)) AS t(elem)", pointCount)); }</code><technical_language>format this query a more lines is easier read</technical_language>

Target:
public void setUp() throws IOException { queryRunner = new MemoryLocalQueryRunner(); queryRunner.installPlugin(new GeoPlugin()); Path path = Paths.get(BenchmarkSpatialJoin.class.getClassLoader().getResource("us-states.tsv").getPath()); String polygonValues = Files.lines(path) .map(line -> line.split("\t")) .map(parts -> format("('%s', '%s')", parts[0], parts[1])) .collect(Collectors.joining(",")); queryRunner.execute(format("CREATE TABLE memory.default.polygons AS SELECT * FROM (VALUES %s) as t (name, wkt)", polygonValues)); }
Source:
<code>public void testListWithOption() { GenericQueryOptions options = new GenericQueryOptions(); options.options(0, 0, null, "test", null); List<Server> serversWithQuery = serverApi().getList(options); assertNotNull(serversWithQuery); assertFalse(serversWithQuery.isEmpty()); <START> Assert.assertTrue(serversWithQuery.size() > 0); <END> }</code><technical_language>Remove this redundant assertion similar ones</technical_language>

Target:
public void testListWithOption() { GenericQueryOptions options = new GenericQueryOptions(); options.options(0, 0, null, "test", null); List<Server> serversWithQuery = serverApi().list(options); assertNotNull(serversWithQuery); Assert.assertTrue(serversWithQuery.size() > 0); }
Source:
<code>private static ExportPackageDescription[] getExportsWiredTo(ResolverBundle rb, Map<String, List<StateWire>> stateWires) { ResolverImport[] imports = rb.getImportPackages(); List<ExportPackageDescription> exportsWiredTo = new ArrayList<>(imports.length); List<StateWire> importWires = new ArrayList<>(imports.length); for (ResolverImport <START> import1 : <END> imports) { if (import1.getSelectedSupplier() != null) { ExportPackageDescription supplier = (ExportPackageDescription) import1.getSelectedSupplier().getBaseDescription(); exportsWiredTo.add(supplier); StateWire wire = newStateWire(rb.getBundleDescription(), import1.getVersionConstraint(), supplier.getExporter(), supplier); importWires.add(wire); } } if (stateWires != null && !importWires.isEmpty()) stateWires.put(BundleRevision.PACKAGE_NAMESPACE, importWires); return exportsWiredTo.toArray(new ExportPackageDescription[exportsWiredTo.size()]); }</code><technical_language>rename resolverImport</technical_language>

Target:
private static ExportPackageDescription[] getExportsWiredTo(ResolverBundle rb, Map<String, List<StateWire>> stateWires) { ResolverImport[] imports = rb.getImportPackages(); List<ExportPackageDescription> exportsWiredTo = new ArrayList<>(imports.length); List<StateWire> importWires = new ArrayList<>(imports.length); for (ResolverImport resolverImport : imports) { if (resolverImport.getSelectedSupplier() != null) { ExportPackageDescription supplier = (ExportPackageDescription) resolverImport.getSelectedSupplier().getBaseDescription(); exportsWiredTo.add(supplier); StateWire wire = newStateWire(rb.getBundleDescription(), resolverImport.getVersionConstraint(), supplier.getExporter(), supplier); importWires.add(wire); } } if (stateWires != null && !importWires.isEmpty()) stateWires.put(BundleRevision.PACKAGE_NAMESPACE, importWires); return exportsWiredTo.toArray(new ExportPackageDescription[exportsWiredTo.size()]); }
Source:
<code>public EmailWrapper generateUserCourseRegisterEmail(AccountAttributes user, CourseAttributes course) { String emailBody = Templates.populateTemplate(EmailTemplates.USER_COURSE_REGISTER, "${userName}", SanitizationHelper.sanitizeForHtml(user.getName()), "${courseId}", SanitizationHelper.sanitizeForHtml(course.getId()), "${courseName}", SanitizationHelper.sanitizeForHtml(course.getName()), "${googleId}", SanitizationHelper.sanitizeForHtml(user.getGoogleId()), "${supportEmail}", Config.SUPPORT_EMAIL); EmailWrapper email = getEmptyEmailAddressedToEmail(user.getEmail()); email.setSubject(String.format(EmailType.USER_COURSE_REGISTER.getSubject(), <START> course.getId(), course.getName())); <END> email.setContent(emailBody); return email; }</code><technical_language>contents of email subject do match (currently shows name "Course ID:")</technical_language>

Target:
public EmailWrapper generateUserCourseRegisterEmail(AccountAttributes user, CourseAttributes course) { String emailBody = Templates.populateTemplate(EmailTemplates.USER_COURSE_REGISTER, "${userName}", SanitizationHelper.sanitizeForHtml(user.getName()), "${userType}", user.isInstructor() ? "an instructor" : "a student", "${courseId}", SanitizationHelper.sanitizeForHtml(course.getId()), "${courseName}", SanitizationHelper.sanitizeForHtml(course.getName()), "${googleId}", SanitizationHelper.sanitizeForHtml(user.getGoogleId()), "${supportEmail}", Config.SUPPORT_EMAIL); EmailWrapper email = getEmptyEmailAddressedToEmail(user.getEmail()); email.setSubject(String.format(EmailType.USER_COURSE_REGISTER.getSubject(), course.getName(), course.getId())); email.setContent(emailBody); return email; }
Source:
<code>public void setTrustStoreConfigFromDefault() { <START> String trustStore = System.getProperty("java.home") + File.separator + "lib" + File.separator + "security" + File.separator + "cacerts"; <END> String trustStorePassword = "changeit"; m_sslConfig = new SSLConfiguration.SslConfig(null, null, trustStore, trustStorePassword); }</code><technical_language>Is another place define this, this need defined in RealVoltDB here</technical_language>

Target:
public void setTrustStoreConfigFromDefault() { String trustStore = Constants.DEFAULT_TRUSTSTORE_RESOURCE; String trustStorePassword = Constants.DEFAULT_TRUSTSTORE_PASSWD; m_sslConfig = new SSLConfiguration.SslConfig(null, null, trustStore, trustStorePassword); }
Source:
<code>public void waitForSlotCollapsed(WebElement slot) { <START> wait.until(CommonExpectedConditions.elementHasSize(slot, 0, 0)); <END> }</code><technical_language>a problem "implicit wait" interfering "explicit wait" strategy</technical_language>

Target:
public void waitForSlotCollapsed(WebElement slot) { changeImplicitWait(250, TimeUnit.MILLISECONDS); try { wait.until(CommonExpectedConditions.elementHasSize(slot, 0, 0)); } finally { restoreDeaultImplicitWait(); } }
Source:
<code>public void ProcessPartitionExpVariableTest_OptionalCompulsaryPartition() throws Exception { String startTime = TimeUtil.getTimeWrtSystemTime(-4); String endTime = TimeUtil.getTimeWrtSystemTime(30); bundles[0] = bundles[0].getRequiredBundle(bundles[0], 1, 2, 1, baseTestDir, 1, startTime, endTime); bundles[0].setProcessData(bundles[0] .setProcessInputNames(bundles[0].getProcessData(), "inputData0", "inputData")); Property p = new Property(); p.setName("var1"); p.setValue("hardCoded"); bundles[0].setProcessData(bundles[0].addProcessProperty(bundles[0].getProcessData(), p)); bundles[0].setProcessData(bundles[0] .setProcessInputPartition(bundles[0].getProcessData(), "${var1}", "${fileTime}")); for (int i = 0; i < bundles[0].getDataSets().size(); i++) logger.info(Util.prettyPrintXml(bundles[0].getDataSets().get(i))); <START> logger.info((bundles[0].getProcessData())); <END> createDataWithinDatesAndPrefix(cluster, TimeUtil.oozieDateToDate(TimeUtil.addMinsToTime(startTime, -25)), TimeUtil.oozieDateToDate(TimeUtil.addMinsToTime(endTime, 25)), baseTestDir + "/input1/", 1); bundles[0].submitAndScheduleBundle(bundles[0], prism, false); TimeUnit.SECONDS.sleep(20); InstanceUtil.waitTillInstanceReachState(clusterOC, Util.getProcessName(bundles[0].getProcessData()), 2, CoordinatorAction.Status.SUCCEEDED, 20, ENTITY_TYPE.PROCESS); }</code><technical_language>extra ()</technical_language>

Target:
public void ProcessPartitionExpVariableTest_OptionalCompulsaryPartition() throws Exception { String startTime = TimeUtil.getTimeWrtSystemTime(-4); String endTime = TimeUtil.getTimeWrtSystemTime(30); bundles[0] = bundles[0].getRequiredBundle(bundles[0], 1, 2, 1, baseTestDir, 1, startTime, endTime); bundles[0].setProcessData(bundles[0] .setProcessInputNames(bundles[0].getProcessData(), "inputData0", "inputData")); Property p = new Property(); p.setName("var1"); p.setValue("hardCoded"); bundles[0].setProcessData(bundles[0].addProcessProperty(bundles[0].getProcessData(), p)); bundles[0].setProcessData(bundles[0] .setProcessInputPartition(bundles[0].getProcessData(), "${var1}", "${fileTime}")); for (int i = 0; i < bundles[0].getDataSets().size(); i++) logger.info(Util.prettyPrintXml(bundles[0].getDataSets().get(i))); logger.info(Util.prettyPrintXml(bundles[0].getProcessData())); createDataWithinDatesAndPrefix(cluster, TimeUtil.oozieDateToDate(TimeUtil.addMinsToTime(startTime, -25)), TimeUtil.oozieDateToDate(TimeUtil.addMinsToTime(endTime, 25)), baseTestDir + "/input1/", 1); bundles[0].submitAndScheduleBundle(bundles[0], prism, false); TimeUnit.SECONDS.sleep(20); InstanceUtil.waitTillInstanceReachState(clusterOC, Util.getProcessName(bundles[0].getProcessData()), 2, CoordinatorAction.Status.SUCCEEDED, 20, ENTITY_TYPE.PROCESS); }
Source:
<code>public void HeroModuleTests_001_VerifyImageVisibilityAfterRefresh() { ModularMainPageObject mom = new ModularMainPageObject(driver); mom.logInCookie(credentials.userNameStaff, credentials.passwordStaff, wikiURL); String prevMoMSrc = mom.getMoMSrc(); <START> mom.selectFileToUpload(PageContent.FILE); <END> mom.verifyDragToRepositionText(); mom.clickPublishButton(); String newMoMSrc = mom.getMoMSrc(); mom.verifySrcTxtAreDifferent(prevMoMSrc, newMoMSrc); mom.refreshPage(); mom.verifySrcTxtAreDifferent(prevMoMSrc, newMoMSrc); }</code><technical_language>I a proper fix this: ModularMainPageObject mom = new ModularMainPageObject(driver); mom.openWikiPage(wikiURL);</technical_language>

Target:
public void HeroModuleTests_001_VerifyImageVisibilityAfterRefresh() { ModularMainPageObject mom = new ModularMainPageObject(driver); mom.openWikiPage(wikiURL); mom.logInCookie(credentials.userNameStaff, credentials.passwordStaff, wikiURL); String prevMoMSrc = mom.getMoMSrc(); mom.selectFileToUpload(PageContent.FILE); mom.verifyDragToRepositionText(); mom.clickPublishButton(); String newMoMSrc = mom.getMoMSrc(); mom.verifySrcTxtAreDifferent(prevMoMSrc, newMoMSrc); mom.refreshPage(); mom.verifySrcTxtAreDifferent(prevMoMSrc, newMoMSrc); }
Source:
<code>public void onResponse(Call<ApiMeasurement> call, Response<ApiMeasurement> response) { if (response.isSuccessful() && response.body() != null && response.body().results != null && response.body().results.size() == 1 && response.body().results.get(0).measurement_url != null) { onSuccess(response.body().results.get(0)); } else { <START> onError(Integer.toString(response.code())); <END> } }</code><technical_language>I this onError clause trigger for 200 status code if response JSON information expect have. I suggest change error mention HTTP error a malformed JSON. (We include response code JSON.) &mdash; I missing something</technical_language>

Target:
public void onResponse(Call<ApiMeasurement> call, Response<ApiMeasurement> response) { if (response.isSuccessful() && response.body() != null && response.body().results != null && response.body().results.size() == 1 && response.body().results.get(0).measurement_url != null) { onSuccess(response.body().results.get(0)); } else { onError(Integer.toString(R.string.Modal_Error_JsonEmpty)); } }
Source:
<code>public void createEncounter(DateTime encounterDatetime, String encounterType, String locationName, String patientUuid, String providerUuid, Map<String, String> observations) { Location location = getLocationByName(locationName); Patient patient = patientService.getPatientByUuid(null, patientUuid); Provider provider = providerService.getProviderByUuid(null, providerUuid); <START> List<Observation> observationList = convertObservationMapToList(observations, encounterDatetime); <END> EncounterType type = new EncounterType(encounterType); Encounter encounter = new Encounter(location, type, encounterDatetime.toDate(), patient, provider.getPerson(), observationList); encounterService.createEncounter(null, encounter); }</code><technical_language>Observations null, is required field in action. in method convertObservationMapToList NPE</technical_language>

Target:
public void createEncounter(DateTime encounterDatetime, String encounterType, String locationName, String patientUuid, String providerUuid, Map<String, String> observations) { Location location = getLocationByName(locationName); Patient patient = patientService.getPatientByUuid(null, patientUuid); Provider provider = providerService.getProviderByUuid(null, providerUuid); List<Observation> observationList = MapUtils.isNotEmpty(observations) ? convertObservationMapToList(observations, encounterDatetime) : null; EncounterType type = new EncounterType(encounterType); Encounter encounter = new Encounter(location, type, encounterDatetime.toDate(), patient, provider.getPerson(), observationList); encounterService.createEncounter(null, encounter); }
Source:
<code>public void onNext(final AllocatedEvaluator allocatedEvaluator) { <START> final int evalCount = activeEvaluatorCount.getAndIncrement(); <END> final Configuration partialContextConf = ContextConfiguration.CONF .set(ContextConfiguration.IDENTIFIER, CONTEXT_ID_PREFIX + evalCount) .build(); final Configuration contextConf = Configurations.merge( partialContextConf, emConf.getContextConfiguration()); final Configuration serviceConf = emConf.getServiceConfiguration(); allocatedEvaluator.submitContextAndService(contextConf, serviceConf); LOG.info((evalCount + 1) + " evaluators active!"); }</code><technical_language>move changes another PR :)</technical_language>

Target:
public void onNext(final AllocatedEvaluator allocatedEvaluator) { final Configuration partialContextConf = ContextConfiguration.CONF .set(ContextConfiguration.IDENTIFIER, CONTEXT_ID_PREFIX + activeEvaluatorCount.getAndIncrement()) .build(); final Configuration contextConf = Configurations.merge( partialContextConf, emConf.getContextConfiguration()); final Configuration serviceConf = emConf.getServiceConfiguration(); allocatedEvaluator.submitContextAndService(contextConf, serviceConf); LOG.info(activeEvaluatorCount.get() + " evaluators active!"); }
Source:
<code>public JExpression generateReader(JBlock body, JVar parcelParam, ASTType type, JClass returnJClassRef, JDefinedClass parcelableClass, JVar identity, JVar readIdentityMap) { ASTType componentType = type.getGenericArgumentTypes().get(0); ReadWriteGenerator generator = generators.getGenerator(componentType); JExpression readExpression = generator.generateReader(body, parcelParam, componentType, generationUtil.ref(componentType), parcelableClass, identity, readIdentityMap); JClass mutableLiveDataType = generationUtil.ref(TYPE); JVar outputVar = body.decl(mutableLiveDataType, namer.generateName(mutableLiveDataType)); <START> body.assign(outputVar, JExpr._new(generationUtil.ref(TYPE))); <END> body.invoke(outputVar, "setValue").arg(readExpression); return outputVar; }</code><technical_language>reuse multableLifeDataType instead of generationUtil.ref(TYPE)</technical_language>

Target:
public JExpression generateReader(JBlock body, JVar parcelParam, ASTType type, JClass returnJClassRef, JDefinedClass parcelableClass, JVar identity, JVar readIdentityMap) { ASTType componentType = type.getGenericArgumentTypes().get(0); ReadWriteGenerator generator = generators.getGenerator(componentType); JExpression readExpression = generator.generateReader(body, parcelParam, componentType, generationUtil.ref(componentType), parcelableClass, identity, readIdentityMap); JClass mutableLiveDataType = generationUtil.ref(TYPE); JVar outputVar = body.decl(mutableLiveDataType, namer.generateName(mutableLiveDataType)); body.assign(outputVar, JExpr._new(mutableLiveDataType)); body.invoke(outputVar, "setValue").arg(readExpression); return outputVar; }
Source:
<code>public DarkClusterConfigMap getDarkClusterConfigMap(String clusterName) throws ServiceUnavailableException { FutureCallback<DarkClusterConfigMap> clusterCountFutureCallback = new FutureCallback<>(); _state.listenToCluster(clusterName, (type, name) -> { ClusterProperties clusterProperties = _state.getClusterProperties(clusterName).getProperty(); <START> if (clusterProperties != null) { clusterCountFutureCallback.onSuccess(clusterProperties.getDarkClusters()); } else { clusterCountFutureCallback.onSuccess(new DarkClusterConfigMap()); } <END> }); try { return clusterCountFutureCallback.get(_timeout, _unit); } catch (ExecutionException | TimeoutException | IllegalStateException | InterruptedException e ) { die("ClusterInfo", "PEGA_1018, unable to retrieve dark cluster info for cluster: " + clusterName + ", exception: " + e); return new DarkClusterConfigMap(); } }</code><technical_language>simplify 2 lines: DarkClusterConfigMap darkClusterConfigMap = clusterProperties != null ? clusterProperties.getDarkClusters() : new DarkClusterConfigMap(); clulsterCountFutureCallback.onSuccess(darkClusterConfigMap); rename clusterCountFutureCallback -> darkClusterConfigMapFutureCallback;</technical_language>

Target:
public DarkClusterConfigMap getDarkClusterConfigMap(String clusterName) throws ServiceUnavailableException { FutureCallback<DarkClusterConfigMap> darkClusterConfigMapFutureCallback = new FutureCallback<>(); _state.listenToCluster(clusterName, (type, name) -> { ClusterProperties clusterProperties = _state.getClusterProperties(clusterName).getProperty(); DarkClusterConfigMap darkClusterConfigMap = clusterProperties != null ? clusterProperties.getDarkClusters() : new DarkClusterConfigMap(); darkClusterConfigMapFutureCallback.onSuccess(darkClusterConfigMap); }); try { return darkClusterConfigMapFutureCallback.get(_timeout, _unit); } catch (ExecutionException | TimeoutException | IllegalStateException | InterruptedException e ) { die("ClusterInfo", "PEGA_1018, unable to retrieve dark cluster info for cluster: " + clusterName + ", exception: " + e); return new DarkClusterConfigMap(); } }
Source:
<code>public void testPublishWithNullMessage() { byte[] testBytes = {}; <START> String testString = "Test"; <END> when(urlService.inputStreamToByteArray(inputStream)).thenReturn(testBytes); when(facebookService.publishToFacebook(any(byte[].class), any(String.class), any(String.class), any(String.class), any(String.class))).thenReturn(testString); Response response = resource.publish(key, "Test", inputStream, contentDisposition, "photo", null, "Test"); String result = (String) response.getEntity(); assertEquals(response.getStatusInfo(), Response.Status.OK); assertEquals(result, "Test"); }</code><technical_language>needing local variable</technical_language>

Target:
public void testPublishWithNullMessage() { byte[] testBytes = {}; when(urlService.inputStreamToByteArray(inputStream)).thenReturn(testBytes); when(facebookService.publishToFacebook(any(byte[].class), any(String.class), any(String.class), any(String.class), any(String.class))).thenReturn("Test"); Response response = resource.publish(key, "Test", inputStream, contentDisposition, "photo", null, "Test"); String result = (String) response.getEntity(); assertEquals(response.getStatusInfo(), Response.Status.OK); assertEquals(result, "Test"); }
Source:
<code>public static void oneTimeSetup() { <START> sessionFactory = new SessionFactory(baseConfiguration, "org.neo4j.ogm.persistence.relationships.direct.aa."); <END> }</code><technical_language>package a trailing .</technical_language>

Target:
public static void oneTimeSetup() { sessionFactory = new SessionFactory(baseConfiguration, "org.neo4j.ogm.persistence.relationships.direct.aa"); }
Source:
<code>private void configureChannelPipelineHandlers() { this.httpClient = this.httpClient.tcpConfiguration(tcpClient -> { if (this.httpClientConfig.getProxy() != null) { tcpClient = tcpClient.proxy(typeSpec -> typeSpec.type(ProxyProvider.Proxy.HTTP).address(this.httpClientConfig.getProxy())); } tcpClient = tcpClient.secure(sslContextSpec -> sslContextSpec.sslContext(this.httpClientConfig.getConfigs().getSslContext())); if (LoggerFactory.getLogger(REACTOR_NETWORK_LOG_CATEGORY).isTraceEnabled()) { tcpClient = tcpClient.wiretap(REACTOR_NETWORK_LOG_CATEGORY, LogLevel.INFO); } <START> tcpClient = tcpClient.option(ChannelOption.CONNECT_TIMEOUT_MILLIS, 30 * 1000); <END> return tcpClient; }); }</code><technical_language>consider 30 *100 a constant</technical_language>

Target:
private void configureChannelPipelineHandlers() { this.httpClient = this.httpClient.tcpConfiguration(tcpClient -> { if (this.httpClientConfig.getProxy() != null) { tcpClient = tcpClient.proxy(typeSpec -> typeSpec.type(ProxyProvider.Proxy.HTTP).address(this.httpClientConfig.getProxy())); } tcpClient = tcpClient.secure(sslContextSpec -> sslContextSpec.sslContext(this.httpClientConfig.getConfigs().getSslContext())); if (LoggerFactory.getLogger(REACTOR_NETWORK_LOG_CATEGORY).isTraceEnabled()) { tcpClient = tcpClient.wiretap(REACTOR_NETWORK_LOG_CATEGORY, LogLevel.INFO); } tcpClient = tcpClient.option(ChannelOption.CONNECT_TIMEOUT_MILLIS, httpClientConfig.getConfigs().getConnectionAcquireTimeoutInMillis()); return tcpClient; }); }
Source:
<code>protected Cause createUpstreamCause(Run<?, ?> build) { if(Jenkins.getInstance().getPlugin("promoted-builds") != null) { if(build instanceof Promotion) { Promotion promotion = (Promotion)build; return new UpstreamCause((Run<?,?>)promotion.getTarget()); } } <START> return new UpstreamCause((Run) build); <END> }</code><technical_language>remove obsolete class casts</technical_language>

Target:
protected Cause createUpstreamCause(Run<?, ?> build) { if(Jenkins.getInstance().getPlugin("promoted-builds") != null) { if(build instanceof Promotion) { Promotion promotion = (Promotion)build; return new UpstreamCause((Run<?,?>)promotion.getTarget()); } } return new UpstreamCause(build); }
Source:
<code>private void postToGoogleAnalytics(int position) { String fragmentName = viewPagerAdapter.getItem(position).getClass().getSimpleName(); <START> googleAnalytics(fragmentName); <END> }</code><technical_language>reason for GoogleAnalytics.post(this, fragmentName); here</technical_language>

Target:
private void postToGoogleAnalytics(int position) { String fragmentName = viewPagerAdapter.getItem(position).getClass().getSimpleName(); GoogleAnalytics.post(this, fragmentName); }
Source:
<code>public Predicate<ChangeData> file(String file) throws QueryParseException { if (allowFileRegex) { if (file.startsWith("^")) { return new RegexFilePredicate(args.dbProvider, args.patchListCache, file); } else { throw new IllegalArgumentException(); } } else { if (file.startsWith("^")) { throw error("regular expression not permitted here: file:" + file); } if (args.indexManager == ChangeIndex.Manager.DISABLED) { <START> throw error("secondary index support is disabled, please enable it first"); <END> } return new EqualsFilePredicate(args.dbProvider, args.patchListCache, file); } }</code><technical_language>This message is shown end user. point telling enable indexing</technical_language>

Target:
public Predicate<ChangeData> file(String file) throws QueryParseException { if (allowFileRegex) { if (file.startsWith("^")) { return new RegexFilePredicate(args.dbProvider, args.patchListCache, file); } else { throw new IllegalArgumentException(); } } else { if (file.startsWith("^")) { throw error("regular expression not permitted here: file:" + file); } if (args.indexManager == ChangeIndex.Manager.DISABLED) { throw error("supplied query is using secondary index while it is disabled"); } return new EqualsFilePredicate(args.dbProvider, args.patchListCache, file); } }
Source:
<code>protected static DefaultConfiguration createRootConfig(Configuration config) { final DefaultConfiguration dc = new DefaultConfiguration(ROOT_MODULE_NAME); <START> if (config != null) { <END> dc.addChild(config); } return dc; }</code><technical_language>check for null required, do pass null AbstractXpathRegressionTest</technical_language>

Target:
protected static DefaultConfiguration createRootConfig(Configuration config) { final DefaultConfiguration dc = new DefaultConfiguration(ROOT_MODULE_NAME); dc.addChild(config); return dc; }
Source:
<code>private void readFully(ZipFileEntry entry, long position, byte[] buffer, int bufferOffset, int bufferLength) throws IOException { long offset = entry.getLocalHeaderOffset(); byte[] fileHeader = new byte[LocalFileHeader.FIXED_DATA_SIZE]; dataInputSource.readFully(offset, fileHeader); offset += fileHeader.length; if (!ZipUtil.arrayStartsWith(fileHeader, ZipUtil.intToLittleEndian(LocalFileHeader.SIGNATURE))) { throw new PrestoException(DRUID_SEGMENT_LOAD_ERROR, <START> format("The file '%s' is not a correctly formatted zip file: Expected a File Header at file offset %d, but was not present.", entry.getName())); <END> } int nameLength = ZipUtil.getUnsignedShort(fileHeader, LocalFileHeader.FILENAME_LENGTH_OFFSET); int extraFieldLength = ZipUtil.getUnsignedShort(fileHeader, LocalFileHeader.EXTRA_FIELD_LENGTH_OFFSET); offset += (nameLength + extraFieldLength); int compressedSize = (int) entry.getCompressedSize(); byte[] compressedData = new byte[compressedSize]; dataInputSource.readFully(offset, compressedData); InflaterInputStream inflaterInputStream = new InflaterInputStream(new ByteArrayInputStream(compressedData), new Inflater(true)); try { inflaterInputStream.skip(position); int size = chunkedRead(inflaterInputStream, buffer, bufferOffset, bufferLength); checkState(size == bufferLength); } catch (IOException e) { throw new PrestoException(DRUID_SEGMENT_LOAD_ERROR, format("Malformed zip file: %s", entry.getName())); } }</code><technical_language>format argument match</technical_language>

Target:
private void readFully(ZipFileEntry entry, long position, byte[] buffer, int bufferOffset, int bufferLength) throws IOException { long offset = entry.getLocalHeaderOffset(); byte[] fileHeader = new byte[LOCAL_HEADER_FIXED_DATA_SIZE]; dataInputSource.readFully(offset, fileHeader); offset += fileHeader.length; if (!ZipUtil.arrayStartsWith(fileHeader, ZipUtil.intToLittleEndian(LOCAL_HEADER_SIGNATURE))) { throw new PrestoException( DRUID_SEGMENT_LOAD_ERROR, format("The file '%s' is not a correctly formatted zip file: Expected a File Header at offset %d, but not present.", entry.getName(), offset)); } int nameLength = ZipUtil.getUnsignedShort(fileHeader, LOCAL_HEADER_FILENAME_LENGTH_OFFSET); int extraFieldLength = ZipUtil.getUnsignedShort(fileHeader, LOCAL_HEADER_EXTRA_FIELD_LENGTH_OFFSET); offset += (nameLength + extraFieldLength); int compressedSize = (int) entry.getCompressedSize(); byte[] compressedData = new byte[compressedSize]; dataInputSource.readFully(offset, compressedData); InflaterInputStream inflaterInputStream = new InflaterInputStream(new ByteArrayInputStream(compressedData), new Inflater(true)); try { inflaterInputStream.skip(position); int size = chunkedRead(inflaterInputStream, buffer, bufferOffset, bufferLength); checkState(size == bufferLength); } catch (IOException e) { throw new PrestoException(DRUID_SEGMENT_LOAD_ERROR, format("Malformed zip file: %s", entry.getName())); } }
Source:
<code>private void initUniqueOutcomeEventsSentSets() { attributedUniqueOutcomeEventsSentSet = OSUtils.newConcurrentSet(); <START> unattributedUniqueOutcomeEventsSentSet = OSUtils.newConcurrentSet(); <END> Set<String> tempAttributedUniqueOutcomeEventsSentSet = OneSignalPrefs.getStringSet( OneSignalPrefs.PREFS_ONESIGNAL, OneSignalPrefs.PREFS_OS_UNIQUE_OUTCOME_EVENTS_SENT, null ); if (tempAttributedUniqueOutcomeEventsSentSet != null) attributedUniqueOutcomeEventsSentSet.addAll(tempAttributedUniqueOutcomeEventsSentSet); }</code><technical_language>if i close open 30 seconds in theory unattributedUniqueOutcomeEventsSentSet uniqueOutcomes, agree ignoring this case</technical_language>

Target:
private void initUniqueOutcomeEventsSentSets() { if (unattributedUniqueOutcomeEventsSentSet == null) unattributedUniqueOutcomeEventsSentSet = OSUtils.newConcurrentSet(); attributedUniqueOutcomeEventsSentSet = OSUtils.newConcurrentSet(); Set<String> tempAttributedUniqueOutcomeEventsSentSet = OneSignalPrefs.getStringSet( OneSignalPrefs.PREFS_ONESIGNAL, OneSignalPrefs.PREFS_OS_UNIQUE_OUTCOME_EVENTS_SENT, null ); if (tempAttributedUniqueOutcomeEventsSentSet != null) attributedUniqueOutcomeEventsSentSet.addAll(tempAttributedUniqueOutcomeEventsSentSet); }
Source:
<code>public void failFastContextInitialized(ServletContextEvent event) { ApplicationRouteRegistry registry = ApplicationRouteRegistry .getInstance(new VaadinServletContext( event.getServletContext())); Stream<Class<? extends Component>> hasErrorComponents = findBySuperType( getErrorParameterPackages(), HasErrorParameter.class) .filter(Component.class::isAssignableFrom) .filter(clazz -> clazz != RouteNotFoundError.class) .map(clazz -> (Class<? extends Component>) clazz); final Set<Class<? extends Component>> errorComponents = hasErrorComponents.collect(Collectors.toSet()); if (errorComponents.stream().noneMatch(RouteNotFoundError.class::isAssignableFrom)) { <START> errorComponents.add(SpringRouteNotFoundError.class); <END> } registry.setErrorNavigationTargets(errorComponents); }</code><technical_language>Do I understand correctly, if application a custom RouteNotFoundError view (subclass), this spring view is shown</technical_language>

Target:
public void failFastContextInitialized(ServletContextEvent event) { ApplicationRouteRegistry registry = ApplicationRouteRegistry .getInstance(new VaadinServletContext( event.getServletContext())); Set<Class<? extends Component>> errorComponents = findBySuperType( getErrorParameterPackages(), HasErrorParameter.class) .filter(Component.class::isAssignableFrom) .filter(clazz -> clazz != RouteNotFoundError.class) .map(clazz -> (Class<? extends Component>) clazz) .collect(Collectors.toSet()); if (errorComponents.stream().noneMatch(RouteNotFoundError.class::isAssignableFrom)) { errorComponents.add(SpringRouteNotFoundError.class); } registry.setErrorNavigationTargets(errorComponents); }
Source:
<code>public void repeatedProcessBatchDoesNotBreakOnItsOwnSuccessfulWrites() { KeyAlreadyExistsException originalException = new KeyAlreadyExistsException( "boo", ImmutableList.of(ENCODING_STRATEGY.encodeStartTimestampAsCell(3L)), ImmutableList.of(ENCODING_STRATEGY.encodeStartTimestampAsCell(2L))); <START> KeyAlreadyExistsException newExistingKeyException = new KeyAlreadyExistsException( <END> "boo", ImmutableList.of(ENCODING_STRATEGY.encodeStartTimestampAsCell(2L))); doThrow(originalException) .doThrow(newExistingKeyException) .doNothing() .when(mockTransactionService) .putUnlessExistsMultiple(anyMap()); TestTransactionBatchElement elementNotExisting = TestTransactionBatchElement.of(2L, 200L); TestTransactionBatchElement elementAlreadyExisting = TestTransactionBatchElement.of(3L, 300L); WriteBatchingTransactionService.processBatch(mockTransactionService, ImmutableList.of( elementNotExisting, elementAlreadyExisting)); verify(mockTransactionService).putUnlessExistsMultiple(ImmutableMap.of(2L, 200L, 3L, 300L)); assertThatThrownBy(() -> elementAlreadyExisting.result().get()).hasCause(originalException); assertThatCode(() -> elementNotExisting.result().get()).doesNotThrowAnyException(); verify(mockTransactionService, atLeastOnce()).getEncodingStrategy(); }</code><technical_language>This is thrown list of timestamps empty (once success, failure). add a timestamp succeeded failed attempt verify retry for</technical_language>

Target:
public void repeatedProcessBatchDoesNotBreakOnItsOwnSuccessfulWrites() { List<Cell> failed = ImmutableList.of(ENCODING_STRATEGY.encodeStartTimestampAsCell(3L)); List<Cell> succeeded = ImmutableList.of(ENCODING_STRATEGY.encodeStartTimestampAsCell(2L)); KeyAlreadyExistsException originalException = new KeyAlreadyExistsException("boo", failed, succeeded); doThrow(originalException) .doNothing() .when(mockTransactionService) .putUnlessExistsMultiple(anyMap()); TestTransactionBatchElement elementNotExisting = TestTransactionBatchElement.of(2L, 200L); TestTransactionBatchElement elementAlreadyExisting = TestTransactionBatchElement.of(3L, 300L); TestTransactionBatchElement elementToRetry = TestTransactionBatchElement.of(4L, 400L); WriteBatchingTransactionService.processBatch(mockTransactionService, ImmutableList.of( elementNotExisting, elementAlreadyExisting, elementToRetry)); verify(mockTransactionService).putUnlessExistsMultiple(ImmutableMap.of(2L, 200L, 3L, 300L, 4L, 400L)); verify(mockTransactionService).putUnlessExistsMultiple(ImmutableMap.of(4L, 400L)); assertThatThrownBy(() -> elementAlreadyExisting.result().get()).hasCause(originalException); assertThatCode(() -> elementNotExisting.result().get()).doesNotThrowAnyException(); assertThatCode(() -> elementToRetry.result().get()).doesNotThrowAnyException(); verify(mockTransactionService, atLeastOnce()).getEncodingStrategy(); }
Source:
<code><START> protected List<Character> getConfiguredLenientGroupingSeparators() { <END> return CollectionUtility.arrayList('’', '\'', '´'); }</code><technical_language>Collection instead of List</technical_language>

Target:
protected Set<Character> getConfiguredLenientGroupingSeparators() { return CollectionUtility.hashSet( '\'', '´', '`', '’', "\u00B7".charAt(0), "\u0020".charAt(0), "\u2009".charAt(0), "\u202F".charAt(0) ); }