Skip to content

Commit

Permalink
Merge pull request element-hq#3318 from element-hq/feature/bma/checkM…
Browse files Browse the repository at this point in the history
…ethodName

Add test on function name which may start or end with spaces
  • Loading branch information
bmarty authored Aug 21, 2024
2 parents 0686745 + f2b38e5 commit c01bbe7
Show file tree
Hide file tree
Showing 4 changed files with 35 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -495,7 +495,7 @@ class RoomListPresenterTest {
}

@Test
fun `present - when room service returns no room, then contentState is Empty `() = runTest {
fun `present - when room service returns no room, then contentState is Empty`() = runTest {
val scope = CoroutineScope(coroutineContext + SupervisorJob())
val roomListService = FakeRoomListService()
roomListService.postAllRoomsLoadingState(RoomList.LoadingState.Loaded(0))
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ class MediaViewerPresenterTest {
}

@Test
fun `present - check all actions `() = runTest {
fun `present - check all actions`() = runTest {
val matrixMediaLoader = FakeMatrixMediaLoader()
val mediaActions = FakeLocalMediaActions()
val snackbarDispatcher = SnackbarDispatcher()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -306,7 +306,7 @@ class DefaultPushHandlerTest {
}

@Test
fun `when diagnostic PushData is received, the diagnostic push handler is informed `() =
fun `when diagnostic PushData is received, the diagnostic push handler is informed`() =
runTest {
val aPushData = PushData(
eventId = DefaultTestPush.TEST_EVENT_ID,
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,32 @@
/*
* Copyright (c) 2024 New Vector Ltd
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/

package io.element.android.tests.konsist

import com.lemonappdev.konsist.api.Konsist
import com.lemonappdev.konsist.api.verify.assertTrue
import org.junit.Test

class KonsistMethodNameTest {
@Test
fun `Ensure that method name does not start or end with spaces`() {
Konsist.scopeFromProject()
.functions()
.assertTrue {
it.name.trim() == it.name
}
}
}

0 comments on commit c01bbe7

Please sign in to comment.