Merge pull request #17036 from dongjoon-hyun/fix_typo_in_test
Fix typos in comments/strings of `test` module.
This commit is contained in:
commit
64c4b5f3ca
|
@ -27,7 +27,7 @@ subprojects {
|
|||
apply plugin: 'elasticsearch.build'
|
||||
|
||||
|
||||
// the main files are actually test files, so use the appopriate forbidden api sigs
|
||||
// the main files are actually test files, so use the appropriate forbidden api sigs
|
||||
forbiddenApisMain {
|
||||
bundledSignatures = ['jdk-unsafe', 'jdk-deprecated']
|
||||
signaturesURLs = [PrecommitTasks.getResource('/forbidden/all-signatures.txt'),
|
||||
|
|
|
@ -36,7 +36,7 @@ dependencies {
|
|||
compileJava.options.compilerArgs << '-Xlint:-cast,-rawtypes,-try,-unchecked'
|
||||
compileTestJava.options.compilerArgs << '-Xlint:-rawtypes'
|
||||
|
||||
// the main files are actually test files, so use the appopriate forbidden api sigs
|
||||
// the main files are actually test files, so use the appropriate forbidden api sigs
|
||||
forbiddenApisMain {
|
||||
bundledSignatures = ['jdk-unsafe', 'jdk-deprecated']
|
||||
signaturesURLs = [PrecommitTasks.getResource('/forbidden/all-signatures.txt'),
|
||||
|
|
|
@ -166,7 +166,7 @@ public class BootstrapForTesting {
|
|||
}
|
||||
|
||||
/**
|
||||
* we dont know which codesources belong to which plugin, so just remove the permission from key codebases
|
||||
* we don't know which codesources belong to which plugin, so just remove the permission from key codebases
|
||||
* like core, test-framework, etc. this way tests fail if accesscontroller blocks are missing.
|
||||
*/
|
||||
@SuppressForbidden(reason = "accesses fully qualified URLs to configure security")
|
||||
|
|
|
@ -1698,7 +1698,7 @@ public abstract class ESIntegTestCase extends ESTestCase {
|
|||
|
||||
/** Helper method to create list of plugins without specifying generic types. */
|
||||
@SafeVarargs
|
||||
@SuppressWarnings("varargs") // due to type erasure, the varargs type is non-reifiable, which casues this warning
|
||||
@SuppressWarnings("varargs") // due to type erasure, the varargs type is non-reifiable, which causes this warning
|
||||
protected final Collection<Class<? extends Plugin>> pluginList(Class<? extends Plugin>... plugins) {
|
||||
return Arrays.asList(plugins);
|
||||
}
|
||||
|
@ -1706,7 +1706,7 @@ public abstract class ESIntegTestCase extends ESTestCase {
|
|||
/**
|
||||
* This method is used to obtain additional settings for clients created by the internal cluster.
|
||||
* These settings will be applied on the client in addition to some randomized settings defined in
|
||||
* the cluster. These setttings will also override any other settings the internal cluster might
|
||||
* the cluster. These settings will also override any other settings the internal cluster might
|
||||
* add by default.
|
||||
*/
|
||||
protected Settings transportClientSettings() {
|
||||
|
@ -1842,7 +1842,7 @@ public abstract class ESIntegTestCase extends ESTestCase {
|
|||
}
|
||||
@Override
|
||||
public String description() {
|
||||
return "a test plugin that registeres index.tests.seed as an index setting";
|
||||
return "a test plugin that registers index.tests.seed as an index setting";
|
||||
}
|
||||
public void onModule(SettingsModule module) {
|
||||
module.registerSetting(INDEX_TEST_SEED_SETTING);
|
||||
|
@ -1983,7 +1983,7 @@ public abstract class ESIntegTestCase extends ESTestCase {
|
|||
@After
|
||||
public final void after() throws Exception {
|
||||
printTestMessage("finished");
|
||||
// Deleting indices is going to clear search contexts implicitely so we
|
||||
// Deleting indices is going to clear search contexts implicitly so we
|
||||
// need to check that there are no more in-flight search contexts before
|
||||
// we remove indices
|
||||
super.ensureAllSearchContextsReleased();
|
||||
|
|
|
@ -109,7 +109,7 @@ public abstract class ESSingleNodeTestCase extends ESTestCase {
|
|||
public void setUp() throws Exception {
|
||||
super.setUp();
|
||||
// Create the node lazily, on the first test. This is ok because we do not randomize any settings,
|
||||
// only the cluster name. This allows us to have overriden properties for plugins and the version to use.
|
||||
// only the cluster name. This allows us to have overridden properties for plugins and the version to use.
|
||||
if (NODE == null) {
|
||||
startNode();
|
||||
}
|
||||
|
@ -154,7 +154,7 @@ public abstract class ESSingleNodeTestCase extends ESTestCase {
|
|||
|
||||
/** Helper method to create list of plugins without specifying generic types. */
|
||||
@SafeVarargs
|
||||
@SuppressWarnings("varargs") // due to type erasure, the varargs type is non-reifiable, which casues this warning
|
||||
@SuppressWarnings("varargs") // due to type erasure, the varargs type is non-reifiable, which causes this warning
|
||||
protected final Collection<Class<? extends Plugin>> pluginList(Class<? extends Plugin>... plugins) {
|
||||
return Arrays.asList(plugins);
|
||||
}
|
||||
|
|
|
@ -131,7 +131,7 @@ public abstract class ESTestCase extends LuceneTestCase {
|
|||
protected void afterIfFailed(List<Throwable> errors) {
|
||||
}
|
||||
|
||||
/** called after a test is finished, but only if succesfull */
|
||||
/** called after a test is finished, but only if successful */
|
||||
protected void afterIfSuccessful() throws Exception {
|
||||
}
|
||||
|
||||
|
|
|
@ -31,7 +31,7 @@ public final class InternalSettingsPlugin extends Plugin {
|
|||
|
||||
@Override
|
||||
public String description() {
|
||||
return "a plugin that allows to set values for internal settings which are can't be set via the ordinary API without this pluging installed";
|
||||
return "a plugin that allows to set values for internal settings which are can't be set via the ordinary API without this plugin installed";
|
||||
}
|
||||
|
||||
public static final Setting<Integer> VERSION_CREATED = Setting.intSetting("index.version.created", 0, false, Setting.Scope.INDEX);
|
||||
|
|
|
@ -56,7 +56,7 @@ public class BlockClusterStateProcessing extends SingleNodeDisruption {
|
|||
}
|
||||
logger.info("delaying cluster state updates on node [{}]", disruptionNodeCopy);
|
||||
boolean success = disruptionLatch.compareAndSet(null, new CountDownLatch(1));
|
||||
assert success : "startDisrupting called without waiting on stopDistrupting to complete";
|
||||
assert success : "startDisrupting called without waiting on stopDisrupting to complete";
|
||||
final CountDownLatch started = new CountDownLatch(1);
|
||||
clusterService.submitStateUpdateTask("service_disruption_block", new ClusterStateUpdateTask(Priority.IMMEDIATE) {
|
||||
|
||||
|
|
|
@ -144,7 +144,7 @@ public class ElasticsearchAssertions {
|
|||
}
|
||||
|
||||
/**
|
||||
* Checks that all shard requests of a replicated brodcast request failed due to a cluster block
|
||||
* Checks that all shard requests of a replicated broadcast request failed due to a cluster block
|
||||
*
|
||||
* @param replicatedBroadcastResponse the response that should only contain failed shard responses
|
||||
*
|
||||
|
@ -716,7 +716,7 @@ public class ElasticsearchAssertions {
|
|||
|
||||
/**
|
||||
* Applies basic assertions on the SearchResponse. This method checks if all shards were successful, if
|
||||
* any of the shards threw an exception and if the response is serializeable.
|
||||
* any of the shards threw an exception and if the response is serializable.
|
||||
*/
|
||||
public static SearchResponse assertSearchResponse(SearchRequestBuilder request) {
|
||||
return assertSearchResponse(request.get());
|
||||
|
@ -724,7 +724,7 @@ public class ElasticsearchAssertions {
|
|||
|
||||
/**
|
||||
* Applies basic assertions on the SearchResponse. This method checks if all shards were successful, if
|
||||
* any of the shards threw an exception and if the response is serializeable.
|
||||
* any of the shards threw an exception and if the response is serializable.
|
||||
*/
|
||||
public static SearchResponse assertSearchResponse(SearchResponse response) {
|
||||
assertNoFailures(response);
|
||||
|
|
Loading…
Reference in New Issue