[Asterisk-code-review] tests: Spelling fixes (asterisk[18])

Josh Soref asteriskteam at digium.com
Mon Nov 15 09:39:34 CST 2021


Josh Soref has uploaded this change for review. ( https://gerrit.asterisk.org/c/asterisk/+/17377 )


Change subject: tests: Spelling fixes
......................................................................

tests: Spelling fixes

Correct typos of the following word families:

mounting
jitterbuffer
thrashing
original
manipulating
entries
actual
possibility
tasks
options
positives
taskprocessor
other
dynamic
declarative

ASTERISK-29714

Change-Id: I6b94659d045eec5d8d020fce2e9b6e2f593dfeb6
---
M tests/CI/gates.jenkinsfile
M tests/CI/periodics-daily.jenkinsfile
M tests/CI/ref_debug.jenkinsfile
M tests/CI/unittests.jenkinsfile
M tests/test_abstract_jb.c
M tests/test_aoc.c
M tests/test_astobj2_thrash.c
M tests/test_config.c
M tests/test_hashtab_thrash.c
M tests/test_media_cache.c
M tests/test_optional_api.c
M tests/test_sorcery.c
M tests/test_sorcery_memory_cache_thrash.c
M tests/test_stasis.c
M tests/test_strings.c
M tests/test_taskprocessor.c
16 files changed, 26 insertions(+), 26 deletions(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/77/17377/1

diff --git a/tests/CI/gates.jenkinsfile b/tests/CI/gates.jenkinsfile
index a5465ce..48861ab 100644
--- a/tests/CI/gates.jenkinsfile
+++ b/tests/CI/gates.jenkinsfile
@@ -8,7 +8,7 @@
  * pipeline, you can define the trigger in the pipeline file.  This keeps
  * everything in one place.  We transition to scripted pipeline later on because
  * we need to dynamically determine which docker image we're going to use and
- * you can't do that in a delcarative pipeline.
+ * you can't do that in a declarative pipeline.
  */
 def timeoutTime = 60
 def timeoutUnits = 'MINUTES'
@@ -139,7 +139,7 @@
 					def r = currentBuild.startTimeInMillis % images.length
 					def ri = images[(int)r]
 					def randomImage = env.DOCKER_REGISTRY + "/" + ri
-					/* FYI... Jenkins takes care of mouting the workspace for the container */
+					/* FYI... Jenkins takes care of mounting the workspace for the container */
 					def dockerOptions = "--privileged --ulimit core=0 --ulimit nofile=10240 " +
 						" --tmpfs /tmp:exec,size=1G -v /srv/jenkins:/srv/jenkins:rw -v /srv/cache:/srv/cache:rw " +
 						" --entrypoint=''"
diff --git a/tests/CI/periodics-daily.jenkinsfile b/tests/CI/periodics-daily.jenkinsfile
index 3ba8dfc..3fbffaf 100644
--- a/tests/CI/periodics-daily.jenkinsfile
+++ b/tests/CI/periodics-daily.jenkinsfile
@@ -8,7 +8,7 @@
  * pipeline, you can define the trigger in the pipeline file.  This keeps
  * everything in one place.  We transition to scripted pipeline later on because
  * we need to dynamically determine which docker image we're going to use and
- * you can't do that in a delcarative pipeline.
+ * you can't do that in a declarative pipeline.
  */
 def timeoutTime = 3
 def timeoutUnits = 'HOURS'
diff --git a/tests/CI/ref_debug.jenkinsfile b/tests/CI/ref_debug.jenkinsfile
index 742d5f8..c8e621b 100644
--- a/tests/CI/ref_debug.jenkinsfile
+++ b/tests/CI/ref_debug.jenkinsfile
@@ -8,7 +8,7 @@
  * pipeline, you can define the trigger in the pipeline file.  This keeps
  * everything in one place.  We transition to scripted pipeline later on because
  * we need to dynamically determine which docker image we're going to use and
- * you can't do that in a delcarative pipeline.
+ * you can't do that in a declarative pipeline.
  */
 def timeoutTime = 24
 def timeoutUnits = 'HOURS'
diff --git a/tests/CI/unittests.jenkinsfile b/tests/CI/unittests.jenkinsfile
index 1a0876d..24ef2f9 100644
--- a/tests/CI/unittests.jenkinsfile
+++ b/tests/CI/unittests.jenkinsfile
@@ -8,7 +8,7 @@
  * pipeline, you can define the trigger in the pipeline file.  This keeps
  * everything in one place.  We transition to scripted pipeline later on because
  * we need to dynamically determine which docker image we're going to use and
- * you can't do that in a delcarative pipeline.
+ * you can't do that in a declarative pipeline.
  */
 def timeoutTime = 30
 def timeoutUnits = 'MINUTES'
diff --git a/tests/test_abstract_jb.c b/tests/test_abstract_jb.c
index 76eb4bf..7826c9f 100644
--- a/tests/test_abstract_jb.c
+++ b/tests/test_abstract_jb.c
@@ -246,7 +246,7 @@
 		info->description = \
 			"This tests putting a single frame into a " literal_type_name " jitterbuffer " \
 			"when the jitterbuffer is empty and verifying that it is indeed " \
-			"the first frame on the jitterbufffer"; \
+			"the first frame on the jitterbuffer"; \
 		return AST_TEST_NOT_RUN; \
 	case TEST_EXECUTE: \
 		break; \
diff --git a/tests/test_aoc.c b/tests/test_aoc.c
index 0adc28d..6be64c4 100644
--- a/tests/test_aoc.c
+++ b/tests/test_aoc.c
@@ -453,7 +453,7 @@
 		}
 	}
 
-	/* Test every billing id possiblity */
+	/* Test every billing id possibility */
 	{
 		int billid[9] = {
 			AST_AOC_BILLING_NA,
diff --git a/tests/test_astobj2_thrash.c b/tests/test_astobj2_thrash.c
index 8a7e64c..1c1c30f 100644
--- a/tests/test_astobj2_thrash.c
+++ b/tests/test_astobj2_thrash.c
@@ -22,7 +22,7 @@
  * \author\verbatim David M. Lee, II <dlee at digium.com> \endverbatim
  *
  * Inspired by the original hashtest2.c by Steve Murphy <murf at digium.com>.  This test runs
- * several threads manipulatings a concurrent astobj2 container to see if they maintain
+ * several threads manipulating a concurrent astobj2 container to see if they maintain
  * consistency. While the tests attempt to check consistency and error normally, threading
  * errors often result in segfaults.
  * \ingroup tests
@@ -59,7 +59,7 @@
 	struct ao2_container *to_be_thrashed;
 	/*! Number of entries to insert in the grow thread. */
 	int max_grow;
-	/*! Number of enteries added by the grow thread. */
+	/*! Number of entries added by the grow thread. */
 	int grow_count;
 	/*! Entries preloaded into the hashtab; to be deleted by the shrink thread */
 	int preload;
diff --git a/tests/test_config.c b/tests/test_config.c
index c429bbe..770aa15 100644
--- a/tests/test_config.c
+++ b/tests/test_config.c
@@ -1493,7 +1493,7 @@
 	case TEST_INIT:
 		info->name = "config_options_test";
 		info->category = "/config/";
-		info->summary = "Config opptions unit test";
+		info->summary = "Config options unit test";
 		info->description =
 			"Tests the Config Options API";
 		return AST_TEST_NOT_RUN;
diff --git a/tests/test_hashtab_thrash.c b/tests/test_hashtab_thrash.c
index ee77c92..854c00c 100644
--- a/tests/test_hashtab_thrash.c
+++ b/tests/test_hashtab_thrash.c
@@ -22,7 +22,7 @@
  * \author\verbatim David M. Lee, II <dlee at digium.com> \endverbatim
  *
  * Inspired by the original hashtest.c by Steve Murphy <murf at digium.com>.  This test runs
- * several threads manipulatings a concurrent hastab to see if they maintain
+ * several threads manipulating a concurrent hastab to see if they maintain
  * consistency. While the tests attempt to check consistency and error normally, threading
  * errors often result in segfaults.
  * \ingroup tests
@@ -51,7 +51,7 @@
 	struct ast_hashtab *to_be_thrashed;
 	/*! Number of entries to insert in the grow thread. */
 	int max_grow;
-	/*! Number of enteries added by the grow thread. */
+	/*! Number of entries added by the grow thread. */
 	int grow_count;
 	/*! Entries preloaded into the hashtab; to be deleted by the shrink thread */
 	int preload;
diff --git a/tests/test_media_cache.c b/tests/test_media_cache.c
index c35e43f..14438f0 100644
--- a/tests/test_media_cache.c
+++ b/tests/test_media_cache.c
@@ -170,7 +170,7 @@
 		info->summary = "Test off nominal existance of resources in the cache";
 		info->description =
 			"This test verifies that checking for bad resources (NULL, bad "
-			"scheme, etc.) does not result in false positivies.";
+			"scheme, etc.) does not result in false positives.";
 		return AST_TEST_NOT_RUN;
 	case TEST_EXECUTE:
 		break;
diff --git a/tests/test_optional_api.c b/tests/test_optional_api.c
index fdf5d0c..0bac9a5 100644
--- a/tests/test_optional_api.c
+++ b/tests/test_optional_api.c
@@ -20,7 +20,7 @@
  * \file
  * \brief Test optional API.
  *
- * This tests exercise the underlying implementation functions. Acutal usage
+ * This tests exercise the underlying implementation functions. Actual usage
  * won't look anything like this; it would use the wrapper macros.
  *
  * \author\verbatim David M. Lee, II <dlee at digium.com> \endverbatim
diff --git a/tests/test_sorcery.c b/tests/test_sorcery.c
index 9f5cc7e..1f22040 100644
--- a/tests/test_sorcery.c
+++ b/tests/test_sorcery.c
@@ -842,7 +842,7 @@
 		ast_test_status_update(test, "A new ast_variable was not created for 'jim'\n");
 		res = AST_TEST_FAIL;
 	} else if (copy->jim == obj->jim) {
-		ast_test_status_update(test, "Created copy of 'jim' is actually the ogirinal 'jim'\n");
+		ast_test_status_update(test, "Created copy of 'jim' is actually the original 'jim'\n");
 		res = AST_TEST_FAIL;
 	} else if (strcmp(copy->jim->value, obj->jim->value)) {
 		ast_test_status_update(test, "Value of 1st 'jim' on newly created copy is not the same as original\n");
diff --git a/tests/test_sorcery_memory_cache_thrash.c b/tests/test_sorcery_memory_cache_thrash.c
index dfe7a7c..d9ad3d1 100644
--- a/tests/test_sorcery_memory_cache_thrash.c
+++ b/tests/test_sorcery_memory_cache_thrash.c
@@ -61,7 +61,7 @@
 	unsigned int average_execution_time;
 };
 
-/*! \brief Structure for memory cache thrasing */
+/*! \brief Structure for memory cache thrashing */
 struct sorcery_memory_cache_thrash {
 	/*! \brief The sorcery instance being tested */
 	struct ast_sorcery *sorcery;
@@ -160,7 +160,7 @@
 
 /*!
  * \internal
- * \brief Set up thrasing against a memory cache on a sorcery instance
+ * \brief Set up thrashing against a memory cache on a sorcery instance
  *
  * \param cache_configuration The sorcery memory cache configuration to use
  * \param update_threads The number of threads which should be constantly updating sorcery
diff --git a/tests/test_stasis.c b/tests/test_stasis.c
index 08c3d1e..5efb1ec 100644
--- a/tests/test_stasis.c
+++ b/tests/test_stasis.c
@@ -259,7 +259,7 @@
 		if (r == ETIMEDOUT) {
 			break;
 		}
-		ast_assert(r == 0); /* Not expecting any othet types of errors */
+		ast_assert(r == 0); /* Not expecting any other types of errors */
 	}
 	return consumer->messages_rxed_len;
 }
