Skip to content

chore: package cleanup#659

Merged
jpenilla merged 21 commits intomasterfrom
chore/2/rename
Jan 24, 2024
Merged

chore: package cleanup#659
jpenilla merged 21 commits intomasterfrom
chore/2/rename

Conversation

@Citymonstret
Copy link
Copy Markdown
Member

@Citymonstret Citymonstret commented Jan 24, 2024

annotations.injection -> injection
annotations.specifier -> parser.specifier
annotations.* -> util.annotation
arguments.parser -> parser
arguments.suggestions -> suggestion
arguments -> component
CommentComponent -> component.CommandComponent
arguments.(Standard)CommandSyntaxFormatter -> syntax
captions -> caption
types -> type
types.tuples -> type.tuple
+bean
CommandSuggestionProcessor -> SuggestionProcessor

annotations.injection -> injection
annotations.specifier -> parser.specifier
annotations.* -> util.annotation
arguments.parser -> parser
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jan 24, 2024

Test Results

 80 files  ±0   80 suites  ±0   13s ⏱️ ±0s
400 tests ±0  400 ✅ ±0  0 💤 ±0  0 ❌ ±0 
438 runs  ±0  438 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 9cacdbb. ± Comparison against base commit 65dfc9d.

This pull request removes 121 and adds 121 tests. Note that renamed tests count towards both.
cloud.commandframework.CommandBeanTest ‑ testCommandBeanRegistration()
cloud.commandframework.arguments.StandardCommandSyntaxFormatterTest ‑ accountsForSenderType()
cloud.commandframework.arguments.StandardCommandSyntaxFormatterTest ‑ respectsPermissions()
cloud.commandframework.arguments.aggregate.AggregateParserTest ‑ testExceptionForwarding()
cloud.commandframework.arguments.aggregate.AggregateParserTest ‑ testFailureComponentParsingFailure()
cloud.commandframework.arguments.aggregate.AggregateParserTest ‑ testFailureMissingInput()
cloud.commandframework.arguments.aggregate.AggregateParserTest ‑ testMultiLevelAggregateParsing()
cloud.commandframework.arguments.aggregate.AggregateParserTest ‑ testParsing()
cloud.commandframework.arguments.aggregate.AggregateParserTest ‑ testSuggestionsFirstArgument()
cloud.commandframework.arguments.aggregate.AggregateParserTest ‑ testSuggestionsSecondArgument()
…
cloud.commandframework.bean.CommandBeanTest ‑ testCommandBeanRegistration()
cloud.commandframework.exception.NoPermissionExceptionTest ‑ testCommandPermission()
cloud.commandframework.exception.NoPermissionExceptionTest ‑ testSucceededResult()
cloud.commandframework.exception.handling.ExceptionControllerTest ‑ HandleException_NewExceptionThrown_ExceptionHandled()
cloud.commandframework.exception.handling.ExceptionControllerTest ‑ HandleException_NoFallback_ExceptionReThrown()
cloud.commandframework.exception.handling.ExceptionControllerTest ‑ HandleException_ParentType_ExceptionHandled()
cloud.commandframework.exception.handling.ExceptionControllerTest ‑ HandleException_PassThroughHandlerWithConsumer_ExceptionHandled()
cloud.commandframework.exception.handling.ExceptionControllerTest ‑ HandleException_PassThroughHandler_ExceptionHandled()
cloud.commandframework.exception.handling.ExceptionControllerTest ‑ HandleException_PreciseType_ExceptionHandled()
cloud.commandframework.exception.handling.ExceptionControllerTest ‑ HandleException_UnwrappingHandler_ExceptionHandled()
…

♻️ This comment has been updated with latest results.

@Citymonstret Citymonstret marked this pull request as ready for review January 24, 2024 20:11
@jpenilla jpenilla merged commit fe826e2 into master Jan 24, 2024
@jpenilla jpenilla deleted the chore/2/rename branch January 24, 2024 21:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants