Skip to content

Commit

Permalink
SINT: fix various typos in javadoc and comment
Browse files Browse the repository at this point in the history
  • Loading branch information
guusdk committed Mar 14, 2024
1 parent ec4caf6 commit 8061065
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ protected AbstractSmackLowLevelIntegrationTest(SmackIntegrationTestEnvironment e
* Get a connected connection. Note that this method will return a connection constructed via the default connection
* descriptor. It is primarily meant for integration tests to discover if the XMPP service supports a certain
* feature, that the integration test requires to run. This feature discovery is typically done in the constructor of the
* integration tests. And if the discovery fails a {@link TestNotPossibleException} should be thrown by he constructor.
* integration tests. And if the discovery fails a {@link TestNotPossibleException} should be thrown by the constructor.
*
* <p> Please ensure that you invoke {@link #recycle(AbstractXMPPConnection connection)} once you are done with this connection.
*
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -494,7 +494,7 @@ private void runConcreteTest(ConcreteTest concreteTest)
return;
}
Throwable nonFatalFailureReason;
// junit assert's throw an AssertionError if they fail, those should not be
// junit asserts throw an AssertionError if they fail, those should not be
// thrown up, as it would be done by throwFatalException()
if (cause instanceof AssertionError) {
nonFatalFailureReason = cause;
Expand Down Expand Up @@ -594,8 +594,8 @@ private static Exception throwFatalException(Throwable e) throws Error, NoRespon
throw (InterruptedException) e;
}

// We handle NullPointerException as a non fatal exception, as they are mostly caused by an invalid reply where
// a extension element is missing. Consider for example
// We handle NullPointerException as a non-fatal exception, as they are mostly caused by an invalid reply where
// an extension element is missing. Consider for example
// assertEquals(StanzaError.Condition.foo, response.getError().getCondition())
// Otherwise NPEs could be caused by an internal bug in Smack, e.g. missing null handling.
if (e instanceof NullPointerException) {
Expand All @@ -613,9 +613,9 @@ private static Exception throwFatalException(Throwable e) throws Error, NoRespon
public static final class TestRunResult {

/**
* A short String of lowercase characters and numbers used to identify a integration test
* A short String of lowercase characters and numbers used to identify an integration test
* run. We use lowercase characters because this string will eventually be part of the
* localpart of the used JIDs (and the localpart is case insensitive).
* localpart of the used JIDs (and the localpart is case-insensitive).
*/
public final String testRunId = StringUtils.insecureRandomString(5).toLowerCase(Locale.US);

Expand Down

0 comments on commit 8061065

Please sign in to comment.