@@ -280,7 +280,7 @@
 		if (r == ETIMEDOUT) {
 			break;
 		}
-		ast_assert(r == 0); /* Not expecting any othet types of errors */
+		ast_assert(r == 0); /* Not expecting any other types of errors */
 	}
 	return consumer->complete;
 }
@@ -306,7 +306,7 @@
 		if (r == ETIMEDOUT) {
 			break;
 		}
-		ast_assert(r == 0); /* Not expecting any othet types of errors */
+		ast_assert(r == 0); /* Not expecting any other types of errors */
 	}
 	return consumer->messages_rxed_len;
 }
diff --git a/tests/test_strings.c b/tests/test_strings.c
index 63ac856..f3c7e56 100644
--- a/tests/test_strings.c
+++ b/tests/test_strings.c
@@ -22,7 +22,7 @@
  *
  * \author Mark Michelson <mmichelson at digium.com>
  *
- * This module will run some dyanmic string tests.
+ * This module will run some dynamic string tests.
  *
  * \ingroup tests
  */
diff --git a/tests/test_taskprocessor.c b/tests/test_taskprocessor.c
index c9a40df..537f77c 100644
--- a/tests/test_taskprocessor.c
+++ b/tests/test_taskprocessor.c
@@ -139,7 +139,7 @@
 	case TEST_INIT:
 		info->name = "default_taskprocessor";
 		info->category = "/main/taskprocessor/";
-		info->summary = "Test of default taskproccesor";
+		info->summary = "Test of default taskprocessor";
 		info->description =
 			"Ensures that a queued task gets executed.";
 		return AST_TEST_NOT_RUN;
@@ -363,7 +363,7 @@
 	case TEST_INIT:
 		info->name = "default_taskprocessor_load";
 		info->category = "/main/taskprocessor/";
-		info->summary = "Load test of default taskproccesor";
+		info->summary = "Load test of default taskprocessor";
 		info->description =
 			"Ensure that a large number of queued tasks are executed in the proper order.";
 		return AST_TEST_NOT_RUN;
@@ -546,7 +546,7 @@
 /*!
  * \brief Test for a taskprocessor with custom listener.
  *
- * This test pushes tasks to a taskprocessor with a custom listener, executes the taskss,
+ * This test pushes tasks to a taskprocessor with a custom listener, executes the tasks,
  * and destroys the taskprocessor.
  *
  * The test ensures that the listener's callbacks are called when expected and that the data
@@ -563,7 +563,7 @@
 	case TEST_INIT:
 		info->name = "taskprocessor_listener";
 		info->category = "/main/taskprocessor/";
-		info->summary = "Test of taskproccesor listeners";
+		info->summary = "Test of taskprocessor listeners";
 		info->description =
 			"Ensures that listener callbacks are called when expected.";
 		return AST_TEST_NOT_RUN;
@@ -760,7 +760,7 @@
 	case TEST_INIT:
 		info->name = "taskprocessor_shutdown";
 		info->category = "/main/taskprocessor/";
-		info->summary = "Test of taskproccesor shutdown sequence";
+		info->summary = "Test of taskprocessor shutdown sequence";
 		info->description =
 			"Ensures that all tasks run to completion after the taskprocessor has been unref'ed.";
 		return AST_TEST_NOT_RUN;

-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/17377
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 18
Gerrit-Change-Id: I6b94659d045eec5d8d020fce2e9b6e2f593dfeb6
Gerrit-Change-Number: 17377
Gerrit-PatchSet: 1
Gerrit-Owner: Josh Soref <jsoref at gmail.com>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20211115/6f22140c/attachment-0001.html>


More information about the asterisk-code-review mailing list